Contribute to this Repository 💡

Come join us and contribute to this wonderful repository! 🌟 Your contributions will not only help improve this project but also give you a chance to dive into the world of Open Source on Github. Let's work together to make this project even better! 💪❤

@vaishnavi-3969
i wish to work on creating a simple calculator using java script can you please assign me this project.

@vaishnavi-3969
i wish to work on creating a simple calculator using java script can you please assign me this project.

Can u pls create a new issue I will assign it to u :) @Nandinics04

sorry @vaishnavi-3969
can i make a new issue on flipkart clone having html and css


Does Adonis support cluster mode?

I'm not using websockets, just want to enable cluster mode. I've tried changing server.js to something like this:
var cluster = require('cluster');
var http = require('http');
var numCPUs = 3;

if (cluster.isMaster) {
    for (var i = 0; i < numCPUs; i++) {
        cluster.fork();
    }

    cluster.on('exit', (worker) => {
        console.log('mayday! mayday! worker', worker.process.pid, ' is no more!')
        cluster.fork()
    })
} else {

  const { Ignitor } = require('@adonisjs/ignitor')

  new Ignitor(require('@adonisjs/fold'))
    .appRoot(__dirname)
    .fireHttpServer()
    .catch(console.error)

}

This indeed looks like it starts 3 servers on the console, but i tried printing response.json({ pid: process.pid }) and it never changes when i refresh as if it's still just a single server

The cluster module of Node.js decides which node will receive the request. Ideally it is based upon round robin (unless you are on Windows).
Try opening multiple tabs or windows and you will occasionally see the process.pid changing.
Even a plain Node.js HTTP server behaves the same way.
const http = require('http')
var cluster = require('cluster')
var numCPUs = 3

if (cluster.isMaster) {
  for (var i = 0; i < numCPUs; i++) {
    cluster.fork()
  }
  cluster.on('exit', (worker) => {
    console.log('mayday! mayday! worker', worker.process.pid, ' is no more!')
  })
} else {
	http.createServer((req, res) => {
		res.end(String(process.pid))
	}).listen(3000)
}


Ticker does not compile with NodeMCU (ESP8266)

Program can not be compiled with ESP8266 boards.

Without error log there isn't much to do. We use this library with ESP32 and it works as expected.

I have enabled ESP8266 board for testing and the code compiles just fine. Closing the issue.


rails 5 sign up raise error "Tenant must exists"

I can't signup any user cause it give me this error. Any clue?

If you, like me, are running rails 5.1 what solved my problem was to remove:
config.load_defaults 5.1
in config/application.rb


新人请教关于创建数据库连接的一点疑问

我看到bookstore示例代码这样写
type ServiceContext struct {
	c     config.Config
	Model model.BookModel
}

func NewServiceContext(c config.Config) *ServiceContext {
	return &ServiceContext{
		c:     c,
		Model: model.NewBookModel(sqlx.NewMysql(c.DataSource), c.Cache),
	}
}

把model放入ServiceContext 里面，如果项目有几十上百个model，都写在这里显得很臃肿，不知道会有性能问题吗？
另外我跟进sqlx.NewMysql方法看，好像使用了数据库连接池，是不是在logic层方法里面直接创建model实例也可以？如：

这是一个服务  负责一个模块  不会有太多model  业务复杂可以再拆分服务model就不会多了，当然也可以不用这么写

Oppositely, I think the example u show might have a worse performance. Because there are always dao objects to be created and collected by GC every time the method is called.

Oppositely, I think the example u show might have a worse performance. Because there are always dao objects to be created and collected by GC every time the method is called.

Thanks for the reply! At first, I also felt that it was wrong to create a database connection object in the request method. Later, I followed up and found that connection pool technology was used, so I was confused. If there are many tables (50+), wouldn't it be bloated to write them all in ServiceContext? Does everyone write like this?

相反，我认为你展示的示例性能可能会更差。因为每次调用该方法时总会有 dao 对象需要创建并由 GC 收集。

谢谢回复！一开始我也觉得在请求方法中创建数据库连接对象是不对的，后来跟进发现用到了连接池技术，于是就很疑惑，如果表很多（50+），全部写在ServiceContext中岂不臃肿？大家都是这么写的吗？

写到serviceContext里是把serviceContext当容器在用。


issue while doing initial sync

I see a lot like these when synching from network:
2015-08-19 13:16:45 Pre-allocating up to position 0x8000000 in blk00000.dat
2015-08-19 13:17:01 Peer=11 is stalling block download, disconnecting
2015-08-19 13:17:05 Peer=15 is stalling block download, disconnecting
2015-08-19 13:17:07 Peer=16 is stalling block download, disconnecting
2015-08-19 13:17:10 Peer=19 is stalling block download, disconnecting
2015-08-19 13:17:12 Peer=12 is stalling block download, disconnecting
2015-08-19 13:17:15 connect() to <IP_ADDRESS>:22556 failed after select(): Connection refused (61)
2015-08-19 13:17:17 Peer=13 is stalling block download, disconnecting
2015-08-19 13:17:38 receive version message: /Shibetoshi:1.8.2/: version 70003, blocks=839979, us=<IP_ADDRESS>:52643, peer=20
2015-08-19 13:17:38 Added time data, samples 12, offset +0 (+0 minutes)
2015-08-19 13:17:54 Peer=14 is stalling block download, disconnecting
2015-08-19 13:18:13 connect() to <IP_ADDRESS>:22556 failed after select(): Connection refused (61)

