[Breaking] Remove the "pre-commit hooks" and "update manifests" features

Closes #111
Closes #170
Closes #172
Closes #194
Closes #213
Summary
This pull request makes the following changes:

Remove the "pre-commit hook" functionality.
Remove the "update manifests" functionality.
Bump the version number from 1.x.y to 2.0.0, because this is a breaking change.

Motivation
There are multiple reasons why I want to make this change:

Security concerns, e.g. #213. In order to update a manifest, we call Pkg.update. This means that package build code gets run as part of the CompatHelper job. In theory, a malicious package could add some build code that tries to steal secure environment variables such as GITHUB_TOKEN.
There are other (better) options for people that want to update their manifests. I won't make an exhaustive list here, but perhaps someone could make a Discourse post of all of the manifest-updater GitHub Actions.
It is out of scope of the original intention of CompatHelper. The original goal of CompatHelper was simply to bump [compat] entries in the Project.toml file when your dependencies make new breaking releases. In the spirit of "do one thing, and do it well", I'd like to focus on this core mission.
It causes CompatHelper to break if there is a "bad" Manifest.toml file in the repo. This can be quite confusing to users - why should CompatHelper break due to an unrelated Manifest.toml file?

Future
I am not opposed to adding this functionality back, but we would need to fix the security first. For example, maybe we would run the Pkg.update inside some kind of secure sandbox that has restricted privileges and does not have access to any secure environment variables  such as GITHUB_TOKEN.

bors try

bors try

bors merge

Until https://github.com/JuliaRegistries/CompatHelper.jl/pull/282 the suggested workflow wasn’t versioned, so I’d be pretty careful about breaking changes! It seems like this doesn’t break that workflow though, only custom ones, is that right?

Yeah, it will only break custom workflows that pass a custom precommit hook.
If people still need the precommit hook functionality, they will need to use CompatHelper v1.
I'll make a Discourse post asking people to update the CompatHelper workflow files.

There are other (better) options for people that want to update their manifests. I won't make an exhaustive list here, but perhaps someone could make a Discourse post of all of the manifest-updater GitHub Actions.

It is out of scope of the original intention of CompatHelper. The original goal of CompatHelper was simply to bump [compat] entries in the Project.toml file when your dependencies make new breaking releases. In the spirit of "do one thing, and do it well", I'd like to focus on this core mission.

re. these two points: in CUDA.jl I have the Manifest committed, and CI uses it. If CompatHelper now bumps the compat entries without updating the Manifest, I can't be sure that bump is safe because the CI run on that PR used an older Manifest. Sure, there's separate actions that update the Manifest, but that then happens in a different PR, so doesn't help here. Or am I missing something?

The short answer is that you can still use CompatHelper v1. Just copy the workflow file from https://github.com/JuliaRegistries/CompatHelper.jl/blob/master/.github/workflows/CompatHelper.yml, and change the line that looks like version = "2" to be version = "1" instead.
The long answer has a couple different parts:

Even if you have a manifest checked in to source control, and even if CompatHelper updates the manifest, this does not guarantee that your test suite will run with the updated versions of dependencies. For example, suppose that your [compat] entry is Foo = "0.1, 0.2". And suppose that the Manifest.toml file uses Foo version 0.2.0. This does not guarantee that Pkg.test will use Foo version 0.2.0. When you run Pkg.test, Pkg is allowed to re-resolve your dependencies. Therefore, Pkg.test is still allowed to use Foo version 0.1.0. More discussion is available in https://github.com/JuliaLang/Pkg.jl/pull/2439 and https://github.com/JuliaLang/Pkg.jl/pull/2176. This issue has been solved on GitHub Actions; see https://github.com/JuliaRegistries/CompatHelper.jl/issues/160 for details of the fix. However, if you don't use GitHub Actions, you won't get this fix. At some point, we should think about adding this fix to Buildkite - basically, we would need to implement https://github.com/julia-actions/julia-runtest/pull/20 for Buildkite.
Honestly, I would be happy to add the "update manifests" feature back to CompatHelper. However, we need to figure out a secure way to do it. For example, when we run the Pkg.update function, we need to make sure of the following:

The process that runs Pkg.update does not have access to the GITHUB_TOKEN or COMPATHELPER_PRIV secure environment variables. Since processes can access the environment variables of any other process started by the same user, this also means that we need to somehow run this process under a different user.
The process that runs Pkg.update does not have access to SSH private keys, e.g. it doesn't have access to the $HOME/.ssh directory.

If we can get https://github.com/JuliaLang/Pkg.jl/pull/2504 merged, we can add the "update manifests" feature back to CompatHelper.


Fix Comment Formatting

I think it looks neater when addresses are all in monospace formatting. Make sure that addresses are always monospaced.
Some recent examples where its normal text formatting:

https://github.com/ubiquity/bounty-bot/issues/146#issuecomment-1451355192
https://github.com/ubiquity/bounty-bot/issues/148#issuecomment-1452008402

An exception for the claim button comment formatting: the address should be monospace AND truncated

https://github.com/ubiquity/ubiquity-dollar/issues/592#issuecomment-1452238250

Oh @Steveantor you forgot to use the /assign command

Hey @0xcodercrane how can I have the bot generate a payment permit? I thought the conditions were correct here (unless @Steveantor has not yet registered their wallet address?)

/help

Oh @Steveantor you forgot to use the /assign command for automatic payouts.

Yeah wasn't around for a quite while @pavlovcik

/help

/wallet 0x4841e8153a7b9E8B1F218E42d3cBaEb3e99C28eE

lol

@0xcodercrane please look into this

https://etherscan.io/tx/0xc5b0909d0194fc511042dd63a60791444d3a7a9b327bca7cc32536dd6ccd0182


Devcontainer

I ran some experiments with devcontainers in hopes of a better localhost Docker developer experience.  It's not perfect, but it shows promise.
In my experiments, it is currently functional, but too slow to be usable (full test cycle is 10x longer, running basic terminal commands like ls can take seconds to complete).
One of the recommended performance tips is to keep the container completely isolated -- it lives only in Docker, no bind mounts to the host file system.
That might be a "2 steps forward, 1 step back" solution.  All development happens in Docker, but to do anything useful it requires VS Code and the Remote Development VS Code extension.
Filing this issue to track the experiment.
Links:
Developing inside a container
https://code.visualstudio.com/docs/remote/containers
Containers tutorial
https://code.visualstudio.com/docs/remote/containers-tutorial
Performance tips
https://code.visualstudio.com/remote/advancedcontainers/improve-performance
Non-root user
https://code.visualstudio.com/remote/advancedcontainers/add-nonroot-user
JavaScript and Postgres example
https://github.com/microsoft/vscode-dev-containers/tree/main/containers/javascript-node-postgres

When running 100% in a devcontainer, performance is much better.  Delightful actually.
Requirements:

Good experience for VS Code devcontainers
Good experience for Docker command line users

VS Code devcontainer
Prerequisites
Install Docker
https://docs.docker.com/get-docker/
Install VS Code
https://code.visualstudio.com/
Install the "Remote Containers" extension:
https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.remote-containers
Start the devcontainer
Open VS Code
Open the "Command Palette"

Choose "Remote Containers: Clone Repository in Container Volume..."

When prompted for a git repo, enter "medplum/medplum" (you can use "codyebberson/medplum" for now)

You should see "Starting Dev Container" in the lower right corner

In the background, it is doing the following:

Starting a Docker container for development
Starting a Docker container for Postgres for development
Starting a Docker container for Postgres for tests
Starting a Docker container for Redis for development

This can take a few minutes.
Run a build
When the devcontainer is running, you should be prompted with a terminal:

Run ./scripts/build.sh to run a build.  This can take a few minutes.
Run the API server
Click the "+" button to open a new terminal:

Start the API server:
cd packages/server
npm run dev

Run the front end server
Click the "+" button to open a new terminal:

Start the webpack dev server:
cd packages/app
npm run dev

You will receive a notification that the application is running.  You can either click "Open in Brower", or manually open http://localhost:3000:

Tried this and encountered the following issues:
(1) Asked me to "Create a unique volume" or a "new volume" and I didn't know what to choose

(2) Got this terminal error

@reshmakh Thanks for trying!

I'll add instructions to choose "Create a unique volume", although I'm not sure why that dialog appears

I need to research more, but it appears to be related to how bind mounts work:
https://github.com/microsoft/vscode-remote-release/issues/5808
https://github.com/microsoft/vscode-remote-release/issues/5388

Abandoned


tippecanoe: 1.35.0 -> 1.36.0

Motivation for this change
Changelog
Things done

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

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

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

WFM on macos 10.14. Wondering what the aarch64 failure is about though.
@ofborg build tippecanoe

Yup, seems to persist.

I have no idea what is going wrong on aarch64... What is better way to proceed? Mark as not supported on aarch64?

Sad, but I guess it's the best thing to do. Might be worth opening a ticket upstream letting them know that their coalesce-tract test may have issues on non-x86.
It doesn't help that:

They appear to be doing bytewise comparisons of json output, which will obviously be unnecessarily sensitive to e.g. object key order.
We only have access to ofborg for aarch64, and can't inspect its filesystem after failure.

Sad, but I guess it's the best thing to do. Might be worth opening a ticket upstream letting them know that their coalesce-tract test may have issues on non-x86.

marked as broken on aarch64
upstream issue https://github.com/mapbox/tippecanoe/issues/881

@risicle I've just tried to build tippecanoe on my aarch64 machine and it works just fine. Can we unbroke it on aarch64 and perhaps disable this test on aarch64 to not fail on CI?

Having worked a bit more with aarch64 packages now, there are quite a few which have slight disagreements in float precision with "expected" results that were presumably generated on x86. I think this is due to the existence of the x87 80bit extended precision mode.
Short answer: yes, but I would raise the issue upstream too if it isn't already.

Short answer: yes, but I would raise the issue upstream too if it isn't already.

Here is upstream issue: https://github.com/mapbox/tippecanoe/issues/881.


Large delay before displaying weather data

