unecessary redownload of istioctl slows down dev+test cycle

each time I run my test and the test code change or the test fails, for instance
bazel run //tests/e2e/tests/simple:go_default_test -- -alsologtostderr -test.v -v 2 -test.run TestSvc2Svc --skip_setup --skip_cleanup --auth_enable --namespace=e2e-test4

the same istioctl is re-downloaded:
I0918 22:33:44.783560   68809 commonUtils.go:110] Start downloading from https://storage.googleapis.com/istio-artifacts/pilot/438acd5c5a0069b707e392f86f44e40946a59fad/artifacts/istioctl/istioctl-osx to /var/folders/c3/4bc8zz5s5hj13nnr718w91xr00f56m/T/istio.e2e.999444277/istioctl ...

cc @andraxylia to explain "by design, it’s correct behaviour" (oh can it be correct behavior to redownload the exact same istioctl by sha more than once)

It is correct to download the version specified in istio.VERSION, and test against it.

It is correct to download the version specified in istio.VERSION, and test against it.

that's not what I asked about on slack nor wrote here

source if the download is https://github.com/istio/istio/blob/6f17cf513217c13422cc55724a4ef2dfbc3398b6/tests/e2e/framework/istioctl.go#L113
and a workaround is to download manually and then use -istioctl location but that doesn't work inside bazel run
once I have a short version I'll update https://github.com/istio/istio/blob/master/tests/e2e/README.md
(if there is any Bazel guru listening, there should be a way to make bazel do the download so it will cache it/not download it over and over again no ?)


[linkis-spring-cloud-gateway] Modification of scala file floating red

[linkis-spring-cloud-gateway] Modification of scala file floating red
Related issues: https://github.com/apache/incubator-linkis/issues/2880

the code has been merged into dev-1.3.0


figures cannot be displayed

Thank you for your contribution!
I meet a  problem that figures cannot be displayed, when I use your template on overleaf. After I add \usepackage{graphicx} and \includegraphics{fig.png}, there is only an empty box with label"fig.png" in it.
Can you help me? Thanks!

Ohh... I use wrong Compile Mode. Sorry to bother you.


feature/date_type

Coverage increased (+0.7%) to 94.619% when pulling cb9a8648ca2f1e4ff2b493473b0c29cb97b5f5dd on feature/date_type into e63f6ea221ff5016297cf3ca76c0c7be3f51b78d on master.

Coverage increased (+1.0%) to 94.886% when pulling e93c778adc65406e8bfd7130b5aabaa2b2624cf6 on feature/date_type into e63f6ea221ff5016297cf3ca76c0c7be3f51b78d on master.

Coverage increased (+0.7%) to 94.67% when pulling 2146385afa2c61ff50a366d41f79ccae832e9873 on feature/date_type into e63f6ea221ff5016297cf3ca76c0c7be3f51b78d on master.

Coverage increased (+0.7%) to 94.67% when pulling 744f1dfb88379d12e242d8595efbdf230b5e397c on feature/date_type into e63f6ea221ff5016297cf3ca76c0c7be3f51b78d on master.


Faster fuzz test: teach the fuzz test programs to handle directories

Instead of invoking the fuzz test programs once for every corpora
file, we invoke them once for each directory of corpora files.  This
dramatically reduces the number of program invikations, as well as the
time 99-test_fuzz.t takes to complete.
fuzz/test-corpus.c was enhanced to handle directories as well as
regular files.

This is an alternative to #5771.  The speed reduction is still dramatic, but "only" of a factor of 10.

Could there be a problem in the event of a memory leak? At the moment if we build with crypto-mdebug the executable aborts on exit if memory has been leaked. Doesn't this approach mean that we won't abort until we have run all the fuzz files? That would make it difficult to figure out which one caused the problem.

@mattcaswell, I'm working with test-corpus.c as is.  If we're going to enhance it with crypto-mdebug support, I think it should be done in a separate PR.  Feel free.

I've added another commit to handle way appveyor complained about

(closed and opened to give Travis a kick...)

@mattcaswell, I'm working with test-corpus.c as is. If we're going to enhance it with crypto-mdebug support, I think it should be done in a separate PR. Feel free.

It's not about enhancing it with support. This PR is taking something away, i.e. the ability to know which file caused an issue.
Come to think of it, isn't that a more general problem with this PR as it stands. If one of the files causes a failure - what happens? How do we know which file caused the problem?

When there's a failure, there's usually a crash of some sort...  a segfault.  That's the reason I'm having test-corpus.c print out the file name it's running on

When there's a failure, there's usually a crash of some sort... a segfault. That's the reason I'm having test-corpus.c print out the file name it's running on

Ok, that works, except in the crypto-mdebug case where the segfault doesn't happen until the program exits.

Huh? How does crypto-mdebug stop a segfault caused by, say, a null dereference?

Huh? How does crypto-mdebug stop a segfault caused by, say, a null dereference?

That's not what I meant. crypto-mdebug causes its own abort on process exit if there has been a memory leak. With the old model that would mean we load the fuzzer file, exit the process and then seg fault immediately if a mem leak occurred meaning that we would know that that fuzzer file caused the memory leak. With this new model the memory leak checking still won't happen until process exit, i.e. after all the fuzzer files have been run. If we get a seg fault then (due to a mem leak) we won't know the cause.