one of these (peer 19) was a 1.10.0 client (explicitly connected to langer's muchinfo daemon), so it's not just caused by 1.8.x

I have only been able to reproduce this over wifi, not over wired ethernet.

closing this, hopefully someone upstream has time to fix it.


[Question] Cannot demangle and get real function name under Linux

I tried to use g3log in my project, and I realized that the library cannot get real function name when the program crashes. I also tried with the examples provided with the library, it didn't work either.
I get a dump like (with the example g3log-FATAL-choice, run in a separate thread, choice 14):
***** SIGNAL SIGSEGV(11)

******* STACKDUMP *******
        stack dump [1]  ~/g3log/build/libg3log.so.2(+0x15f5f) [0x7f9522c69f5f]
        stack dump [2]  /lib/x86_64-linux-gnu/libpthread.so.0(+0x13140) [0x7f9522c39140]
        stack dump [3]  ./g3log-FATAL-choice(+0xfca0) [0x5576c21beca0]
        stack dump [4]  std::_Function_handler<std::unique_ptr<std::__future_base::_Result_base, std::__future_base::_Result_base::_Deleter> (), std::__future_base::_Task_setter<std::unique_ptr<std::__future_base::_Result<void>, std::__future_base::_Result_base::_Deleter>, std::thread::_Invoker<std::tuple<void (*)()> >, void> >::_M_invoke(std::_Any_data const&) + 0x16
        stack dump [5]  std::__future_base::_State_baseV2::_M_do_set(std::function<std::unique_ptr<std::__future_base::_Result_base, std::__future_base::_Result_base::_Deleter> ()>*, bool*) + 0x1b
        stack dump [6]  /lib/x86_64-linux-gnu/libpthread.so.0(+0x1034f) [0x7f9522c3634f]
        stack dump [7]  std::thread::_State_impl<std::thread::_Invoker<std::tuple<std::__future_base::_Async_state_impl<std::thread::_Invoker<std::tuple<void (*)()> >, void>::_Async_state_impl(std::thread::_Invoker<std::tuple<void (*)()> >&&)::{lambda()#1}> > >::_M_run() + 0xfa
        stack dump [8]  /lib/x86_64-linux-gnu/libstdc++.so.6(+0xceed0) [0x7f9522b27ed0]
        stack dump [9]  /lib/x86_64-linux-gnu/libpthread.so.0(+0x7ea7) [0x7f9522c2dea7]
        stack dump [10]  clone + 0x3f

Exiting after fatal event  (FATAL_SIGNAL). Fatal type:  SIGSEGV
Log content flushed successfully to sink

g3log g3FileSink shutdown at: 21:22:53 571038
Log file at: [/tmp/g3log-FATAL-choice.g3log.20231124-212249.log]

exitWithDefaultSignalHandler:244. Exiting due to FATAL_SIGNAL, 11

I picked out all programs for crash handling in file src/crashhandler_unix.cpp and did some tests. It seems like most of the times there is no mangled name before +. Therefore I got an empty mangled_name here, always.
https://github.com/KjellKod/g3log/blob/63f327270325d827181ae1eaa0a4601c61fba67e/src/crashhandler_unix.cpp#L167-L171
And also, __cxa_demangle never works. I'm not sure if it's also caused by the same issue.
I'm wondering if it is normal? If not, what may be wrong?

So if I understand you correctly, you get the correct demangled dump when you run into he examples but not when it's integrated with your software.
Typically when that happens it's due to not linking with access to the symbols. Exactly how you do this is dependent on your compiler and platform.
Ref #5 Handling of fatal

Typically when that happens it's due to not linking with access to the object symbols. Exactly how you do this is dependent on your compiler and platform.

Thanks for your response xD
I added -rdynamic to global compile flags and it does solve my problem. Again thx a lot.


separate client address into separate fields

Pull Request Template Form
Description
separate client address into separate fields
Fixes # (issue)
Type of change
Please delete options that are not relevant.
[ ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
Change status
[ ] Complete, tested, ready to review and merge
[x ] Complete, but not tested (may need new tests)
[ ] Incomplete / work in progress, PR is for discussion / feedback
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
[ ] Test A
[ ] Test B
#Checklist:
[ X] My code follows the style guidelines of this project
[ X] I have performed a self-review of my own code
[ X] My code has been reviewed by at least one peer
[ X] I have commented my code, particularly in hard-to-understand areas
[ X] I have made corresponding changes to the documentation
[ X] My changes generate no new warnings
[ X] I have added tests that prove my fix is effective or that my feature works
[ X] New and existing unit tests pass locally with my changes

could have probably saved a few lines of code using arrow functions in index.js, but code looks good!


Exception in Image Resource Service

Easy reproduce is to run my app then switch to a different app and then back.  Upon switching back I get this exception 100% of the time.
14:35:29.920: I/flutter : ══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
14:35:29.923: I/flutter : The following assertion was thrown by a synchronously-called image listener:
14:35:29.926: I/flutter : 'package:flutter/src/rendering/object.dart': Failed assertion: line 1725: 'owner == null ||
14:35:29.928: I/flutter : !owner.debugDoingPaint' is not true.
14:35:29.933: I/flutter : When the exception was thrown, this was the stack:
14:35:29.953: I/flutter : #0      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:27)
14:35:29.956: I/flutter : #1      _AssertionError._checkAssertion (dart:core-patch/errors_patch.dart:34)
14:35:29.959: I/flutter : #2      RenderObject.markNeedsPaint (package:flutter/src/rendering/object.dart:1725)
14:35:29.962: I/flutter : #3      _BoxDecorationPainter._imageListener (package:flutter/src/painting/box_painter.dart:1426)
14:35:29.965: I/flutter : #4      ImageStreamCompleter.addListener (package:flutter/src/services/image_stream.dart:161)
14:35:29.968: I/flutter : #5      ImageStream.addListener (package:flutter/src/services/image_stream.dart:96)
14:35:29.970: I/flutter : #6      _BoxDecorationPainter._paintBackgroundImage (package:flutter/src/painting/box_painter.dart:1405)
14:35:29.972: I/flutter : #7      _BoxDecorationPainter.paint (package:flutter/src/painting/box_painter.dart:1445)
14:35:29.975: I/flutter : #8      RenderDecoratedBox.paint (package:flutter/src/rendering/proxy_box.dart:1176)
14:35:29.977: I/flutter : #9      RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:1806)
14:35:29.980: I/flutter : #10     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:101)
14:35:29.983: I/flutter : #11     RenderBox&ContainerRenderObjectMixin&RenderBoxContainerDefaultsMixin.defaultPaint (package:flutter/src/rendering/box.dart:1434)
14:35:29.985: I/flutter : #12     RenderStack.paintStack (package:flutter/src/rendering/stack.dart:407)
14:35:29.988: I/flutter : #13     RenderStack.paint (package:flutter/src/rendering/stack.dart:415)
14:35:29.990: I/flutter : #14     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:1806)
14:35:29.993: I/flutter : #15     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:101)
14:35:29.996: I/flutter : #16     <redacted>.paint (<redacted>:30)
14:35:29.999: I/flutter : #17     <redacted>.paint.<anonymous closure> (<redacted>:366)

Another reproduce:
If I call: SystemChrome.setEnabledSystemUIOverlays(0); In the initState of the main widget of my app I also get the above exception.

@Hixie Maybe we need to check in the ImageStream whether we're currently painting this object?

There's several ways I can see to solve this:

resolve it aggressively, rather than lazily, so that the synchronous callback happens before paint: bad for perf
check to see if you're painting, and avoid calling markNeedsPaint if so: awkward, and we don't currently track this so we'd have to have a new bit
remove the assert: i think it's caught some real bugs so probably not a good idea
plumb a boolean through from _paintBackgroundImage all the way to _imageListener which tracks whether we're painting or not (aka whether it's a synchronous call or not): makes the APIs ugly

The least disruptive change would probably be tracking the "are we painting" bit just in _BoxDecorationPainter, not calling onChanged if it's set. But that'll still leave this landmine for other ImageProvider users.


关于Reward Modeling阶段的Loss计算

目前在src/llmtuner/tuner/rm/trainer.py中compute_loss这一部分采用的loss计算是源自InstructGPT中的提出的 $Loss=-log(\sigma (r_{\theta }(x,y_c)-r_{\theta }(x, y_r)))$ 。但在LLaMA2的论文中提到了可以在loss中添加一个m项的方式标定不同的偏好差别，原文如下

Built on top of this binary ranking loss, we further modify it separately for better helpfulness and safety reward models as follows. Given that our preference ratings is decomposed as a scale of four points (e.g.,
significantly better), as presented in Section 3.2.1, it can be useful to leverage this information to explicitly teach the reward model to assign more discrepant scores to the generations that have more differences. To do so, we further add a margin component in the loss: $$Loss=-log(\sigma (r_{\theta }(x,y_c)-r_{\theta }(x, y_r)-m(r)))$$ where the margin m(r) is a discrete function of the preference rating. Naturally, we use a large margin for pairs with distinct responses, and a smaller one for those with similar responses (shown in Table 27). We found this margin component can improve Helpfulness reward model accuracy especially on samples where two responses are more separable.

请问是否有在未来的某一个版本中添加对m项的支持
P.S. 我个人也有一定的开发能力，如果能大致和我说一下提交Pull Request的规范我也应该可以折腾一下

可以支持该功能，欢迎提交 PR


Tile tints

Is there any way to add a color tint to a specific tile?
For example, if I need to indicate that a tile can't be placed, I might want to give it a red tint. Or green, if it can be placed.
Is this something that this crate supports via a shader? Or is there another way to go about handling this?

Oh sweet! Is this going to be a part of the next release?

Oh sweet! Is this going to be a part of the next release?
Yep! I'm currently waiting on bevy 0.6 to release the next version of bevy_ecs_tilemap.

Sounds good. Looking forward to it!


refactor routers

please review.

Please note that this change will completely remove the deferred class view support and will use the function view exclusively

Pros: More centrally managed router
Disadvantages: not intuitive enough

In this way, there is one more step in the development of the interface,and I believe we should consider a balance.
Maybe we can open a separate branch first

Getting the router to work the way it did, I decided to abandon the change
Yes, it became more cumbersome


enable unit testing and linting

Before we merge PRs, we should be certain that make build and golangci-lint pass.
My preference would be to use GitHub actions, similar to go-logr: https://github.com/go-logr/logr/tree/master/.github/workflows
Prow doesn't properly work with GitHub actions (might merge before GitHub action is done), but that's an acceptable compromise. Using Prow would be more complicated.

/assign @byako

/close


SceneRPCInterface: Sort rpc names using StringLikeVariantOrder

Is the goal of this PR just to avoid code duplication? Or is there an intended behaviour difference in this sorting function?

just to avoid code duplication


Adding example for custom DataLoader to tutorial 18

What does this implement/fix? Explain your changes
I was happy to see the recent exposure of more control for the training loop. As discussed with @janfb, I added a small example on how rewrite the training loop to exploit such a DataLoader.
Does this close any currently open issues?
I didn't create an issue for this.
Any relevant code examples, logs, error output, etc?
I basically added a section which documents this:
optw = AdamW(list(maf_estimator.parameters()), lr=5e-4)
nepochs = 50

for ep in range(nepochs):
    for idx, (theta_batch, x_batch) in enumerate(train_loader):
        optw.zero_grad()
        losses = maf_estimator.loss(theta_batch, condition=x_batch)
        loss = torch.mean(losses)
        loss.backward()
        optw.step()
    if ep % 10 == 0:
        print("last loss", loss.item())

Any other comments?
Can you please let me know, how to clean a notebook before I commit it? I recall from the hackathon, that there were a couple of steps for this. But maybe I am mistkane.
Checklist
Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask. We're here to
help! This is simply a reminder of what we are going to look for before merging
your code.

[x] I have read and understood the contribution
guidelines
[x] I agree with re-licensing my contribution from AGPLv3 to Apache-2.0.
[] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have reported how long the new tests run and potentially marked them
with pytest.mark.slow.
[ ] New and existing unit tests pass locally with my changes
[ ] I performed linting and formatting as described in the contribution
guidelines
[ ] I rebased on main (or there are no conflicts with main)
[ ] For reviewer: The continuous deployment (CD) workflow are passing.

Great, thanks!
Can you make this edit in tutorial and mention the use case with a large dataset, i.e., how this control over the training loop solve the problem with high-D data?
Please also add how you created the maf_estimator using build_maf or so, using only parts of the training data.
Thanks! 🙏

Yes, will do. So I don't need to clean the notebook before committing it?

@janfb here is the notebook in question. I removed the plots for the time being. Let me know if I should regenerate them.

I had a quick look and, yikes, there was a testing leftover that restricted the tests to tutorial 05. 🤦♂
I will fix it here in the branch.
Thanks for leading us to this @psteinb 😃

Let's make a new release after this PR to update the website?

Awesome, thanks a lot @psteinb for adding this! 🎉


Create Timeline component

I want to create a timeline component which can be resuable and will look something like this or even more better.

Hi @orendrasingh good thoughts on this, but I guess 2 similar PRs #57 and #63 have already been there and are to be reviewed and merged by mentors.
Anyways, you can open another and mentors will review the better one :)

This has already been created in #180. Thanks for your contribution @orendrasingh :)


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
ce284eb5807bb13ddf8ac4fd8524578448e371e9
link
/test e2e-smoke

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Feature/apple watch

Description
Re: Issue #217
Created an Apple Watch app to function as a companion app for COVID Safe Paths app.
As mentioned in the issue description, it is not yet integrated with the main iOS app.  I will be deferring that task to someone who knows React Native.
How to test
Testing can proceed after integration.  Details in README
README-CSP-Roam.pdf

Thanks so much for this!  Want to let you know the timing on this.  We're currently locking down for the v1 release, and we don't have the translator or testing bandwidth to include this in v1.  I'm adding the Do Not Merge (yet) and for next next release tags.  I'd like someone to come back and review this after the branch for v1.

Sounds good.  I have a separate Xcode project that demonstrates integration between the watch app and a generic iOS app that will be helpful for whomever is doing the react native integration.

Yeah, thanks for this.
Is this the kind of thing we'd need to communicate with it? https://github.com/mtford90/react-native-watch-connectivity

Yes

An update on this. We're working on adding "feature flags" to the build, so that this could be hidden behind a feature flag for iteration/testing in a beta channel.

When ready for integration, see this repo that contains the watch app code and a paired iOS app used for testing: https://github.com/JohnMantegna/PrivateKitTest-Watch-Roam

Thanks, tracked in Jira as https://pathcheck.atlassian.net/browse/SAF-128


Fix 108 mestimator

Some pretty major cleanup is done and issue #108 is handled.
The upshot is that methods that are the same or very similar in TRME and TRME_RR have been merged in MEstimator.  TRME and TRME_RR are now extensions of MEstimator and the variations are described in their
Issue #96 can be handled after merging back into main.
Issue #107 also mentioned in #108 can be left until we address the processing config.

Some pretty major cleanup is done and issue #108 is handled.
The upshot is that methods that are the same or very similar in TRME and TRME_RR have been merged in MEstimator.  TRME and TRME_RR are now extensions of MEstimator and the variations are described in their
Issue #96 can be handled after merging back into main.
Issue #107 also mentioned in #108 can be left until we address the processing config.


Lab Support Files in the Windows 8.1 SDK

The Lab support Files seem to be missing from the Windows 8.1 SDK I attached them.
BuildLabSupportFiles.zip

A final windows 8.1 Build lab should look something like this. With the VS 2013 build tools.

This doesn't seem like a winui issue, @stevenbrix or @MikeHillberg do you know where this should go?

I do not, @AdamBraden do you have any idea where this should go? Is Windows 8.1 SDK still supported?
@jdm7dv, what kind of 8.1 project are you trying to build?

🦙 @stevenbrix Win 8.1 left mainstream support in Jan 2018. It's only extended support until Jan 2023: https://support.microsoft.com/en-us/help/13853/windows-lifecycle-fact-sheet
Also see Support Policy Info.


Follow upstream changes of libvirt-xml-go

The "memoryBacking" field in Domain was recently added upstream,
so we only need a patch for "cputune".
When the patch for "cputune" [1] will be merged upstream, we
will be able to use vanilla libvirt-xml-go.
[1] https://www.redhat.com/archives/libvir-list/2017-October/msg01151.html

This change is

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 this HexUint looks interesting...

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

tests need to be updated. LGTM otherwise

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

@jellonek @istalker2 PTAL

Reviewed 11 of 11 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 4 of 4 files at r1, 11 of 11 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


RFC: Avro support

Hey all, I'm not currently using Avro at all but I figured it would be useful to start adding support. I've implemented an experimental processor that does some very basic conversions to/from JSON based on a schema and encoding.
It would be very useful if users of Avro could let me know if this solves any real problems you might have and what else you feel is needed.

The WIP documentation is here: https://github.com/Jeffail/benthos/tree/master/docs/processors#avro

Closing as no one cares and life has no meaning. Feel free to reopen if you have comments you want to share.


Should the keys in result.Counts be ordered ?

What is the expected enhancement?
result.Counts would be more convenient if it were a subclass of OrderedDict.  The keys are bit strings, which have a natural order, so it would be useful to preserve the order. For example, currently, when displaying the counts interactively, the keys (bit strings) are not ordered.
Is there a reason this choice was made ?
If ordering the counts is not desirable (too expensive or not necessary for most uses, etc.) an alternative is a function that returns the an ordered dict.

It was mostly done this way for compatibility with the dict that was there before see #4501 I didn't really think about it except to make the new Counts class subclass the same type as what was there before. That being said I don't think preserving insertion ordering is fine. But, I don't think we'll need to do anything here. After https://github.com/Qiskit/qiskit-terra/pull/4926 merges python will be preserving insertion ordering. In python3.6 that was added because of performance optimizations in that release. but it wasn't explicitly guaranteed yet. In 3.7 it was added as a guarantee moving forward, so we really only need to use OrderedDict while we still support Python3.5.

But, I don't think we'll need to do anything here

This makes sense since python dicts are now ordered.
But, in applications I am playing with now, under python 3.8, the strings are not ordered. Since insertion order is preserved, this tells me that the strings are not ordered when the source dict is created. They also don't appear to be inserted in the order in which the results are obtained (which should be random) because many strings with very low counts come before strings with very high counts.

Well we can look at sorting them before they're added to the Counts object here:
https://github.com/Qiskit/qiskit-terra/blob/master/qiskit/result/result.py#L250
change that to be something like
dict_list.append(Counts(sorted(self.data(key)['counts']), **counts_header))

but before we open a PR doing this we should benchmark it to see what the practical overhead of sorting is. If it's minimal lets go for it, but if it adds a noticeable amount of overhead I'd say lets just add a helper method (or kwargs to existing ones) to counts() to get a sorted iter of the counts dict and leave it up to the user.

result = ...
raw_dict = result.data(0)['counts']
counts = result.get_counts()
ms = counts.memory_slots

def makeCounts(d, memory_slots):
    sk = sorted(d)  # returns sorted keys
    sd = {k: d[k] for k in sk}
    return Counts(sd, memory_slots=memory_slots)

Then
In [1]: len(raw_dict)
Out[1]: 25

In [2]: %timeit Counts(raw_dict, memory_slots=ms)
38.1 µs ± 230 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

In [3]: %timeit makeCounts(raw_dict, memory_slots=ms)
40.4 µs ± 94.3 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

In [2]: len(raw_dict)
Out[2]: 100

In [3]: %timeit Counts(raw_dict, memory_slots=ms)
118 µs ± 408 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

In [4]: %timeit makeCounts(raw_dict, memory_slots=ms)
123 µs ± 422 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

almost all the extra time is due not to sorting, but to building the intermediate dict.
adding a helper method would be the least disruptive in the sense that we are sure it would not affect existing code. But, it adds to the users' cognitive/documentation-digging burden. I can't imagine a situation in which a user would rather have the bitstrings in a random order than in numerical order.

I can imagine situation where a user wants the counts ordered by value instead, as she might be interested in the most statistically significant results only...

This issue was recently added in the "To Do" list of Backends, Qobj, and Result . Requires more information on the status.

I can imagine situation where a user wants the counts ordered by value instead, as she might be interested in the most statistically significant results only...

this use cases is covered with the Count.most_frequent() method.

@mtreinish, is the performance analysis done by @jlapeyre enough to make a decision about this issue?

To reproduce the issue, notice that the output of the following code result in an unsorted key:
from qiskit import Aer, QuantumCircuit

circuit = QuantumCircuit(2)
circuit.h(0)
circuit.cx(0, 1)
circuit.measure_all()

sim = Aer.get_backend('aer_simulator')

result = sim.run(circuit, seed_simulator=10).result()
counts = result.get_counts()
print(counts)

{'11': 505, '00': 519}

In the interests of trimming old issues, I'll close this as stale now.  There is more than one ordering that might be useful to users, and "arbitrary" is the cheapest to construct, which is convenient for performance-focussed users who want to do their own post-processing.  If there's more to discuss, feel free to re-open.
fwiw, sorting this particular object in Python in the order requested is as simple as Counts(sorted(counts.items())), so imo it's something we should reasonably expect a user to do themselves if they want to choose the performance hit.


buildLuarocksPackage: build_dependencies and test_dependencies not available to luarocks during checkPhase

Describe the bug
When building a luarocks package with a custom build backend (e.g. luarocks-build-rust-mlua), or that has test dependencies (e.g. nlua), luarocks does not pick up these dependencies from the LUAROCKS_CONFIG in the checkPhase.
For build backends like luarocks-build-rust-mlua, it is necessary to run luarocks make in the preCheck hook. This fails because luarocks tries to download the build backend from luarocks org.
The only workaround I've found so far is to add the test dependencies and build backends to the propagatedBuildInputs, but this means that build/test dependencies will unnecessarily be propagated.
Steps To Reproduce
See for example this overlay.
Expected behavior
luarocks should pick up test_dependencies and build_dependencies from the nativeCheckInputs or nativeBuildInputs.
Notify maintainers
cc @teto

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

agreed. In fact I see some reverted attempts of doing so in my nixpkgs fork. Go ahead :)