I can set the weather data by calling Settings.System.putString(context.getApplicationContext ().getContentResolver (), "WeatherInfo", jsonData) and these data are instantly displayed by standard clock widgets: a weather widget and an information screen that slides out from the top of the main screen.
However, GreatFit displays new data with a large delay. Unlocking the screen does not help, the data is not updated.
Inside the code, 'WatchDataListener.onDataUpdate` is not called by the host for some reason.
I tried to call ISpringBoardHostStub.postDataToProvider(this, 8, new Bundle ()) from my plugin, after calling System.putString but unsuccessfully, WatchDataListener.onDataUpdate is still not called.
Then I added ContentObserver to catch changes in Settings.System and call WatchDataListener.onDataUpdate from there - the weather data began to be updated, but only on the unlocked screen, on the locked one is still not updated.
Even if you unlock the screen and see new data - after locking the screen, old data will still be shown.
How can I force to weather data on locked screen to be updated?
Please take a look on my PR

The observer for the watchfaces is registered on "WeatherCheckedSummary" and not on "WeatherInfo" (take a look at com.huami.watch.watchface.AbstractWatchFace in the libs). This probably why you change doesn't trigger an update.
Try updating that value too... I will also try to test it...

I can confirm that updating WeatherCheckSummary instantly refreshes GreatFit. The same way stock watchfaces work, thus I would suggest updating that system values too.

Great, the use of WeatherCheckSummary really works! Thank you. I will then cancel the PR and update my weather plugin.


REG_EXPAND_SZ values are expanded in SYSTEM user's context

I found an issue which is unfortunately not easy to fix (research told me that with WMI it is actually impossible to fix). I still want to point it out. It is actually a duplicate of #6 but that issue was closed due to inactivity without further analysis.
When reading a REG_EXPAND_SZ value, I would expect it to be expanded with the current process' environment variables. For example, when reading the value Desktop from HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Explorer\User Shell Folders, whose raw value is %USERPROFILE%\Desktop, I would expect to get C:\Users\david\Desktop in my case.
Unfortunately, the string is expanded by the WMI service in its own context, using the environment variables of the service's process which runs under the SYSTEM account. This leads to unexpected behavior because in my example above I get C:\WINDOWS\system32\config\systemprofile\Desktop returned (which doesn't even exist because the system account doesn't need a desktop).
As a very dirty workaround, I currently check whether a value I read is of type REG_EXPAND_SZ and if it is, I do a .replace(process.env.SYSTEMROOT + '\\system32\\config\\systemprofile', process.env.USERPROFILE) on it and cross my fingers - this works because most of the expandable registry strings I'll encounter will have something to do with a user profile path, and luckily variables like LOCALAPPDATA exist for the system account too, just as C:\WINDOWS\system32\config\systemprofile\AppData\Local et al. so replacing the profile path part works.
This issue seems to be impossible to fix - the straight-forward strategy of reading the raw value and manually replacing the environment variables is not doable because WMI expands the string even when GetStringValue (and not GetExpandedStringValue) is used, for some reason, which makes it impossible to obtain the raw value before it is expanded.
The only possible way I could think of is to use WScript.Shell's RegRead method (which has different semantics unfortunately, and where 32-bit/64-bit translation issues must be taken care of manually) for REG_EXPAND_SZ because this method does not expand the string, so it can be fetched in its raw format, and then environment variables could be replaced manually (or not at all, maybe with an option).

@CherryDT TYVM for the investigation. I will go deeper on this during the upcoming weekend

@nhz-io any news here?

This problem is now addressed with listUnexpandedValues


Let one character search effective

On the tag search box, one character search seems to be ignored. But many Japanese (also Chinese) characters have enough meanings by one character. Please make it effective. It's very important for quick mapping.
ex. "酒"(alcohol), "木"(tree), "店"(shop), "道"(street)...

Done in 3.1.


New Module : Windows Path interactive

Description of the new feature / enhancement
Hello
I want to create a new module with this 4 firsts features :

Open a file or its parent folder from a string path
Past the file/folder path from files copied in the clipboard
Past a file or folder inside the windows explorer if a string path are present in the clipboard
In the explorer, send Ctrl+V with some text or bitmap inside the clipboard will create a file and fill it with the clipboard content

Scenario when this would be used?
Open a file or its parent folder from a string path
Imagine this log file :
Error : C:\fold\filename1.ext has not the right syntax
Error : //server/dir1/dir2/randomFileName.ext is missing

I select the first line (from 'Error' to 'syntax'), and I send "Win + Enter" :

PowerToy copy the selected txt
PowerToy found the windows path inside the txt (I already write a Regex for that : https://regex101.com/r/zWGLMP/17 )
If path exist :

PowerToy open the file with his default software

else if path does not exist :

PowerToy warn the user that the path does not exist, and offer to open in explorer the first valid parent directory

We can add some variation :

If PowerToy found more than one path, he ask user wich one he want to open, or if he want to open all.
If user do Win+Alt+Enter, PowerToy open the parent path in explorer rather than open it in default software

Open a file or its parent folder from a windows control
We can do the same with all windows control who hook a Ctrl+C.
Imagine a software who run a MessageBox who say "look at this log file : C:\a\random\path\what\Im\too\lazy\to\find\log.txt".
You just have to send Win+Enter, and PowerToy can get the path with Ctrl+C, parse the path and open it !
Past the file/folder path from files copied in the clipboard
Imagine you edit an .ini or .cfg file. You want to set a path to paramFile :
[ParamSection]
paramFile = //randomServer/random/path/that/im/tired/to/get/inputFile.ext

So, to simply add the path, there is already Maj+RightClick from the explorer who offer to copy the path, and past it inside the .ini file with your favorite editor...
But I want more ergonomic way : I want, when I have a file or a dir copied in my clipboard, past the path when I send Ctrl+V !
We just have to add an exception for the explorer and dialog box like "Save As" or "Open..."
Copy a file or folder inside the windows explorer if a string path are present in the clipboard
Imagine, you have build something in Visual Studio, and at the success, the compilator write in the console
Build success in "C:/path/too/long/to/your/project/and/more/to/binary/software.exe".

Now, you want to send the exe to a friend. You just copy the message, and send Ctrl+V in your OneDrive explorer to share your .exe.
In the explorer, send Ctrl+V with some text or bitmap inside the clipboard will create a file and fill it with the clipboard content

If want to quickly save some txt, copy it, and past in the explorer with Ctrl+Maj+V to save the clipboard content in a new file.

Maj is needed to mark the difference when a windows path is present in the clipboard

If you have some screenshoot in your clipboard, you just have to send Ctrl+Maj+V in your explorer to create a file with the clipboard content (bitmap, png, jpg...).

Supporting information
Hello
I already made a proof of concept of this 4 feature with an AutoHotkey project.
You can try it here :
https://gitlab.com/nitrateag/windowsexplorertools
I recently discover PowerToy and I want to contribute with adding this 4 main feature who I use every day !
I'm thinking of redo this job in C#.
I hope that my contribution will interest you !
=)

@crutskas
This sounds interesting. What do you think about including this?


Generic connection-string parser

In case you might find it interesting, there is generic connection-string, which would let you avoid doing any parsing, so you can simplify your code a lot.
And if you have any questions -  just ask me ;)

Thanks for the interest, and your generic connection-string project looks really interesting. As with many projects, some of the gold only appears when the raw material goes into the fire. I will certainly be looking at this as an alternative parsing option.


Global Color Map Model

We need a globally-accessible color map that we can:

Access by any view rendering that type of thing
Edit by the Group Key Table
Cache if turned off, and recover if turned back on

Resolved?


Happy to help to deploy Hocus in Kubernetes env

Hocus is very promising and I'm a kubernetes enthusiast.
If you need some support to port Hocus to k8s don't hesitate to ping me 😄
I am currently trying to do this.

@jbanety Thank you for finding the time to check out Hocus :)
We want to fully support k8s deployments in the future and provide helm charts for it. It's one of our long term goals but right now it doesn't make any sense to work on it as we first need to make single node deployments stable on top of other priorities.
If you want to deploy Hocus right now on K8s please keep us updated on how it went down. Please have at most one Hocus agent running in an priviledged pod as we temporarily don't support multiple agents per one control plane.

Hi,
I have already deployed Hocus in a hacky way on my dev cluster.
Of course, at the moment, Hocus isn't designed to work with k8s and I need to have a dind side-car to be able to have something on my screen.
This is not the best way but it works.


Ability to Hide / Show only ONE home on accounts with multiple homes in Carrier account

We have two homes that each have carrier systems in them.  The plugin finds them both, but we have separate HomeKit worlds for each of these devices.  Is there a way via config to just focus on one location?

From the homebridge UI you can hide individual accessories from homekit. (this is a thing built into homekit, not specific to the plugin or config for the plugin.
I think it makes sense to have the plugin also support this natively, at least for individual locations within one account. So I'll keep this open for tracking that feature request.
In the meantime, hiding the accessories from the other location from each subbridge should work.

@grivkees Thanks for keeping this open as a future enhancement. The hiding of individual accessories in Homebridge UI doesn't seem to hide them from HomeKit itself, it seems to only remove it from the Accessories tab in the web UI.  Would love it if this could be extended to filter by location so that we don't use Siri to accidentally turn on the heat at the summer home in the middle of winter :-)


Voice model and sample code not working for Hindi

I've tried to use the sample code to build an alexa skill in Hindi but every time I pass a valid user input I'm not getting any proper output. I'm using alexa hosted back-end.
Example:
Me: ( A valid user invoke statement )
Alexa: "< Audio Only Response >"

@Souvik-py which browser are you using to test the code. If it is Firefox then it needs permission to use microphone (to take input in audio format from user). Alternative: you can type your invoke statement. for detailed answer you can navigate to: https://developer.amazon.com/docs/custom-skills/handle-requests-sent-by-alexa.html #118

Are you using Alexa simulator? Alexa simulator doesn't support Audio Player. https://developer.amazon.com/en-US/docs/alexa/devconsole/alexa-simulator.html#alexa-simulator-limitations

Could you give me what phrase you used for testing?

Closing due to inactivity.


Fix a README typo

Coverage remained the same at 97.389% when pulling 0388e673e1d0cd25d312d64094cac18fac49ca36 on almog:master into 09483e025f5e70425940d05348b78a49b0218c5a on jondot:master.


Release Notes - 2021-02

This issue was created to capture the continuous contribution notes. Insight from comments will be used to create the regular release notes.
Example comments could be:
name of new metric and date of release
descriptions of changes to existing metric
Changes to focus areas or restructuring of WG

Metrics Name changes:

Update and rename Reviews_Duration.md to Change_Requests_Duration.md #377

Update and rename Reviews_Declined.md to Change_Requests_Declined.md #375