@mattcaswell, I have no idea what you're talking about.  As far as I know, memory leak checking is participatory, i.e. the program that wants this reporting must actively enable it.  And no, the programs do not abort, they exit with a failure code.
However, seeing how this has all been hidden in testutil (here and here), I can understand how this is becoming forgotten knowledge.
The fuzz testers are not linked with testutil (and they shouldn't), and they don't contain one bit of code that activates memory leak checking (and they shouldn't, because then they are suddenly testing two things instead of just one).

Additional commit addresses @dot-asm's comments

@levitte : while you are in this corner, you may try to reuse the buffer and so avoid to call malloc in a loop  ?

As far as I know, memory leak checking is participatory, i.e. the program that wants this reporting must actively enable it. And no, the programs do not abort, they exit with a failure code.

I must be confusing the behaviour with what happens in the 1.0.2 mem leak checking. I know I have seen aborts happen there.

The fuzz testers are not linked with testutil (and they shouldn't), and they don't contain one bit of code that activates memory leak checking (and they shouldn't, because then they are suddenly testing two things instead of just one).

That surprises me. I'm not sure I agree that they shouldn't. But I guess that's not a relevant discussion for this PR.

@levitte : while you are in this, you may try to reuse the buffer and so avoid to call malloc in a loop ?

Not sure how much we'll gain...  basically, we'll change malloc to realloc, and considering how much the files differ in size (in all directions), that'll not be much less work, says my gut feeling.  Not worth it in this PR at least.  But hey, if someone wants to optimize test-corpus.c (without making it overly complex and without having it rely heavily on libcrypto), I'm looking forward to the PRs ;-)

Should I backport this to 1.1.0?

Merged.
5d322036b4afbd16afa2714539cf3f85eb7669c2 .travis.yml: with fast fuzz testing, there is no point avoiding it
9d740909595546ebd34a845e6328cb60afa388b8 Faster fuzz test: teach the fuzz test programs to handle directories

Should I backport this to 1.1.0?

I'd say yes.

I must be confusing the behaviour with what happens in the 1.0.2 mem leak checking. I know I have seen aborts happen there.

Ahhhh, right, I had forgotten about this!
But either way, engaging in CRYPTO_MDEBUG is still participatory, and the fuzz stuff doesn't go near it.

No, do not backport.  Making tests run faster is not a bugfix.


How to implement an image edit popup

Thank you for the time spent developing this wrapper but I found very difficult to implement our own extensions due to the lack of documentation and tutorials.
I am looking to implement a small popup/modal that will appear next to an inserted image once clicked : this to have the possibility to either add a style/class to the image (left - right - float) or to change the src.
How could i achieve this ?? I am desperately looking for at least and idea how to start/ or being shown which functions to use.

I need an click event handler on the image, that will get the position for the popup to be shown next to the image
how to programatically add a class to the image (or add a style) and change the src attribute ??

Many thanks for any infos...

You may share your main pain points in the open discussion to improve the documentation. In the meantime, I would recommend you to take a look at the structure of the Links example because it does something similar (but injecting a different Node (a tag) in the document.
Spending a bit of time reading through the Prosemirror docs also helps a lot because tiptap is really using a lot of its APIs instead of defining new things; as a starting point, the Commands is an essential part to create/patch how the editor works.

@francoisdop I would suggest making a custom node with Vue components. Copy everything from the image node, and start from there. You can check the example https://github.com/scrumpy/tiptap/tree/master/examples/Components/Routes/Embeds

another approach, if you want a detached oberver from the editor could be the implementation of a mutation observer, but destroy it wisely after usaged ;)
https://github.com/dev7ch/app/blob/wysiwyg-replacement/src/interfaces/wysiwyg-full/components/EditorContent.vue#L111


Disallow stm32wl rng operations

The stm32wl does not reliably provide a random number on the initial attempt; depending on when the initial attempt is executed, the result is often 0.

@jbeaurivage :
I tested 2 other sx126x boards successfully.  The stm32wl is a special case, since it handles the internal sx1262 differently.  For the stm32wl, it would be better to depend on the random number generation feature the MCU provides.
I will merge this without review.


Won't create a Cookie after click on "Accept"

Hi,
I am using your module with Nuxt 2.11.0 and It looks perfect.
It also hides after I click on "Accept" but it won't create a cookie for the user.
Therefor after a reload the box appears again.
Do you have any idea what the problem could be?
Thank you!

Hi Stanislav,
I am not using Nuxt on daily basis, but I presume that you are rendering it on server side - probably it shouldn't be rendered there, especially that is not something that should be grabbed by bots or SERPs.
This package uses js-cookies from npm, which uses document.cookie behind the scenes, that most likely is not available in Nuxt.
Probably that's not much helpful to you, anyways, let me know if you'll figure out issue - I am curious what this actually is.

Hey,
yes that's correct I am using it in SSR mode, but I use this as a "plugin" in Nuxt which is only rendered on client side. Otherwise it wouldn't even display the banner, because it wouldn't find the document root.
Sure, if I find a solution I will let u know!


Create REST servlet for machine to machine query

It should take all the parameters defined in the spec, perform the search and then return the result as JSON.

Is live

This was handled with #25


AllNodesShouldHaveGraphBuildGlobalProperty Test Failure On MacOS

This test randomly failed on macOS .NET Core 2.1. Re-running the test resulted in a success. Likely similar to #4310
Exception
System.Collections.Generic.KeyNotFoundException : The given key 'Microsoft.Build.BackEnd.ConfigurationMetadata' was not present in the dictionary.

Call Stack
   at System.Collections.Concurrent.ConcurrentDictionary`2.ThrowKeyNotFoundException(Object key)
   at System.Collections.Concurrent.ConcurrentDictionary`2.get_Item(TKey key)
   at Microsoft.Build.Experimental.Graph.ProjectGraph.<CreateEdgesAndDetectCycles>g__CreateEdgesAndDetectCyclesForRoot|14_0(ProjectGraphNode node, ConfigurationMetadata nodeConfig, IDictionary`2 nodeState, <>c__DisplayClass14_0& ) in /_/src/Build/Graph/ProjectGraph.cs:line 314
   at Microsoft.Build.Experimental.Graph.ProjectGraph.<CreateEdgesAndDetectCycles>g__CreateEdgesAndDetectCyclesForRoot|14_0(ProjectGraphNode node, ConfigurationMetadata nodeConfig, IDictionary`2 nodeState, <>c__DisplayClass14_0& ) in /_/src/Build/Graph/ProjectGraph.cs:line 341
   at Microsoft.Build.Experimental.Graph.ProjectGraph.CreateEdgesAndDetectCycles(List`1 entryPointConfigurationMetadata, ProjectInterpretation sdkInfo, ConcurrentDictionary`2 allParsedProjects) in /_/src/Build/Graph/ProjectGraph.cs:line 293
   at Microsoft.Build.Experimental.Graph.ProjectGraph.LoadGraph(List`1 entryPointConfigurationMetadata, ProjectCollection projectCollection, ProjectInstanceFactoryFunc projectInstanceFactory, ProjectInterpretation sdkInfo) in /_/src/Build/Graph/ProjectGraph.cs:line 248
   at Microsoft.Build.Experimental.Graph.ProjectGraph..ctor(IEnumerable`1 entryPoints, ProjectCollection projectCollection, ProjectInstanceFactoryFunc projectInstanceFactory) in /_/src/Build/Graph/ProjectGraph.cs:line 204
   at Microsoft.Build.Experimental.Graph.ProjectGraph..ctor(IEnumerable`1 entryProjectFiles, IDictionary`2 globalProperties) in /_/src/Build/Graph/ProjectGraph.cs:line 108
   at Microsoft.Build.UnitTests.Helpers.CreateProjectGraph(TestEnvironment env, IDictionary`2 dependencyEdges, IDictionary`2 globalProperties, CreateProjectFileDelegate createProjectFile, IEnumerable`1 roots) in /Users/vsts/agent/2.149.2/work/1/s/src/Shared/UnitTests/ObjectModelHelpers.cs:line 1635
   at Microsoft.Build.Experimental.Graph.UnitTests.ProjectGraphTests.AllNodesShouldHaveGraphBuildGlobalProperty(Dictionary`2 edges, Int32[] roots, Dictionary`2 globalProperties) in /Users/vsts/agent/2.149.2/work/1/s/src/Build.UnitTests/Graph/ProjectGraph_Tests.cs:line 1610

@cdmihai, can you take a look at this or route?


support node-red-node-pi-gpiod

Currently there are 2 different nodes to interface with the raspberry pi hardware (GPIO pins):

node-red-node-pi-gpio
node-red-node-pi_gpiod

This issue is raised to support the 2nd node (=node-red-node-pi_gpiod) to interface with the raspberry pi hardware.  This node can easily be installed via the node-red editor (via the manage palette menu) but in order to use this node the daemon pigpiod must be running.
So this issue is about how to get the daemon pigpiod running.

Regarding

So this issue is about how to get the daemon pigpiod running.

I think that the best way (using balena) is to create an extra docker service that runs this daemon.


Cannot download SoundCloud files on Ubuntu 16.04.2

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.05.29. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.05.29

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Error Output
root@forgehe:~# youtube-dl --verbose https://soundcloud.com/ntreboot/suki
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://soundcloud.com/ntreboot/suki  ']
WARNING: Assuming --restrict-filenames since file system encoding cannot encode   all characters. Set the LC_ALL environment variable to fix this.
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968,   pref ANSI_X3.4-1968
[debug] youtube-dl version 2017.05.29
[debug] Python version 2.7.12 - Linux-2.6.32-042stab093.5-x86_64-with-Ubuntu-16.  04-xenial
[debug] exe versions: ffmpeg 2.8.11-0ubuntu<IP_ADDRESS>, ffprobe 2.8.11-0ubuntu0.16  .04.1, rtmpdump 2.4
[debug] Proxy map: {}
[soundcloud] ntreboot/suki: Resolving id
[soundcloud] ntreboot/suki: Downloading info JSON
ERROR: Unable to download JSON metadata: HTTP Error 504: Gateway Time-out (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 502, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2106, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Description of your issue, suggested solution and other information
Fails when trying to download off soundcloud. Youtube works.
Windows exe works when downloading the same link. Fails on Ubuntu 16.04.2. Haven't tested it on any other system.
I just noticed that youtube-dl uses python version 2.7, but I also have python 3 (and 3.5?) installed in my VPS as well. Maybe that causes problems?
Found this issue that looks like it's having the same problem
https://github.com/rg3/youtube-dl/issues/9198

Most likely your VPS is blocked by soundcloud.


[WIP] feat: add option to clean archived repos

We added an option (--clean-archived), off by default, that makes snyk-scm-refresh clean all manifests of a project if the corresponding repository has been archived.
We've tested this on our tenant and it works OK.
The only problem is that we don't know how to delete the target, after all its projects has been deleted.
Using the web UI we've seen that, when using the button, this is done by issuing a DELETE to:

https://app.snyk.io/org/{{org-name}}/targets/{{target-UUID}}

But we cannot really find how to do that using the API. Any help/ideas @scotte-snyk ?
If merged, this fixes #97

the concept of targets is only available in the new REST API https://apidocs.snyk.io/?version=2022-04-06~beta#tag--Targets
we added a generic low-level client for this in pysnyk https://github.com/snyk-labs/pysnyk#experimental-rest-low-level-client

this was resolved in https://github.com/snyk-labs/snyk-scm-refresh/pull/123


Fixed typo in comment

This typo was also carried over to in the melpa package description.

thanks!


🐛 BUG: CTRL +C sometimes doesn't cancel operations with C3

Which Cloudflare product(s) does this pertain to?
C3
What version of Wrangler are you using?
c3 2.0.7
What operating system are you using?
Windows 10
Describe the Bug

npm create cloudflare
select chatgpt plugin
when asked to deploy application, hit CTRL + C.

It will deploy anyway.
This also happens when being asked about TypeScript when doing a hello world - hitting CTRL + C just proceeds and doesn't cancel.

This was fixed by https://github.com/cloudflare/workers-sdk/pull/3525 and will be included in 2.0.12.
The root cause here was that the confirm and select inputs weren't correctly handling the cancel state of clack prompts.


Bump Helm to 3.9.0

This adds support for Kubernetes 1.24. See
https://github.com/helm/helm/releases/tag/v3.9.0 for details.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

Isn’t this the same as #861?

Yes, it's even your commit, but by ordering the same three approvals in this way we can get it merged today.


chore: repro problem with provide/inject

Fail to reproduce error for #1597

This was just demo-ing a bug (that isn't a bug, I don't think


refactor: use css vars for tabs component

#448  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


Documentation is unavailable

Documentation is fall:
https://patchthecode.com
it cant be displayed.

i'll get something up this weekend.
i stop paying for webhosting

I also need it, my app uses this library but it has bugs I gotta fix, @patchthecode can you provide us the new docs link please?

+1, even if you could drop the site in a repo that we could run locally that'd be helpful. Thanks

before removing the site, i zipped every thing in to a file.
site.zip
I am really pressed for time, so just unzip the file, and you'll see all pages of the site saved as html tha tyou can view in your browser.
If any one knows a quick way i can just dump these on a quick FREE site then let me know.
I am tied up with other projects at the moment.

Thank you for the zip file support.
I haven't used it myself, but maybe Github Pages could help? https://pages.github.com/

I have remove it from the website because of cost.
I'll have to upload it again.
In the mean time does the link here not provide it? -> https://github.com/patchthecode/JTAppleCalendar/issues/1397#issuecomment-2054113374

Hope you find the time to uploade the documentation somewhere soon.
It works with the zip, but it is very cumbersome..

@mortenholmgaard i had it up for years... then google just yanked my ad sense.
So now i was paying more to keep the site up and getting nothing for it.
Will get an alternative free one soon.

Hey @patchthecode I submitted a PR last month (see above) that might help at least partially resolve this documentation issue for people, let me know what you think!

@orangegrove1955 did some good work trying to port it to github.
Ive merged it for now until i have the time where i put it back up on a site again.
thanks @orangegrove1955


Support for password, but not in the filename

Hello!! I'm so excited to find this as I've been looking for something like this for a while now. I've been checking if it supports password protected files, but I could only find it is when is in the name of the file.
Could it be possible to have another way to support passwords? I was thinking maybe a file in the same folder like "password.txt" that unpackerr could use as a "list" of passwords to try.
I get a lot of rar & zip files from the same source that all have the same password, but it's not on the name of the file itself, like the other case.
Thank you so much.

Not an unreasonable idea. Would storing the list of passwords in the config file seem sufficient, or would a separate password file work better?

Sorry for the very late reply,
Yes, on the config file is okay, I can just add them there without any problem.
Thank you so much

I started working on this, and will eventually get it in. Just giving you an update.


Fix default mappings unmapping

There is a problem with unmapping default extension keys.
Example:
mappings = {
  i = {
    ["<M-m>"] = false,
  },
},

will end up with something like this eventually:
mappings = {
  i = {
    ["<M-m>"] = false,
    ["<A-m>"] = fb_actions.move,
  },
},

I didn't retrieved all the results, that depend on which mapping will be proceeded first, but my experience is that I have my global mapping muted (it doesn't work and the default mapping doesn't work either)
So I've rewritten part with mappings merging. Current algorithm:

retrieves all ignored mappings (in internal termcodes)
merges all mappings
removes all ignored mappings (comparing with internal termcodes)

Sorry for the late response.
I'm not sure how to feel about this. The current behavior seems to be consistent with telescope's builtin pickers.
require("telescope").setup({
  defaults = {
    mappings = {
      i = {
        ["<A-m>"] = false,
      },
    },
  },
  pickers = {
    find_files = {
      mappings = {
        i = {
          ["<M-m>"] = function() print("hello") end,
        },
      },
    },
  },
})

I think if we were to implement a change like this, it should probably be in upstream telescope.
What do you think?

I've just tested it with telescope's mappings, and telescope's mappings work as expected with <A-m> and <M-m> either.
My original problem was that I tried to disable all of the plugin's mappings and I didn't figure out how I can do this.
for telescope for example, I can simply use default_mappings = {}:
telescope.setup({
  defaults = {
    default_mappings = {},
  },
})

Unfortunately, this plugin doesn't support this approach.

Ah okay. For disabling default mappings see https://github.com/nvim-telescope/telescope-file-browser.nvim/issues/322#issuecomment-1793981447

Thanks. I did it at the time of the pull request.
It it isn't required, I'll close it


create a footer template

create a header template using html,bootstrap and twig

the issue did not represent a use case


(PUP-3567) Remove current_environment check in Puppet::Indirector::Reque...

...st#environment=
Prior to this commit, there was an environment setter that allowed an
environment to be set if it existed in Puppet.lookup(:current_environment). This commit changes that by only
allowing environments to be set from what exists within Puppet.lookup(:environments).

@joshcooper The code path that is being removed allowed the remote environment to be passed to an indirector request by name (e.q. Puppet::Resource::Catalog.indirection.find(..., :environment => :foo)) and let that work if it so happened that the name was the name of the current environment. This is important on the agent because the environment that the master is using is usually not in the environments known by Puppet.lookup(:environments).get on the agent.
The remote environments on the agent should be handled by only ever passing around the environment by a Puppet::Node::Environment instance instead of by name. In fact, a more drastic approach here is to only allow the environment to be specified by a reference and never let the indirector look one up.

so... by just removing the lines in question do we actually achieve anything without also taking the more drastic approach? The only case I see where a lookup of name would fail is if the now current environment no longer can be loaded (i.e. removed from disk while in use by the runtime), or am I missing something?

Changed my mind; the removed code was a crutch for some reason that is no longer apparent. Merging.


Remove jboss deps

This PR removes all of the JBoss dependencies that the project has and replaces them with Tomcat's JULI logger and StringManager for messages. It also removes one feature of the vault which was to extract class information from JBoss Modules, which Tomcat doesn't have, so no harm there.
Please review and let me know what you think.

The scripts still work but I need to remove the jboss deps in them and move the jar location and such, so don't accept this just yet.

I updated the scripts so that they don't use jboss modules anymore and cleaned up the module.xml files that were left before. All of the dependencies on JBoss are now removed. I'll probably follow up with a PR at some point changing the directory structure from modules to something like lib.


fix buffered events bug

fix buffered event bug

Coverage decreased (-0.02%) to 66.225% when pulling f7018a7a8db11f7f336789c8d2104331fe03bbe7 on yiminc:fix_buffered_bug into f7e2ff19bf245590f899d38664033e0c99688470 on uber:master.

Coverage increased (+0.004%) to 66.254% when pulling f7018a7a8db11f7f336789c8d2104331fe03bbe7 on yiminc:fix_buffered_bug into f7e2ff19bf245590f899d38664033e0c99688470 on uber:master.


Messaging that external developers should use integration.

I was mistaken in thinking that external devs should be using production. Apparently they're supposed to be using integration still.

Reviewed by @dompuiu


useDeepCompareEffect should not show the warning when the values are not primitive but null/undefined at the moment

I have this code that uses useDeepCompareEffect and when I have on the deps a value that is not a primitive value but at the moment it is null or undefined, it shows the following message:
`useDeepCompareEffect` should not be used with dependencies that are all primitive values. Use React.useEffect instead.

In my opinion this message should be removed for null and undefined values.

Super late but useDeepCompareEffectNoCheck was created for this use case.
From the README:
NOTE: Only use this if your values are objects or arrays that contain objects. Otherwise, you should just use React.useEffect. In case of "polymorphic" values (eg: sometimes object, sometimes a boolean), use useDeepCompareEffectNoCheck, but do it at your own risk, as maybe there can be better approaches to the problem.

@italodeandra I agree with you actually.  It's normal to expect an object variable to possibly be  null or undefined but not a boolean. So yeah although useDeepCompareEffectNoCheck gets around the problem, I don't think it should be required. It also confuses things with the naming.

useDeepCompareEffectNoCheck doesn't exist anymore and useCustomCompareEffect has the same code inside, so there's no way to get around this besides rolling your own implementation.


Additional Support Added

Add BB8 Support
Clean up BB9E
Add Mini Support
Clean Sphero
Add Mini to Utils
Add Test Scripts for BB8 and Mini
Fix spelling error in firmware
Add missing call to SystemInfo

Note: Further support of Mini and BB8 will need to be added to support all calls but LED is a good testing feature to support for responses

I think you need to pull main and resolve the conflicts first.

Fixed it! sorry, made progress on bb8 animations and completely forgot to look into the merge errors

Thank you! I've given you write access to the repo so you can directly dev on a new branch instead of forking.


TrackerField with JSONField AttributeError with deferred fields

In my model i use JSONField (https://pypi.python.org/pypi/jsonfield)
I got error with deferred fields:
/usr/local/lib/python2.7/dist-packages/django/db/models/query.py:256: in __iter__
    self._fetch_all()
/usr/local/lib/python2.7/dist-packages/django/db/models/query.py:1087: in _fetch_all
    self._result_cache = list(self.iterator())
/usr/local/lib/python2.7/dist-packages/django_mysql/models/query.py:160: in iterator
    for row in super(QuerySetMixin, self).iterator():
/usr/local/lib/python2.7/dist-packages/django/db/models/query.py:66: in __iter__
    obj = model_cls.from_db(db, init_list, row[model_fields_start:model_fields_end])
/usr/local/lib/python2.7/dist-packages/django/db/models/base.py:565: in from_db
    new = cls(*values)
/usr/local/lib/python2.7/dist-packages/django/db/models/base.py:557: in __init__
    signals.post_init.send(sender=self.__class__, instance=self)
/usr/local/lib/python2.7/dist-packages/django/dispatch/dispatcher.py:191: in send
    response = receiver(signal=self, sender=sender, **named)
/usr/local/lib/python2.7/dist-packages/model_utils/tracker.py:163: in initialize_tracker
    tracker = self.tracker_class(instance, self.fields, self.field_map)
/usr/local/lib/python2.7/dist-packages/model_utils/tracker.py:37: in __init__
    self.init_deferred_fields()
/usr/local/lib/python2.7/dist-packages/model_utils/tracker.py:104: in init_deferred_fields
    field_obj = getattr(self.instance.__class__, field)
/usr/local/lib/python2.7/dist-packages/jsonfield/subclassing.py:33: in __get__
    raise AttributeError('Can only be accessed via an instance.')
E   AttributeError: Can only be accessed via an instance.

Hi @MikeVL ! Which version of Django are you using? Can you copy/paste some code sample that triggers this error?

Hi @romgar As I remember I tested the version 1.10

Also having this. Any ideas?

+1

+1

Any updates on this? Testing on both Python 2 and 3 with Django 1.10 and 1.11. Only instances with 1.11 cause this error.

1

I don't think this is a bug in django-model-utils. Descriptors should return themselves when accessed from the class, not raise an error. Also, I think this has been fixed in django-jsonfield: https://github.com/dmkoch/django-jsonfield/commit/a9190f6320ee03a330c0dd392c8112f220217bf0#diff-39f2ef2d3835084e14fbdd002375d263


Get Arc from ShapeHandle

Doing some C interop. ShapeHandle appears to wrap Arc<dyn Shape>, and while there's a method to convert an Arc<dyn Shape> to a ShapeHandle, there's no way to get the Arc<dyn Shape> from an existing ShapeHandle.
I'd like this because then I can expose the pointer provided by Arc::into_raw to the C API, however since I can't do that, I have to box a ShapeHandle, which adds another layer of allocation and indirection.

Hi @ColonelThirtyTwo!
If that can be useful to you, we could add an into_inner method to ShapeHandle:
impl<N: RealField> ShapeHandle<N> {
     pub fn into_inner(self) -> Arc<dyn Shape<N>> {
         self.0
     }
}


Feature | PEO-826 | Fill technology stack section

PEO-826
Before submitting the merge request make sure the following are checked

[x] Followed the style guidelines of this project
[x] Performed a self-review of own code
[x] Wrote the tests that prove that fix is effective/that feature works

https://github.com/rubygarage/inquisition/pull/124
https://github.com/rubygarage/inquisition/pull/129

@rostikkkk2 your branch has conflicts, can you resolve them, thx


middleMarker can't drag in Leaflet 1.0.0-rc.2

when middleMarker become normal marker , it can't drag in Leaflet 1.0.0-rc.2.

@wstygk thanks for the report.
Looks like the event changes messed something up. What still works is when you click to add a middle marker, and then drag.
The instant drag doesn't work - I'm working on a fix now!

This seems to be a leaflet bug. The drag is stuck when changing the marker inside a dragstart event.
https://github.com/Leaflet/Leaflet/issues/4824
I'm on it, maybe I have a workaround until it's fixed by leaflet

Ok this was already reported here:
https://github.com/Leaflet/Leaflet/issues/4484

Fixed


Build a search performance test harness

Build a search performance test harness to include reasonable variations of search, Leverage the Windtunnel project as appropriate
The goal is to help us find slow queries or queries which scale poorly with the amount of data, especially in advanced search features like:

whole-system search
chaining
reverese chaining (_has) - #1218
includes
reverse includes
compartment search

Once, ready, we will use this framework to get a baseline for our search performance (#677).
Then, we need some way to track this over time (and spot regressions).

Is the goal here to have something that execute these searches every time we each run a build?  or is it intended to be something that someone would run based on a time interval (maybe weekly)?  or just when new fhir code is committed?  My opinion is that there needs to be a consistent chunk of data found and returned when these searches are done in order to better understand performance ... that would require mock data designed with specific relationships and content ... not sure it makes sense to include in the build process to be run all the time

Is the goal here to have something that execute these searches every time we each run a build?  or is it intended to be something that someone would run based on a time interval (maybe weekly)? or just when new fhir code is committed?

Running it on pull requests (and being notified of slowdowns) would be great, but at this point I'd just be happy to have a baseline of the various search queries against a sizeable database.

My opinion is that there needs to be a consistent chunk of data found and returned when these searches are done in order to better understand performance ... that would require mock data designed with specific relationships and content ... not sure it makes sense to include in the build process to be run all the time

I agree.  Perhaps we should just run this ourselves against one of our internal deployments (and probably not from the public CI).  Although hopefully we could still share the results in some publicly-available way.

At connectathon, I became aware of a nice Postman collection of "complex but realistic" queries for Provider Directory (PDEX PlanNet):  https://github.com/Jammjammjamm/plan-net-postman-collection
These queries, or queries like them, but form a good basis for some queries to measure.
For convenience, I'll list some of the more advanced ones here:

Scenario 3: Request data on an individual Organization

{{url}}/Organization?name=GREATER LAWRENCE FAMILY HEALTH CENTER, INC.&_revinclude=Location:organization&_revinclude=HealthcareService:organization&_revinclude=Endpoint:organization

Scenario 4: Request data on Organizations

{{url}}/Organization?name=BAY STATE COMMUNITY SERVICES, INC.&_revinclude=OrganizationAffiliation:participating-organization&_revinclude=Endpoint:organization&_revinclude=HealthcareService:organization

Scenario 5a: Request Organization data for a specific Provider

{{url}}/Practitioner?identifier=1230782571&_revinclude=PractitionerRole:practitioner&_include:iterate=PractitionerRole:organization

Scenario 5b: Request Locations, HealthcareServices, and Endpoints for Provider Organization

{{url}}/Organization?_id={{pract_assoc_orgs}}&_revinclude=Location:organization&_revinclude=HealthcareService:organization&_revinclude=Endpoint:organization

Scenario 6a: Identify Organization and Practitioner data associated with a Network

{{url}}/Organization?name=State of Massachusetts Preferred Provider Network&_revinclude=OrganizationAffiliation:network&_revinclude=PractitionerRole:network

Scenario 6b: Identify location and healthcare services data associated with an Organizational Provider Network
Member

{{url}}/Organization?_id={{network_org_members}}&_revinclude=Location:organization&_revinclude=HealthcareService:organization

Scenario 7a: Identify HealthcareServices in a state

{{url}}/HealthcareService?location.address-state=RI&_include=HealthcareService:organization

Scenario 7b: Identify HealthcareServices in a city

{{url}}/HealthcareService?location.address-state=RI&location.address-city=Providence&_include=HealthcareService:organization

Scenario 7c: Identify particular types of HealthcareServices in a state

{{url}}/HealthcareService?service-type=respiratory-therapy&location.address-state=RI&_include=HealthcareService:organization

Scenario 7d: Identify particular types of HealthcareServices in an area

{{url}}/HealthcareService?service-type=respiratory-therapy&location.near=-71.4|41.8|100|mi_us

Scenario 8: Find Practitioners whose license will expire in the next 90 days

{{url}}/Practitioner?qualification-period=eb2019-12-14

Scenario 9a: Identify Organizations in a Network

{{url}}/OrganizationAffiliation?network=plannet-network-HPID110000&_include=OrganizationAffiliation:participating-organization

Scenario 9b: Identify Practitioners in a Network

{{url}}/PractitionerRole?network=plannet-network-HPID110000&_include=PractitionerRole:practitioner

Mock Data Search example:
search sample that covers chaining and include
https://localhost:7000/fhir-server/api/v4/ExplanationOfBenefit?claim:Claim.created=2015-10-16&patient:Patient._id=17546b5cd4a-cae29bb0-d6d7-4a1a-a8df-505e2e3a975e&_include=ExplanationOfBenefit:*
Currently fails.
Sample with just chaining works
https://localhost:7000/fhir-server/api/v4/ExplanationOfBenefit?claim:Claim.created=2015-10-16&patient:Patient._id=17546b5cd4a-cae29bb0-d6d7-4a1a-a8df-505e2e3a975e

tests added and merged to performance/fhir-meter/searchTest.jmx JMeter tests in git


Add optional inner translation usageMatch for ember

Is your feature related to a specific framework or general for this extension
ember
Is your feature request related to a problem? Please describe.
Sometimes I'll have translations inside translations:
{{t "some.key"
  htmlSafe=true
  property=(t "some.other.key")
}}

Gets usageMatched to
{{t [some translation here! neat!]
  htmlSafe=true
  property=(t "some.other.key")
}}

Describe the solution you'd like
{{t [some translation here! neat!]
  htmlSafe=true
  property=(t [another translation was matched!)
}}

Additional context
Figured it out, as well as added some refactorTemplates to help with all the URL translations:
# An array of strings which contain Language Ids defined by VS Code
# You can check avaliable language ids here: https://code.visualstudio.com/docs/languages/overview#_language-id
languageIds:
  - javascript
  - typescript
  - javascriptreact
  - typescriptreact
  - handlebars

# An array of RegExes to find the key usage. **The key should be captured in the first match group**.
# You should unescape RegEx strings in order to fit in the YAML file
# To help with this, you can use https://www.freeformatter.com/json-escape.html
usageMatchRegex:
  # The following example shows how to detect `t("your.i18n.keys")`
  # the `{key}` will be placed by a proper keypath matching regex,
  # you can ignore it and use your own matching rules as well
  - "(?:{{\\s*t\\s+|[^\\w\\d]intl\\.t\\(\\s*|=\\s*\\(\\s*t[\\s\\n]+)['\"`]({key})['\"`]"

# An array of strings containing refactor templates.
# The "$1" will be replaced by the keypath specified.
# Optional: uncomment the following two lines to use

refactorTemplates:
  - "{{t '$1' }}"
  - "{{t '$1' htmlSafe=true }}"
  - "this.intl.t('$1')"
  - "this.intl.t('$1', { htmlSafe:true })"
  - "$1"

This shouldn't conflict tho, did you try to add regex for (t "{key}"?

I'm not sure what you mean; I changed that regex so that it does additionally include checking for (t key. I just didn't know how to officially change it for your ember version so I created a feature request issue.

Thanks, updated!

Thanks, updated!


python310Packages.arcam-fmj: 1.2.1 -> 1.3.0

Description of changes
Diff: https://github.com/elupus/arcam_fmj/compare/refs/tags/1.2.1...1.3.0
Changelog: https://github.com/elupus/arcam_fmj/releases/tag/1.3.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 227187 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.arcam_fmj
python310Packages.arcam-fmj
python310Packages.arcam-fmj.dist
python311Packages.arcam-fmj
python311Packages.arcam-fmj.dist


n_i5 Winner's share

I am asked to code this indicator.
 Correctly, this is a pre-post Gini (or any index of the bounded entropy family).
Concentration is not the same thing as "Winner's shares" (e.g. of a market).
Gini makes sense only as an indicator for the agency and not for the Winner. It makes no sense to flag a company as suspicious of corruption if after a crisis it wins ONLY in some Agencies while before it used to win in many more. Think about a urn process where I draw 5 blue balls and 5 red balls. My concentration is 0. Then I add a selection bias in the process and I draw 1 blue ball and 5 red balls. My concentration raises, but I am actually winning less overall, so how can I be accused of corruption???
It makes sense only for agencies that both raised their frequency of wins and concentration.
For Agencies it makes sense because they actually control who wins the bid, so they are imputable of the selection bias. Even if they reduced the number of balls in the stochastic process, why does one color seems to keep being drawn more?

Proposal:

Rename it, while keeping the explanation that mentions Gini itself.
Simplify this just as a test on Agencies only, on pre-post differences in Gini, which I guess is in line with @simonedelsa original proposal. Simone's code does much more passages so I want to be sure my understanding is correct.

Gini is not a proportion, do we need a specific test for pre-post Gini? Please indicate me the correct test for pre-post Gini, which is still the difference between the 2 sample estimates, not 2 distributions (so, I am confused about mentioning KS test).
P.S. I know very well that this discussion has been made at least 3 times but I think it was never put down written with explicit considerations.

Thanks @giuliogcantone for this issue.
I'm totally agree with you for each point and with your proposals. The indicator explanation in CORE deliverable 4.3 needs to be adjusted according to your comments (I'm working on it, as other parts need adjustments as well).
Currently, I am not sure on how testing the pre-post for this indicator, but no KS test and no Barnard for sure! We can discuss about it on next Thurdsay at 9.30.

I implemented the indicator like @giuliogcantone suggested in f9072e75142269dc33085019a966837cc320f46a.


Windows 10 support

-Installer not working
-Themes not switching
-Notification problems

https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/304
https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/308

https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/304
https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/308

Seems to work with new version of nsis


Added finding PR from refs update

Closes #23

Why do all cassettes changes? This make it a huge commit for only a few lines of code :(

Why do all cassettes changes? This make it a huge commit for only a few lines of code :(

I don't remember exactly but I had to change some of them for some reason, also I messed up and had two tests with the same name so I had to redo the cassettes


Windows Server 2016 compose install instructions?

From: https://forums.docker.com/t/where-do-you-get-docker-compose-exe-for-windows-2016/31313
The current install instructions say that since Docker for Windows and Docker Toolbox already include compose that windows instructions aren't needed. Neither of these solutions encompass running a docker windows daemon/client directly on windows server 2016.
File: compose/install.md, CC @londoncalling

Invoke-WebRequest "https://github.com/docker/compose/releases/download/$dockerComposeVersion/docker-compose-Windows-x86_64.exe" -UseBasicParsing -OutFile $Env:ProgramFiles\docker\docker-compose.exe

Fixed in #3035


Fyle 1vcx46z : Fixed change the split expense success pop up to success toast message

the error toast should also be above the bottom nav (above the pink camera) it shouldn't hide the bottom sections. Can we fix this @siddharth0-0 ?


replace copied code with imports

The pull request #136  has copied a lot of code from go-ethereum project, so it should be corrected to use it as imports instead of copying it.

outdated


Rails 4.2.0 compatible

Hi!
Is this gem already rails 4.2 compatible?
My Gemfile:
gem "rails", "4.2.0"
gem 'monologue', :git => "git://github.com/jipiboily/monologue", :branch => "master"

if I try to  install the bundle I get following error:
Fetching git://github.com/jipiboily/monologue
Fetching gem metadata from https://rubygems.org/.........
Fetching additional metadata from https://rubygems.org/..
Resolving dependencies...
Bundler could not find compatible versions for gem "sass-rails":
  In Gemfile:
    monologue (>= 0) ruby depends on
      sass-rails (~> 4.0.0) ruby

    sass-rails (5.0.1)

I can see the dependencies in the monologue.gemspec file:
s.add_dependency "rails", ">= 4.0.4"
s.add_dependency "sass-rails",'~> 4.0.0'

Could this simple be updated to rails 4.2 or will such an update cause spec failures? Did you try it out already?

I didn't try it myself but I am not expecting any major problems, updating the gemspec should cover that to start with.

Oh, and PR welcomed.

Rails also removes controller level respond_to calls in 4.2

This should now be fixed. Please give it a shot and let me know if you still have issues


Asset validation failed (90266) Install folder failed

Asset validation failed (90266)
Your application bundle must install to '/Applications'. (ID: e648ecde-f424-44e4-a6e3-e55e13a44549)
How is installation folder is configured?

just a typo in the terminal


From v0.14.32 this code is not working anymore

From v0.14.32 this code is not working anymore
Tables: []core.Table{
  {Name: "me", Table: "users"},
},

error:
error="table not found for selector: me".
Same code with v0.14.31 works.

Do you have a blocklist?

Oh! Wow! This was the problem! My blocklist was:
"password",
"account_id",
"tokens",
"token",
"users",
"user",

But this is a problem now if I have a column called this way, right?

It is a general blocklist for now. I could add one for tables and one for columns

Fyi you don't really need to use blocklist in production the allow list ensures only saves queries ca. Be run no one can add cloumns or modify the query in any way from the client


Color as type

Hi,
Is it an option to add Color as a type. Like
type Color func(s ...string) string

So it can be used like.
package main

import (
	"fmt"
	"github.com/jwalton/gchalk"
)

type Message struct {
	message string
	color gchalk.Color
}

func printMessage(m Message) {
	fmt.Println(m.color(m.message))
}

func main() {
	warning := Message{
		message: "Warning",
		color: gchalk.Yellow,
	}
	printMessage(warning)
	error := Message{
		message: "Error",
		color: gchalk.Red,
	}
	printMessage(error)
}

Else the user has to add the type itself. It's just convince but I think it is beter.
Regards, Marcel

I can definitely add that.  Any objection to ColorFn instead of just Color?

Released in v1.3.0.


Failed to parse CMake console output to locate block start/end markers. (when I want to install literally anything)

I am not sure is it a "me" problem or a vcpkg bug (or maybe sth else), so to be safe, I opened this in here
I did all steps of installing vcpkg but whenever I want to install any package (that I tested yet, including pybind11, boost, sqlite3, and some parts of boost discretely) I constantly get this error:
Computing installation plan...
Failed to parse CMake console output to locate block start/end markers.

vcpkg is installed in E drive inside the project (I changed its path, and it didn't affect result)
Am I doing sth wrong? I googled and found nothing remotely useful

Do you use (bootstrap) the latest version of vcpkg?
Is the cmake version managed by vcpkg?
In the past, problems occured with CLICOLOR_FORCE=1, but this should be resolved (#20430).

I cloned it from GitHub yesterday, so I guess, it's new?!
"Is the cmake version managed by vcpkg?"
I'm not sure, but I'm sure cmake was installed prior to vcpkg, therefore I guess no, cmake is not managed by vcpkg

I tested solutions offered on the other thread, non seem to work,
because it seems like it's a popen parse issue, just to inform, I'm using PowerShell, on a Windows device, with Windows Terminal, but error persisted in cmd in wt as well
It may be an old cmake tho (cmake version 3.22.3)
can it be the cause?

CMake 3.22.3 is fine.
For more output, try .\vcpkg install --debug --debug-env zlib.

[DEBUG] The following environment variables are currently set:
ALLUSERSPROFILE=C:\ProgramData
ANDROID_SDK_ROOT=C:\Users\AppData\Local\Android\Sdk
APPDATA=C:\Users\aarmn\AppData\Roaming
ChocolateyInstall=C:\ProgramData\chocolatey
ChocolateyLastPathUpdate=132709909149260773
ChocolateyToolsLocation=C:\tools
CHROME_EXECUTABLE=C:\Program Files\Google\Chrome Dev\Application\chrome.exe
CLICOLOR=0 // due to test got added here (post note)
CommonProgramFiles=C:\Program Files (x86)\Common Files
CommonProgramFiles(x86)=C:\Program Files (x86)\Common Files
CommonProgramW6432=C:\Program Files\Common Files
ComSpec=C:\WINDOWS\system32\cmd.exe
DriverData=C:\Windows\System32\Drivers\DriverData
GIT_INSTALL_ROOT=C:\Users\aarmn\scoop\apps\git\current
GOPATH=C:\Users\aarmn\go
GTK_BASEPATH=C:\Program Files (x86)\GtkSharp\2.12
HAXEPATH=C:\HaxeToolkit\haxe
HOMEDRIVE=C:
HOMEPATH=\Users\aarmn
IntelliJ IDEA=d:\Program Files\JetBrains\IntelliJ IDEA 2020.3.2\bin;
IntelliJ IDEA Community Edition=C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2021.3.1\bin;
INTEL_DEV_REDIST=C:\Program Files (x86)\Common Files\Intel\Shared Libraries
JAVA_HOME=C:\Program Files\Android\Jdk\microsoft_dist_openjdk_<IP_ADDRESS>
JetBrains Rider=d:\Program Files\JetBrains\JetBrains Rider 2020.3.2\bin;
LOCALAPPDATA=C:\Users\aarmn\AppData\Local
MIC_LD_LIBRARY_PATH=C:\Program Files (x86)\Common Files\Intel\Shared Libraries\compiler\lib\mic
NEKO_INSTPATH=C:\HaxeToolkit\neko
NUMBER_OF_PROCESSORS=12
NVTOOLSEXT_PATH=C:\Program Files\NVIDIA Corporation\NvToolsExt
OS=Windows_NT
PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC;.PY;.PYW;.CPL
PORTABLE_APPS=D:\Semi-Portable
POWERSHELL_DISTRIBUTION_CHANNEL=MSI:Windows 10 Pro
PROCESSOR_ARCHITECTURE=x86
PROCESSOR_ARCHITEW6432=AMD64
PROCESSOR_IDENTIFIER=Intel64 Family 6 Model 165 Stepping 2, GenuineIntel
PROCESSOR_LEVEL=6
PROCESSOR_REVISION=a502
ProgramData=C:\ProgramData
ProgramFiles=C:\Program Files (x86)
ProgramFiles(x86)=C:\Program Files (x86)
ProgramW6432=C:\Program Files
PSModulePath=C:\Users\aarmn\Documents\WindowsPowerShell\Modules;C:\Program Files\WindowsPowerShell\Modules;C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules
PUBLIC=C:\Users\Public
PyCharm=c:\Program Files\JetBrains\PyCharm 2020.3.3\bin;
PyCharm Community Edition=C:\Program Files\JetBrains\PyCharm Community Edition 2021.1\bin;
Qemu=C:\Program Files\qemu
SESSIONNAME=Console
STARSHIP_SESSION_KEY=olNucqvbmYL6ntSA
STARSHIP_SHELL=powershell
Startmenu=C:\ProgramData\Microsoft\Windows\Start Menu\Programs
SystemDrive=C:
SystemRoot=C:\WINDOWS
TEMP=C:\Users\aarmn\AppData\Local\Temp
TMP=C:\Users\aarmn\AppData\Local\Temp
USERNAME=aarmn
USERPROFILE=C:\Users\aarmn
VBOX_MSI_INSTALL_PATH=C:\Program Files\Oracle\VirtualBox
VCPKG_COMMAND=E:/vcpkg/vcpkg/vcpkg.exe
VIRTUAL_ENV_DISABLE_PROMPT=1
windir=C:\WINDOWS
X_VCPKG_RECURSIVE_DATA={}
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'manifests' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Failed to open: E:\vcpkg\vcpkg\vcpkg-bundle.json
[DEBUG] Using scripts-root: E:\vcpkg\vcpkg\scripts
[DEBUG] Bundle config: readonly=0, usegitregistry=0, embeddedsha=nullopt
[DEBUG] Using builtin-ports: E:\vcpkg\vcpkg\ports
[DEBUG] Using installed-root: E:\vcpkg\vcpkg\installed
[DEBUG] Using buildtrees-root: E:\vcpkg\vcpkg\buildtrees
[DEBUG] Using packages-root: E:\vcpkg\vcpkg\packages
[DEBUG] Using vcpkg-root: E:\vcpkg\vcpkg
[DEBUG] Using scripts-root: E:\vcpkg\vcpkg\scripts
[DEBUG] Using builtin-registry: E:\vcpkg\vcpkg\versions
[DEBUG] Using downloads-root: E:\vcpkg\vcpkg\downloads
[DEBUG] Default binary cache path is: C:\Users\aarmn\AppData\Local\vcpkg\archives
Computing installation plan...
[DEBUG] Found path: C:\Users\aarmn\AppData\Roaming\Python\Python39\Scripts\cmake.exe
[DEBUG] Found path: D:\Program Files\CMake\bin\cmake.exe
[DEBUG] CreateProcessW("C:\Users\aarmn\AppData\Roaming\Python\Python39\Scripts\cmake.exe" --version)
[DEBUG] ReadFile() finished with GetLastError(): 109
[DEBUG] 20112: cmd_execute_and_stream_data() returned 0 after   181924 us
[DEBUG] CreateProcessW("C:\Users\aarmn\AppData\Roaming\Python\Python39\Scripts\cmake.exe" -DVCPKG_ROOT_DIR=E:/vcpkg/vcpkg -DPACKAGES_DIR=E:/vcpkg/vcpkg/packages -DBUILDTREES_DIR=E:/vcpkg/vcpkg/buildtrees -D_VCPKG_INSTALLED_DIR=E:/vcpkg/vcpkg/installed -DDOWNLOADS=E:/vcpkg/vcpkg/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P "E:\vcpkg\vcpkg\buildtrees\0.vcpkg_tags.cmake")
[DEBUG] ReadFile() finished with GetLastError(): 109
[DEBUG] 20112: cmd_execute_and_stream_data() returned 0 after    87674 us
Failed to parse CMake console output to locate block start/end markers.
[DEBUG] D:\a_work\1\s\src\vcpkg\cmakevars.cpp(297):
[DEBUG] Time in subprocesses: 269598 us
[DEBUG] Time in parsing JSON: 49 us
[DEBUG] Time in JSON reader: 81 us
[DEBUG] Time in filesystem: 6847 us
[DEBUG] Time in loading ports: 3256 us
[DEBUG] Exiting after 445.4 ms (440483 us)

Path env var is not up there because it's super big and messy and may contain some info that I don't want to share to the world
but probably nothing much is going on there (the cmake and vcpkg are checked, actually I run vcpkg by absolute path)

C:\Users\aarmn\AppData\Roaming\Python\Python39\Scripts\cmake.exe
Looks very strange and not like a normal cmake installation

GIT_INSTALL_ROOT=C:\Users\aarmn\scoop\apps\git\current

Just another issue in the context of scoop.
https://github.com/microsoft/vcpkg/search?q=scoop&type=issues

GIT_INSTALL_ROOT=C:\Users\aarmn\scoop\apps\git\current

Just another issue in the context of scoop. https://github.com/microsoft/vcpkg/search?q=scoop&type=issues

Ok, and how can I fix it? Should I uninstall cmake and git, then reinstall them directly by their setup (in case it's a scoop issue)
while writing, I didn't check the link because it seems it's a broad list of issues related to it.
I feel a bit confuse, what should be my next action to resolve?

I would start with getting C:\Users\aarmn\AppData\Roaming\Python\Python39\Scripts out of the PATH, so that vcpkg uses its own download, or otherwise ensure a trustable CMake.

Ok, issue is resolved by moving cmake (freshly official install) priority higher in my path, I don't know what was the conflicting binary and at this point I don't care either!
I guess it had sth to do with scoop or python, as you said (I moved it over C:\Users\aarmn\AppData\Roaming\Python\Python39\Scripts and C:\Users\aarmn\AppData\Roaming\Python\Python310\Scripts)
anyway, thanks for your help


omnisharp-roslyn: 1.39.11 -> 1.39.12

Description of changes
Changelog: https://github.com/OmniSharp/omnisharp-roslyn/releases/tag/v1.39.12

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.


PasswordHasherOptions isn't registered properly via DI

services.AddOptions(identityConfig) cannot find PasswordHasherOptions as they are not part of IdentityOptions or anything discoverable.
'Microsoft.Framework.OptionsModel.OptionsManager`1' is waiting for the following dependencies:
- Service 'System.Collections.Generic.IEnumerable`1[[Microsoft.Framework.OptionsModel.IConfigureOptions`1[[Microsoft.AspNet.Identity.PasswordHasherOptions, Microsoft.AspNet.Identity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]], Microsoft.Framework.OptionsModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]' which was not registered.

Can you include more of the repro which triggered this error?  Can you show entire method that's configuring your service collection?

Maybe it works with default DI container, I am using a custom Castle.Windsor container. All in all the problem is fixed simply by adding
services.Configure<PasswordHasherOptions>(options =>
            {
            });

Services registration method:
app.UseServices(services => {
            // Add EF services to the services container.
            services
                .AddEntityFramework(Configuration)
                .AddSqlServer()
                .AddDbContext<ApplicationDbContext>();

            // !! This fixes the problem described !!
            services.Configure<PasswordHasherOptions>(options =>
            {
            });

            // Add Identity services to the services container.
            services
                .AddIdentity<ApplicationUser, IdentityRole>(Configuration)
                .AddEntityFrameworkStores<ApplicationDbContext>()
                .AddDefaultTokenProviders();

            // Add MVC services to the services container.
            services
                .AddMvc();

            container.Populate(services);

            container.BeginScope();

            return container.Resolve<IServiceProvider>();
        });

Closing unless we can get a repro, options don't need to be registered in DI, only the IOptions itself which is done via services.AddOptions() or using any framework which does this for you (AddIdentity/AddMvc)

It is hard to get a repro, since it requires a different DI container to be shown, but services.AddOptions(); isn't enough, PasswordHasherOptions do not get registered. If I have time I will create a repro project.

Ok, here is a repro https://github.com/egorpavlikhin/IdentityIssue319

@HaoK please have a look at the repro project, when you get a chance.

So it looks like you are doing your own DI here, could it be an issue with your implementation behaving differently from the rest in how it tries to resolve IEnumerable of a generic type?  Basically Options is just querying DI IEnumerable<ConfigureOptions<TOptions>>.  Nothing is registered by default for this, it should just return an empty enumerable by default.

This container passes all DI unit tests though. Do you think this should be added as a test case or each option should be registered explicitly with the container?

I'm not sure, but perhaps these lines might be the issue?
What does the HasComponent check do?   https://github.com/egorpavlikhin/IdentityIssue319/blob/master/WindsorRegistration.cs#L108

Do you think this should be added as a test case ...?

Sounds like a good idea (although other things maybe missing this shouldn't hurt). @HaoK can you take care of it?

@HaoK if a type is not registered within the container Windsor will not resolve IEnumerable, that's just how it works. Autofac works differently, but I don't think it is right to assume that every DI implementation returns empty Enumerable for unregistered types.

@davidfowl @lodejard I'm pretty sure K assumes that all DI implementations will return an empty IEnumerable when requested.  Can you guys confirm?  Certainly options is built on this assumption, and if the DI implementations throws an exception saying T is not registered, that is the DI implementation not living up to the implicit contract.  I can certainly add a unit test in DI verifying that asking for an IEnumerable of another generic type doesn't blow up once we have consensus.

The DI implementation has to return empty IEnumerable


Use rowspan to properly visualize double-length talks in grid view

e.g. https://joind.in/event/phpworld-2019/schedule/grid
This would be useful for conferences that have either double-length workshops or half-length uncon talks running alongside a standard talk length.
Below is an example that I got my adding rowspans and deleting cells manually:

Hello! Can you assign this one for me? :-)

@lfbittencourt Done. When should I check in, if I haven't seen a PR from you by then?

@iansltx I'm working on it! I will send the PR as soon as I can.


Continue reorganization of Storybook

Could be nice if longer term we shifted more towards a more flattened hierarchy like Carbon’s, especially since our v2 does not include the CDAI and Security libraries.
Might look something like

Overview (welcome, examples)
Components
Patterns
Experimental (separate canary from stable)
Novice-to-pro / onboarding / PLG
Internal (not meant for global usage)
Community

Possibly include Hooks with usePrefix() someday ... or could be a way to rethink Datagrid structure, but also maybe not for now. 😅

Originally posted by @elycheea in #3986

https://github.com/carbon-design-system/ibm-products/pull/4534#issuecomment-1995402962

Carbon React Storybook currently does not use sentence case. We can keep it for now but should revisit as a topic with Taylor/Kenny later to see if how we want to structure this going forward.


Added Binary search

Added binary search implementation in Kotlin

Thanks so much for your contribution to The All ▲lgorithms Project.  You rock!


Draw test

This PR is aimed to test determining a game as a draw by the AutoRef. It relates to issue #13 .

This is the actual log of running this scenario:
[0000.001|0000.000] Error: Host is not correctly defined in game.json file, it should be <IP_ADDRESS> instead of <IP_ADDRESS>.
[0000.002|0000.000] Info: Real time factor is set to 0. Simulation will run as fast as possible, real time waiting times will be minimised.
[0001.080|0000.000] Info: Spawned RED_PLAYER_1 RobocupRobot on port 10001 at halfTimeStartingPose: translation (3.5 -3.06 0.24), rotation (0 0 1 1.571592653589793).
[0001.726|0000.000] Info: Spawned RED_PLAYER_2 RobocupRobot on port 10002 at halfTimeStartingPose: translation (3.5 3.06 0.24), rotation (0 0 1 4.711592653589793).
[0002.381|0000.000] Info: Spawned RED_PLAYER_3 RobocupRobot on port 10003 at halfTimeStartingPose: translation (0.75 -3.06 0.24), rotation (0 0 1 1.571592653589793).
[0002.924|0000.000] Info: Spawned RED_PLAYER_4 RobocupRobot on port 10004 at halfTimeStartingPose: translation (0.75 3.06 0.24), rotation (0 0 1 4.711592653589793).
[0003.463|0000.000] Info: Spawned BLUE_PLAYER_1 RobocupRobot on port 10021 at halfTimeStartingPose: translation (-3.5 -3.06 0.24), rotation (0 0 1 1.57).
[0004.009|0000.000] Info: Spawned BLUE_PLAYER_2 RobocupRobot on port 10022 at halfTimeStartingPose: translation (-3.5 3.06 0.24), rotation (0 0 1 -1.57).
[0004.578|0000.000] Info: Spawned BLUE_PLAYER_3 RobocupRobot on port 10023 at halfTimeStartingPose: translation (-0.75 -3.06 0.24), rotation (0 0 1 1.57).
[0004.582|0000.000] Info: Connected to GameControllerSimulator at localhost:8750.
[0004.582|0000.000] Info: Game type is NORMAL.
[0004.582|0000.000] Info: Red team is Funny Dingos.
[0004.582|0000.000] Info: Blue team is Agile Lynxes.
[0004.582|0000.000] Info: Left side is blue.
[0004.582|0000.000] Info: Kickoff is blue.
[0004.582|0000.000] Info: Beginning of first half.
[0004.582|0000.000] Info: Sending 1:SIDE_LEFT:2 to GameController.
[0004.582|0000.000] Info: Sending 2:KICKOFF:2 to GameController.
[0005.083|0000.216] Info: New state received from GameController: STATE_INITIAL.
[0007.639|0009.808] Info: Sending 1230:STATE:READY to GameController.
[0008.080|0012.352] Info: New state received from GameController: STATE_READY.
[0016.592|0058.224] Info: New state received from GameController: STATE_SET.
[0016.595|0058.224] Info: Sending 7283:PENALTY:1:1:INCAPABLE to GameController.
[0016.598|0058.224] Info: INCAPABLE penalty for red player 1: did not move to kickoff position. Sent to translation (3 3.11 0.24), rotation (0.0 0.0 1.0 4.713185307179587).
[0016.598|0058.224] Info: Sending 7284:PENALTY:1:2:INCAPABLE to GameController.
[0016.600|0058.224] Info: INCAPABLE penalty for red player 2: did not move to kickoff position. Sent to translation (4.2 3.11 0.24), rotation (0 0 1 4.711592653589793).
[0016.600|0058.224] Info: Sending 7285:PENALTY:1:3:INCAPABLE to GameController.
[0016.603|0058.224] Info: INCAPABLE penalty for red player 3: did not move to kickoff position. Sent to translation (3.6 3.11 0.24), rotation (0.0 0.0 1.0 4.713185307179587).
[0016.603|0058.224] Info: Sending 7286:PENALTY:1:4:INCAPABLE to GameController.
[0016.605|0058.224] Info: INCAPABLE penalty for red player 4: did not move to kickoff position. Sent to translation (2.4 3.11 0.24), rotation (0 0 1 4.711592653589793).
[0016.605|0058.224] Info: Sending 7287:PENALTY:2:1:INCAPABLE to GameController.
[0016.608|0058.224] Info: INCAPABLE penalty for blue player 1: did not move to kickoff position. Sent to translation (-3 3.11 0.24), rotation (0.0 0.0 1.0 4.711592653589793).
[0016.608|0058.224] Info: Sending 7288:PENALTY:2:2:INCAPABLE to GameController.
[0016.610|0058.224] Info: INCAPABLE penalty for blue player 2: did not move to kickoff position. Sent to translation (-4.2 3.11 0.24), rotation (0 0 1 -1.57).
[0016.610|0058.224] Info: Sending 7289:PENALTY:2:3:INCAPABLE to GameController.
[0016.613|0058.224] Info: INCAPABLE penalty for blue player 3: did not move to kickoff position. Sent to translation (-3.6 3.11 0.24), rotation (0.0 0.0 1.0 4.711592653589793).
[0019.907|0063.216] Info: Sending 7914:STATE:PLAY to GameController.
[0020.598|0066.016] Info: New state received from GameController: STATE_PLAYING.
[0023.103|0075.664] Info: Ball in play.
[0027.608|0094.712] Info: New state received from GameController: STATE_READY.
[0040.124|0141.112] Info: New state received from GameController: STATE_SET.
[0046.132|0167.544] Info: New state received from GameController: STATE_PLAYING.
[0048.635|0177.768] Info: Ball in play.
[0051.138|0188.640] Info: New state received from GameController: STATE_READY.
[0062.152|0235.208] Info: New state received from GameController: STATE_SET.
[0064.654|0245.856] Info: New state received from GameController: STATE_PLAYING.
[0067.158|0255.464] Info: Ball in play.
[0378.093|0584.160] Error: GameController sent "600 seconds remaining"!
[0602.885|0794.416] Info: Sending 99315:STATE:FINISH to GameController.
[0602.885|0794.416] Info: End of first half.
[0602.889|0794.416] Info: red player 1 reset to halfTimeStartingPose: translation (-3.5 -3.06 0.24), rotation (0 0 1 1.57).
[0602.891|0794.416] Info: red player 2 reset to halfTimeStartingPose: translation (-3.5 3.06 0.24), rotation (0 0 1 -1.5700000000000003).
[0602.894|0794.416] Info: red player 3 reset to halfTimeStartingPose: translation (-0.75 -3.06 0.24), rotation (0 0 1 1.57).
[0602.897|0794.416] Info: red player 4 reset to halfTimeStartingPose: translation (-0.75 3.06 0.24), rotation (0 0 1 -1.5700000000000003).
[0602.899|0794.416] Info: blue player 1 reset to halfTimeStartingPose: translation (3.5 -3.06 0.24), rotation (0 0 1 1.571592653589793).
[0602.901|0794.416] Info: blue player 2 reset to halfTimeStartingPose: translation (3.5 3.06 0.24), rotation (0 0 1 4.711592653589793).
[0602.903|0794.416] Info: blue player 3 reset to halfTimeStartingPose: translation (0.75 -3.06 0.24), rotation (0 0 1 1.571592653589793).
[0603.386|0794.576] Info: New state received from GameController: STATE_FINISHED.
[0603.387|0794.576] Info: Sending 99336:STATE:SECOND-HALF to GameController.
[0603.387|0794.576] Info: Beginning of second half.
[0603.888|0794.776] Info: New state received from GameController: STATE_INITIAL.
[0605.281|0795.968] Info: Sending 99511:STATE:READY to GameController.
[0605.889|0796.536] Info: New state received from GameController: STATE_READY.
[0642.937|0842.592] Info: New state received from GameController: STATE_SET.
[0642.937|0842.592] Info: Sending 105340:PENALTY:1:1:INCAPABLE to GameController.
[0642.940|0842.592] Info: INCAPABLE penalty for red player 1: did not move to kickoff position. Sent to translation (-3 3.11 0.24), rotation (0.0 0.0 1.0 4.711592653589793).
[0642.940|0842.592] Info: Sending 105341:PENALTY:1:2:INCAPABLE to GameController.
[0642.942|0842.592] Info: INCAPABLE penalty for red player 2: did not move to kickoff position. Sent to translation (-4.2 3.11 0.24), rotation (0 0 1 -1.5700000000000003).
[0642.942|0842.592] Info: Sending 105342:PENALTY:1:3:INCAPABLE to GameController.
[0642.944|0842.592] Info: INCAPABLE penalty for red player 3: did not move to kickoff position. Sent to translation (-3.6 3.11 0.24), rotation (0.0 0.0 1.0 4.711592653589793).
[0642.944|0842.592] Info: Sending 105343:PENALTY:1:4:INCAPABLE to GameController.
[0642.946|0842.592] Info: INCAPABLE penalty for red player 4: did not move to kickoff position. Sent to translation (-2.4 3.11 0.24), rotation (0 0 1 -1.5700000000000003).
[0642.946|0842.592] Info: Sending 105344:PENALTY:2:1:INCAPABLE to GameController.
[0642.948|0842.592] Info: INCAPABLE penalty for blue player 1: did not move to kickoff position. Sent to translation (3 3.11 0.24), rotation (0.0 0.0 1.0 4.713185307179587).
[0642.948|0842.592] Info: Sending 105345:PENALTY:2:2:INCAPABLE to GameController.
[0642.950|0842.592] Info: INCAPABLE penalty for blue player 2: did not move to kickoff position. Sent to translation (4.2 3.11 0.24), rotation (0 0 1 4.711592653589793).
[0642.950|0842.592] Info: Sending 105346:PENALTY:2:3:INCAPABLE to GameController.
[0642.952|0842.592] Info: INCAPABLE penalty for blue player 3: did not move to kickoff position. Sent to translation (3.6 3.11 0.24), rotation (0.0 0.0 1.0 4.713185307179587).
[0646.224|0847.584] Info: Sending 105971:STATE:PLAY to GameController.
[0646.941|0850.840] Info: New state received from GameController: STATE_PLAYING.
[0649.443|0861.800] Info: Ball in play.
[0788.611|1450.240] Info: Sending 181304:STATE:FINISH to GameController.
[0788.611|1450.240] Info: End of second half.
[0789.110|1452.184] Info: New state received from GameController: STATE_FINISHED.
[0789.111|1452.184] Info: End of the game.
[0789.111|1452.184] Info: The score is 1-1.
[0789.111|1452.184] Info: This is a draw.

Same comment here as for #49.


Error message Convert error! struct data not recognized as molecule, query, reaction or reaction query on opening structure with atoms that have (0,0) coordinates

Steps to Reproduce
1.Open files with  structure with atoms that have (0,0) coordinates
files.zip
Expected behavior
File is opened, Structure is placed to the canvas
Actual behavior
Error is shown
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser Chrome Version 93.0.4577.63, Firefox Version 94.0.1, Microsoft Edge Version 93.0.961.47

Related issue:
https://github.com/epam/Indigo/issues/581

After investigation from backend side: - "Attached mol-files are corrupted. Chemaxon is unable to load them as well." and re-checking from the front side the issue should be closed.

waiting for indigo update


The share/embed popup is styled incorrectly

Current:

Mock:

@kravets-levko the embed is now correct, but the "Link" section should still go to the main page (not the embedded able graph)


Normalizing field values causes the caret to jump to the end of the input

Normalizing field values causes the caret to jump to the end of the input
When Normalizing the value of a field it causes the caret/ cursor to jump to the end of the input. This is frustrating if you want to amend a characters after entering an initial value because as soon as the user starts to type the caret immediately jumps to the end of the input.
Your environment
Using 8.2.2
Steps to reproduce
You can see this bug in the Normalizartion example in the redux-form docs.
https://redux-form.com/8.2.2/examples/normalizing/

Type a value into either the phone or "shout" field
Try and amend the first two characters you entered into that field
After typing the first character, the caret/ cursor jumps to the end of the input

Expected behaviour
The caret will not jump to the end of the input
Actual behaviour
The caret jumps to end of the input after changing or adding a new value to the input.

It seems this issues has been reported a few times in the past but has been continuously closed with no feedback. Lastest example can be found here: https://github.com/redux-form/redux-form/issues/4480

Hmm, strange behaviour. If you look in the example first two fields are almost the same, but jumping only happens on second field. On first field cannot reproduce.

It does happen on the first field (username) but only if you try and type and uppercase character into that field.

It seems this issues has been reported a few times in the past but has been continuously closed with no feedback. Lastest example can be found here: #4480

I got closed but not because of the feedback

One advice..do not use that lib use it is trash.

That's rude. Better to say.

One advice..do not use this lib, it has a lot of issues.

Well, a bit, sorry to say but that's not my word. Many people have lost a lot of time because of that lib, the author of redux-form itself recommend to use the redux-final-form (which he also wrote) because of all the issues. You should definitely not use it, and the author judged this library should be thrown to trash in profit of final-form.

We are using this library almost two years in production, and don't bother about small issues :slightly_smiling_face:
The main generator of the issues, is because this library tried to make every user happy, and added a lot of fatures.

I strongly advise you to migrate your form to use final-form, this will drastically improve your application performance.

this will drastically improve your application performance.

I don't understand what kind of forms people have, that they struggle about performance. :smile:
redux-form immutablejs :eyes:

I strongly advise you to migrate
I have a dream to rewrite redux-form@v9 with hooks and drop immutablejs support

After some investigation, this behaviour will happen also without redux-form. Is the way React and DOM works. redux-form can't fix this issue because we don't know what exactly happened with the value.
Here is a sandbox with the issue
https://codesandbox.io/s/react-cursor-jump-wmqp1
Here is a sandbox with an attemp to restore cursor
https://codesandbox.io/s/react-cursor-jump-fixed-zh5lp

https://github.com/facebook/react/issues/955

Solution would be to use a react input mask library that takes care of cursor restoring


Auto-start with "Unstick from corners" only

Hi there,
thank you very much for this application. I use 4 monitors and I was getting exhausted with the sticky corners when moving between monitors.
I just have a quick feature request. is it possible to auto-start with just "Unstick from corners".

Many thanks.

Hi @dookei, glad you're enjoying it! This is actually already implemented but not yet released (and also needs to be documented, see #11 for that). Basically the feature is the retention of the settings (i.e. those checked options) on next app start.
I do have an unofficial build on my own fork that merges in the latest code and includes pull requests that I have waiting to be merged here if you'd like to run with that for now until a new official version is released (probably v2.2). https://github.com/patricknelson/MouseUnSnag/releases

Ok v2.2 is now released! 😊  You can check it out here: https://github.com/MouseUnSnag/MouseUnSnag/releases


Iterating through DistributedDataset created from dataset with raggedTensor raises InvalidArgumentError

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04.5
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): dockerhub 2.5.0-gpu
TensorFlow version (use command below): 2.5.0
Python version: 3.6.9
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: cuda_11.2
GPU model and memory:  4 * Tesla T4  with 15109MiB memory

Describe the current behavior
After creating a DistributedDataset instances from dataset with raggedTensor elements inside, I tried to iterate through it
but InvalidArgumentError raised.  What' more, I found that only raggedTensor with string dtype has this problem and int raggedtenor works fine.
Describe the expected behavior
Contributing

Do you want to contribute a PR? (yes/no): no
Briefly describe your candidate solution(if contributing):

Standalone code to reproduce the issue
strategy = tf.distribute.MirroredStrategy(["GPU:0", "GPU:1"])

# it works
ragged_dict = {'ragged':tf.ragged.constant([[3, 1, 4, 1], [], [5, 9, 2], [6], [], [3, 1, 4, 1], [3, 1], [2, 1, 4, 1]])}
ragged_ds = tf.data.Dataset.from_tensor_slices(ragged_dict ).batch(2)
dist_dataset = strategy.experimental_distribute_dataset(ragged_ds)
ds = iter(dist_dataset)
next(ds)

# it doesn't work
ragged_dict = {'ragged' : tf.ragged.constant([['3', '1', '4', '1'], [], ['5', '9', '2'], ['6'], [], ['3', '1', '4', '1'], ['3', '1'], ['2', '1', '4', '1']])}
ragged_ds = tf.data.Dataset.from_tensor_slices(ragged_dict).batch(2)
dist_dataset = strategy.experimental_distribute_dataset(ragged_ds)
ds = iter(dist_dataset)
next(ds)

Other info / logs
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/distribute/input_lib.py", line 686, in __next__
    return self.get_next()
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/distribute/input_lib.py", line 730, in get_next
    self, self._strategy, return_per_replica=False)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/distribute/input_lib.py", line 604, in _get_next_as_optional
    iterator._iterators[i].get_next_as_list())  # pylint: disable=protected-access
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/distribute/input_lib.py", line 1948, in get_next_as_list
    self._iterator.get_next_as_optional())
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/data/ops/multi_device_iterator_ops.py", line 604, in get_next_as_optional
    result.append(self._device_iterators[i].get_next_as_optional())
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/data/ops/iterator_ops.py", line 825, in get_next_as_optional
    self.element_spec)), self.element_spec)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/gen_dataset_ops.py", line 2809, in iterator_get_next_as_optional
    _ops.raise_from_not_ok_status(e, name)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 6897, in raise_from_not_ok_status
    six.raise_from(core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError: 2 root error(s) found.
  (0) Invalid argument: During Variant Host->Device Copy: non-DMA-copy attempted of tensor type: string
  (1) Invalid argument: During Variant Host->Device Copy: non-DMA-copy attempted of tensor type: string
0 successful operations.
0 derived errors ignored.
         [[{{node RemoteCall}}]] [Op:IteratorGetNextAsOptional]

@kenyonke
I was able to reproduce the code shared in tf-2.5 and tf-nightly versions  with no errors.Please find the gist here.Thanks

@kenyonke
I was able to reproduce the code shared in tf-2.5 and tf-nightly versions with no errors.Please find the gist here.Thanks

Hi @UsharaniPagadala , it sitll doesn't work on my equipment. I have no idea about this problem. I work on tensorflow docker with Tesla T4 Gpu so I think the environment is fine.

@kenyonke
Looking at the error log it is similar to  #28007  and #71 and let us know if it helps.Thanks

@kenyonke
Looking at the error log it is similar to #28007 and #71 and let us know if it helps.Thanks

You need to run this code on gpus, it works on cpu or single gpu. Colab doesn't use multiple gpu even if you add
strategy = tf.distribute.MirroredStrategy(["GPU:0", "GPU:1"])
You can see this link
https://colab.research.google.com/drive/1tEpZ1TQxyfh-Mz5uDPPXy3RDnX4_yCBM?usp=sharing

@kenyonke
I was not able to access the gist provided,Could you Please provide the access.Thanks

@UsharaniPagadala
Sorry, now you should have right of access
https://colab.research.google.com/drive/1tEpZ1TQxyfh-Mz5uDPPXy3RDnX4_yCBM?usp=sharing
I think it's necessary to check whether gpu is available before mirroredstrategy

@Saduf2019
I  was able to replicate the issue reported here.Please find the gist.Thanks

After investigation, commits https://github.com/tensorflow/tensorflow/commit/55a1146af14d38bcad9afe3c3aa392e94c5e07cc and https://github.com/tensorflow/tensorflow/commit/9418c87eaa8a927fbdec2b09fbde1ca461e2579c (and related supporting commits) seem sufficient to addressed expected use cases. Some corner cases around pure eager execution may remain.
Please re-open this bug if the same symptom re-occurs or create a new bug if there's a failure with a different symptom.

PS. Please see  https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/ragged/ragged_factory_ops_test.py for examples of working cases of ragged tensors with DistributedDataset.


[radio] Add API for resetting CSL parameters

Add ResetCslParameters and otPlatRadioResetCslParameters.
This allows handling stack reset as separate call instead of using kShortAddrInvalid as otShortAddress and
nullptr as otExtAddress which can be difficult to handle on vendor's side.
Default implementation still calls
otPlatRadioEnableCsl(aInstance,0, Mac::kShortAddrInvalid, nullptr) So no action is needed if vendor has already implemented handling this case.

CC @canisLupus1313

Hi @maciejbaczmanski, I'd like to understand it more clearly. What's the difficulty for vendor code to handle kShortAddrInvalid for otShortAddress and nulltpr and otExtAddress?

Hi @maciejbaczmanski, I'd like to understand it more clearly. What's the difficulty for vendor code to handle kShortAddrInvalid for otShortAddress and nulltpr forotExtAddress?

Hi @Irving-cl . nulltpr is the main issue for us, as on Radio Driver layer, Ext Address is checked if it is not NULL to prevent handling pointer incorrectly. For current implementation we would need to consider a special case, which would accept NULL pointer and it doesn't sound very secure.
Also, EnableCsl and otPlatRadioEnableCsl in documentation don't specify the case of resetting the values, they only specify setting them (with no explanation of the special case described before in particular). We are working on separate APIs in Radio Driver to purge ENH_ACK header IE on stack reset, and would like it to be handled separately (preventing accidental purging while calling functions that should set the value).
IMO this change makes the code clearer as well as it explicitly shows the case when we should clear parameters as part of clearing the stack on RCP reset and separate it from the case where we should just set the values. Default weak implementation makes sure that other vendors can, but don't have to implement separate handler for ResetCslParameters as  well.


Add create topics support

This is a first attempt at adding support for the create topics request. I have been wanting this for a while since I work on some "management" tools for Kafka. Most other Kafka clients in other languages also seem to rely on the "metadata" solution.
This supports specifying number of partitions and replication factor. Explicit replica assignments and config entries are left out for now.
A create topic request must be sent to the controller. To find the current controller, this also adds a version 1 metadata request, which includes the controller Id in the response.
There are a couple of things that are less-than-optimal and could use some input for improvement:

Added a "V2" suffix to resolve the conflict with current versions of createTopics(..) and loadMetadataForTopics(..).
KafkaClient includes these new functions, but the Admin constructor may be more appropriate for create topics, if this is intended to host more admin functionality in the future (would also resolve the createTopics(..) conflict on the client)?
The metadata structure returned from loadMetadataForTopics(..) could use a different structure. The first item in the array is an object with all the broker nodes. Second item is an object with a metadata property containing the topics. I've added the clusterMetadata property here, to avoid breaking the existing structure.
As a follow up to this, I've left the general handling of meta data load/refresh, but the new one could replace the existing one as I see it, so controller id is always available.

Let me know of anything we need to change before this can hopefully get merged in.
Once this is wrapped up, I have delete topics request ready next, and will then add support for config entries on creation of topics.

I am looking into the build issue with Kafka 0.9 (metadata version 1 was added in 0.10)

Updated PR with a few changes to handle clusters of version < 0.10.
Will now return an error if a controller is not found + tests are skipped for version 0.9.
Let me know what you think @hyperlink

Bunch of changes are in:

Removed WIP parts (log, delete topics request encoder/decoder - will come in later PR with the full feature)
For version 1+ calls of metadata, [] values are sent as null to keep client behaviour of fetching all topics
Both createTopics and loadMetadataForTopics had their V2 suffix removed, so they replace the original implementations on Client.
I had to add support for existing calls to createTopic - if an array of strings is detected as input, the call is forwarded to original Client implementation. Lot's of tests depend on this, so rather than risking blowing it all up with this change, this will keep backwards compatibility for now.
loadMetadataForTopics on KafkaClient now uses getSupportedForRequestType (please check this bit, I added additional logic to make ensure broker is ready and the API version support is resolved).
Added loadMetadata function that refreshes metadata without topics.
Make sure of createErrorGroup when errors are detected in decoder*
Replaced README part on createTopics with new function documentation (we can also have both version described?)

note for the createTopics decoder, I distinguish between two types of errors. Overall request errors and individual topic creation errors. Overall request errors are returned as an instance of Error (e.g. timeout or the request somehow got sent to a non-controller). Topic errors are returned as a normal result with the error message provided by Kafka (e.g. setting a replication factor higher than available brokers).

Updated with a sendControllerRequest method on KafkaClient. It wraps the provided callback with logic that handles a NotControllerError error. In this case the cached controller id is removed and the request is redone (based on the getController method that will then be forced to fetch latest metadata).

I'll merge once conflicts are resolved. thanks @jlandersen

Rebased on master and all checks out! Thanks! Looking forward to adding more things with the controller request in place!

Published as 3.0.0


No CPU capping in test zones

Try without CPU capping to see if it helps with first query after start taking a long time.

If this works we need to followup somehow since staging tests should support performance validation.

This value is already controlled by the container-cpu-cap feature flag, except for the method changed in this PR which bypasses the flag entirely. In this case I'd recommend just using the flag, then change the code if you want it to be permanent.
Though ideally we should remove the method and always use the feature flag.

Nice, then we can control this per app in staging and this will have no impact on other apps. I'll try to remove the method and always use the flag

https://github.com/vespa-engine/vespa/pull/27343 changes this to only use the feature flag


enable LoRA + FSDP2

how to run it

pytorch nightly after 05/08 with https://github.com/pytorch/pytorch/pull/125339
7B: tune download meta-llama/Llama-2-7b-hf --output-dir /tmp/Llama-2-7b-hf --hf-token <HF_TOKEN> && tune run --nproc_per_node 8 lora_finetune_distributed --config recipes/configs/llama2/7B_lora.yaml
70B: tune download meta-llama/Llama-2-70b-hf --output-dir /tmp/Llama-2-70b-hf --hf-token <HF_TOKEN> && tune run --nproc_per_node 8 lora_finetune_distributed --config recipes/configs/llama2/70B_lora.yaml

recipe tests: pytest tests/recipes/test_lora_finetune_distributed.py -m integration_test
unit test: pytest tests/torchtune/utils/test_distributed.py -k test_state_dict
FSDP2 changes

[x] init models on device=meta instead of cpu: 4.5 x speedup from 270s to 60s
[x] activation checkpointing happens before register FSDP hooks

checkpoint changes

[x] loading: load HF checkpoint on rank0 and broadcast to other ranks
[x] saving: gather full state dict on rank0 from distributed state dict
[ ] resume finetuning from previous checkpoints using rank0 load and broadcast

optional

[ ] foreach=True for adam + DTensor through optimizer.foreach=True since pytorch PR have not land yet https://github.com/pytorch/pytorch/pull/123394

Codecov Report
Attention: Patch coverage is 10.37736% with 95 lines in your changes are missing coverage. Please review.

Project coverage is 64.60%. Comparing base (a978956) to head (672aabb).
Report is 27 commits behind head on main.

Files
Patch %
Lines

torchtune/utils/_distributed.py
14.75%
52 Missing :warning:

recipes/lora_finetune_distributed.py
0.00%
42 Missing :warning:

torchtune/utils/_profiler.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #855      +/-   ##
==========================================
- Coverage   66.39%   64.60%   -1.80%
==========================================
  Files         155      171      +16
  Lines        6484     7153     +669
==========================================
+ Hits         4305     4621     +316
- Misses       2179     2532     +353

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

I see meaningful numeric difference between this PR and torchtune trunk. will debug it before publishing

synced with @kartikayk on action items

add docstring for meta init, state dict, migration to DCP
place FSDP2 in receipes/dev/
attach wandb loss curve
pending feedbacks on unit tests


Update extensionsGallery.json for ML 0.8.0

Extension file is already uploaded and used in Insider marketplace

@cssuh Do you have time to update the gallery?

yes I can do that

Gallery updated!


build: add nix build/dev environment

This introduces a Nix flake, which provides:

A Nix devShell with the Rust toolchain and formatting tools,
etc.
Nix instructions for building the package
(by overriding the nixpkgs ttags derivation).
Checks:

Formatting.
Version check (which runs ttags --version and validates
the output).

Plus it adds a packaging status badge to the readme.

@npezza93 which version of rustfmt are you using? When I run version 1.6.0, it formats a bunch of files (which is why I have it disabled in the nix flake).

rustfmt 1.6.0-nightly (e0d7ed1f 2023-10-01)
Not a huge fan of adding the extra builder


Implement an EVERYONE principal in WebACAuthorizationDelegate.

Name of the principal is the string value of the URI for foaf:Agent

Looks good, pending response to code review comment:
https://github.com/fcrepo4-labs/fcrepo-module-auth-webac/pull/11#discussion_r38455464

:+1: pending @awoods review comment.

:+1:

Merged via 6a6b0ff293eac5e9a14ae62733db31e7c3204af2


Increase Emscripten stack size (+options update)

Default stack was decreased in https://github.com/emscripten-core/emscripten/commit/157fcd4650161d9e0261966e2d3b529d62e1babc. 64k results in screen corruption in even the most minimal example, 256k is enough to fix this but increase to 1MB to be safe. (Old default was 5MB)
Also clean up to use the preferred -sTHING= syntax and remove the EXTRA_ from EXPORTED_RUNTIME_METHODS.

Thank you!
I went to grab a zip of build artifacts to try- force of habit from Pico- but there are none :cry: I don't think there's a reason for this? I'll raise a PR to add them when I get a moment.
Built it locally- taking an opportunity to test the guide- and it all looks good!

You can just add install(FILES ... instructions to CMake to get it to add in the extra bits though (I think I've done this in Another Project and it seemed to work)

You can do that... and I forgot that we already are:
https://github.com/32blit/32blit-sdk/blob/3c6de84d4dac618246babc86106085a6457d4fea/32blit-sdl/CMakeLists.txt#L187-L189
So uh, ignore that comment.


[MacosAlertDialog] Fix incorrect suppress placement

Per updated guidelines from Apple, the suppress widget should be placed above the buttons:

The official macOS 12 Sketch library has the suppress option below the buttons, like we've done. So I think we don't need to make any changes.


Move the CF tweaks from nav-secondary

Looks like nav-secondary includes some tweaks to Capital Framework that should make their way up to that project.

Since the secondary nav has been completely re-written to use the atomic expandable, I'm pretty sure this can be closed. Feel free to re-open if I'm wrong.


Handle simultaneous keyboard and mouse streams - Alternative 2

Alternative to PR #184 for fixing #182.
Two main changes to master branch:

Pulse interrupt lines to ensure rising edge.
Supports for pulse:

Under Interrupt Processing, paragraph 2, from here
Paragraph 1 from here

Against pulse:

Under Reading Keyboard Input, paragraph 2, from here. It says that the interrupt is cleared after reading port 0x60. It might be referring to AT controllers instead of PS/2.

Before data in port 0x60 gets read, don't switch between kbd and mouse queues.
In hardware terms: inhibit keyboard and mouse communication and don't overwrite the DBBOUT output buffer until OBF gets cleared.
Useful refs:

Under Reading Keyboard Input, paragraph 1, from here
Under Main Entry, line 0102, from here
VirtualBox behaves this way and Kolibri OS works fine on it.
Qemu doesn't seem to behave this way. Mouse can jump around and hang in Kolibri OS running on qemu due to the keyboard controller giving keyboard values to the mouse driver.
Testing KolibriOS on real machine: works :ok_hand:.

Difference between this PR and PR #184:

Keyboard data gets priority over mouse data
Works without timers! :)

Thanks! This looks good to me, I tested it and it fixes the problem.


Fixes from static-analysis tools.

Makes sense. I thought the findbugs plugin is already setup in the pom.xml. So maybe just a matter to run the right goal.

I've rolled back the changes regarding exceptions and I'll start a new pull request. Sorry about that, I thought it was a safe refactoring.
Most of these static analysis rules come from IntelliJ community edition, so they're not easy to run in an automated way unless your build tool is TeamCity. CheckStyle has a few rules in common, like "sort modifiers" but not many.