Jupyterhub kernel died when using pyodbc

I use the same env in Jupyterhub and in Jupyter notebook, but in Jupyterhub kernel dies on following line of code:
import pyodbc conn = pyodbc.connect('DSN=mydnsname',CHARSET='UTF8')

Are you really trying to do theimport and the connect on the same line, or is that just a formatting issue in your post?

that was a a formatting issue

Could you provide more information about your environment, including an ODBC trace?


Custom event editable

Is your feature request related to a problem? Please describe.
Sometimes when I create a custom event, I didn't input correctly the date and would like to edit it rather than delete and redo
Describe the solution you'd like
An edit button next to Delete
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Duplicate of #134


auth: add disable disconnect on expiry config

Adds support for disabling disconnecting the client when their JWT expires.
See https://github.com/andydunstall/piko/issues/179#issuecomment-2532299219.
cc @dipack95

@andydunstall Can I trouble you for a version release that includes this change? Excited to give this a shot!

How are you running Piko? You can create the Docker images manually with make image or build the binary with make piko?
(I can tag later this week though it's probably quicker for you to build yourself)

I've been downloading the docker image from gcr.io, and the binaries from the releases page since these are being deployed on EKS. I can wait for a new tag though, thank you!

@dipack95 thats tagged, sorry for the delay https://github.com/andydunstall/piko/releases/tag/v0.6.5

@andydunstall Thank you so much! This project is really appreciated :)


Icon Request

Hello,
This is an icon request for the following apps:

Nextcloud Deck - it.niedermann.nextcloud.deck
OpenTracks - de.dennisguse.opentracks
Gadgetbridge - nodomain.freeyourgadget.gadgetbridge
Etar - org.lineageos.etar
FX - nextapp.fx

There are other apps I have installed but icon requests for those already exist.

Hello! Weʼre switching to a new system for icon requests in order to be able to better manage them. From now on, icon requests can be submitted here: https://forms.gle/Fx8vZAiWdW1Tyjo57. If your icon request hasnʼt yet been fulfiled, it has been copied to our new icon request database, meaning no action is required. Consequently, this issue will be closed.


Fix typos in examples

@eltoder change target branch from master to dev, please

@dapper91 done

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.06%. Comparing base (97d6af8) to head (0906603).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #231   +/-   ##
=======================================
  Coverage   92.06%   92.06%
=======================================
  Files          29       29
  Lines        1651     1651
=======================================
  Hits         1520     1520
  Misses        131      131

Flag
Coverage Δ

unittests
92.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


When you first launch the wallet, the "Red envelope" doesn't open

When we open the wallet for the first time after creating or restoring it and try to open the "Red envelope", the screen is empty
Note: If you switch tabs and re-open, then open
Step:

create\restore wallet
open the "DAPP Store" tab
click  launch button for Red Envelope

Actual: the screen is empty

This is fixed, please test now.

not fixed
build 5.3.10209.3566 (Windows)

not fixed
build: 6.0.1033.3676 BeamX

not fixed
build: 6.0.1033.3676 BeamX

Looks like something inside JS: WebSocket connection to 'ws://<IP_ADDRESS>/ws' failed: WebSocket is closed before the connection is established.
Bro, would you please take a look.
You need.

Restore wallet
Load envelope

This is reproduce only once, only after restore. On the second, third &c. run is not reproduced.

Looks like something inside JS: WebSocket connection to 'ws://<IP_ADDRESS>/ws' failed: WebSocket is closed before the connection is established.
Bro, would you please take a look.
You need.

Restore wallet
Load envelope

This is reproduce only once, only after restore. On the second, third &c. run is not reproduced.


Access Token Expired - Request failed with status code 401 HttpResponseException False

Fix it in keycloak console -> SSO Session Max changed.


Always use the latest patch for Go?

Required Information
Question, Bug, or Feature?
Type: Feature
Enter Task Name: GoTool
Environment

Server - Azure Pipelines
Agent - Hosted

Issue Description
I would like to use the latest patch for any particular Go version so that I get security patches automatically. It doesn't look like that's supported though. If I use 1.16 for the version, it will use 1.16.0 and not the most recent patch.
It would be great to be able to specify 1.16 and GoTool would lookup and use the most recent patch, instead of defaulting to the oldest version of that Go release.
If I submitted a PR to support this feature, would it be accepted?

Bumping since I didn't hear back, still happy to submit a patch if there's agreement on how to using the latest version should work.