Update and rename Reviews_Accepted.md to Change_Requests_Accepted.md #374

Update and rename Reviews.md to Change_Requests.md #373

Update and rename Code_Review_Iteration.md to Code_Change_Request_Ite…#370

Update and rename Code_Review_Efficiency.md to Code_Change_Request_Ef…

@sgoggins  are 5 and 6 actually being released? I don't see them in them in https://docs.google.com/spreadsheets/d/1tAGzUiZ9jdORKCnoDQJkOU8tQsZDCZVjcWqXYOSAFmE/edit#gid=1004270137

1 to 4 are now live on the website

closing for release

New metrics include:
Branch lifecycle
Change Request Acceptance ratio
Name Change Revision
Change Requests accepted
Change Requests declined
Change Requests Duration
Change Requests


ValueError: Could not find directory in config for picard

Version info

bcbio version (bcbio_nextgen.py --version): 1.1.1
OS name and version (lsb_release -ds): Ubuntu 18.04.6 LTS

To Reproduce
Exact bcbio command you have used:
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/tools/software/bcbio/anaconda/lib/python3.6/multiprocessing/pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/joblib/_parallel_backends.py", line 608, in __call__
    return self.func(*args, **kwargs)
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/joblib/parallel.py", line 256, in __call__
    for func, args, kwargs in self.items]
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/joblib/parallel.py", line 256, in <listcomp>
    for func, args, kwargs in self.items]
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/bcbio/utils.py", line 55, in wrapper
    return f(*args, **kwargs)
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/bcbio/distributed/multitasks.py", line 327, in concat_variant_files
    return vcfutils.concat_variant_files(*args)
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/bcbio/variation/vcfutils.py", line 372, in concat_variant_files
    out_file = _run_concat_variant_files_gatk4(input_file_list, out_file, config)
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/bcbio/variation/vcfutils.py", line 404, in _run_concat_variant_files_gatk4
    broad_runner = broad.runner_from_config(config)
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/bcbio/broad/__init__.py", line 518, in runner_from_config
    return BroadRunner(_get_picard_ref(config), pdir, config)
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/bcbio/broad/__init__.py", line 508, in _get_picard_ref
    picard = config_utils.get_program("picard", config, "dir")
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/bcbio/pipeline/config_utils.py", line 193, in get_program
    return _get_program_dir(name, pconfig)
  File "/tools/software/bcbio/anaconda/lib/python3.6/site-packages/bcbio/pipeline/config_utils.py", line 245, in _get_program_dir
    raise ValueError("Could not find directory in config for %s" % name)