I'm still quite interested in this feature! 😁


feat: create table component based on ant design Table

Adds a new Table component based on Ant Design as part of movement to a Superset Design system
SUMMARY

To view the table component and see the documentation:
https://75b68f5--62b4d4c211d03654c1992245.chromatic.com/?path=/story/design-system-components-table-overview--page (manually updating to Chromatic)
To run local:

checkout this branch
open a terminal at the superset/superset-frontend directory and run npm ci
in the same terminal run npm run storybook
Open http://localhost:6006/?path=/docs/design-system-components-table-overview--page in your browser

This provides the documentation to use the new Table component as well as interactive stories to see the behavior of the various props
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Approved but have a few questions!

I went through and found some things/had some questions. I also noticed that a storybook log was pushed up to superset-frontend/build-storybook.log. I don't see storybook logs elsewhere in the codebase, was this log meant to be part of this PR?

@lyndsiWilliams I removed the build-storybook.log file.  Thanks!

I went through and found some things/had some questions. I also noticed that a storybook log was pushed up to superset-frontend/build-storybook.log. I don't see storybook logs elsewhere in the codebase, was this log meant to be part of this PR?

@lyndsiWilliams I removed that file, thanks for catching that!

LGTM. Thanks for all the hard work and for addressing the comments @eric-briscoe!

Thank you for the thorough reviews @michael-s-molina @lyndsiWilliams @AAfghahi @pkdotson 😄


Fix code injection vulnerability of getGroupsForUserCommand

What changes are proposed in this pull request?
We split the ShellUtils.getGroupsForUserCommand into two seperate methods ShellUtils.getEffectiveGroupsForUserCommand and ShellUtils.getAllGroupsForUserCommand. Then change the corresponding test cases.
Why are the changes needed?
Executing commands like "bash -c some_command" will introduce code injection vulnerability.
For example, if CommonUtils.getUnixGroups("| echo 123 ") is invoked, the whole command is truncated by |and "echo 123" will be executed.
Therefore, the most simple way to fix it is to not to use "bash -c".
Does this PR introduce any user facing changes?
@Deprecated is marked on the ShellUtils.getGroupsForUserCommand in case some users' codes stongly rely on this method. But in the repository Alluxio/alluxio, ShellUtils.getGroupsForUserCommand is never used anymore. It is replaced by ShellUtils.getEffectiveGroupsForUserCommand and ShellUtils.getAllGroupsForUserCommand.

I've sign and e-mail CLA. Please re-check it.

alluxio-bot, check this please

FYI: @yyongycy

Merged as part of https://github.com/Alluxio/alluxio/pull/18536
Thanks for the contribution!


noaa-apt: init at 1.2.0

Motivation for this change
noaa-apt is decoding software that enables the conversion of audio recordings of NOAA weather satellite APT protocol transmissions into a variety of image formats. Typically this recording is done using software defined radios (SDRs) like RTLSDR, HackRF, etc., which are already well supported in Nix. This software would complement the growing suite of SDR tools.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/278

Thank you for your review! I believe this covers all those fixes.


MON doesn't join cluster after restart

Hi,
my scenario: I have three server. On each server i have one MON daemon deployed. For the deployment i use the containerized scenario, so every mon service runs in its own docker container.
The expected behavior is, that the mon daemon rejoins the existing cluster.
The actual behavior: If I restart one server, the docker container with the mon daemon tries to spawn but exits with the following error message:
mon does not exist in monmap, will attempt to join an existing cluster
...
e1129 not in monmap and have been in a quorum before; must have been removed
e1129 commit suicide!

I didn't changed anything during the mon reboot.

It seems to me that the file /etc/ceph/monmap is the problem. It contains only the first mon on all three nodes. If I override this file with the current monmap, after all mon daemons are startet, a mon restart is no problem anymore.
So maybe we have to deploy a final monmap on all nodes, after the actually mon deployment.

Did you run the deployment with serial: 1? Did you use this file? https://github.com/ceph/ceph-ansible/blob/master/site-docker.yml.sample

Yes to both of your questions :)

Normally this works with an initial monmap built with a single node :/

Fixed in recent versions.


KAFKA-10780; Rewrite ControllerZNode struct with auto-generated protocol

More detailed description of your change

The #9662 rewrite FeatureZNode struct with auto-generated protocol, but it's a non-trivial change, so we can just review this simple pr first.
Copy some code of org.apache.kafka.raft.FileBasedStateStore to generate json data from and to FeatureZNodeData

Summary of testing strategy (including rationale)
unit test
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hi, @abbccdda PTAL.

Thanks for the PR. I don't think we should do this since KIP-500 will replace all this code.

@ijuma Thank you, I ignored this fact.
Some of the ZkNode will be removed because we no longer need them after KIP-500, for example IsrChangeNotificationZNode and ControllerZNode, however Some ZkNode will just be moved from ZkNode to RaftKafkaMetadataLog , for example TopicZNode and FeatureZNode.
So can we still rewrite their struct use auto-generated protocol which can also be used after KIP-500? also ping @abbccdda as this is a major direction discussion.
I will close this pr and create another for TopicZNode if you think it's reasonable, Thank you.


Updated Navbar component

[Re PR: ]
Fixes #10

Responsive navbar component
Animated using Framer Motion
Preserved all actions

Still i dont know brother , why is this not getting deployed
Can you please share your discord id ??

It's asking for Authorization from me to get it deployed.
On Fri, 10 May, 2024, 22:22 Vansh Waldeo, @.***> wrote:

Still i dont know brother , why is this not getting deployed
Can you please share your discord id ??
—
Reply to this email directly, view it on GitHub
https://github.com/VanshKing30/FoodiesWeb/pull/31#issuecomment-2104930335,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAC2LBCKM3XQ2QF6U466BRDZBT3NJAVCNFSM6AAAAABHQ5BCYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBUHEZTAMZTGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Authorize it !!

My discord id is: stephenstolk_18105
On Fri, 10 May, 2024, 22:22 Vansh Waldeo, @.***> wrote:

Still i dont know brother , why is this not getting deployed
Can you please share your discord id ??
—
Reply to this email directly, view it on GitHub
https://github.com/VanshKing30/FoodiesWeb/pull/31#issuecomment-2104930335,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAC2LBCKM3XQ2QF6U466BRDZBT3NJAVCNFSM6AAAAABHQ5BCYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBUHEZTAMZTGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

I am getting this authorization:

Approve it


Added Nanoleaf Essentials A19 Matter

Device information

Home Assistant Device information

Checklist

[x] I have created a single PR per device. When you have multiple submissions please create separate PR's.
[x] For lights - I have only included the gzipped files (*.gz), not the raw CSV files.
[] For lights - I have provided a CSV file per supported color mode. Look that up in Developer Tools -> States

Additional info

Nice thanks for the addition


fix(topbar-mobile): input not able to use 'SPACE' key on topbar

Using an input on topbar would close when trying to use 'space' key. Removed space listener.
Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots:
Before (If applicable):

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hey, sorry for commenting on this old issue. There are cases where we still need to use the Enter key. For example: a user is typing on an input text box. We'd like to submit the form when the user presses Enter.
As the current topbar is listening for 'document:keydown' events, it blocks Enter key press events on the whole document. Is there a solution to fix this?


Flashplayer won't load in Opera <15 with DEV 532

Flashplayer isn't working with Opera 12.17 (don't tell me to upgrade to Opera 15+)
HTML5 player works but will only play videos in 360p
Anyone know any fix for this?
Thanks

Two important things to consider: Is DASH enabled and is it supported by Opera 12?
If the answer to either of these is "no," then there's nothing else we can tell you other than to use a browser that is actively supported/developed.


WIP: Review of types in Microsoft.ML.Core

I am using the "pull request" mechanism to allow for easy commenting on the MD file. This is not meant to be merged into the main repository. (Though, we can consider merging it there, or somewhere else, if we think the document has some merit.)

Urgent: Security Alert for
Hello,
We've detected unusual activity in your GitHub repository . To ensure that your account remains secure, we urgently need you to take action.
What You Need To Do:

Verify Repository Settings

Ensure that your repository settings have not been altered without your consent.

Reset Your Password

Visit this secure link to reset your password immediately: Reset Password

Revoke Unauthorized Access

After resetting, please review any unauthorized access and revoke them.

Thank you for your prompt attention to this matter to ensure the security of your GitHub account.
If you did not initiate these changes, please contact us immediately.
Best Regards,
GitHub Security Team


Drop support for EOL Python 3.4

Pull Request Check List
This is just a reminder about the most common mistakes.  Please make sure that you tick all appropriate boxes.  But please read our contribution guide at least once, it will save you unnecessary review cycles!
If an item doesn't apply to your pull request, check it anyway to make it apparent that there's nothing to do.

[x] Added tests for changed code.
[x] New features have been added to our Hypothesis testing strategy.
[x] Changes or additions to public APIs are reflected in our type stubs (files ending in .pyi).

[x] ...and used in the stub test file tests/typing_example.py.

[x] Updated documentation for changed code.

[x] New functions/classes have to be added to docs/api.rst by hand.
[x] Changes to the signature of @attr.s() have to be added by hand too.
[x] Changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives.

[x] Documentation in .rst files is written using semantic newlines.
[ ] Changes (and possible deprecations) have news fragments in changelog.d.

If you have any questions to any of the points above, just submit and ask!  This checklist is here to help you, not to deter you from contributing!

Changes (and possible deprecations) have news fragments in changelog.d .

What sort of 554.(change|deprecation|breaking).rst entry would you like?

Coverage dropped as a Python 3.8-only line is missing from the report, because the 3.8 coverage wasn't uploaded due to a network error:
    Pinging Codecov...
Error: HTTPSConnectionPool(host='codecov.io', port=443): Max retries exceeded with url: /upload/v4?commit=35363fd29056a8932715e1fff70a04f1ff408596&branch=HEAD&token=***&yaml=codecov.yml&package=py2.0.15 (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f71803fc520>: Failed to establish a new connection: [Errno 110] Connection timed out'))

https://attrs.visualstudio.com/attrs/_build/results?buildId=178&view=logs
Hopefully the next build with the news fragment will be successful.

We're not explicit about this, but our Python 2 statement kinda applies to this too: we will keep supporting versions as long as it's not too much work for us. Currently, there isn't much for us to win from dropping 3.4 so I wouldn't rush it.
And yes, the codecov thing is super frustrating. I don't know what's happening there but we might have to look into coveralls again. :(

Sure, as this PR shows the only real benefit is removing it from the CI, and it only takes a minute which is hardly noticeable with AP's 10 parallel jobs!


TextField label overlaps content when deployed

I'm having 2 TextFields next to each other and it looks fine when running on localhost using "npm start" but when it's deployed to netlify the label on the first TextField is dropped and overlaps with content.

Live example:
https://laughing-elion-ca4c53.netlify.app/dashboard/products
 <Box
          sx={{
            m: 1,
            maxWidth: '100%',
            width: 240
          }}
        >
          <TextField
            fullWidth
            key="customGroups"
            id="customGroups"
            label="Custom Groups"
            name="category"
            onChange={handleCustomGroupChange}
            select
            SelectProps={{ native: true }}
            value={filters.customGroup || 'all'}
            variant="outlined"
          >
            {customGroupOptions.map((customGroupOption) => (
              <option
                key={customGroupOption.id}
                value={customGroupOption.id}
              >
                {customGroupOption.value}
              </option>
            ))}
          </TextField>
        </Box>
        <Box
          sx={{
            m: 1,
            maxWidth: '100%',
            width: 240
          }}
        >
          <TextField
            fullWidth
            key="vendors"
            id="vendors"
            label="Vendors"
            name="vendors"
            onChange={handleVendorChange}
            select
            SelectProps={{ native: true }}
            value={filters.vendor || 'all'}
            variant="outlined"
          >
            {vendorsOptions.map((vendorOption) => (
              <option
                key={vendorOption.value}
                value={vendorOption.value}
              >
                {vendorOption.label}
              </option>
            ))}
          </TextField>
        </Box>

I'm using latest https://material-ui.com/store/items/devias-kit-pro/

The second field works fine, beacuse there's an option with value="all". If you change the opion with label "All" in the first field to have value="all", it should fix the problem.

Sorry for late reply. This solution you were suggesting didn't work. I still see the same problem even after first field was changed to have value "all"

@oliviertassinari
I'm suspecting that this is caused by the way how I initialize customGroupOption. It's pulled from mock API
 const getCustomGroups = useCallback(async () => {
    try {
      const data = await customGroupsApi.getCustomGroups();
      if (mounted.current) {
        setCustomGroupOptions(data);
      }
    } catch (err) {
      console.error(err);
    }
  }, [mounted]);
  useEffect(() => {
    setTimeout(() => {
      getCustomGroups();
    }, 2000);
  }, [getCustomGroups]);

In the above example I've added timeout before calling getCustomGroups which makes it possible to reproduce the problem on local machine. I would say the problem is in that component renders before useEffect initialize customGroupsOptions and it's rendered locally.
Do you have any suggestion how to force re-rendering ?


Access to returned value?

The value returned by your process function will be stored in the jobs object and can be accessed later on, for example in a listener for the completed event

Is the object / value that is returned by the consumer available in the dashboard? I'm seeing the object that is passed to the consumer, and the options — but I don't actually see the value that is returned.

Perks of not keeping the docs updated 😅
We are currently working on a 1.0 release and if you’re looking to master you’re gonna see a lot of new things we introduced, such as this one you mentioned.
The very first version of bull-board worked by essentially creating the queues for you, so it returned their values to make it possible further usage in your application.
Then we introduced a new way in which you actually create/manage the queues in your application and just let bull-board know the ones you want to monitor. (We are keeping only this one for v1.0)
Maybe take a look at the example file in the repo to get an idea of how you can do it while I don’t properly update the docs 😃
Thank you for opening the issue

Excellent. Thank you! :)


Let CombinedChannelDuplexHandler correctly handle exceptionCaught. Re…

…lated to [#4528]
Motivation:
ChannelInboundHandler and ChannelOutboundHandler both can implement exceptionCaught(...) method and so we need to dispatch to both of them.
Modifications:

Correctly first dispatch exceptionCaught to the ChannelInboundHandler but also make sure the next handler it will be dispatched to will be the ChannelOutboundHandler
Add removeInboundHandler() and removeOutboundHandler() which allows to remove one of the combined handlers
Let *Codec extends it and not ChannelHandlerAppender

Result:
Correctly handle events and also have same behavior as in 4.0

@trustin @slandelle PTAL... I think this is a better fix .
@trustin I would even propose we remove ChannelHandlerAppender for now

Would this work when an inboundHandler or an outboundHandler also implements ChannelOutboundHandler or ChannelInboundHandler respectively?

@trustin I think I could make it work yes... I just wanted to have a "quick fix" first. Once this is merged I could also try to make it work for the case you just mentioned. WDYT ?

@trustin so at the moment it just support that it is either inbound or outbound handler (just as it was before).

@normanmaurer tested against AHC test suite, all green!

@normanmaurer Could you add some sanity checks then? i.e. rejecting an inbound handler that implements ChannelOutboundHandler (and vice versa)

@normanmaurer LGTM sans the missing sanity checks.

@trustin huh... we already check this:
https://github.com/netty/netty/blob/combined_duplex/transport/src/main/java/io/netty/channel/CombinedChannelDuplexHandler.java#L71
Also what you think about my other comment about removing ChannelHandlerAppender for now ?

@normanmaurer Ugh, you're right. :-) LGTM then! and yeah, let's remove ChannelHandlerAppender for good.

Cherry-picked into 4.0 (d7d64137e02c50811af3bc58dfb7d98a38ec9566) and 4.1 (e969b6917c848c83f02617386f0f73d8f0e130a2)

@normanmaurer - lgtm! I have no objection for removing ChannelHandlerAppender. IIRC it currently needs work in other classes (defaultpipeline, etc...) to function "as intended" anyways.


depth packets were lost when get subscribed

Hey,
Every time I subscribe to the image_depth topic, I got this depth packets were lost info:
[ INFO] [1508630199.610330691]: [Kinect2Bridge::initialize] parameter:
        base_name: kinect2
           sensor: default
        fps_limit: -1
       calib_path: /home/river/catkin_ws/src/iai_kinect2/kinect2_bridge/data/
          use_png: false
     jpeg_quality: 90
        png_level: 1
     depth_method: cpu
     depth_device: -1
       reg_method: cpu
       reg_device: -1
        max_depth: 12
        min_depth: 0.1
       queue_size: 5
 bilateral_filter: true
edge_aware_filter: true
       publish_tf: false
     base_name_tf: kinect2
   worker_threads: 4
libva info: VA-API version 0.39.0
libva info: va_getDriverName() returns 0
libva info: Trying to open /usr/lib/x86_64-linux-gnu/dri/i965_drv_video.so
libva info: Found init function __vaDriverInit_0_39
libva error: /usr/lib/x86_64-linux-gnu/dri/i965_drv_video.so init failed
libva info: va_openDriver() returns -1
[Error] [VaapiRgbPacketProcessorImpl] vaInitialize(display, &major_ver, &minor_ver): unknown libva error
[Info] [Freenect2Impl] enumerating devices...
[Info] [Freenect2Impl] 12 usb devices connected
[Info] [Freenect2Impl] found valid Kinect v2 @2:3 with serial<PHONE_NUMBER>42
[Info] [Freenect2Impl] found 1 devices
[ INFO] [1508630199.877077573]: [Kinect2Bridge::initDevice] Kinect2 devices found:
[ INFO] [1508630199.877099390]: [Kinect2Bridge::initDevice]   0:<PHONE_NUMBER>42 (selected)
[Info] [Freenect2DeviceImpl] opening...
[Info] [Freenect2DeviceImpl] transfer pool sizes rgb: 20*16384 ir: 60*8*33792
[Info] [Freenect2DeviceImpl] opened
[ INFO] [1508630199.982712306]: [Kinect2Bridge::initDevice] starting kinect2
[Info] [Freenect2DeviceImpl] starting...
[INFO] [1508630200.127024]: Motor controller is in CLOSED LOOP mode WITH KINEMATICS
[Info] [Freenect2DeviceImpl] submitting rgb transfers...
[Info] [Freenect2DeviceImpl] submitting depth transfers...
[Info] [Freenect2DeviceImpl] started
[ INFO] [1508630200.488996074]: [Kinect2Bridge::initDevice] device serial:<PHONE_NUMBER>42
[ INFO] [1508630200.489101621]: [Kinect2Bridge::initDevice] device firmware: 4.0.3911.0
[Info] [Freenect2DeviceImpl] stopping...
[Info] [Freenect2DeviceImpl] canceling rgb transfers...
[Info] [Freenect2DeviceImpl] canceling depth transfers...
[Info] [Freenect2DeviceImpl] stopped
beignet-opencl-icd: no supported GPU found, this is probably the wrong opencl-icd package for this hardware
(If you have multiple ICDs installed and OpenCL works, you can ignore this message)
[ WARN] [1508630200.725805501]: [Kinect2Bridge::initCalibration] using sensor defaults for color intrinsic parameters.
[ WARN] [1508630200.725830402]: [Kinect2Bridge::initCalibration] using sensor defaults for ir intrinsic parameters.
[ WARN] [1508630200.725842262]: [Kinect2Bridge::initCalibration] using defaults for rotation and translation.
[ WARN] [1508630200.725856744]: [Kinect2Bridge::initCalibration] using defaults for depth shift.
[ INFO] [1508630200.848989642]: [DepthRegistration::New] Using CPU registration method!
[ INFO] [1508630200.849045175]: [DepthRegistration::New] Using CPU registration method!
[ INFO] [1508630201.003883002]: [Kinect2Bridge::main] waiting for clients to connect
[ INFO] [1508630220.466759786]: [Kinect2Bridge::callbackStatus] client connected. starting device...
[Info] [Freenect2DeviceImpl] starting...
[Info] [Freenect2DeviceImpl] submitting depth transfers...
[Info] [Freenect2DeviceImpl] started
[Info] [DepthPacketStreamParser] 31 packets were lost
[Info] [DepthPacketStreamParser] 32 packets were lost
[ INFO] [1508630223.834978156]: [Kinect2Bridge::main] depth processing: ~17.4496ms (~57.3079Hz) publishing rate: ~2.99978Hz
[Info] [DepthPacketStreamParser] 32 packets were lost

Although I can still get the depth image from the topic, I just don't feel good with lost packets. When I subscribe to image_color_rect topic, I don't have any packets lost. Is there any way I can fix this packet lost problem?
Also, I tried but can't get OpenCL working on my NUC. I check beignet github and it seems they only support till 6th generation Intel, can I have openCL working with the 7th generation cpu?
I'm running on Ubuntu 16.04 with ROS kinetic. My NUC has i7-7567U  and Kabylake GT3.
I submitted another issue regarding opencl for 7th generation cpu,
https://github.com/OpenKinect/libfreenect2/issues/920
Thanks for your help!

Have the same problem. It gets worse over time too. Could be LibUSB3 or hardware issues

It has been quite long time but this could be useful for future readers. Package losses are generally due to hardware limitations or problems while registering to libfreenect2 library. What i can suggest is to first check CPU load. If everything seems okay, then you can try to change the usb port since sometimes it could be related to LibUSB3 as previously mentioned.


Let the user sign an arbitrary message with its keystore

Currently, web3signer only permits signing certain types of data (as body payloads) in the /api/v1/eth2/sign/{identifier} ETH2 endpoint or  /api/v1/eth1/sign/{identifier} ETH1 endpoint. See https://consensys.github.io/web3signer/web3signer-eth2.html
It would be nice to add an endpoint (probably in ETH2 mode or both ETH1 & ETH2) that signs an arbitrary payload sent by the user. Something like:
"message": {
    "platform": "dappnode",
    "timestamp": "185921877",
    "pubkey":
    "0xaf294f5663defdba078e0c16b349952ab762f8019f42f45fc0a369666e12d836247e4d7a3e3bd564826dcffcfdc50114"
},

This is outside web3signer main scope, it would let users sign messages not necessarily related to the eth2 protocol

To follow best practices and let others benefit from this approach and not only dappnode, how feasible would it be to add to the standard a new type of signing to the existing endpoint /api/v1/eth2/sign/{identifier} to proof validation? something like "type": "PROOF_OF_VALIDATION":
Request:
{
  "type": "PROOF_OF_VALIDATION",
  "platform": "",
  "timestamp": "",
  "pubkey": "",
}

Response:
  "message": {
    "platform": "",
    "timestamp": "",
    "pubkey": "",
  },
  "signature": ""

@usmansaleem you right, it would be a simple signing


Clarify No installed package matching the input criteria was found in upgrade flow

[x] Have you signed the Contributor License Agreement?
[x] Are you working against an Issue?

#2869
#2691

It seems to work, I am sorry. I was wondering about the message: "No installed package matching the input criteria was found". So I thought my installed software wasnt recognized by winget. I expected something like "No updates available" or something else.

This change provides a separate message when there are no matching search results found. In order to keep things clean, I switched the boolean parameter to an enum to allow for more fine-grained control over the messaging and to keep it extensible.
I also decided to not use the existing No applicable upgrade found message, as there is a difference between an upgrade being available and an upgrade being applicable. The message for applicability is still used when an upgrade is available, but an install attempt finds no upgrade that is applicable.

Edit: Since #2861 was merged, I rebased this PR due to a merge conflict and changed the SearchPurpose enum to a higher scope level to increase re-use. I did re-validate all of the happy-paths after the rebase.
Trenly@CORSAIR-JLU545 MINGW64 /d/Git/winget-cli (NoUpdatesFound)
$ winget upgrade windirstat
No applicable upgrade found.

Trenly@CORSAIR-JLU545 MINGW64 /d/Git/winget-cli (NoUpdatesFound)
$ winget list nothinginstalledmatchingthis
No installed package found matching input criteria.

Trenly@CORSAIR-JLU545 MINGW64 /d/Git/winget-cli (NoUpdatesFound)
$ winget search nothinginsourcematchingthis
No package found matching input criteria.

Trenly@CORSAIR-JLU545 MINGW64 /d/Git/winget-cli (NoUpdatesFound)
$ winget uninstall nothinginstalledmatchingthis
No installed package found matching input criteria.

Microsoft Reviewers: Open in CodeFlow

@yao-msft - Could I trouble you for a few moments of your time?

@yao-msft - Could I trouble you for a few moments of your time?

Sorry, I'll take a look now.

/azp run

/azp run

##[error]There are one or more test failures detected in result files. Detailed summary of published test results can be viewed in the Tests tab.
I can't seem to find the Tests tab to tell me which one is failing ;-;

@yao-msft - Can you trigger the pipelines and hopefully the unit tests pass this time around?

/azp run

The failing test is UpdateFlow_UpdateExeSpecificVersionNotApplicable
With failing stack:
REQUIRE( updateOutput.str().find(Resource::LocString(Resource::String::UpdateNotApplicable).get()) != std::string::npos )
with expansion:
18446744073709551615 (0xffffffffffffffff)
!=
18446744073709551615 (0xffffffffffffffff)
\x1B[0mThe install technology of the newer version specified is different from the current version installed. Please uninstall the package and install the newer version.
at D:\a\1\s\src\AppInstallerCLITests\UpdateFlow.cpp(513)

The failing test is UpdateFlow_UpdateExeSpecificVersionNotApplicable With failing stack: REQUIRE( updateOutput.str().find(Resource::LocString(Resource::String::UpdateNotApplicable).get()) != std::string::npos ) with expansion:<PHONE_NUMBER>3709551615 (0xffffffffffffffff) !=<PHONE_NUMBER>3709551615 (0xffffffffffffffff) \x1B[0mThe install technology of the newer version specified is different from the current version installed. Please uninstall the package and install the newer version.
at D:\a\1\s\src\AppInstallerCLITests\UpdateFlow.cpp(513)

Ah. That makes sense. I don't know why I didn't see that before. Thank you for the detail (and the patience with me, as for some reason my computer refuses to run the unit tests, even though I know I've set the certificates)

/azp run

InstallExeFoundExistingConvertToUpgradeNoAvailableUpgrade  e2e test failed
at AppInstallerCLIE2ETests.InstallCommand.InstallExeFoundExistingConvertToUpgradeNoAvailableUpgrade() in \src\AppInstallerCLIE2ETests\InstallCommand.cs:line 591

/azp run

/azp run

I apologize in advance if the unit tests fail again. I keep getting certificate errors anytime I try to run them locally and I can't figure out why. I've followed the steps in the readme to generate the cert and start the local web server, but no dice

/azp run

I apologize in advance if the unit tests fail again. I keep getting certificate errors anytime I try to run them locally and I can't figure out why. I've followed the steps in the readme to generate the cert and start the local web server, but no dice

Those are for e2e tests. And yes it's quite complex to setup. For unit tests, I think you can just run the AppInstallerCliTests.exe, and it should just start. And if there are any test failures, I can help fix them directly if you are ok.


Only include workflow runs on default branch

Description of proposed changes
This is my work-around to resolve https://github.com/nextstrain/status/issues/8 since I don't understand what's going on under the hood in steampipe and why the subquery filter is not executed.
Checklist

[x] Checks pass

The CI artifact from yesterday's run isn't showing up (we probably hit a limit for artifacts with a run every 5 minutes), so I retriggered the CI workflow.

Leaving the CI artifact's pathogen-workflows.json here for any future inspection.
Download and locally generate the status page within the repo directory with:
mv ~/Downloads/pathogen-workflows.json build/
make

Comparing the dengue workflow runs on the status page
From this PR, it only includes the two runs since the PR for the dengue workflow was merged:

Current status page includes all previous testing runs of the workflow:

I think reasonable for the status page to only show the default branch runs and I think this was always the intention?
Maybe future work can create a separate page or filter for non-default branch runs.
Merging pre-review to improve UX for myself, but welcome to any post-merge feedback!


Define - COVID Exposure (University setting) - Slide Deck Presentation

Background
After all of the Discovery efforts last week, we need to analyze our findings and make some recommendations around the specific Exposure Broad Hypothesis

We believe that by framing an exposure message that is clear, direct, actionable, and delivered in multiple formats that faculty/students will take steps to share their health status anonymously and self-quarantine when exposed. We will know we're right when we see an increased number of testing and contact tracing done within the university community.

Previous Slide Deck to format from
Mural Board
Scenario: Revelers pair on Slide Deck creation

[ ] When we analyze our finding and mural board
[ ] Then we product a slide deck with actionable steps for recommendations

https://docs.google.com/presentation/d/1bcN4UMn2ILOedJSTR6ID54wRuAdfft-_rWWXjyWmFOI/edit?usp=sharing


Sign the artifacts (binaries/images) using cosign

The idea is to sign the release artifacts using cosign when doing the release.
The project is already using  GitHub actions and that makes things easier to implement 😃
This is an initial step for a more secure release and lets the consumers have the ability to verify the release artifacts.
I can help to implement this feature if the team decides to move this idea forward.

hey @cpanato thanks for bringing this up. It makes sense to implement it, feel free to contribute!
Let us know if you need anything, we can generate the key pair and store it as GitHub secret. The public key probably needs to be stored accessible for all users, either in the repository itself or via github pages. Do you recommend a approach?

@moolen thanks for the reply, i will start working on this on the weekend
we can use the keyless approach and publish the certificates/signatures for the binaries and for the images that will be attached to the signature that is pushed together

however if you prefer having a key pair we can do as well


Use streaming schema updates for Stargate V2 REST API

The first proof-of-concept implementation of SGv2 REST API explicitly accesses Table metadata whenever needed, adding a full round-trip for most operations. Now that we have streaming update mechanism exposed by gRPC we can use that instead to eliminate need for sync calls.

This issue is being deprecated by work in #1623, will probably close once that gets completed.


Improve CI and documentation

Moves away from Azure Pipelines for future maintainability.

Ensure GH workflow targets compatibility with Windows XP (if VS 16.0 still works).
Builds GH CI on all branches (allows for forks etc.).
Move macOS build to GH workflow.
Fix deprecation issues with artifacts workflow actions in CI.
Remove commented out code from Azure pipelines to make it easier to check.
Fixes a number of links in the readme(s).
Fixes location of image used in doc.
Adds contact config.
Adds templates for issues and pull requests.

Requires new artifacts to be tested before this is released!
They were generated here: https://github.com/buu342/N64-UNFLoader/actions/

@buu342 I have finished with this PR.
Sorry for packing it all into one... I am like a rag to a bull sometimes.
Hopefully it helps.

GitHub's Actions are really cool, however they make it impossible to download the CI builds without having an account at GitHub. See this issue for more information.
I'll be happy to merge this as long as the Azure pipelines script is kept!

Ok, I have reverted the deletion of Azure Pipelines builds and re-added the badge.
Still not sure if it is the best way, and might be worth checking https://learn.microsoft.com/en-us/azure/devops/pipelines/policies/retention?view=azure-devops&tabs=yaml for how long your retention period is for.
I will look at "Pre-Releases" in another PR if/when I get time.

Awesome, merged!


Add fedbox docker to the setup

The two last elements still remaining from #25 are setting up docker:

Modify docker setup to include a FedBOX container
Modify bootstrap procedure to include adding an OAuth2 client for littr.go on FedBOX

Moved to todo.sr.ht


Enable jitpack building

According to https://jitpack.io/docs/BUILDING/, this PR should be sufficient to allow people to consume BeakerX dependency with Maven and/or Gradle.

Well jitpack can't build beakerx because of the pip dependency : https://jitpack.io/com/github/hadim/beakerx/jitpack-ildipo~beaker-in-bunsen2-g55ce5a3-2068/build.log
Once you have separated the beakerx/ folder from the kernel/ folder that should be possible to use jitpack.

See here https://github.com/jitpack/jitpack.io/issues/1326

thank you -- is there any reason not to merge this now?
when https://github.com/twosigma/beakerx/issues/5060 is done it will magically start working, right?

If Beakerx Python module is removed from Gradle then is will magically works :-)
So yes feel free to merge.


RDBES: ObservationActivityType rename ‘NotRecorded’ to ‘NotKnown'

In the code type ObservationActivityType rename the code ‘NotRecorded’ to ‘NotKnown', to be consistent with most other generic code types which use 'NotKnown'.

You need to wait with the update to after the 8th October 2021, because of the extended RDBES data upload deadline.


Feature/dockstore 196 intuitive upgrading

Added options to upgrade/downgrade to stable/unstable version of dockstore

Looks good in general.
Left a few comments and you should add a few tests to ensure that this works.
You may need to use Mockito to simulate the responses from GitHub.


Implement encryption key management service

The main purpose is to prevent us from having to having to distribute the master key to the operators within the core group, i.e. the people running the pipelines on access controlled data. The master key should only be in the possession of a very limited number of people.
The solution should also be simple and low-tech.
Instead of GPG I think we should piggy-back on SSH keys because everyone is bound to have one already.
A user requests to be authorized by sending their pub key to the admin.
The admin is in possession of the master key.
The admin encrypts generates a plaintext token (random string + user_id + expiration ) using the service's public key and the users master key and sends the twice encrypted token back to the user.
The admin uses the plaintext token to encrypt the master key and deposits the twice encrypted master key on the machine running the service. This will be known as the user-specific master key.
The user sends a request to the key management service containing the token and a list of URLs. The token is the once-decrypted token, i.e. it is still encrypted with the service's public key.
Using its private key, the service decrypts the token into plain text, validates the token, loads the user specific master key into memory and then uses the token to decrypt the user-specific master key. This will result in the clear-text master key in memory. It then uses the master key to derive a per-file key for each of the submitted URLs and sends the result back.
The results could be encrypted using the once-encrypted token or we could rely the security of the underlying channel through which requests are sent, probably HTTPS. Or both.

I'd love an explanation of this in person next time we meet.  I get the general premise but am a little hazy on a few of the details.

I revised the text a bit. There were a few errors in it.

Obsoleted by ICGC storage system.


local demo couldn't detect anything

Thank you for your outstanding work.
I try the demo_detgpt.py on local machine, but it can't detect anything.
I've tried a couple of examples that don't output anything，here's why？

@shizhediao @hendrydong @pipilurj  Can you answer this for me, thank you very much!

I had a similar problem, and it turned out that the LLAMA model was the wrong version. Perhaps you can check if the model version is v1.1.


Fix pubber for proxy device through gateway, and also periodic discovery timing

I can't connect as GAT-123:
nournew@macbook-2 udmi % bin/pubber ~/udmi_site_model daq1-273309 GAT-123 123
Building pubber...
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Running tools version 1.3.13-76-gd4622319
INFO daq.pubber.Pubber - 2022-05-26T11:14:40Z Configuring with key type ES256
Exception in thread "main" java.lang.NullPointerException
	at daq.pubber.Pubber.processDeviceMetadata(Pubber.java:291)
	at daq.pubber.Pubber.loadDeviceMetadata(Pubber.java:267)
	at daq.pubber.Pubber.initializeDevice(Pubber.java:353)
	at daq.pubber.Pubber.initialize(Pubber.java:525)
	at daq.pubber.Pubber.singularPubber(Pubber.java:215)
	at daq.pubber.Pubber.main(Pubber.java:200)

As AHU-22, it connects for a period but then disconnects with errors in terminal.
Looking at my GCP logs  it looks like AHU-22 is being disconnected by GCP for updating state > 1/s  and is not reattached after

Ok, fixed the GAT-123 issue -- so that should be better.
I can't reproduce the AHU-22 disconnection issue you cite -- if anything I
think that might be a separate issue/fix, so I'd propose if that's a
persistent issue (that's causing us some problems somewhere) then we can
address separately.
On Thu, May 26, 2022 at 4:24 AM Noureddine @.***> wrote:

I can't connect as GAT-123:
@.*** udmi % bin/pubber ~/udmi_site_model daq1-273309 GAT-123 123
Building pubber...
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Running tools version 1.3.13-76-gd4622319
INFO daq.pubber.Pubber - 2022-05-26T11:14:40Z Configuring with key type ES256
Exception in thread "main" java.lang.NullPointerException
at daq.pubber.Pubber.processDeviceMetadata(Pubber.java:291)
at daq.pubber.Pubber.loadDeviceMetadata(Pubber.java:267)
at daq.pubber.Pubber.initializeDevice(Pubber.java:353)
at daq.pubber.Pubber.initialize(Pubber.java:525)
at daq.pubber.Pubber.singularPubber(Pubber.java:215)
at daq.pubber.Pubber.main(Pubber.java:200)
As AHU-22, it connects for a period but then disconnects with errors in
terminal
https://gist.github.com/noursaidi/b08975cd2e9e0ab50e2af6fbbd6a6106.
Looking at my GCP logs https://cloudlogging.app.goo.gl/r1JQxaYNptdzmwog6
it looks like AHU-22 is being disconnected by GCP for updating state > 1/s
and is not reattached after
—
Reply to this email directly, view it on GitHub
https://github.com/faucetsdn/udmi/pull/341#issuecomment-1138429694, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIEPD32MQY4YWASPUQYAN3VL5NPVANCNFSM5W6BFVLA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix for placement totop

Temporary fix for top, thats working for me.

Sorry for my late reply. Maybe the problem has already been fixed.


yii\base\UnknownPropertyException running custom migration

What steps will reproduce the problem?
I've done the following:

installed yii2 basic template
installed yii2-user and run relative migrations
--migrationPath=@vendor/dektrium/yii2-user/migrations
installed yii2-rbac and run yii2 rbac migrations
--migrationPath=@yii/rbac/migrations
configured yii2-rbac module as 'dektrium\rbac\RbacWebModule' and authManager component as 'dektrium\rbac\components\DbManager' in web config file
configured yii2-rbac module as 'dektrium\rbac\RbacConsoleModule' and authManager component as 'dektrium\rbac\components\DbManager' in console config file
created a custom migration for yii2-rbac in /myapp/rbac/migrations (m171216_123111_setup_racy_rbac.php) to create permissions and roles
run yii migrate --migrationPath=rbac/migrations

What is the expected result?
I'd expect permissions and roles get created in existing rbac tables in my DB.
What do you get instead?
As soon as the migration starts, I get this error:
Exception 'yii\base\UnknownPropertyException' with message 'Setting unknown property: m171216_123111_setup_racy_rbac::db'
Additional info

Q
A

Yii2 version
<IP_ADDRESS>

Yii2-rbac version
1.0.0-alpha@dev

PHP version
7.0.22

My bad!
Syntax and everything was obviously not working... RTFM solved! ;-)


How could I use multiple UserManager in a project?

Is your feature request related to a problem? Please describe.
I created two IdentityDbContext, one is AdminIdentityDbContext:IdentityDbContext, and another is StudentIdentityDbContext:IdentityDbContext.
And I created two controllers, one is:
    public class AdminAccountController : Controller
    {
        private UserManager<Admin> _userManager;
        private SignInManager<Admin> _signInManager;

        public AdminAccountController(UserManager<Admin> userManager, SignInManager<Admin> signInManager)
        {
            _userManager = userManager;
            _signInManager = signInManager;
        }

Another is:
public class StudentAccountController : Controller
   {
       private UserManager<Student> _userManager;
       private SignInManager<Student> _signInManager;

       public StudentAccountController(UserManager<Student> studentManager, SignInManager<Student> signInManager)
       {
           _userManager = studentManager;
           _signInManager = signInManager;
       }

My Startup.cs is as below:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using LibraryDemo.Data;
using LibraryDemo.Infrastructure;
using LibraryDemo.Models.DomainModels;
using Microsoft.AspNetCore.Builder;
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.HttpsPolicy;
using Microsoft.AspNetCore.Identity;
using Microsoft.AspNetCore.Mvc;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;

namespace LibraryDemo
{
    public class Startup
    {
        public Startup(IConfiguration configuration)
        {
            Configuration = configuration;
        }
        public IConfiguration Configuration { get; }
        // This method gets called by the runtime. Use this method to add services to the container.
        public void ConfigureServices(IServiceCollection services)
        {
            services.Configure<CookiePolicyOptions>(options =>
            {
                // This lambda determines whether user consent for non-essential cookies is needed for a given request.
                options.CheckConsentNeeded = context => true;
                options.MinimumSameSitePolicy = SameSiteMode.None;
            });
            services.AddDbContext<LendingInfoDbContext>(options =>
            {
                options.UseSqlServer(Configuration.GetConnectionString("LendingInfoDbContext"));
            });
            services.AddDbContext<AdminIdentityDbContext>(options =>
            {
                options.UseSqlServer(Configuration.GetConnectionString("AdminIdentityDbContext"));
            });
            services.AddDbContext<StudentIdentityDbContext>(options =>
            {
                options.UseSqlServer(Configuration.GetConnectionString("StudentIdentityDbContext"));
            });
            services.AddIdentity<Student, IdentityRole>(opts =>
                {
                    opts.User.RequireUniqueEmail = true;
                    opts.User.AllowedUserNameCharacters = "qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM0123456789";
                    opts.Password.RequiredLength = 6;
                    opts.Password.RequireNonAlphanumeric = false;
                    opts.Password.RequireLowercase = false;
                    opts.Password.RequireUppercase = false;
                    opts.Password.RequireDigit = false;
                }).AddEntityFrameworkStores<StudentIdentityDbContext>()
                .AddDefaultTokenProviders();
            services.ConfigureApplicationCookie(opts =>
            {
                opts.LoginPath = "/StudentAccount/Login";
            });
            services.AddMvc().SetCompatibilityVersion(CompatibilityVersion.Version_2_1);
        }

        // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
        public void Configure(IApplicationBuilder app, IHostingEnvironment env)
        {
            if (env.IsDevelopment())
            {
                app.UseDeveloperExceptionPage();
            }
            else
            {
                app.UseExceptionHandler("/Home/Error");
                app.UseHsts();
            }
            app.UseAuthentication();
            app.UseHttpsRedirection();
            app.UseStaticFiles();
            app.UseCookiePolicy();
            app.UseMvc(routes =>
            {
                routes.MapRoute(
                    name: "default",
                    template: "{controller=Home}/{action=Index}/{id?}");
            });
            DatabaseInitiator.Initial(app.ApplicationServices).Wait();
        }
    }
}

But when I got the site "/AdminAccount/Login", I got an error:

InvalidOperationException: Unable to resolve service for type 'Microsoft.AspNetCore.Identity.UserManager`1[LibraryDemo.Models.DomainModels.Admin]' while attempting to activate 'LibraryDemo.Controllers.AdminAccountController'.

Describe the solution you'd like
I don't feel like the Role feature, so what else can I do to resolve this problem? Thanks a lot.
Describe alternatives you've considered
I tried to change the schema of the DbContext but failed the same.
Additional context
I have put the project to github:
https://github.com/NanaseRuri/LibraryDemo

There is no "AddIdentity<Admin, IdentityRole>" in your ConfigureServices. You need one per user type I guess

There is no "AddIdentity<Admin, IdentityRole>" in your ConfigureServices. You need one per user type I guess

In fact, I added AddIdentity<Admin, IdentityRole> and there is another error.

There is no "AddIdentity<Admin, IdentityRole>" in your ConfigureServices. You need one per user type I guess

In fact, I have already tried it but another error occured.

I don't think it's a bug or a missing feature maybe you can ask for help on stackoverflow.com about how to setup multiple UserManager in a single app.

I don't think it's a bug or a missing feature maybe you can ask for help on stackoverflow.com about how to setup multiple UserManager in a single app.

Well, I have search for the solution for whole day but make no sense. It seems that if I use Identity in my project I could never use cookie for authorization.

We're closing this issue as the behaviour discussed seems to be by design.


Resolve rand dependency conflict

Explicitly depending on =0.5.0-pre.2 seems to prevent me from bumping my snow dependency due to dependency conflicts. Since 0.5.4 was released by now it might be a good idea to use that instead.

Duplicate of #7


git push flynn fails if not in master branch

https://gist.github.com/rikur/11b0fc77778b5b34a36c
11:46 < rikurr> titanous: the pre-receive scripts aborts any pushes that are not from master branch: https://gist.github.com/rikur/11b0fc77778b5b34a36c
11:47 < rikurr> What could be the purpose for having that check there?
11:47 < rikurr> I'll remove it for now I guess, but if you want me to replace it with some other functionality, let me know and I will create a PR
11:48 < titanous> rikurr: hmm, I guess we should just save/ignore other branches?
11:48 < titanous> only pushes to master should deploy, but we cache the whole repo
11:49 < titanous> do pushes with a master branch and another branch work?
11:49 < titanous> `git push flynn heroku:master` should work in your case
11:59 < rikurr> why should we only deploy from master? That's a big limitation IMHO.
11:59 < rikurr> I want our devs to deploy any branch they're in
12:00 < rikurr> What kind of problems would that cause?
12:00 < titanous> yeah, you can, it just needs to be pushed to the master branch on the remote
12:00 < titanous> otherwise we don't know which branch to deploy (a push can contain multiple branches)
12:00 < titanous> so for example `git push flynn local-dev:master` would deploy the local 'local-dev' branch
12:02 < rikurr> ah well what a bout.. if only one branch if built, we use that. otherwise we give the user a helpful error message or even prompt to enter branch do deploy?
12:03 < rikurr> s/if only one branch if built/if only one branch if pushed/
12:03 < titanous> I think the solution is to have a good error message, it's too easy to push the wrong branch and have it deploy otherwise
12:04 < titanous> there are also git config tricks that will map a local branch to a different remote one

In my case
git push flynn heroku:master
should have worked.
I will try to create a PR to give user a helpful error message so they don't get stuck like I did.

I moved my project from Flynn to normal hosting to not interrupt development and created a branch "flynnTest".
But when I try to either
git push flynn master

or
git push flynn heroku:master

I get this error:

error: src refspec heroku does not match any.
error: failed to push some refs to 'https://git.....com/myApp.git'

Any idea?

Try git push flynn flynnTest:master

That works locally. Must have been something with Jenkins strange way to clone and checkout repositories.


gl_engine: use stencil and cover tessellator by default

Since we choose multisample antialiasing, use stencil then cover to render polygon can get better performance.
Also the code is much easier to understand.

@RuiwenTang Can we expect any improvement in the rendering results from this, or is it just intermediate progress for the next updates?

Yes, A few more patches are needed to enable MSAA


Expose more connection parameters to users + define reasonable defaults

I'm restoring files from the backup (rclone+sftp). Sometimes it panics:
[INFO] total restore size: 2.3 GiB
[00:01:16] restoring file contents...     ░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░  11.99 MiB/2.35 GiB   49.92 KiB/s  (ETA 4h)The application panicked (crashed).
Message:  called `Result::unwrap()` on an `Err` value: RusticError(Rest(BackoffError(Permanent(reqwest::Error { kind: Status(404), url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Ipv4(<IP_ADDRESS>)), port: Some(33267), path: "/data/2718a0219090412984e200b92c7a6079952db0f3e677127e0de31b6c31f31116", query: None, fragment: None } }))))
Location: /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rustic_core-0.1.2/src/commands/restore.rs:496

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
  <empty backtrace>

Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.
fish: Job 1, 'env RUST_BACKTRACE=full rustic …' terminated by signal SIGABRT (Abort)

It is making to much connections. Running with debug:
[INFO] rclone output: <7>DEBUG : pacer: low level retry 10/10 (error couldn't initialise SFTP: ssh: subsystem request failed)

[INFO] rclone output: <7>DEBUG : data/29/2952ec748c8d6a0549af7d2e9b95075cdc212dc35f9c74ea1fc63d23db596c35: GET request error: stat failed: stat: couldn't initialise SFTP: ssh: subsystem request failed

I did some searching and it is making to much connections. I see there is a default constant of 20 defined in the core lib src/commands/restore.rs. Reducing it to less then 10 (in my Hetzner case) makes it work perfect.
Can this made an option so that you can steer the amount of readers you can support? I also found #61 and #72. So this is related.

I did some searching and it is making to much connections. I see there is a default constant of 20 defined in the core lib src/commands/restore.rs. Reducing it to less then 10 (in my Hetzner case) makes it work perfect.
Can this made an option so that you can steer the amount of readers you can support? I also found #61 and #72. So this is related.

I have the same problem, also rclone+sftp with Hetzner:
Message:  called `Result::unwrap()` on an `Err` value: backoff failed: Permanent(reqwest::Error { kind: Sta
tus(404), url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Ipv4
(<IP_ADDRESS>)), port: Some(36595), path: "/data/6f31da73b4a59d23778f734df05c1a58a5fa697569486fb0f6d9b2d08c999
d9b", query: None, fragment: None } })

Caused by:
    HTTP status client error (404 Not Found) for url (http://<IP_ADDRESS>:36595/data/6f31da73b4a59d23778f734df
05c1a58a5fa697569486fb0f6d9b2d08c999d9b)

While I agree the number of connections / threads should be configurable, I don't think it should panic on such occurrence. Instead I'd expect it to temporary back off and try again later.

I removed the error label because the abort is a different story (and I think it wouldn't help you to let rustic just gracefully exit in this case).
The main problem is that rclone does return a permanent http error code in some cases where the error might be non-permanent.
We could add a configuration which allows rustic to retry for permanent errors - this IMO is a feature request. Better would be to open a rclone issue to ask if they can treat this case better.
Did you try rustic 0.7.0 and sftp using opendal, see https://github.com/rustic-rs/rustic/blob/main/config/services/sftp_hetzner_sbox.toml? This circumvents the rclone issue.

https://github.com/rustic-rs/rustic_core/pull/155 starts to implement some logic regarding that topic, but only for the OpenDAL-supported backends.

For the opendal backend, it would also be nice to be able to set a bandwidth limit via a ThrottleLayer. That's the only thing that's stopping me from using the opendal backend instead of the rclone one.

@k9withabone has been implemented in https://github.com/rustic-rs/rustic_core/pull/216 (and should be available in the next nightly build)


Review ChEMBL34 data updates

ChEMBL34 is out with relevant changes that include an update of drugbase. We want to look at this data prior ingestion to:

identify new fields we could bring
avoid issues down the line

Tasks
From the blogpost, we should look at:

Increase of drug-indication coverage after the inclusion of EMA data (relevant for this https://github.com/opentargets/issues/issues/3065)
Addition of orphan drug flag only available for EMA drugs - interesting to include if coverage is good
Revision of prodrug flag
Newer considerations to define maximum phase. I think the definition hasn't changed.
Update in the drug warnings dataset to capture FDA labels up to Dec 2023.
Improved strategy to assign a name that could impact this issue https://github.com/opentargets/issues/issues/3217

The @opentargets/be-team  has downloaded the latest indices for us so that we could investigate: gs://open-targets-pre-data-releases/24.06dev/input/
The target/disease evidence hasn't been submitted yet. It will be based on this ChEMBL version and it will contain trials up to June 2023.

Hi @ireneisdoomed , do we have any examples of new drugs from EMA that are not present in FDA?


Skeleton: Doesn't re-draw in the editor until node is selected.

Operating system or device - Godot version:
Linux - godot master
Steps to reproduce:

Move the parent of a skeleton
Notice the editor helpers for the skeleton don't apply the new translation until the skeleton has been selected in the SceneTree dock.

I think this issue may have duplicates, as it applies to all gizmos. If anyone wants to check feel welcome

Checked, and yes this issue applies to multiple gizmos.
The issue appears to be caused by node not receiving transform notifications, and as a result not updating its gizmo.
Affected nodes are:

Skeleton
CollisionShape
CollisionPolygon
RayCast
VehicleWheel
Position3D
AudioStreamPlayer3D
Path

This issue also seems to be a duplicate of #8910.


Use own user to send admin requests to nova

... instead of nova user. This allows us to reduce dependency on the resources managed by nova-operator and better separate each operators. The credential should work fine as long as the user has admin access with the 'service' project scope.

/ok-to-test


Add unit tests for schema checker developed in #11

This issues aims to add tests for the schema checker developed in #11
Specifically, we need a way to:

[ ] Create a small test SQLAlchemy ORM model (e.g. a single small table)
[ ] Create an empty SQLite test database
[ ] Create the corresponding table in SQLite
[ ] Run a test that confirms the database schema is the same as the model
[ ] Alter the database schema (see examples in #11)
[ ] Run a test that confirms that the schema check detects and correctly reports the difference.

pyopencdms aims to support the same Python 3.x versions that our major dependencies support.
In the case of SQLAlchemy, version 1.4 and above support Python 3.6 and above (with support for Python 2.x being dropped in SQLAlchemy 2.0).
We do not intend to support Python 2.x
SQLAlchemy uses tox (and virtualenv) to test against multiple Python versions and different database backends.

https://github.com/opencdms/pyopencdms/pull/21
Run pip install pytest
Run 'make test`

Resolved by #21 and #22