ValueError: Could not find directory in config for picard
"""

Your yaml configuration file:
details:
- algorithm:
     aligner: bwa
     mark_duplicates: true
     variantcaller: vardict
     effects: snpeff
     effects_transcripts: canonical_cancer
     vcfanno: somatic
   analysis: variant2
   description: PD39456-normal
   files: ../input/HUMAN_1000Genomes_hs37d5_genomic_PD39456b.dupmarked.bam
   genome_build: hg38
   metadata:
     batch: bcbio
     phenotype: normal
- algorithm:
     aligner: bwa
     mark_duplicates: true
     variantcaller: vardict
     effects: snpeff
     effects_transcripts: canonical_cancer
     vcfanno: somatic
   analysis: variant2
   description: PD39456-tumor
   files: ../input/HUMAN_1000Genomes_hs37d5_genomic_PD39456a.dupmarked.bam
   genome_build: hg38
   metadata:
     batch: bcbio
     phenotype: tumor
 fc_date: '2021-12-05'
 fc_name: PD39456
 upload:
   dir: ../final_simplified
 resources:
   default:
     memory: 8G
     cores: 8
     jvm_opts: ["-Xms4G", "-Xmx8G"]

Log files (could be found in work/log)
Please attach (10MB max):  bcbio-nextgen-commands.log, and bcbio-nextgen-debug.log.

Hi @hocinebendou !
Thanks for reporting and sorry about the issue!
You are running bcbio1.1.1 which is from Nov 2018.
If it is your production freeze, just make sure nothing changed in the environment
(whether bcbio/tools/bin in the PATH, which picard).
And/Or you may install the latest 1.2.9a bcbio and contribute to 1.2.9 release!
Sergey

Hi @naumenko-sa
I installed a fresh bcbio 1.2.9 and I get the same error message
ValueError: Could not find directory in config for picard

which picard does not produce anything and picard --version says that it does not exists.

Hi @kokyriakidis !
Are you sure you have added bcbio/tools/bin to the PATH after installation?
on my system:
$ which picard
/n/app/bcbio/1.2.9/tools/bin/picard
$ readlink -f `which picard`
/n/app/bcbio/1.2.9/anaconda/envs/java/share/picard-2.26.9-0/picard
$ echo $PATH
/n/app/bcbio/1.2.9/anaconda/bin:/n/app/bcbio/1.2.9/tools/bin:some_other_staff

Sergey

Oups... I had wrong path. Thanks for the reminder!


Move vovsbr to br0 port 3 rather than 9

It's dumb that we use port 9 for vovsbr, and then containers get ports 3-8 and 10+...
Changing this in kube would require an upgrade path, but we don't promise upgradeability with multitenant yet.

Also update kube setup and openshift-docs [...vovsbr is added to br0 (port 9)... in https://github.com/openshift/openshift-docs/blob/master/architecture/additional_concepts/sdn.adoc]

Changing this in kube would require an upgrade path, but we don't promise upgradeability with multitenant yet.

After some discussion on IRC, Ravi and I realized this doesn't actually break compatibility, it just means that you could end up with a cluster where some nodes have vovsbr on port 3 and others have it on port 9. (But no one cares what port any other node has it on, so that's OK.) So we can update kube too.

LGTM

(The comment about updating kube too was wrong, because after an update, if it got restarted without re-running setup.sh, then ovs would still be using port 9, but kube.go would think it was using port 3. So it got merged without that part.)


13 word Mnemonic

Im trying to recover some coins for a client who used Electrum circa 2015.  According to https://bitcoinelectrum.com/frequently-asked-questions/#why-wont-electrum-accept-my-seed-when-i-attempt-to-restore-my-wallet-from-it Electrum allowed 13 words for a period of time.
Im happy to make a pr with switch to override this logic https://github.com/MetacoSA/NBitcoin/blob/master/NBitcoin/BIP39/Mnemonic.cs#L22

The so-called 13-world is simply the passphrase. NBitcoin already supports that.

I think Electrum has another algorithm completely different from BIP39.


CmdLets for PowerShellCore - Xrm.Framework.CI.PowerShellCore.Cmdlets

Would you be interested in .NET Core build of CRM365/Xrm.Framework.CI.PowerShell.Cmdlets ?

Copied Xrm.Framework.CI.PowerShell.Cmdlet commands to a .NET 6 project
Using Microsoft.PowerPlatform.Dataverse.Client instead of Xrm core assemblies. Microsoft.PowerPlatform.Dataverse.Client is .NET core compatible. But is only for D365 Online versions and not on-prem
Only few things had to be fixed - namely CrmServiceClient renamed to ServiceClient and there is no OrganizationServiceProxy property - but there are others, for timeout and stuff.

I was able to successfully use Get-XrmSolution -UniqueSolutionName Default -ConnectionString "AuthType=OAuth;Url=https://org.......api.crm4.dynamics.com;Username=;Password=;AppId=51f81489-12ee-4a9e-aaae-a2591f45987d;RedirectUri=app://58145B91-0C36-4500-8554-080854F2AC97;LoginPrompt=Auto" from PowerShell 7 to get a response.

Or perhaps whole CRM365 project should be .net core compatible? What are your thoughts, @WaelHamze ?


add date type to editor.type in ng2-smart-table

editor.type in ng2-smart-table only accept  these('text' | 'textarea' | 'completer' | 'list' | 'checkbox') types.
how could i add date type to editor ?
such as(  1989/ 04/ 02)

Not even that it should support basic html input types.

bump for any info?

I ALSO HAVE SAME ISSUE PLEASE HELP


Feature/panda 3216

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Qingyu-Yang:x: Jerify02Jerify02 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Option to upload multiple datasets

Requested by @Rensvandeschoot in #398
Feature Request
Is your feature request related to a problem? Please describe.
The possibility to upload multiple datasets. Reference managers are well equipped to combine records from multiple sources into one dataset, but in some cases this is not sufficient:  When you want to upload a (partly) labeled records, or when you've already started screening in ASReview. Then, you will need some Excel or R/Python skills to add a 'label_included' column to your data.
Describe the solution you'd like
It would be very convenient to the user to be able to upload multiple datasets at the Select dataset step. These are then combined by ASReview into one pool that needs to be screened. A column called 'dataset' or 'file' should be added to trace the record's original file.
Extension 1: Let's say you have prepared a set of 20 'key papers'. If you can drag this set to the 'Drag file with prior inclusions' box, there is no need to type and search for these records in the Select prior knowledge step. This saves a lot of time.

Please don't mind my design skills.
Extension 2: the option to upload a new file with records while screening has already started. At the moment, if a user needs to add a couple of additional records to their project the only solution is to export the project, paste the additional records in de excel export, and start a new project where you upload this file. This process is also described here in the FAQ. It would be nice to have the option to just upload a file with some additional records in the project dashboard:

Drawbacks:
The feature of adding additional records while screening has already started probably has major consequences for reproducibility and the state file (@PeterLombaers). Another point of attention of this feature is that there are probably a lot of possibilities to misuse it. For example, with uploading multiple datasets deduplication has probably not taken place while this is a very important step.
Describe alternatives you've considered
Alternatives are explained in the documentation,

https://asreview.readthedocs.io/en/latest/features/pre_screening.html#partly-labeled-data
https://asreview.readthedocs.io/en/latest/intro/faq.html#i-already-started-labeling-before-i-came-across-asreview-how-can-i-keep-my-former-screening-decisions-when-starting-a-new-project-in-asreview
https://asreview.readthedocs.io/en/latest/intro/faq.html#related-question-how-can-i-add-more-publications-while-i-are-already-started-screening-in-asreview

Teachability, Documentation, Adoption, Migration Strategy
See above.

Thanks for reporting and the proposed design. Relates to #450.


Add CKAN backend development environment

This PR adds the Docker Compose CKAN development environment, along with the initial Cypress integration test scaffolding.

@tanderegg @dmannarino This is the backend CKAN dev environment with initial tests included (just wanted to keep you in the loop).
Let me know if you'd prefer I just tag you instead of assigning you as reviewers moving forward and feel free to ping me (here or on Slack) if you have any questions.

@dmannarino No problem, thanks!


Make token optional in all HfApi methods.

This PR makes the token argument of all HfApi methods optional and default to the saved token. As you can see in the updated docs, the result is quite nice in the sense a user never has to retrieve their token (and thus get some chance of leaking it by mistake).
Since this changes the order of the argument and is thus a breaking change, some internal magic is implemented to detect if the argument at the old place of the token is the token, when the argument at the place of the token is not a valid token, and then do some switch.

Also cc @stevhliu, could you link which documentation pages should be updated to reflect this change? Thanks a lot

I think these are the only two pages that will need to be updated:

Integrate a library with the Hub

How to integrate upstream utilities in your library

Let me know if I can help update these :)

I've already updated those to the new API, normally.


Explain missing email issue in error message

The current error message is not very clear and we've had multiple users in the PyTorch repo having problems with resolving this.

Thank you for your PR. We settled on the message specified in the last comment on this issue. #3847 to specify the missing GitHub ID. We also thought it best not to add a solution into the message text, since many times the reverse is true (needing to update the git config file to the correct email.)

The message from #3847 is definitely better then the current one in main.
My concern is that people will not realize that they used a different email and still will not be able to resolve the issue themselves.
Maybe still mention the .patch page so they quickly see the exact email address used?
And I understand the hesitation to add a solution into the message text, although in my experience helping multiple people with this issue in the PyTorch repo, in all but one case the desired solution was to add the email to the GitHub profile.

The email address for the commit (xxxxxx) is not linked to the GitHub account, preventing the EasyCLA check. Consult Article Help and GitHub Help to resolve. (To view the commit email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please submit a support request ticket.

Hi @kit1980  would you like to amend your commit to the above and resubmit?

Hi @kit1980 would you like to amend your commit to the above and resubmit?

@WillsonHG updated, please take a look.

I'll rebase and sign the DCO if the change looks good.

Rebased and force-pushed to fix DCO.

Thanks @kit1980 for your contribution,
Verified issue on Dev environment & found that above changes are proper & accepted by @mlehotskylf , @jarias-lfx , @WillsonHG
Refer testing details

Link added on commit are proper & redirecting user to correct page

this Help Article : https://confluence.linuxfoundation.org/pages/viewpage.action?pageId=86641302
GitHub Help : https://docs.github.com/en/pull-requests/committing-changes-to-your-project/troubleshooting-commits/why-are-my-commits-linked-to-the-wrong-user
please submit a support request ticket. : https://jira.linuxfoundation.org/plugins/servlet/desk/portal/4

MISSING ID ON COMMIT & NOT COVERED  cla boat appears on same commit .
Refer PR : https://github.com/sun-test-org/repo1/pull/69

Refer missing email error message :

❌ The email address for the commit (https://github.com/sun-test-org/repo1/commit/2d6eeec7147eb74450832f027c10c559fae783e7) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please submit a support request ticket.

Fixes are properly deployed on production , Refer PR https://github.com/CLATestOrg/VulDetect1/pull/33


onJoinChannelSuccess listener is not working in release mode android

Describe the bug
The listener onJoinChannelSuccess is not working in release mode but on the other user side it shows that the user has joined  who is in debug mode
To Reproduce
Steps to reproduce the behavior:

join a channel from a debug build android
join the same channel from release build
onJoinChannelSuccess won't get fired on release mode but works fine in debug mode

@tayyabasikander
Can you please check as follow steps:

Please check if your two devices are using the same app ID.
Please check if your two devices are using the same UID data type. Number UIDs can only communicate with number UIDs, and string UIDs can only communicate with string UIDs.


Reworked Stream API and implementation

Closes #71.

Added more useful functions to Stream, mostly inspired from the JavaScript Array functions.
Removed the ArrayLikeStream type as we're not using it anywhere and it's hard to maintain full compatibility between Array and Stream.
Reworked the internal implementation.
Added tests for all functions except some that are implicitly tested in other tests.

I added more functionality and rewrote the tests. @msujew could you look again?


refactor: remove legacy subcommand syntax

Drop CI test for legacy commands
Drop test cases for legacy commands
Drop logics for legacy commands

Call for code reviewer.


Detect media/mime type and return in Runner response

The idea is to buffer initial bytes of the runner to auto-detect the media/mime type associated with the output.
Perhaps using libraries such as https://github.com/gabriel-vasile/mimetype or comparable.
This will allow the Notebook client to auto-set the renderer type instead of requiring the user to do it.

Lemme know if you have questions @adambabik. Thx!


Index out of range exception

GORM Playground Link
Sorry this is an EMPTY REPO, but I can't reliably reproduce this error, so there's no way for me to provide a Playground link for you. I'm just trying to not have my issue closed right away.
Description
reflect: slice index out of range
Stacktrace:
| goroutine 48979 [running]:
| runtime/debug.Stack(0x1466100, 0xc00a16e000, 0x0)
| 	/usr/local/go/src/runtime/debug/stack.go:24 +0x9d
| cool-package/log.StacktraceIndented(0xcafe7f, 0x1, 0xc007c0f420, 0x1)
| 	/local/path/to/cool-package/log/logger.go:350 +0x34
| cool-package/log.(*Logger).Errorf(0xc0003c6c30, 0xc00030a7e0, 0x53, 0xc007c0f420, 0x1, 0x1)
| 	/local/path/to/cool-package/log/logger.go:324 +0xa1
| cool-package/utils.(*LoggerTagged).Errorf(0xc008614240, 0xcd873a, 0x32, 0xc007c0f420, 0x1, 0x1)
| 	/local/path/to/cool-package/utils/logger.go:55 +0x12b
| cool-package/broker/core.saveDiscoveryResult.func1(0xc008614240)
| 	/local/path/to/cool-package/broker/core/rpc_methods.go:414 +0xbb
| panic(0xbbeb60, 0xdbbe10)
| 	/usr/local/go/src/runtime/panic.go:967 +0x15d
| reflect.Value.Index(0xc0025cfb00, 0xc00b64fae0, 0x97, 0xa, 0x0, 0x0, 0x196)
| 	/usr/local/go/src/reflect/value.go:939 +0x1e7
| gorm.io/gorm/callbacks.CreateWithReturning(0xc00895fd10)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/callbacks/create.go:155 +0x7ad
| gorm.io/gorm.(*processor).Execute(0xc003e02b40, 0xc00895fd10)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/callbacks.go:105 +0x224
| gorm.io/gorm.(*DB).Create(0xc00895fce0, 0xc0025cfb00, 0xc00b64fae0, 0x1)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/finisher_api.go:24 +0xb9
| gorm.io/gorm/callbacks.saveAssociations(0xc00895fad0, 0xc0000dc630, 0xc0025cfb00, 0xc00b64fae0, 0xc00895fbf0, 0xc00b64fa00, 0x0, 0x0, 0x0, 0x456a2c, ...)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/callbacks/associations.go:379 +0x84e
| gorm.io/gorm/callbacks.SaveBeforeAssociations(0xc00895fad0)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/callbacks/associations.go:70 +0x926
| gorm.io/gorm.(*processor).Execute(0xc003e02b40, 0xc00895fad0)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/callbacks.go:105 +0x224
| gorm.io/gorm.(*DB).Create(0xc00b1b2300, 0xb9bb60, 0xc00b64ee20, 0xc00895f830)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/finisher_api.go:24 +0xb9
| cool-package/broker/scopedb.SaveDiscoveryResult.func1(0xc00b1b2300, 0x0, 0x0)
| 	/local/path/to/cool-package/broker/scopedb/scopedb_discovery.go:353 +0x23b1
| gorm.io/gorm.(*DB).Transaction(0xc00301a2a0, 0xc00b19a100, 0x0, 0x0, 0x0, 0x0, 0x0)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/finisher_api.go:530 +0x3dc
| cool-package/broker/scopedb.SaveDiscoveryResult(0xde4440, 0xc008614240, 0xc00b0d5400, 0x3c65, 0xc008614270, 0x4, 0x0)
| 	/local/path/to/cool-package/broker/scopedb/scopedb_discovery.go:148 +0x256
| cool-package/broker/core.saveDiscoveryResult(0xc00062c640)
| 	/local/path/to/cool-package/broker/core/rpc_methods.go:463 +0x8cd
| created by /cool-package/broker/core.(*Broker).SubmitScanResult
| 	/local/path/to/cool-package/broker/core/rpc_methods.go:195 +0x8c

It seems like
| gorm.io/gorm/callbacks.CreateWithReturning(0xc00895fd10)
| 	/local/path/to/go/pkg/mod/gorm.io/gorm@v1.20.11/callbacks/create.go:155 +0x7ad

is the reason for this error. I would guess that simply inserting
if int64(db.Statement.ReflectValue.Len()) <= db.RowsAffected {
    break
}

would solve this issue, but I'm not to sure whether this would have any side effects or whether this is another issue earlier.

Hi, this line is used to avoid the issue https://github.com/go-gorm/gorm/blob/3d3208ed602cdf219cc0501a05bd9f00c6b4bd12/callbacks/create.go#L166-L167, so not sure what caused the panic, it would be helpful if you can reproduce this issue and post the value / code you used when this error happen, Or it is hard to me to make the change, thank you.

Sorry, I must have overseen that check.
So the way I see it there are two possible ways how we can fail even with this check:

We try to access an empty slice (i.e. no 0 index)
RowsAffected gets incremented at the end of the loop
https://github.com/go-gorm/gorm/blob/3d3208ed602cdf219cc0501a05bd9f00c6b4bd12/callbacks/create.go#L176
where we have no check afterwards

However in the second case we should not enter the loop again, because there is no next row.
Therefore I think it would be sufficient (and equivalent to the current implementation) to move the check before accessing the slice.

1 We try to access an empty slice (i.e. no 0 index)

should returns error gorm.ErrEmptySlice in this case. https://github.com/go-gorm/gorm/blob/3d3208ed602cdf219cc0501a05bd9f00c6b4bd12/callbacks/create.go#L259

Closing due to failed to reproduce the issue. feel free to reopen it if possible to reproduce it.

Sorry to comment to this old thread, but I am having the same issue using the method Pluck in a concurrent environment.


package --all-platforms doesn't match docs

As noted in #4295, the docs for --all-platforms say that we'll resolve and cache all platforms, without mentioning that "all" means platforms this Gemfile has previously been resolved on. We should either document exactly how to package onto multiple platforms or fix the option to actually resolve and cache all platforms.

I vote fix the docs and do something separate for "add a foreign platform to my lockfile"

I think this option should probably be more specific, maybe --all-locked-platforms. I can then imagine something like lock --all-platforms meaning that, with cache --all-platforms meaning to lock and cache all platforms.

[ ] fix the docs (https://github.com/bundler/bundler/pull/4331)
[ ] update --all-platforms (WIP)

I vote for not changing the --all-platforms behavior, given my recent changes to platform support and the existence of lock --add-platform

Agreed. I think we will want cache to default to this behavior of caching for all locked platforms in 2.0, with management of which platforms done via lock.

👍 In which case we can close this issue


No completion for "export default exports" modules

From @ithinkihaveacat on July 4, 2016 8:20

VSCode Version: 1.2.1
OS Version: OS X 10.11.5

Steps to Reproduce:

git clone https://gist.github.com/ithinkihaveacat/ed5fc36db488b6e5ecb209fb2d31aa68
typings install
open main.ts in VSCode
Note that there are no completions for $.. ($.greet should be a completion.)

Copied from original issue: Microsoft/vscode#8711

@ithinkihaveacat  JQuery is not added in your typings.json. why do you think it should be installed?
after installing JQuery defintions, please set "allowSyntheticDefaultImports" : true in your tsconfig.json.

@mhegazy Wait, what? This doesn't have anything to do with jQuery--the same thing happens if e.g. $ is replaced with quux.
My issue is that I can't get completion/intellisense working on what (I think) is 100% valid TypeScript code. It compiles and runs just fine:
$ typings install
$ tsc *.ts
$ node main.js
Hello, World

However completions/intellisense is not available.
(The code; I've changed $ to quux in case that was confusing.)

From https://gist.github.com/ithinkihaveacat/ed5fc36db488b6e5ecb209fb2d31aa68#file-typings-json,
{
  "globalDependencies": {
    "node": "registry:dt/node#6.0.0+20160621231320"
  }
}

you are only installing definitions for node. $, quux or any thing else is not installed. why do you expect it to work any differently?

I'm not sure why I need to import definitions for TypeScript code; I thought the point of the ambient definitions was to add types to existing JS code?
import * as quux from "./foo" generates completions just fine:

import quux from "./foo" (i.e. utilising export default exports) does not:

How can I make the import * as quux case (no need to explicitly list names to import) work the same as import quux (with respect to completion/intellisense)?

I thought the point of the ambient definitions was to add types to existing JS code?

Exactly, but how were you expecting to acquire those ambient definitions?  TypeScript cannot magic them out of thin air.  The information you provided indicated that you have not included any ambient typing information in the scope of your project.  TypeScript is good, but it isn't magical.

How can I make the import * as quux case (no need to explicitly list names to import) work the same as import quux (with respect to completion/intellisense)?

Generally, as @mhegazy instructed:

please set "allowSyntheticDefaultImports" : true in your tsconfig.json.

When external libraries do not use the ES6 module syntax, and therefore don't have a default export, that compiler flag allows you to do that.  On the other hand, if the module actually has a default export declared in the ambient typings, it will provide code completion for that default export.

First of all, thanks for looking into this--I do appreciate your time even though I haven't managed to convince you that this is an issue.
However, I still don't understand why import * as quux behaves differently to import as quux, or more precisely why this behaviour is desirable or necessary, rather than undesirable.
(I think it happens because TypeScript does not understand what exports refers to in the context of a module--it only knows that exports exists because it appears in (Node's) ambient definitions, where its type is any. But if TypeScript was given a better understand of what exports means (c.f. its special knowledge of export =) it might be able to provide completion in the export default exports case as well.)

… how were you expecting to acquire those ambient definitions? TypeScript cannot magic them out of thin air.

I don't see why TypeScript would need to magic definitions out of thin air. From looking at the reproduce code, a programmer can say what quux.greet() does, and that quux.baz() does not exist, so in principle TypeScript could figure it out too. (It's not like the function is dynamically generated at runtime.)

… please set "allowSyntheticDefaultImports" : true in your tsconfig.json

I did try this, but it doesn't make any difference to the completion behaviour.
After poking around a bit, I discovered that this construction does activate completion:
// for the `import * as quux` case
export function greet() {
  console.log("Hello, World");
}

// for the `import quux` case
export default {
  greet: greet,
};

However this requires manually listing each of the functions to be exported by default. (And only for completion--it compiles without error and runs just fine without it.) Is there any way to avoid this?

Generally, TypeScript follows the ES6 module format.
export something is a named export, meaning that to import it you should import { something } from 'module';  If you utilise import * as anything from 'module' you will get all the exports of the module.
export default something is the default export, to import it you use import anything from 'module' where.
export = something is legacy syntax to support "legacy" modules, to import that type of export, you should use import * as anything from 'module'.  allowSyntheticDefaultImports should change the behaviour of TypeScript so that import anything from 'module' is equivalent to import * as anything from 'module'.  If the module has a default export though, even with allowSyntheticDefaultImports, if the module has a default export, you will get that when using the import anything from 'module' construct.
You cannot have an export = something and a export default something in the same module.


Duplex: using undeclared variable when packing

Reported here:
http://forum.renoise.com/index.php/topic/27886-duplex-beta-versions/?p=360436

Should be fixed in this branch
https://github.com/renoise/xrnx/tree/bug/duplex-monome-nil-issue


Question: Possibility of making items accept only certain children?

Is it possible to configure an item so it can be merged only with other specific items?
I need a behavioutr similar to HTML5 Drag and Drop when you call preventDefault() in an dragover event handler, which results in a drop enabled item.
More detailed description:

I have 3 items. Lets say 2 of them are marked as groupA, and the last one as groupB (probably in the item's state).
When the user starts dragging the item from groupA over groupB a callback is fired which checks the group condition and returns true/false.
If it returns false the item can't be dropped inside, and the highlighter isn't indicating such possibility. It would be even better if the highlighter would mark that there is no possibility of dropping the item.
If the callback returns true everything works as normal.

I really love this plugin, and I would like to use it in my project, but this functionality is critical for me and unfortunately i can't find anything in the docs that could help me achieve this. Am I missing something or this isn't possible?
I have found this example http://golden-layout.com/examples/#30979a043f34750045dbe57aba4b1ee5 but I think it doesn't work at all, and I'm not quite sure that this is the thing i'm looking for.

The example you linked does what you are asking for? what wrong with that?

It's not working at all - some console errors are being thrown. I can't place any tab in neither "Drop" nor "No Drop Zone". Error - https://www.screencast.com/t/Mokb6Jbcgizr appears when i start to drag an item.

Although the example didn't work, it pushed me on the right track i think. But one thing isn't clear to me - What is the best way to find out which item is being dragged over the parent element?


CLI

Description
Add CLI options
nava alarm.wav

nava --file="alarm.wav"

FWIW, I created a small command line interface after reading this issue :)
https://github.com/lewoudar/son

FWIW, I created a small command line interface after reading this issue :) https://github.com/lewoudar/son

Hey Kevin, that's so cool. Thanks for sharing your package with us. I saw Pomodoro action in your programs list. You may want to check our other project my timer:
https://github.com/sepandhaghighi/mytimer

Hey Kevin, that's so cool. Thanks for sharing your package with us. I saw Pomodoro action in your programs list. You may want to check our other project my timer:
https://github.com/sepandhaghighi/mytimer

Hi @sadrasabouri, thank you for pointing me this project, I will check it when I have some free time :)


boto3 library not fully completing with function names.

Hi, I am using 2019.1 of the language server with coc.nvim.
Lately, I've been programming with this library:  https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/iam.html
When I make a new client, autocomplete/suggestions work:
https://i.imgur.com/u01nYJg.png
However, once I have the client object and want to use its methods, such as "attach_user_policy()", nothing/no suggestions does not show up:
https://i.imgur.com/AvgKI21.png
Am I doing something wrong, or perhaps bad settings?

Nvm, answered my own question here: https://acloud.guru/forums/aws-certified-sysops-administrator-associate/discussion/-LFvOYJhH7mSGVjesay4/how-to-configure-autocompletion-for-boto3-functions-it-is-tedious-to-look-docume?answer=-LRG-zk95VIksLfsXRQx


allow passing an actual IAuthenticator instead of just config

Codecov Report

Merging #1021 into master will decrease coverage by <.01%.
The diff coverage is 20%.

@@            Coverage Diff             @@
##           master    #1021      +/-   ##
==========================================
- Coverage   61.28%   61.27%   -0.01%
==========================================
  Files          74       74
  Lines       13139    13141       +2
  Branches     2090     2091       +1
==========================================
  Hits         8052     8052
- Misses       4384     4386       +2
  Partials      703      703

Impacted Files
Coverage Δ

autobahn/wamp/component.py
48.59% <20%> (-0.28%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aedf45b...4c7c14a. Read the comment docs.


Some updates of deps

Some updates that renovate had issues with because of cargo deny.

Why do you include clap but not tempfile? There are open renovate PRs for is_terminal and tempfile because of cargo-deny and rustix. There is no renovate issue with clap.

I must've gotten tempfile and some other name confused. I added it. I did clap to anticipate a renovate PR, because it is out of date (a bit).

We could specify a patch version for clap in Cargo.toml, then we would get more renovate PRs.

I think that's unrelated. Renovate works on Cargo.lock versions as well. We just have a limit on Renovate PRs I think


Add phone extension field to new contact wizard

We need to add the phone extension as a separate field for both the company and person slides.

Do you think that Organizations should have phone extensions? Seems like a Person attribute only.

Also, you cache Phone in Party. Do you want to add a cache field for Extension?

Good point. Extension should apply only to a person for now.
Dave
From: bcooney93<EMAIL_ADDRESS>Reply-To: karbonfm/karbonfm<EMAIL_ADDRESS>Date: Friday, October 12, 2018 at 10:48 AM
To: karbonfm/karbonfm<EMAIL_ADDRESS>Cc: Dave Graham<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [karbonfm/karbonfm] Add phone extension field to new contact wizard (#43)
Do you think that Organizations should have phone extensions? Seems like a Person attribute only.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/karbonfm/karbonfm/issues/43#issuecomment-429406472, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATtiGQNe6pIGAO6XQ4Iwq0f2BLU56KOwks5ukNYCgaJpZM4XZsMK.

We should add the extension to the phone when we cache it.
Dave
From: bcooney93<EMAIL_ADDRESS>Reply-To: karbonfm/karbonfm<EMAIL_ADDRESS>Date: Friday, October 12, 2018 at 10:54 AM
To: karbonfm/karbonfm<EMAIL_ADDRESS>Cc: Dave Graham<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [karbonfm/karbonfm] Add phone extension field to new contact wizard (#43)
Also, you cache Phone in Party. Do you want to add a cache field for Extension?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/karbonfm/karbonfm/issues/43#issuecomment-429407802, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATtiGQX2o4MYxFbmiRxX_fkB0FQif_sLks5ukNc3gaJpZM4XZsMK.

Looks good, but please include the extension label (e.g., " ext. " & Phone::Extension)


refactor & optimize

This PR applies a great deal of refactoring to the module. I originally tried to plug in a simple in-memory cache for parsed nodes when _put() is called, but there was a lot of complex semi-caching happening in the insert and _split[Region|Point]Node functions. The bulk of the work was refactoring those methods to act more like pure functions that don't modify in-memory state or mutate outside of themselves. Once that work was done, the rest fell into place.
Here are some numbers on speed comparing master to this branch:
Benchmarks
kdb-tree-store /w memory store

insertion
full map query

before
620ms
15ms

after
202ms
11ms

insertion
full map query

before
12555ms
77ms

after
2536ms
45ms

Digital Democracy datasets (all indexes)

dataset
before
after

siekopai
74043ms
55199ms

waorani
72840ms
56091ms

Addresses #1.

Merged in 3.2.1.


grid and downscaling images then predictions at scale

Hi,
I am training startdist 3D with images taken at 63x .
The sizes are (50, 1440, 1920) it was takin too much memory and the FOV was too small so I downscale to (50, 240, 320) .
It is learning ok , but the median size object is [ 37.5 156.  160.5] and the FOV (MAX before OOM)  is [ 64 128 128].
I increased the grid and the unet to 3 .
After learning, when I apply to predictions on new images the issues is that the Z is right (because below 64) but the X and Y are way too small because of the FOV of [ 64 128 128] .
Hence it can only see object below that shape : so many small nucleus detected on 1 single nucleus
Hence my question is how to use Stardist when trained on downscale object onto bigger images with full scale (50, 1440, 1920) ?
Could it be possible to downscale, detect and rescale for the predictions ?
Is that clear or not ? (not sure if I explained well! ).
Thanks a lot,

Hi @Nal44 ,
I am not entirely sure if I understand everything correctly, but if you downscale the training data by a certain factor, you should do the same for at the prediction step, i.e. you would have to do

downscale, detect and rescale for the predictions ?

by yourself.
Hope that helps,
M

the median size object is [ 37.5 156. 160.5]

Before or after downscaling the input images?

I increased the grid and the unet to 3 .
After learning, when I apply to predictions on new images the issues is that the Z is right (because below 64) but the X and Y are way too small because of the FOV of [ 64 128 128] .
Hence it can only see object below that shape : so many small nucleus detected on 1 single nucleus
Hence my question is how to use Stardist when trained on downscale object onto bigger images with full scale (50, 1440, 1920) ?

It is difficult to understand what you mean. Ideally show us the code/notebook.

Hi Both,
Before or after downscaling the input images : After downscaling.
It is difficult to understand what you mean. Ideally show us the code/notebook : sure !
I saved it as .html and zip it , if that helps, if not let me know how to share it .
I added the predictions too.
Thanks a lot,
prediction tiles-Copy1 cells 63x 3d test.zip
antho
prediction tiles-Copy1 cells 63x 3d test.zip
CELL VARIOUS 63x 3d-Copy1 with synthetic-bigger grid trial1.zip
CELL VARIOUS 63x 3d-Copy1 with synthetic-bigger grid trial1.zip

In the CELL VARIOUS 63x 3d-Copy1 with synthetic-bigger grid trial1 I did the training with unet 3 and the grid at 2,4,4  but then I tried to increase the FOV so I changed to unet 4 and grid, hence the OOM error that would will see.
But it trained well with unet 3 and grid 2,4,4. but the objects detection are too small when using the predictions on new images .
Thanks for the support and help !
antho

What you see at prediction time is expected, as your training didn't result in a well performing model: If you have a look into your training notebook, you will see that you final accuracy is ~0.1 which is far from anything useful.
Try to

downsample tou data along y and x (e.g. 4 fold) before training, and
using backbone=unet.

I did downscale 6 fold on X and Y before training and use the unet backbone ... ?
Should I keep the depth to 2 ? what about the grid ? keep the same ?
The issue is that the median object will be larger and the FOV will be smaller I will get the  #WARNING: median object size larger than field of view of the neural network.# .
I am puzzled and cant find the right parameters, perhaps could I send you my training files to check if you can find the right parameters , I may have done something wrong with the annotation because the median object is very large after downscaling isn't it ?
Let me know and thanks for you help !


Github Action

Do we need to create github action only for the unit test and linting?

Do we need to create github action only for the unit test and linting?

Yes, AFIK.

Update: Since there is already a unit testing Action added by the Professor, we have to add the lining workflow only and reuse the unit test Action that is already added.

@Mosaddeq107 Got you.


API for Web UI

Is there any plan in place to create an API for the web UI, or any other type of API wrapper for the validator?
My primary motivation is in being able to build the service into an automated pipeline; export gtfs -> POST to a running instance of the validator -> Receive results.json
I can see an argument for building it into the existing code, since there is already a Springboot web application, but the API also would not necessarily need the UI aspect, and could also be built as a separate module, something like gtfs-validator-service

@Cristhian-HA

As a "random stranger", I'm going to provide my perspective here:
From my experience, having an "official" UI that is known to work with the backend is nice to have, but often one UI doesn't fit all needs. Having a publicly documented and stable API is a big benefit, both for the official UI as well as third-party integrations.
I'm therefore in favour of decoupling the UI enough from the backend to allow for a decent development experience with third-party integrations.

Hello Naomi and Jannis
Thank you for your comments. We are currently working on the new version of the GTFS Validator (v2.0) that can handle larger datasets and also the processing speed has been improved. Once we will conclude the work on the validator, we will start defining the scope of work that is needed for the UI.
We plan to split the work in 2 parts. In the first one, the UI will present the results and the second one we will integrate the validator in the we UI.
All the feedback and comments that we receive on this project, will be taken in consideration when defining the scope of work so do not hesitate to write to us and provide your valuable feedback.
Have a great day !

Welcome around @derhuerst ! :) Thks for your input, they read everything here.
From a "random stranger" to another ;)


feat: Add '/meta' endpoing for deployment information

Context
The team keeps having issues with figuring out what is deployed, so I added a little endpoint to help us with that
Demo
https://preprod-api.dev.lw.iog.io/meta

Fix the lint errors. Thought I resolved them earlier
Can we skip the tests for this? I am not sure how to run this application locally and I'd probably would need to get DB snapshot and run it locally

@rhyslbw No type definition was intentional to keep it flexible in case I need to add/remove something.
I don't have it deployed anywhere as of right now but I got this screenshot with the sample response:

I am not sure how to run this application locally and I'd probably would need to get DB snapshot and run it locally

Just yarn test. It should do all the setup for you if you have docker installed

No type definition was intentional to keep it flexible in case I need to add/remove something.

This is where we need to discuss the feature and determine a solution that fits all use cases, not just nix :-). In my mental model I have extra being the place for ad-hoc information to be set, but concepts such as git revision is general and could be typed to present a better interface for clients.

Follow-up work to be completed in ADP-2356


Error fatal: morestack on g0

I use go-oci8 with vagrant-ubuntu-oracle-xe on my web based application.
when I run the application, there isn't any problem. but somehow when I stopped my app, there are error messages showing.
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0
fatal: morestack on g0

How do I fix this?

seems to be https://github.com/therecipe/qt/issues/451


cmd/compile: Opportunity for de-pessimization in provably const range loops

What does 'go version' print?

go version go1.3 linux/amd64

What steps reproduce the problem?

Using the 2-return 'range' over a slice of structs will copy each struct, even if the
struct is provably not modified in the loop.  This is a premature pessimization that the
compiler could and should eliminate.  Ideally the spec would guarantee this optimization
(a la RVO in C++) so people could count on it and maybe even warn when it is not true.

The for-each construct is much nicer to read and write, but this behavior sort of makes
it a non-starter for large data.  This is discussed in many places on the web, with the
general recommendation of "just iterate on index".  This is unfortunate for
the language.

Alternatively: some syntax to say explicitly "iterate by reference" or by
pointer.

Ian asked me to file this issue.

/cc @rasky for possibility of doing this in the prove pass.

Could you provide a short example? I can look into whether this would fit into the prove pass.
I don't know much about it, but maybe the copyelim pass would be a possibility?
Would this currenlty be possible earlier on in walkrange (https://github.com/golang/go/blob/9a00e64633ce3636d9ea94660168d96be3416df8/src/cmd/compile/internal/gc/range.go#L155). I'm not sure who that would be.

I don't think this fits prove.

I don't think this fits prove.
Agreed. The more I read about RVO and copy elision, the more I think it's not for prove.


MAC easy installable

currently easy installation only works for linux and windows.
building  executables are done here.
for available builds visit here.

is now available with release https://github.com/cedric05/dothttp-runner/releases/tag/v0.0.11


Possible integration in tests, and stackblitz tutorials (in memory mongodb for X-Framework)

https://github.com/nodkz/mongodb-memory-server

https://www.npmjs.com/package/run-rs


ui changed for getstarted.html

Closes #23
Changes done:

Replaced the images and added hover effect

Kindly review and merge the changes or let me know for any of your further suggestions. Thankyou!

Fix this bug !
screen-recorder-sat-jan-06-2024-10-13-43.webm

Hey @dakshsinghrathore ! Bug is fixed now. Kindly review...

@dakshsinghrathore please review it


Training on coco2014 using retinanet-101 just reached AP(@0.5:0.95)  of 0.302 after 16 Epochs

Is that normal? or I need more Epochs?

hello, I'm sorry to disturbing you due to this is my homework, I havae a question. I encounter this question:
No such file: '/home/mrl/Documents/retinanet/RetinaNet/datasets/COCO/images/train2017/000000171857.jpg, when run the python code: for iter_num, data in enumerate(dataloader_train), did you face the same problem, thanks!

hello, I'm sorry to disturbing you due to this is my homework, I havae a question. I encounter this question:
No such file: '/home/mrl/Documents/retinanet/RetinaNet/datasets/COCO/images/train2017/000000171857.jpg, when run the python code: for iter_num, data in enumerate(dataloader_train), did you face the same problem,   thanks!

sorry i have never met this erro.but i think this problem may cause by the picture<PHONE_NUMBER>57.jpg) is not in the folder.check it.if it's not there,download it.
---Original---
From<EMAIL_ADDRESS>Date: 2020/4/24 18:40:23
To<EMAIL_ADDRESS>Cc: "Author"<author<EMAIL_ADDRESS>Subject: Re: [yhenon/pytorch-retinanet] Training on coco2014 using retinanet-101 just reached AP(@0.5:0.95)  of 0.302 after 16 Epochs (#142)
hello, I'm sorry to disturbing you due to this is my homework, I havae a question. I encounter this question:
No such file: '/home/mrl/Documents/retinanet/RetinaNet/datasets/COCO/images/train2017/000000171857.jpg, when run the python code: for iter_num, data in enumerate(dataloader_train), did you face the same problem, thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I'm sorry to disturb you, when i see the code, i saw the retinanet.train() followed by retinanet.module.freeze_bn() in line 109 and 110 in train.py, i don't understood. I check the pytorch documentation, eval is used to model evaluation, why author use freeze_bn in train.py? thanks!

Hi, have you solved this problem, I have trained 40+ epoches and AP(@0.5:0.95)  is still 0.3. Will more epoches be better?

@SISTMrL  I heard that when training batch size is small (e.g. <=4), then it's not suitable to use batch normalization.


The RADIUS timeout window needs to be increased

We need to allow for a method to update the RADIUS timeout window in config.
There is an issue with using MFA with SoftEther as the RADIUS timeout window in RADIUS.h:119 it is currently hard-coded to 10 x 1000, however we should be able to update this in configuration, or at the very least be larger.
I could update this myself in my own fork, however it would be better to have this in the main code repo.

There's also an open bug with long radius delays, that causes ppp based protocols to break. - https://github.com/SoftEtherVPN/SoftEtherVPN/pull/282

Agreed, but if these can be changed in configuration, it will allow users to make it longer than 10 seconds then we can ensure people will be able to use these in enterprise environments.
Even 30 seconds should be good enough, just enough to allow the OOB verification to complete, the options that people will need are cellular phone calls, push notifications etc.

I could use this feature for MFA authentication.

Duplicate of issue #80 . Closing


[#2529] Implement damage enricher

Implements a damage enricher in the format [[/damage 1d6 bludgeoning]] or [[/damage formula=1d6 type=bludgeoning]] which will present the normal damage rolling dialog, allowing criticals to be rolled and damage types to be passed through.

Closes #2529

There are some (rare) cases where multiple damage types are called for. Meteor Swarm and Flame Strike come to mind, but I'm sure there are more.

A creature takes 20d6 fire damage and 20d6 bludgeoning damage on a failed save

That could of course be expressed with [[/damage 20d6 fire]] and [[/damage 20d6 bludgeoning]], but for the purposes of applying damage and managing saving throws, etc., it becomes a lot easier if they could be included in a single chat card. Perhaps we could support a syntax like [[/damage 20d6 fire, 20d6 bludgeoning]], separating on a comma or a pipe |, or something. What do you think?

There are some (rare) cases where multiple damage types are called for. Meteor Swarm and Flame Strike come to mind, but I'm sure there are more.

A creature takes 20d6 fire damage and 20d6 bludgeoning damage on a failed save

That could of course be expressed with [[/damage 20d6 fire]] and [[/damage 20d6 bludgeoning]], but for the purposes of applying damage and managing saving throws, etc., it becomes a lot easier if they could be included in a single chat card. Perhaps we could support a syntax like [[/damage 20d6 fire, 20d6 bludgeoning]], separating on a comma or a pipe |, or something. What do you think?

I think it makes sense to be able to handle multiple formulas in one enricher, though maybe we should hold off implementing that portion until we can handle multiple rolls in a damage roll properly.
That format looks good, and I don't think it would be a problem to add support for multiple rolls later. We could even modify the formatter to still support keyed values (e.g. [[/damage 20d6 type=fire, 20d6 type=bludgeoning]] or even [[/damage formula=20d6 type=fire, formula=20d6 type=bludgeoning]] if you want to be really specific).
One question about the multiple damages format is what do we make the roll link. Right now I only have the dice itself as the link, but that becomes a bit more of an issue with multiple types. We could put a link on each of the dice values, but that might be confusing if they both produce the same damage roll with both types.


In VS Code Workspace Include Models functionality does not work in DMN Editor & Custom Tasks are empty in BPMN Editor

When developing decisions using VS Code Workspaces DMN Editor cannot include other DMN Models.
Also BPMN Editor Custom Tasks list is empty.
Steps to reproduce;

Open a KIE Project in VS Code
Add a second project in the same VS Code Workspace
Try to "Import Models" in DMN Editor
Try to add a Custom Task in BPM Editor

The steps 3, 4 fail as DMN Models list and Custom BPMN Tasks list is empty.

@bouzasvas Hi,
the include model functionality works only if the file you want to import, is on the same directory of the current opened file.
Eg. if you are editing ../resource/dmn/A.dmn and you want to include a B.dmn, that file must be in the same A.dmn directory.

Hi @bouzasvas, ok thank you for giving me more details.
Yes, we still don't fully support the multi workspace scope VSCode offers. Thus, my recommendation is to use our extensions in a mono-workspace mode only, in order to don't slip into issues like the one you reported.

@bouzasvas At this moment we don't have plans to implement multi-workspace support. Stay tuned with us, our https://kie.zulipchat.com/ zulip chat and https://blog.kie.org/ kie blog to stay updated :)

@yesamer thanks, have a nice day!


Feat/add access

removes 'isTemporaryPubKey'
adds support to 'addAccess' endpoint at BWS
refactor legacyImport, to use addAccess.
adds mnemonics support (BIP39)

Updated, ready for review. needs https://github.com/bitpay/bitcore-wallet-service/pull/311

Please don't forget the french wordlist :)
https://github.com/bitcoin/bips/blob/master/bip-0039/french.txt

hey @Kirvx, could you send a PR to https://github.com/bitpay/bitcore-mnemonic with the FR wordlist? the words are at https://github.com/bitpay/bitcore-mnemonic/tree/master/lib/words

@matiu Sorry I can't, I'm not a dev :/

@kirvx, see https://github.com/bitpay/bitcore-mnemonic/pull/28 for french mnemonic wordlist

GREAT!! :)
On Mon, Aug 10, 2015 at 3:01 PM, Gregg Zigler<EMAIL_ADDRESS>wrote:

@kirvx https://github.com/kirvx, see bitpay/bitcore-mnemonic#28
https://github.com/bitpay/bitcore-mnemonic/pull/28 for french mnemonic
wordlist
—
Reply to this email directly or view it on GitHub
https://github.com/bitpay/bitcore-wallet-client/pull/107#issuecomment-129550920
.

--
Matías Alejo Garcia
@ematiu
Roads? Where we're going, we don't need roads!

Thanks :smiley:

updated, ready for review.

needs rebase

updated.

updated.

updated.

updated

Amazing job! Thanks!


az login fail (mac m1)

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
The command failed with an unexpected error. Here is the traceback:
Can't listen on port 0. You may try port 0.
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 639, in obtain_token_by_browser
    return self._obtain_token_by_browser(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 677, in _obtain_token_by_browser
    return self.obtain_token_by_auth_code_flow(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/application.py", line 142, in obtain_token_by_auth_code_flow
    return super(_ClientWithCcsRoutingInfo, self).obtain_token_by_auth_code_flow(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oidc.py", line 205, in obtain_token_by_auth_code_flow
    result = super(Client, self).obtain_token_by_auth_code_flow(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 547, in obtain_token_by_auth_code_flow
    return self._obtain_token_by_authorization_code(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 731, in _obtain_token_by_authorization_code
    return self._obtain_token("authorization_code", data=data, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oidc.py", line 116, in _obtain_token
    ret = super(Client, self)._obtain_token(grant_type, *args, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 788, in _obtain_token
    (on_obtaining_tokens or self.on_obtaining_tokens)({
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/application.py", line 643, in <lambda>
    on_obtaining_tokens=lambda event: self.token_cache.add(dict(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/token_cache.py", line 314, in add
    super(SerializableTokenCache, self).add(event, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/token_cache.py", line 117, in add
    return self.__add(event, now=now)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/token_cache.py", line 189, in __add
    self.modify(self.CredentialType.ACCESS_TOKEN, at, at)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal_extensions/token_cache.py", line 64, in modify
    self._reload_if_necessary()
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal_extensions/token_cache.py", line 55, in _reload_if_necessary
    self.deserialize(self._persistence.load())
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal_extensions/persistence.py", line 160, in load
    with open(self._location, 'r') as handle:  # pylint: disable=unspecified-encoding
PermissionError: [Errno 13] Permission denied: '/Users/cubus/.azure/msal_token_cache.json'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/azure/cli/command_modules/profile/custom.py", line 139, in login
    subscriptions = profile.login(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/azure/cli/core/_profile.py", line 154, in login
    user_identity = identity.login_with_auth_code(scopes=scopes, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/azure/cli/core/auth/identity.py", line 153, in login_with_auth_code
    result = self._msal_app.acquire_token_interactive(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/application.py", line 1779, in acquire_token_interactive
    response = _clean_up(self.client.obtain_token_by_browser(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oidc.py", line 281, in obtain_token_by_browser
    return super(Client, self).obtain_token_by_browser(
  File "/opt/homebrew/Cellar/azure-cli/2.41.0/libexec/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 642, in obtain_token_by_browser
    raise ValueError(
ValueError: Can't listen on port 0. You may try port 0.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login

Expected Behavior
Environment Summary
macOS-12.3-arm64-arm-64bit, Darwin 21.4.0
Python 3.10.8
Installer: HOMEBREW

azure-cli 2.41.0

Additional Context

@jiasli for awareness

Delete /Users/cubus/.azure/msal_token_cache.json and try again?

As commented, deleting /Users/cubus/.azure/msal_token_cache.json and trying again solved it. thank you!

@bebound I faced the same problem while using Workspace.from_config()
Thanks, it works!


Add the "suggested password" back into log in page

Add back "suggested password" back into the first time login page.

date - 11/18
Verified fixed


Update TF and tflint versions

Proposed changes

Update Terraform and TFLint Versions

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Refactor (changes to code, which do not change application behavior)

Checklist

[ ] I have filled out this PR template
[ ] I have read the CONTRIBUTING doc
[ ] I have added automated tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (README.md, inline comments, etc.)
[ ] I have updated the CHANGELOG.md under a ## next release, with a short summary of my changes

Relevant Links

Further comments

@eschwartz the terraform plan from master to this version is clean.  The fix will be detected for the SNS policy but other than that there are no other changes.


Adds option to limit the selection, and 2 related callback functions

Option "selectionLimit" with 0 for unlimited or (int) for a limit.
options.limitReached() is called when reaching the limit,
options.limitBlocked() is called when trying to add more than the
limit. Also works for adding entire group and select_all, but
select_all has a new param to respect or ignore the limit.

Perfect!
Thanks a lot.


sklearn compatibility update- renames .feature_importance_ to .featur…

…e_importances_
My editor is set up to remove extraneous hanging whitespaces, so you'll see a fair bit of that edited in the docs file.
Otherwise, simply changes .feature_importance_ to .feature_importances_ to stay consistent with scikit-learn's API, fixing https://github.com/Microsoft/LightGBM/issues/199

Hi @ClimbsRocks, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

what editor are you using?

Sublime Text

@wxchan can I merge it ?

@guolinke although it is a API breaking, but it is safe to merge


Cherry-picked  MigrationTo1_1 class to handle broken app groups (#4711)

Migration to 1.1:

loads root group from the group repository
filters out multiple app entries with the same version
filters out multiple app entries with different version by keeping the oldest
rebuilds the root group and stores it in the group repository

Fixes: DCOS-11627

change log says that this is targetting the 1.1 release stream, but the PR is created against releases/1.3... am I missing something here?

This a migration to 1.1.5 version that was originally developed for 1.1 but has to be part of 1.3.x and current release too.

understood - thanks
On Mon, Dec 5, 2016 at 11:10 AM, Aleksey Dukhovniy<EMAIL_ADDRESS>
wrote:

This a migration to 1.1.5 version that was originally developed for 1.1
but has to be part of 1.3.x and current release too.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/pull/4772#issuecomment-264895801,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPVLO74vLVU31T9EwT6HE_nXSWksC94ks5rFDdwgaJpZM4LEPmW
.

Please update the summary to say 1.1.5.


1.21 Support?

Hello I would like to know if 1.21 support is planned or being developed as this mod seems extremely helpful, as it makes many fun and useful mods more compatible.

Thank you for updating your mod, I really appreciate it ^~^
Is fabric support going to continue or is Forge the only version planned as of 1.21 and onward? Thank you

The fabric version doesn't have anything right now for 1.21.
ViveCraft fixed all of the compatibility issues this mod provided patches for on 1.21.
When create releases I'll port the Create compat to fabric.


The repo is empty

There is no actual code in the repo, neither in the zip nor on GitHub!

Dokumentinformasjon
⚠ Ikke rediger denne delen. Den kreves for koblingen docs.microsoft.com ➟ GitHub-problem.

ID: 0ca3aefd-ffd9-db88-db5d-b6b3d60134cf
Version Independent ID: cc4a730a-70b2-0fa7-e4fd-3490ccf1e8c9
Content: Microsoft identity platform Python web app quickstart
Content Source: articles/active-directory/develop/quickstart-v2-python-webapp.md
Service: active-directory
Sub-service: develop
GitHub Login: @abhidnya13
Microsoft Alias: abpati

@TobiasAsk
Thanks for your feedback! We will investigate and update as appropriate.

Yep, the code is definitely missing.
@abhidnya13 could you look into this?

Hello @abhidnya13 and @navyasric - Do we have a timeline/additional info that we can share on this thread to address the GitHub feedback? Thank you.

Hello @abhidnya13, @rayluo, and @navyasric - Do we have additional info that we can share on this thread to address the GitHub feedback? Thank you.

This issue was reported 20 days ago (which was days after our docs came live), but our actual sample repo only merged its first implementation PR at 15 days ago. Now we already released that sample, so this issue is now solved.
We may want to address the root cause of "docs being published BEFORE that sample", by adding such a step in a checklist, and/or mention an ad-hoc comment in the doc's PR "this docs PR depends on another repo/pr XYZ".

Thank you, @rayluo
Per dev input, issue is resolved. #please-close


Update needed

I believe this article needs to be updated, as the Connect-PnPOnline command is no longer recognized by the management console. It should be substituted with Connect-SPOService.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: fbe84d2b-fe47-958c-7d20-0499b4866158
Version Independent ID: 559aec4d-6837-71a6-294e-4c74f1934302
Content: Add a Geolocation column to a list programmatically in SharePoint
Content Source: docs/general-development/how-to-add-a-geolocation-column-to-a-list-programmatically-in-sharepoint.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

Hi. The document is up-to-date.
The sample is connecting to the environment using PnP PowerShell, not using the SPO Management Shell.  Connect-SPOService is the command for the native SharePoint Management Shell.
If you'd like to use this sample, you'd need to install PnP Powershell first. Please check out the documentation.
https://pnp.github.io/powershell/articles/installation.html

Ok, thanks for the clarification. I think I got misguided by step n.1 "Open the SharePoint Online Management Shell", right before the Connect-PnPOnline command example. Shouldn't it say "Open the Sharepoint PnP Powershell" instead?

I think that's fair :) I'll make an update to say "Open PowerShell"

Ok then, all is well so I think this can now be closed. Thanks again :)


Create an initial Release of the repository for module usage

Please create an initial release of the repository so that module usage can reference a precise version of the code

Hi Daniel, I created a Github release v1.0.0. Let me know if you are able to reference the modules using the release version. Thanks!


Additional Queries thows error

In readme there is some querys:
/{ct}/search?q={query} | Searches for {query} in a specific contenttype.
and
/search?q={query} | Searches for {query} in all contenttypes.
when using them i get an error:
Uncaught Exception: ContextErrorException .
ContextErrorException in ParameterFactory.php line 29:
Notice: Undefined index: contentType

Just made a release. Fixed in v3.0.18


Mobile linking experiment - replace universal links with deep links

What problem does this new feature solve?
There are reports of mobile linking not working.
https://github.com/WalletConnect/walletconnect-monorepo/issues/1165
https://github.com/WalletConnect/walletconnect-monorepo/issues/1609
https://github.com/WalletConnect/walletconnect-monorepo/issues/1529
https://github.com/WalletConnect/walletconnect-monorepo/issues/647
https://github.com/WalletConnect/walletconnect-monorepo/issues/599
https://github.com/WalletConnect/walletconnect-monorepo/issues/563
https://github.com/WalletConnect/walletconnect-monorepo/issues/444
https://github.com/WalletConnect/walletconnect-monorepo/issues/956
This is also especially confusing because it is hard to reproduce and appears somewhat sporadically.
Describe the solution you'd like
We believe this is due (at least in part) to universal links and how iOS handles these. Sometimes the user is directed to the app, and sometimes to the web link.
One solution to try is to use deep links eg metmask:// instead of the universal link eg https://metamask.app.link.
We can create a staging deployment and do some testing.
Some additional research is needed:

How many projects in our explorer have deep links vs explorer links?
What is our strategy to update deep links going forward?
What changes are needed in the cloud app to push deep links?
What happens when deep links fail?

For more context, we received this feedback and concerns on the idea:

Don't see major drawbacks, but a proof-of-concept would help us decide. The differences in that blackbox behavior would be the deciding factor for me. Chrome/Firefox/Opera on mobile each implement different Deep Link prompts and behavior. I'd also like to see if you have any ideas of how the UX should change in Web3Modal when those deep links fail

and

Safari might have similar blackbox logic for Deep Links that mitigates infinite loop redirects and click-jacking (like what we've found for Universal Links). Am happy to prioritize poking at this and testing it. My main concern would be around the scenarios where the iOS system deregisters deep link schemes (like during system storage preservation, app offloads, and updates). In these scenarios, deep links might unexpectedly blip-out and fail and connectors don't yet detect this and tell the user why.

In any case, this would need to be tested with at least 30 major wallets before moving to prod.

Some feedback from wallets and projects:
Wagmi
Approves of changing to deep links.

Seems okay because this is a system-wide prompt. Happens for the GitHub app, if you pop in imessage:// to your URL bar in Safari, etc.
Downside of universal links is much worse imo, but I’m also not an iOS dev so not as familiar with the whole platform.

AlphaWallet

Definitely a yes :)
It’s a slight hit to the UX, but given that it’s prior to signing and as more value attaches to even signing messages (and not just transactions), users aren’t going to be doing a lot of those frequently. The downside of course is if the custom URL scheme is wrong and an ugly error pops up, something like “Safari cannot open the page because the address is invalid.” But that’s already mitigated by having a registry.
In comparison, universal links are too flaky.
So yes, I think custom URL schemes are acceptable especially since it’s more reliable.

Web3-Onboard
Confirmed that there are reports of problems


Running site locally and visiting sdk docs continuously redirects

Under resources:
Visiting blockchain apis works.
Visiting sdk apis causes continuous redirects.
Reproduce.

clone repo
npm i
npm run watch
wait for 8080 and sass to build
visit localhost:8080
click resources
click SDK APIs

The raw SDK link was pointing to a file in a branch that no longer exists. For the future, the API JSON files will only refer to the master branch. That won't get deleted.


Worker options

Hi!
I'm currently trying to use your lib, and it seems that the worker options are not transmitted to the factory. Could you please review a small change I did to do so it allows to use the full worker options and I saw that this is in your todo list :) ?
I don't have the writes to push on your repo, so it was only possible to commit in local. May I ask you to allow me to push on a secondary branch please, and then review the corresponding PR?
Best regards
A.D.

And so at the end I red on gituhub doc how to do it without the rights :) I link the PR

linked PR : https://github.com/xdave/bull-mq-transport/pull/3

@AlexandreDecollas  merged your pr and published 3.0.1