Allow users to select a subset of harness to run

Description of changes:
Users can now select a subset of harnesses to run using --harness multiple times. Previously users could either run a single harness or they had to run them all.
For the example provided in #1778:
// example.rs
#[kani::proof]
fn a() {}
#[kani::proof]
fn b() {}
#[kani::proof]
fn c() {std::unimplemented!();}

Users can select harnesses a and b by running:
$ kani example.rs --harness a --harness b

In case of multiple matches, Kani will run all harnesses that matches at least one of the --harness argument.
Resolved issues:
Resolves #1778
Related RFC:

Optional #ISSUE-NUMBER.
Call-outs:

Testing:

How is this change tested? New tests

Is this a refactor change? No

Checklist

[ ] Each commit message has a non-empty body, explaining why the change was made
[ ] Methods or procedures are documented
[ ] Regression or unit tests are included, or existing tests cover the modified code
[ ] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Awesome change! this will come in really handy for the kani extension. Could you add an example invocation of the change to the description as well?

I have a couple of questions -

Does this change anything about cargo kani? It doesn't seem that way. Afaik cargo kani  always had the option to run multiple harnesses using cargo kani --harness ['a', 'b'] or is that not the case?
This doesn't add the --filter option yet, is that still coming in a future PR? That is something that could be very useful for the kani extension as well.

I have a couple of non-blocking questions -
1. Does this change anything about `cargo kani`? It doesn't seem that way. Afaik `cargo kani` has the option to run multiple harnesses using `cargo kani --harness ['a', 'b']` or is that not the case? (and if not, Why add this change to single file `kani` as opposed to `cargo kani`?)

2. This doesn't add the `--filter [FILTER_STR]` option yet, is that still coming in a future PR? That is something that could be very useful for the kani extension as well.

Great questions! I need to check how this argument configuration will work for configuration via Cargo.toml.

All the changes here should apply to cargo kani as well. I don't believe cargo kani allows you to specify multiple harnesses today.
Since we changed to match against substring, I don't think we need the --filter option. What do you think?

I have a couple of non-blocking questions -
1. Does this change anything about `cargo kani`? It doesn't seem that way. Afaik `cargo kani` has the option to run multiple harnesses using `cargo kani --harness ['a', 'b']` or is that not the case? (and if not, Why add this change to single file `kani` as opposed to `cargo kani`?)

2. This doesn't add the `--filter [FILTER_STR]` option yet, is that still coming in a future PR? That is something that could be very useful for the kani extension as well.

Great questions! I need to check how this argument configuration will work for configuration via Cargo.toml.
1. All the changes here should apply to `cargo kani` as well. I don't believe `cargo kani` allows you to specify multiple harnesses today.

2. Since we changed to match against substring, I don't think we need the `--filter` option. What do you think?

Substrings work great as well. Would it be possible to change the invokation to use the form --harness ['a' 'b'] format instead of --harness 'a' --harness 'b'? Seems more ergonomic but just something to maybe add in the future, not too important for now.

Would it be possible to change the invokation to use the form --harness ['a' 'b'] format instead of --harness 'a' --harness 'b'? Seems more ergonomic but just something to maybe add in the future, not too important for now.

I'd recommend against this.
Allowing multiple --harness arguments gives more flexibility to the user. The other problem is that characters ' and [ often need to be escaped depending on the context (the current proposal doesn't use ' as far as I know).

@jaisnan I totally understand your point, and I totally agree that it is painful to pass multiple --harness arguments, but I think this is more robust, as @adpaco-aws mentioned. It is also not the recommended to accept multiple values if you have positional arguments (see this doc on why). To go around this, we could consider doing one of the following:

Create a short version of --harness, I don't know which letter to use though, since -h is help. 🙂
We could consider using , delimiter to allow users to provide multiple arguments to --harness. So users could use --harness first,second.

I see, makes sense to me. Yeah, the second idea would be ergonomic as well, but honestly what we have works too.

How does this impact the features that are supposed to only work for a single harness (e.g., stubbing)? I couldn't find the code that deals with that case, apologies if I missed it.

It's there, but it's not pretty. I'm currently working on removing that restriction, so I didn't want to spend too much time with this.
Let me improve this to at least throw an error if more than one filter is given and if more than one harness matches the filter. Is that OK?
Also, is there any other feature besides stubbing that relies on a single harness? I remember that we used to have that restriction for concrete playback, but that's no longer the case.

It's there, but it's not pretty. I'm currently working on removing that restriction, so I didn't want to spend too much time with this.
Let me improve this to at least throw an error if more than one filter is given and if more than one harness matches the filter. Is that OK?

Sounds good. I don't think you need to check both conditions though, just the latter.

Also, is there any other feature besides stubbing that relies on a single harness? I remember that we used to have that restriction for concrete playback, but that's no longer the case.

I wasn't sure about concrete playback, but that's the only two that I had mind. Sorry about that.


fix: multiple calls to toJson cause unexpected json patches

Signed-off-by: Nathanael Law<EMAIL_ADDRESS>Fixes cdk8s-team/cdk8s-core#954

💚 All backports created successfully

Status
Branch
Result

✅
1.x

Questions ?
Please refer to the Backport tool documentation


Trivial: Replace "it's" with "its" in several places.

Reviewers: @scottschurr @nbougalis

(got rid of "Trivial: " in the commit title.

  All good things.
:+1:


Cannot get current state

Version
latest
Here's what went wrong:
cannot get the current state. It works only from the second one.
Example
https://codesandbox.io/s/h6vmd

Hi @denisbykovopen,
Not sure I completely follow, are you talking about your console.log in the onChange?
console.log("Val", value, this.state, "vals", this.state.arrayValue);

If so,  setState in React is asynchronous so you will see the current value, not the updated value in the console log.
There is a second argument to setState which is called after the new state has been set:
this.setState({ arrayValue: value }, () => {
  console.log("Val", value, this.state, "vals", this.state.arrayValue);
});

Hello and thank you for your answer at first. And sorry for my newbie misunderstanding of the subject.
If so, cannot you explain, or better show in example how to get the current value in your component to the state? Because i need the current array of values to work with.

You've already gotten the value:
https://codesandbox.io/s/simple-select-dgtjq
this.state.arrayValue is the selected value.


Model_size

Thanks for your nice job! @yihui-he
I wonder that your method can reduce the model size?

You can expect about 2x parameters reduction for a 4x accelerated
model. There's better way for reduce model size like quantization.
On Sun, Nov 26, 2017 at 11:05 PM, liu666666<EMAIL_ADDRESS>wrote:

Did your method reduce the model size?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yihui-he/channel-pruning/issues/66, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AJkBS4YRSN0jtL4d2obduuBLWIZaeSGpks5s6l81gaJpZM4QrNBt
.

--
Best,
Yihui He
yihui-he.github.io

Thanks for your reply. However, I find that the sizes of models have no changed. In here,https://github.com/yihui-he/channel-pruning/releases/tag/channel_pruning_5x , the shared model size is 499M, which is the same as original VGG.
Thanks for your reply.

We only considered conv layers. VGG-16 model size is determined by FC
layers.
On Mon, Nov 27, 2017 at 1:10 AM, liu666666<EMAIL_ADDRESS>wrote:

Thanks for your reply. However, I find that the sizes of models have no
changed. In here,https://github.com/yihui-he/channel-pruning/releases/
tag/channel_pruning_5x , the shared model size is 499M, which is the same
as original VGG.
Thanks for your reply.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yihui-he/channel-pruning/issues/66#issuecomment-347120054,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJkBS7dqteg7bqa5tMyZ9jtsigExwY_Dks5s6nx2gaJpZM4QrNBt
.

--
Best,
Yihui He
yihui-he.github.io

I see that the size of your shared resnet50 is smaller than the original model.
You are so nice! Thank you very much for your kind help.

Duplicate of #32 #41


Boot screen is just black

Hi!
After burning the iso to the USB stick and choosing it from the boot menu, I get a black screen with just ":" and a blinking cursor under it.
I tried waiting for it but nothing happened.
Device: HP ProBook 6450b with Intel HD graphics
Edit: I'm using the latest iso from the releases tab

Booting from USB on a HP Probook 6550b (which is a similar device) I had the same issue. Just : with a blinking cursor under it.
This happens because either your BIOS is an older version that doesn't support UEFI yet or UEFI is turned off (which it is by default).
I solved it by checking the current BIOS-version. You can do that by pressing F10 when you turn the laptop on and then go into System Information. In my case the BIOS-version was not the most recent (it didn't support UEFI at all). I downloaded the most recent BIOS from the HP-website. In my case this was Windows-only software but since there still was a Windows-system on the laptop that was no issue. There seems to be a way to make a bootable USB-disk with a BIOS-update but I didn't figure out how to do that.
Once you have the latest BIOS it is probably suitable for UEFI-booting. If this option is available you'll find it by entering the BIOS again, go into System Configuration and choose Boot Options.
Somewhere in the list you should find an option called UEFI Boot Mode. Turn that on. (see attached screenshot)
Save the changes.
Boot your system pressing F9 (Boot Device Options).
Choose Boot from EFI File.
Select the File System on your USB-stick, click EFI and then click BOOT, then click bootx64.efi (or what's relevant for your system if it's not 64 bit).
If all is well now your laptop will boot from the USB-stick.
This might also be the solution to issue #538


Add event emitters for resume, pause and cued

What does this change?
As part of the sticky video work we need to have greater awareness of the player state of youtube videos.
This PR adds the following event emitters:

resume if the player has already played once but been paused
paused if the player has been paused
cued if the player has been stopped

Given we don't need to send these events to Ophan or GA i'm unsure as to whether we need to include the log() sections?
How to test
Go to storybook, open dev tools and play / pause / stop the video.

Good point, where can we access these logs. looking at the implementation of the log it seems it's just doing console.log but not sure if it's going into any S3 or something 🤔

Yeah i don't think we need it but we should double check!

Given we don't need to send these events to Ophan or GA i'm unsure as to whether we need to include the log() sections?

It might help to clarify that the lines:
eventEmitters.forEach((eventEmitter) => eventEmitter('cued'));

are looping through all the event emitters and sending events (i.e. to Ophan and GA) so if you don't want to send events for resume, pause and cmd then these new event emitters are redundant?
Unless you want to selectively send different events for different emitters?
The log function is just a debug utility so if the event does not need to be sent anywhere then they can both be removed - i.e. the event emitter and log.

Given we don't need to send these events to Ophan or GA i'm unsure as to whether we need to include the log() sections?

For the emitters it might help to clarify that the lines:
eventEmitters.forEach((eventEmitter) => eventEmitter('event'));

will loop through all the event emitters and invoke them, therefore calling the Ophan and GA emitters.
So if you don't need to send events for resume, pause and cmd to GA or Ophan then perhaps these new event emitter calls are redundant?
Unless you are planning to add a new emitter which should only be invoked for specific events?
In which case that will need some logic to selectively invoke the required emitter(s).

For context we need pause, resume and cued for the sticky video sticky state, I envisage we add a new eventEmitter callback called playerSate or similar in YoutubeBlockComponent that can control an isPlaying state or flag that we pass into the StickyVideo wrapper. Apologies if you already knew that!
I think the question I had is around whether there are any costs involved in sending extra events to Ophan or GA that aren't used?
We absolutely could add some kind of logic to check which event emitters should be called on which events, I just quite like the current setup (api?) and how open ended it is.

I think the question I had is around whether there are any costs involved in sending extra events to Ophan or GA

Thanks for the explanation!
Feels more like a product question i.e. :

should we send the new events to Ophan?
does it matter if we do and they are not used?

If there is no hard requirement to specialise the emitters based on the events then I agree, keeping the emitters generic is the best route.

@rtyley could you possibly help us with the above questions please?
Specifically:

should we send the new events to Ophan?
does it matter if we do and they are not used?

I think that Ophan does want to know whether a video is playing, or not, on a page - Ophan's javascript has code in it that attempts to keep track of how long a user is spending actually giving attention to the page - we take the user scrolling the page as a sign of attention, but we also regard a video playing on a page as a sign of attention.
See https://github.com/guardian/ophan/issues/3391 and https://github.com/guardian/ophan/pull/3470 for some related code - can have a video chat about this if it helps?

I think that Ophan does want to know whether a video is playing, or not, on a page - Ophan's javascript has code in it that attempts to keep track of how long a user is spending actually giving attention to the page - we take the user scrolling the page as a sign of attention, but we also regard a video playing on a page as a sign of attention.
See guardian/ophan#3391 and guardian/ophan#3470 for some related code - can have a video chat about this if it helps?

Thanks @rtyley that's very useful to know. I'd be very happy to chat about this in person - for the purposes of this PR though is it safe to assume that it would in fact be beneficial to send each of these new events to Ophan?


Add text border docs

Docs change for https://github.com/flutter/flutter/issues/24108.
This uses images introduced in https://github.com/flutter/assets-for-api-docs/pull/93
Explains how to currently draw a stroke and fill text.

Failed test actually passed, UI issue.


[C++][Parquet] hange Bloom filter implementation to use xxhash

I also strongly recommend doing away with the virtual function calls if possible. We have vendored xxhash in Apache Arrow so we should also remove the murmur3 code while we are at it
Reporter: Wes McKinney / @wesm
Assignee: Xuwei Fu](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=mwish) / @mapleFU
Related issues:

[C++] Move murmur3 hash implementation to arrow/util (relates to)

Note: This issue was originally created as PARQUET-1657. Please see the migration documentation for further details.

Junjie Chen / @chenjunjiedada:
Great, the Bloom filter thrift definition was agreed and It uses the same variant in arrow now (64-bit xxh3). It should be ok to remove murmur3 hash relative files as well as the unit test file. After releasing parquet-format 2.7.0, we will update the binary file in parquet-testing as well.


Update Tox configuration for more replicable testing environments

Related issue: #115

[x] Tests written and passed
[x] Documentation and changelog entry written, docs build passed
[x] All tox checks passed

Hi! Are you still interested in discussing this?
I pulled the obvious improvements to the commit above already.

I also upgraded all of our tooling quite a bunch, so this is unfortunately becoming a bit stale. But I'd love to hear your thoughts on requirements management and tox builds. I'm still contemplating having more isolated tox envs vs. just using the top-level venv for dev.
If you don't respond soonish, I might close the PR (let's reopen though if you want to continue, no biggie). But no pressure, the ideas have been nice to go over already so thank you for that!

Hi @felix-hilden,
Please excuse the delay. I'm a bit preoccupied with other tasks at the moment.
I noted many of the recent changes in the packaging here and agree with your direction. Using pyproject.toml files is the next step for more convenient configuration and package management.
I would still push for isolated tox builds, especially now that the installation process has been completely revamped. Making better use of the section for [project.optional-dependencies] would really simplify the installation of dependencies. I can submit something that works with that approach if you'd like.
I can see that this PR is nearly obsolete, but if there are things you still find valuable here, let me know and I can re-open a new PR with those changes.

Hi! Thanks for the response, I was looking forward to it 🙏 No worries at all. Indeed, I was quite happy with the tooling upgrade 😄 Isolated tox builds do sound alright (it's just mainly that I need to understand the tradeoff between that and having everything use the top env), but I think I will continue to reject the use of optional dependencies for dev tools for the reasons I laid out above. But it would be equally easy to use requires = -r requirements/dev (or whatever the syntax is, if it's available).
If you're unsure and want to avoid unnecessary work, we can discuss it here or in a dedicated issue. But I'll close this MR for now. Thanks again for all the suggestions!


Add tendermint-light-client-js crate

Closes #463
The vast majority of this PR was taken from #671 with very few changes (thanks @romac for breaking ground there!). I basically added a UI to allow for quick/easy tweaking of the inputs, and will be looking at defining a custom data structure to provide a result back to JavaScript as opposed to serializing Rust's Result type (TBD).
:book: Example README

[x] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[ ] Updated CHANGELOG.md

to provide a result back to JavaScript as opposed to serializing Rust's Result type

Result can be used with wasm-bindgen. See https://rustwasm.github.io/docs/wasm-bindgen/reference/types/result.html. I never used it though.

Result can be used with wasm-bindgen. See https://rustwasm.github.io/docs/wasm-bindgen/reference/types/result.html. I never used it though.

I'm quite happy to leave it as-is, if that works for you @webmaster128. The result comes through as per the example in the README. It's just a little weird with all the different naming conventions, but it works :grin:

The one things we may still want to doo, is add a unit test which calls the exported verify function with some JsValues.

Cool! @romac I've now added two basic tests of the verify method which are now tested in CI too against Chrome and Firefox.

Great stuff! Could you also please fix the warnings in light-client-js/src/utils.rs with an #[allow(dead_code)] on set_panic_hook?

Done :grin:


turn.repeatedIds should not be a default adaptive expression element

Describe the bug
The turn.repeatedIds property is not intended to be used by developers.
Version
Version:  2.0.0-nightly.245812.dc07be4
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
Browser

[x] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Go to any property that can take an adaptive expression.
Edit the expression and use the insert property reference widget.
Expand the turn scope.
See error: turn.repeatedIds is displayed as an option.

I have no idea when getting or setting this property would every be relevant.

Expected behavior
Don't expose properties that we don't want users to use.
Screenshots

Additional context
We should not expose UI where the description in the docs will need to be "for internal use only."

(Removing the "UX Design" label -- not a design team call whether this property is relevant. If it is not relevant, seems fine to remove it.)


Function to calculate a ColorRamp's color at a given value

I'd like to ask for a method in ColorRamp to return a color according to a value between 0 and 1. Such as having 0,0,0,0 and 255,255,255,255 as colors in the ramp, and this function would return Color(128,128,128,128) with 0.5 as input.
While the ColorRamp class has methods to get the values to interpolate manually, it'd be really convenient to have that built-in.

:+1:


Why are you disabling the surge's pigman sleeping feature?

Hi, I am one of the authors of Surge. Was wondering why you disable our fix to the pigman sleeping issue? If you would rather talk in private, rather than the issue tracker, feel free to join my Discord Server

It's because I wanted to implement a sleep tweak as well, but it conflicted with Surge's pigman sleep fix. I just backported 1.12's sleep behavior, and added my tweak.


[TASK]: missing LICENSE file

As in the subject: inside the Task it seems that the LICENSE file is missing. Should we add it?

we need to secure that LICENSE, DISCLAIMER and NOTICE file are present as from (here)[https://github.com/apache/openserverless]


Classification notebook

This adds a new notebook classification.ipynb. this is new. simpler, classification note book, that includes univariate/multivariate/equal/unequal length data, ROCKET and HC2 examples and a pipeline classifier. It replaces the two old notebooks 02 and 03.
It also adjusts the single problem loaders to include the PLAID dataset

@TonyBagnall, would you kindly mind moving changes to the datatypes notebook to a different PR? Difficult to chunk & review if a PR touches a lot of files...

@fkiraly I have (kindly) done as you asked. I had wanted to get the problem specific loaders to load into the correct data structure, not default to dataframe, but guess I cant get that past your gate :) This is my first experience with notebooks, not sure why its failing the tests, some weird kernel message. Is reviewnb an alternative?

@TonyBagnall, would you kindly mind moving changes to the datatypes notebook to a different PR? Difficult to chunk & review if a PR touches a lot of files...

@fkiraly I have done so, I need this in before I can deprecate


Sign Language Poses Dataset

Would it be possible to share the dataset of sign language poses that's used during pre-training and fine-tuning? I understand that the raw version is more than 500 GB, so perhaps, the quantized version can be shared? Thank you in advance for your help!

Unfortunately not possible to share the source 500GB.
I could share the quantized version done with (https://github.com/sign-language-processing/sign-vq) but from my experiments, the quantization is not good enough, and therefore the model does not learn well. The translation model available in this repo is not good.


[CELEBORN-1415][HELM] Refactor the charts to achieve better robustness and availability

What changes were proposed in this pull request?
This PR will do some refactoring to the helm chart, include:

Move all  the master/worker resource manifests to charts/celeborn/templates/master and charts/celeborn/templates/worker directories, respectively.
Comment license with helm template syntax
It's better that all resource names are defined once in _helpers.tml and refered in other files.
Use helm-docs to generate helm chart README file.
Add .helmignore file to specify patterns for files and directories that should be ignored when packaging the chart.

Why are the changes needed?
As dicussed in #2499 , I will break it down into serveral smaller dedicated patches.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Local test.

Ping @pan3793 @SteNicholas @RexXiong @zwangsheng .

Move all the master/worker resource manifests to charts/celeborn/templates/master and charts/celeborn/templates/worker directories, respectively.

Does it follow some "best practice"? It produces unnecessary diff, and line-level diff can not be recognized by GitHub tool.

Move all the master/worker resource manifests to charts/celeborn/templates/master and charts/celeborn/templates/worker directories, respectively.

Does it follow some "best practice" or just your habits? It produces unnecessary diff, and line-level diff can not be recognized by the GitHub tool.

@pan3793 When a helm chart contains multiple components, it's better to put them under different directories. As Apache Airflow Helm Chart did that.

When a helm chart contains multiple components, it's better to put them under different directories.

That sounds reasonable, and I respect your professionalism in the cloud-native area, could you make a pure rename PR first, and then make other patches for rest changes, to gain a clear commit history?

When a helm chart contains multiple components, it's better to put them under different directories.

That sounds reasonable, and I respect your professionalism in the cloud-native area, could you make a pure rename PR first, and then make other patches for rest changes, to gain a clear commit history?

@pan3793 I had force-pushed the commits to make a pure renaming PR, PTAL, thanks.

LGTM, please revise the PR title to match the change

@pan3793 I have updated the PR titile.

Thanks, merged to master for 0.5


picard tests fail sometimes

3 tests fail, sometimes. They often pass on a rerun of the build. Here's one example
org.broadinstitute.hellbender.tools.picard.illumina.IlluminaBasecallsToSamTest.testMultiplexedWithAlternateBarcodeName FAILED
    java.lang.AssertionError: SAM file output differs from expected output expected [true] but found [false]
        at org.testng.Assert.fail(Assert.java:94)
        at org.testng.Assert.failNotEquals(Assert.java:494)
        at org.testng.Assert.assertTrue(Assert.java:42)
        at org.broadinstitute.hellbender.utils.read.SamAssertionUtils.assertSamsEqual(SamAssertionUtils.java:27)
        at

I've seen > 3 of these failures at a time -- I remember seeing 5 or 6 of these fail at once the other day. Possibly an issue with test ordering / statefulness in the test class.

This is urgent. We can either:

fix this, or
disable the tests or
remove the code.

We should try to fix it, and if we can't after investing an hour or two we should disable the tests until @mattsooknah or somebody else can take a look at them.

Sorry I wasn't able to look at this sooner. I'd agree with @droazen - unless the fix is obvious, let's disable those tests for now. I won't have much time to look at it this week, though.

Did anyone else notice that one of the tests in the IlluminaBasecallsToSamTest is in the broken test group?

The comment says it actually should be ok, but it doesn't inspire confidence.

I don't see any obvious errors.  @droazen Do you have a saved output from one of your failures?  It looks like @vruano has a failing build that has a similar error in SamComparison failure in BQSR tests, is that actually a bug or is that related to this?

Gradle test > org.broadinstitute.hellbender.tools.walkers.bqsr.ApplyBQSRIntegrationTest.testPR[2](ApplyBQSR(args=' -qq 6')) FAILED
java.lang.AssertionError: SAM file output differs from expected output expected [true] but found [false]
at org.testng.Assert.fail(Assert.java:94)
at org.testng.Assert.failNotEquals(Assert.java:494)
at org.testng.Assert.assertTrue(Assert.java:42)
at

Unfortunately I never saved the output from my failed builds, but the failures were similar to the one in the original post in this ticket.
Perhaps we should leave the tests enabled for a bit longer, but modify them to dump the contents of the mismatching output files to the log so we can see what's different about them?

I opened #375 for that.

It would be generally useful anyway to get that that information anyway even for non-buggy tests.

Check out the most recent failures on Adam's branch (https://travis-ci.org/broadinstitute/hellbender/builds/58486494) -- there is some revealing info in the assertion failures:
org.broadinstitute.hellbender.tools.picard.illumina.parser.IlluminaDataProviderTest.testIlluminaDataProviderBclMethod[0](Bustard Parsing Test(25T8B25T) w/Clocs, 1, 180, [1101, 1201, 2101], [Lorg.broadinstitute.hellbender.tools.picard.illumina.parser.IlluminaDataType;@296e0de9, 25T8B25T, 0, 0, src/test/resources/org/broadinstitute/hellbender/tools/picard/illumina/25T8B25T/Data/Intensities/BaseCalls) FAILED
java.lang.AssertionError: Bustard Parsing Test(25T8B25T) w/Clocs cluster num 10 expected [2159] but found [1903]
at org.testng.Assert.fail(Assert.java:94)
at org.testng.Assert.failNotEquals(Assert.java:494)
at org.testng.Assert.assertEquals(Assert.java:123)
at org.testng.Assert.assertEquals(Assert.java:370)
at org.broadinstitute.hellbender.tools.picard.illumina.parser.IlluminaDataProviderTest.comparePositionalData(IlluminaDataProviderTest.java:75)
at org.broadinstitute.hellbender.tools.picard.illumina.parser.IlluminaDataProviderTest.compareReadData(IlluminaDataProviderTest.java:80)
at org.broadinstitute.hellbender.tools.picard.illumina.parser.IlluminaDataProviderTest.runTest(IlluminaDataProviderTest.java:37)
at org.broadinstitute.hellbender.tools.picard.illumina.parser.IlluminaDataProviderTest.testIlluminaDataProviderBclMethod(IlluminaDataProviderTest.java:175)

Attaching the whole log here: https://gist.github.com/lbergelson/e4babd039228975d3c63

Just noticed that the tool in question is using a thread pool...

(in IlluminaBasecallsConverter)

which might explain why we're seeing non-deterministic failures on these tests

Which would mean there's actually a bug in the tool though, not in the tests.  The results should be deterministic....

Unless the tests are just not aware of ordering issues in the tool...

we'll disable the tests for BlobFish. @nh13 and the team will fix this in their next sprint.

the fix will not happen this week. for now we'll disable them and picard folks will get it fixed asap in next iteration.

Note to whoever takes this issue: need to re-enable tests disabled in https://github.com/broadinstitute/hellbender/pull/379 once they are fixed.

@akiezun This is blocked, since we need to know which three unit tests fail and when (reproducible).  If they fail occasionally also we need to know how can we be sure that it is fixed.

the failures are nondeterministic and that in fact is the problem. All tests were disabled in these classes:
IlluminaBasecallsToFastqTest
IlluminaBasecallsToSamTest
IlluminaDataProviderTest

Have they been failing in the picard repository?

yes they have: https://github.com/broadinstitute/picard/issues/139

@nh13 how can we unblock this?

Does anyone on your team have cycles to take a look?

Ah yes, I remember this now - they haven't been failing in Picard per se, it's just when we tried compiling under Java 8. The info in that issue should make the fix much easier, if it's indeed the same thing.

perfect. Maybe it's as simple as switching all HashSets to LinkedHashSet
(and same for maps)
On Tue, Apr 21, 2015 at 10:02 AM, Matt Sooknah<EMAIL_ADDRESS>wrote:

Ah yes, I remember this now - they haven't been failing in Picard per se,
it's just when we tried compiling under Java 8. The info in that issue
should make the fix much easier, if it's indeed the same thing.
—
Reply to this email directly or view it on GitHub
https://github.com/broadinstitute/hellbender/issues/364#issuecomment-94805095
.

@akiezun Is the work scheduled for your sprint going to fix broadinstitute/picard#139 or should I still plan to throw it in the secon pool? I'm finally catching up on secon work and am prioritizing the inbox so let me know.

@vdauwera My understanding was that the picard folks were on it. If I'm mistaken, decide what you want to do and let me know.

@akiezun This should be resolved by the removal of these classes yet?

yes: #470


Powershell Helper Module for Host Compute Service (HCS)

Is your feature request related to a problem? Please describe.
Meanwhile, there is a Powershell Helper Module Host Networking Service (HNS), it is not the case for Host Compute Service (HCS) - see https://docs.microsoft.com/en-us/virtualization/windowscontainers/container-networking/architecture
Describe the solution you'd like
All Windows Host Networking Service (HNS) and Host Compute Service (HCS) services (API features) should be available through Powershell and for all flavors of Containers on Windows (novel gases)
Describe alternatives you've considered
Using Docker, but Docker should no longer needed at all for running Windows Containers on Windows.

Hey @ohault, could you share more about what you want to accomplish with exposing HNS and HCS (for all container types) through Powershell?

Hey @fady-azmy-msft, yes for sure.

for Windows Admin Center, it will provide a way to create an extension for HNS/HNC
Hyper-V Manager and all other current tools are only showing a very limited part of the whole Virtualisation/Container architecture on Windows (one exception https://github.com/ShaunLawrie/HnsExplorer)
hidden VM, containers .... is a real problem in terms of compliancy
using hnsdiag,exe and hcsdiag.exe is too limitative
it locks innovation
It's a big spending of times and resources for all solutions that need to rely on HNC/HNC (see build session about IoT, Edge and AKS)
it's a very obscure and untrustworthy approach to try to hide such thing as well as all the associated documentation.
....

I can continue, I already confirm that it should be done for both HNS and HCS coherently.
I hope this help.

Thank you for sharing this @ohault. I spoke with my team about this FR and it's certainly something we'll consider in the future but it's not a priority at the moment. I'm going to leave the enhancement tag on but close this issue to keep track of this FR moving forward.

Hi @fady-azmy-msft , can you please add this in the roadmap before closing this feature request


Mega - failure if public-private key creation" was not done.

Trying to list mega account size (using rclone browser and rclone1.42 branch), while the "Public - Private key creation" has not been done in mega, produces :
panic: runtime error: index out of range
goroutine 1 [running]:
github.com/ncw/rclone/vendor/github.com/t3rm1n4l/go-mega.getMPI(0x1816830, 0x0, 0x0, 0x1816830, 0x0, 0x0, 0x0)
C:/gopath/src/github.com/ncw/rclone/vendor/github.com/t3rm1n4l/go-mega/utils.go:174 +0x126
github.com/ncw/rclone/vendor/github.com/t3rm1n4l/go-mega.decryptSessionId(0xc042299898, 0x0, 0x20, 0xc042299878, 0x0, 0x20, 0xc0423bf0e0, 0x10, 0x12, 0x1506a8, ...)
C:/gopath/src/github.com/ncw/rclone/vendor/github.com/t3rm1n4l/go-mega/utils.go:244 +
0x154
github.com/ncw/rclone/vendor/github.com/t3rm1n4l/go-mega.(*Mega).Login(0xc04237e0e0, 0xc0420a29c7, 0x31, 0xc0423328e0, 0xf, 0x0, 0xc0420a6050)
C:/gopath/src/github.com/ncw/rclone/vendor/github.com/t3rm1n4l/go-mega/mega.go:493 +0x4da
github.com/ncw/rclone/backend/mega.NewFs(0xc0420a6050, 0x47, 0xc0420a6050, 0x0, 0x11c3820, 0xc0423700c0, 0x0, 0x0, 0x0, 0x0)
C:/gopath/src/github.com/ncw/rclone/backend/mega/mega.go:200 +0x5fb
github.com/ncw/rclone/fs.NewFs(0xc0420a6050, 0x48, 0xc04205d200, 0xc0420a6050, 0x47, 0xc0420a6050)
C:/gopath/src/github.com/ncw/rclone/fs/fs.go:1002 +0xbc
github.com/ncw/rclone/cmd.NewFsFile(0xc0420a6050, 0x48, 0xc042360800, 0xc042368000, 0xc042299be0, 0x5e1bf7)
C:/gopath/src/github.com/ncw/rclone/cmd/cmd.go:161 +0x99
github.com/ncw/rclone/cmd.newFsFileAddFilter(0xc0420a6050, 0x48, 0xc042299c98, 0xc0420ca800, 0xc042087d10, 0xc042299cc0)
C:/gopath/src/github.com/ncw/rclone/cmd/cmd.go:179 +0x54
github.com/ncw/rclone/cmd.NewFsSrc(0xc042087dd0, 0x1, 0x3, 0xc042087dd0, 0x1)
C:/gopath/src/github.com/ncw/rclone/cmd/cmd.go:201 +0x48
github.com/ncw/rclone/cmd/size.glob..func1(0x17e7f40, 0xc042087dd0, 0x1, 0x3)
C:/gopath/src/github.com/ncw/rclone/cmd/size/size.go:26 +0x88
github.com/ncw/rclone/vendor/github.com/spf13/cobra.(*Command).execute(0x17e7f40, 0xc042087d10, 0x3, 0x3, 0x17e7f40, 0xc042087d10)
C:/gopath/src/github.com/ncw/rclone/vendor/github.com/spf13/cobra/command.go:766 +0x2c8
github.com/ncw/rclone/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0x17dfa40, 0x0, 0x0, 0x0)
C:/gopath/src/github.com/ncw/rclone/vendor/github.com/spf13/cobra/command.go:852 +0x311
github.com/ncw/rclone/vendor/github.com/spf13/cobra.(*Command).Execute(0x17dfa40, 0x0, 0x17e88c0)
C:/gopath/src/github.com/ncw/rclone/vendor/github.com/spf13/cobra/command.go:800 +0x32
github.com/ncw/rclone/cmd.Main()
C:/gopath/src/github.com/ncw/rclone/cmd/cmd.go:545 +0x3d
main.main()
C:/gopath/src/github.com/ncw/rclone/rclone.go:14 +0x27
It is normal that it is unable to get the information or to work properly, but perhaps failure can be avoided and information provided about the error...
Thanks,
CT

This is a bug in t3rm1n4l/go-mega. It isn't checking its errors, so it needs something like this
diff --git a/utils.go b/utils.go
index 959dc5d..7f2e4ae 100644
--- a/utils.go
+++ b/utils.go
@@ -9,6 +9,7 @@ import (
 	"encoding/binary"
 	"encoding/json"
 	"errors"
+	"fmt"
 	"math/big"
 	"net"
 	"net/http"
@@ -79,7 +80,7 @@ func base64urlencode(b []byte) []byte {

 // base64urldecode decodes the byte slice b using base64 url decoding.
 // It adds required '=' padding before decoding.
-func base64urldecode(b []byte) []byte {
+func base64urldecode(b []byte) ([]byte, error) {
 	enc := base64.URLEncoding
 	padSize := 4 - len(b)%4

@@ -92,8 +93,11 @@ func base64urldecode(b []byte) []byte {

 	decSize := enc.DecodedLen(len(b))
 	buf := make([]byte, decSize)
-	n, _ := enc.Decode(buf, b)
-	return buf[:n]
+	n, err := enc.Decode(buf, b)
+	if err != nil {
+		return nil, fmt.Errorf("Failed to base64urldecode(%q): %v", b, err)
+	}
+	return buf[:n], nil
 }

 // base64_to_a32 converts base64 encoded byte slice b to uint32 slice.

And the errors rippled up.
Do you know why the bug happened - is it repeatable?

Create a mega account or use one available which you can delete its contents
Ensure you can log in via web. Logout.
Follow in the web the "I forgot password procedure" Do not log in via browser at the end.
Try with rclone. Breaks.
Then, log in via web. You will see the message "creating encryption key" or alike.
Logout.
Now try with rclone. Success.

Thanks for the explanation.  I'll fix the go-mega library and write some stuff in the rclone docs - unless you wanted to send a PR to patch

feel free...
i am unable to propose a patch...


Analysis: works on lrcProject,

aggregation flaw that doesnt let analysis run before all is finished still needs a fix - comming soon

Don't merge yet. I need to something.


[FEATURE] Allow super-admin to control which system indices are not permission-able

Related to https://github.com/opensearch-project/security/issues/2553
With #2887, a user who has access to modify roles will now have the ability to grant other user with permissions to the system indices. There is a potential of misuse here. If a rogue user, gets access to security index they can delete it causing no users to be able to access the cluster other than the super-admin who will have to create the index again.
What solution would you like?
Create a "denylist" setting which contains a list of system indices that are not permission-able to normal users.

[Triage] Hi @DarshitChanpura, thank you for filing this issue. I am going to leave this issue untriaged pending further review of whether treating the security index separately is adequate.

[Update from Security review] Having security index isolated and documenting it should suffice in satisfying the requirement for this issue.

@DarshitChanpura  "list of system indices that are not permission-able to normal user"
Only super admin would be able to access it by bypassing the checks, is that correct?

[Triage] @DarshitChanpura per your last message "Having security index isolated and documenting it should suffice in satisfying the requirement for this issue." Is there an issue addressing this specific concern? If so we can close this. If there is not, would you want to convert this issue accordingly.

[Triage] @DarshitChanpura following up

This is duplicate of https://github.com/opensearch-project/security/issues/2845

#2845 talks about adding an explicit feature flag that enables/disables the feature. This issue talks about maintaining an explicit deny-list of system indices which are not permission-able (this list will only contain security-index as of now). Let's close it once #2887 is merged.

[Triage] This issue is incorporated into the work of #2887, so we can leave this open as an actionable triaged issue until that PR is merged.

In proposed implementation, only security index is non permission-able and the deny list remains non-configurable (the check will be hardcoded as part of the PR). We can file a new issue in feature if there is a requirement for a broader and more dynamically configurable deny list.


Issue Regarding " HTTP Status: 404 (not found) "

HTTP Status: 404 (not found)
while opening website

It worked if you try this link,   http://ohnlp.org/MedTator/?st=yes

Could you please email me the details you had<EMAIL_ADDRESS>@johith222

Sure. Thank you

I have mailed ...i have tried the link you sent...still same.
link : http://ohnlp.org/MedTator/?st=yes

I did not get your email

https://ohnlp.org/MedTator/?st=yes @johith222  it worked.

I have mailed actually...

and also i am still facing same issue...!!!!

Hello,
Due to some infrastructure changes, the url from medtator is now https://medtator.ohnlp.org
please confirm that this url works for you

yes, it's working...thank you!!!!


Do not change case for user tag value in Configure Approval Workload

note: the User Tag Label is also being lower cased and should not be "Users in xxx" where xxx is the label but all lower case

PR: https://github.com/NationalSecurityAgency/skills-service/pull/1841

:+1:


downgrade node-fetch to version that jupyter labs wants

For #5956

[X] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[X] Title summarizes what is changing
[X] Has a news entry file (remember to thank yourself!)
[ ] Appropriate comments and documentation strings in the code
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated
[ ] Test plan is updated as appropriate
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)
[ ] The wiki is updated with any design decisions/details.

I think this breaks my password code some, but I don't want master broken for another day. So just getting the downgrade in. I tested with a vsix locally that basic function was restored.


Reorganize examples directory

one subdirectory for every exporter we have documentation for
add recommended rules when applicable
use .yaml.snippet extension when snippet shows the recommended delta and cannot be applied as-is as standalone yaml
add minimal README for each subdirectory pointing readers to the docs pages

LGTM. Can you also add directories or READMEs for kubelet/cAdvisor and control plane, even if all they do is point people to the docs?

Yep, done.


Coldstart Optimization: No OkHTTP 3

This removes OkHTTP 3 in favor of another means of uploading events to IOpipe.
Conflicts: #211 (Only one may be pushed)

This works although it takes even longer than OkHTTP3 because HTTPSURLConnection takes awhile for it to open the connection.

Closing and discontinuing, this can have better coldstart times compared to OkHTTP3 however the per-event connection handling is worse (OkHTTP3 has higher performance here).


Is Native Federation "forwards compatible"?

Hi!
I'm considering introducing your package to a large enterprise ecosystem, with many independently deployable apps running on a platform.
I understand native federation makes the host and remotes independent of each other and their dependencies, and even their bundlers. You call native federation "future proof" on that basis. However your package is kind of new and angular doesn't even fully support esbuild yet (though that's the bundler you made examples with). Although all these microfrontends are independent, they'll still depend on using the same version of native federation, won't they? Or at least "compatible" versions of it. What are the outlooks in that regards? Can it be expected that the output of current native federation will play nicely with future versions (to the extent that underlying technology - browser API allows)? It seems like your development API is solid, since it's been kept from module federation. Can the output for this package be considered forward compatible in any matter as well?

Hi!
Thanks for the info. I would like to clarify. In other words, native federation is a library that creates an import map from a manifest. Developers are given API to use this import map (manifest, import, etc). Adapters to the bundlers are to ensure that the generated bundles are ESM modules or a collection of them. You ensure your API is stable, and ESM modules and import maps are also stable to the extent browsers support them. There is nothing in native federation that could cause runtime problems between bundles, because the interface between bundles if fully browser-native. Correct? If so, this is awesome and you may close this issue. Kudos to you and your team!

Yes. Honestly, Native Federation only has dynamic federation. Even though we use it in a way that looks like static federation, dynamic federation is used underneath the covers.


WARNING (MainThread) [homeassistant.components.number]

Hi,
I migrated my thermostat yesterday from Custom-Nefit component to this Bosch component.
I see the following log entry:
WARNING (MainThread) [homeassistant.components.number] custom_components.bosch.number::BoschNumber is overriding deprecated methods on an instance of NumberEntity, this is not valid and will be unsupported from Home Assistant 2022.10. Please report it to the custom component author.

fixed https://github.com/bosch-thermostat/home-assistant-bosch-custom-component/releases/tag/v0.18.2


GEOMESA-1859 Fixing arrow accumulo distributed export

Relocating fasterxml classes to avoid conflict with hadoop

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

Is there a geomesa-test case we could add which would demonstrate that this works properly?

i added an export command to the test package, but verifying the export is kind of tricky b/c it requires the arrow tools...


issue #90

@harshita214 please review it

Merged the PR 🚀
Good work @whiteHatpro
Thanks for contributing 💙


Update to support Angular Core's style CommonJS standard.

This style:
https://www.npmjs.com/package/angular-animate

Coverage decreased (-0.6%) to 96.93% when pulling 46a1a5c95997be181eeaeec8250929100dd64199 on seanhealy:master into 4ff33f06867e3ad8df3c322f754ab6cac9857867 on urish:master.

Thanks!


Collection fixes

This stops the source generator from crashing with a NullReferenceException for ICollection types.
GetEnumeratorSymbol now falls back to implemented interfaces when the provided type symbol does not define one. This method was returning null when given an IReadOnlyCollection and causing the whole generation to fail.
TryGetGenericCollectionInterface was also expanded to support all related collection interfaces.
Tests for both ICollection and IReadOnlyCollection were also added, but are probably overkill, as it was just the generation that was failing and might not be a necessity for ~300 extra tests.

Thank you for the contribution! I'll include it in next release.

I have had some changes with your implementation. I'll leave the note here.

There was issue that ICollection is not in System.Collections.Generic but System.Collections. Fixed this.
As GetEnumeratorSymbol update does not cover explicit interface implementation, updated this accordingly. It will require additional effort for code generation to fall back explicit interface implementation. It is todo for now.
I separated test case to dedicated ones to test generation.


integrate  YouTube player in all courses

Hello I am interested in taking up this issue
Please tell me what needs to be done
And assign me this
Thank you

Please assign under SWOC

Go ahead @prateekmohanty63

Can you please tell me what is the requirement exactly

Like I have to integrate youtube link for the courses?

@arpit456jain can you please explain m doubt

@prateekmohanty63  i want a UI like this
https://codewithharry.com/videos/data-structures-and-algorithms-in-hindi-1/
in which we can links our videos from youtube

@arpit456jain i have made the page
where all the course videos can come in a single page

should i sent a pr to the master branch
or my branch ?

Master @prateekmohanty63

sure


android build creating too many DEX files

I'm getting a ClassNotFoundException.  It seems that the default android build is creating an APK that has over 100 DEX files in it (mine has 138!) which isn't apparently allowed on API 21 and 22 devices.
There some more info here.

This is because there's too many dependencies in your android app usually. We've been running single dex with reducing dependencies and running multi-dex as well when we had more dependencies. But yes we're not targeting 21 and 22. But I can't really see how this could be a implication of react-native really.

My app has ~170k methods, that should fit into about 3 DEX files right?  It seems some combination of gradle or android has a bug in it where DEX files aren't collapsed.  I understand that some versions of android have a 100 DEX file limit.
Reducing my API dependency back to 16 resulted in a more sensible APK.  So it seems that it's just not possible to build a working v21 app at the moment.


404 Errors

I am able to get an access token and my vehicle id, but all other calls result in 404 errors.  I have used both the "vehicle_id" and "id" codes, to no avail.  I have a 2016 Model X with the latest firmware build.  Is there something specific to a Model X that prevents compatibility with this API?  I would appreciate any assistance anyone can offer!
Here is an example of an error I received when trying to retrieve the car's charge state:
Response
404
Headers
•	connection:keep-alive
•	server:nginx
•	date:Thu, 15 Jun 2017 16:58:43 GMT
•	content-type:application/json; charset=utf-8
•	content-length:60
•	status:404 Not Found
•	x-frame-options:SAMEORIGIN
•	x-xss-protection:1; mode=block
•	x-content-type-options:nosniff
•	cache-control:no-cache
•	x-request-id:c5561f18-e1cc-470d-99ab-4ddfa4598ad3
•	x-runtime:0.009613
•	x-newrelic-app-data:PxQDVFVRCQITVlZRDgcFV0YdFGQHBDcQUQxLA1tMXV1dSnwZQRNWERdcRE42LGtnH0tLQBsBGVZUAxoDTFNXVgZRDFMKAQQCA1UJUQBNEVRUFBtDAlYIVVMBWFQFBwRSAgkUHwQHDxFVPw==
•	via:1.1 vegur
Body
{
"response": null,
"error": "not_found",
"error_description": ""
}

You probably have the wrong URL. What URL are you attempting to access?

I am using http://docs.timdorr.apiary.io/#, which is captioned "Tesla Model S JSON API."  Does a different URL exist for the Model X?  I am able to get tokens and my vehicle id, however.

That isn't the correct URL at all. They should be under https://owner-api.teslamotors.com/api/1/

Sorry for the confusion Tim, yes, that is the URL I am using.  So for charge stage, I am using https://owner-api.teslamotors.com/api/1/vehicles/vehicle_id/data_request/charge_state.

It should definitely be the id field: https://github.com/timdorr/model-s-api/blob/master/lib/tesla_api/vehicle.rb#L147 Make sure you're also including the correct Authorization header. That's all I can think of.

Thanks.  I have used both the Id and vehicle_id codes, but both result in 404s.  I know that my token works, because I used that in my auth header to get the vehicle id codes.  Can't figure out why all of the calls fail.

@allante2 Have you got  API working on production mode, i am getting the same error


Fix deps : pytorch/pytorch: No module named 'ultralytics'

Fix the error:
doods.doods - ERROR - Could not create detector pytorch/pytorch: No module named 'ultralytics'

Thanks for the PR! I just realized this was here. I already updated it.. Good to go!


Add index view

On the index page, previously uploaded results should be available for viewing. Preferably with correct temporal results (i e filter series results by date).

Refactor upload to redirect to a results page (that is also possible to link to) with the given event ID after successful upload.


Remediation Host master playbook update

Change(s):

added nested playbook MDEIsolate
added nested playbook Cisco Meraki

Name of the playbook, suggest having DeviceName and Action like CarbonBlack-QuarantineDevice, Crowdstrike_ContainHost

can you give me one example? is whole playbook should be renamed or just directory names?

Playbook name (default name) inside azuredeploy.json and folder name as well. For example

< DeviceName >-< Action >-< Nested >
CiscoMereki-BlockDevice-Nested
MDE-IsolateMachine-Nested
CrowdStrike-ContainHost-Nested
etc

(if not already) Metadata object need to be inside - please refer [Crowdstrike-Remediation-Host] as an example. Please note that certain elements (like prerequisites) are of type array.

Hi @ospeka Please look into the provided comments and update. Thanks

PR updated in this commits in accordance to provided comments.
https://github.com/Azure/Azure-Sentinel/pull/5522/commits/0dc89b2eba3dbb32a4aaf652126aaa03bf7658b9
https://github.com/Azure/Azure-Sentinel/pull/5522/commits/1551ba013c25827d6f039d868bc75e882d2a2045


Landing Page canonical URL defaults to storeUrl

Describe the bug
all VTEX Headless CMS pages of type "Landing Page" without a supplied settings.seo.canonical value default to the root domain in their canonical tag. this can lead to bad SEO outcomes.
Expected behavior
Landing Pages without a supplied value should use their actual addressable path from the CMS as the canonical href when the CMS value is unset
Current behavior
if the canonical value is left blank, LandingPage.tsx uses the storeConfig.storeUrl, i.e. the / route
Steps to reproduce

create a Landing Page in the CMS and give it a path (e.g. test-page)
build a FastStore app against that storefront and navigate to the path
inspect the DOM and see that the canonical link is <link rel="canonical" href="{storeConfig.storeUrl}"> instead of using the actual page URL

Possible Solution
update the logic in LandingPage.tsx to take this into account. it should be fine to use the root domain + CMS configured path similar to how the PDP does it currently

Assigned. Reference: TCM-196

@batzlerg the patch to Update default canonical URLs for landing pages has been merged, and is available in FastStore version 2.2.74 (in rollout). You should see these changes reflected later today.


Update bindings

Continuation of https://github.com/ppy/SDL3-CS/pull/129

Feel free to take over maintaining if I look inactive here!
Relevant osu!framework branch

This bindings update is currently a bit janky mostly because of this PR that changes SDL_bool to bool entirely.
Applied workarounds:

Add --language c in ClangSharp arguments because bool is C++ keyword, and re-define SDL_bool because C/C++ bool cannot be directly converted to C# bool.
Add != NULL / == NULL to compare pointers in header functions as ClangSharp generated pointer && pointer comparison, which is not possible in C#, after --language c.
Define NULL=0 as ClangSharp tried to put null in a variable of nint type after --language c.
Define some constant macros of SDL_pixels.h in manually written code, as adding --language c made SDL_Colorspace macros to have int for some reason.

I will probably report those bugs over ClangSharp repository since upgrading it didn't help...

I don't like the complexity increase here. Most of the bool issues can be solved with --remap bool=SDL_bool. The only errors then are in ClangSharp generated function bodies that try to convert bool to enum SDL_bool, which doesn't work. Better to make SDL_bool a struct with implicit bool conversions.
So I've made https://github.com/ppy/SDL3-CS/pull/147 instead.


Realtek high definition audio(sst) Driver

Is there a Linux driver for realtek high definition audio(sst)?
Here there is a methos to install the Drivere but the link they provide is not working. Maybe someone can play aound with the Native Sound Manager
The device ID is: INTELAUDIO\FUNC_01&VEN_10EC&DEV_0298&SUBSYS_144DC135

see https://bugzilla.kernel.org/show_bug.cgi?id=188411 - driver works, headphones too, but no speaker output.

I have seen it with no Bugfix in sight !


Streaming Capability

Check Mondrian Forest for:

Weighting / Aging of (recent) data points
Unlearning (inverse functions) for sliding windows
Discarding / Extending Mondrian Forest Model criterium

Done, see Streaming Analysis


No document or examples for building a wasm library with js interop

This answer from @SteveSandersonMS points at a very attractive opportunity for many future users of the wasi sdk: build a standalone wasm output to be called from other environments.
It is stated that:

Since this SDK doesn't assume any JavaScript environment exists, there are no built-in JS interop APIs, so you would have to wire up your own way of calling into the host browser to interact with the Chrome extension APIs

It would be fantastic if we could have at least the most basic documentation for building wasm output from a C# library project, and calling out to a host environment. Further discussions in the referenced thread above aim for a higher bar: calling out to a host environment without assuming a browser or WASI. I am not asking for that.
If we could have a bare minimum example of calling out to a browser env and/or WASI, it would allow users such as myself to experiment with potentially the most common use case: building wasm output and calling  it from a browser env, or embedding it into  some WASI friendly cloud service in which it will be provided some interfaces to call.
This project is clearly the way forward for wasm support, so having these fundamentals would be great. I mean, even building a library and calling it from a host environment would be a great example. I may be missing something obvious, and in this case I'll welcome feeling like an idiot if someone can point me at the documentation or example I'm missing 😄

We're exploring this possibility here: https://github.com/extism/csharp-pdk/pull/4


Create third project to install both C and Fortran libraries...

I see some mention of adding a way to install the netcdf-fortran library at the same time the netcdf-c library in installed.
This is cumbersome and not the way autoconf/automake want us to proceed.
Instead we make a third project, very lightweight, which launches first the C library install, then the Fortran library install. Doing a 'make dist' on this third project will create a tarball with both the C and Fortran libraries, and they will be installed in one operation, with ./configure --prefix=/somewhere && make install.
The advantage of this approach is that neither the C nor FORTRAN libraries have to know about the other.
I have recently done this for the GPTL C/Fortran libraries. (https://github.com/NOAA-GSD/GPTL-all). As you can see, this third project does almost nothing. The configure.ac is only about 35 lines of code. It simply launches the C and then Fortran builds.
I would be happy to do one for the netCDF C/Fortran libraries. Any interest?

Incidedntly, the technique using git submodues of creating that third library, which includes both C and Fortran libraries, is fully described in this AMS paper: https://ams.confex.com/ams/2019Annual/meetingapp.cgi/Paper/350095

I’ll take a look at this but we are pretty saturated project wise here. This may be something we revisit in the future but based on support questions, the current system is sufficient. I’ll take this for future review when we get some slack :)


Error when clicking on an item in the tray

Amazing approach to an action tray, bravo!
Foundry V10, DnD5e 2.0.3, all modules updated (lots of modules)
When clicking on any item in the action tray, the following error pops up in the console. Other than throwing an error, it doesn't appear to cause any issues. Otherwise appears to be working as intended.

Unsure if the intent was to make the item description expand on click or not, but if not, this would be a nice enhancement.

Thanks for the report. This is a v10 upgrade regression issue, I will look into it. It is indeed intended to expand the description.


Update teleport node labels cluster

Towards https://github.com/giantswarm/giantswarm/issues/30398
What does this PR do?

Add new ins= label to all nodes, same value as mc= label
Removes conditional check for cluster= label for MC nodes

Any background context you can provide?
Currently, our labels doesn't support listing only MC nodes, because cluster= label is not present on MC nodes, only WC nodes and mc= label is common on all nodes.
In Web UI, we can filter by setting cluster="" but same doesn't work on Terminal tsh client.
$ tsh ssh root@mc=gazelle,cluster=
ERROR: invalid label spec: 'mc=gazelle,cluster=', should be 'key=value'

$ tsh ssh root@mc=gazelle,cluster=""
ERROR: invalid label spec: 'mc=gazelle,cluster=', should be 'key=value'

Trigger e2e tests

/run cluster-test-suites

/run cluster-test-suites


Support Snap as a distribution option on Linux

We should consider using snaps to distribute on Linux. They're pretty trivial to set up and maintain and can be easier/more flexible than using deb/rpm/etc. I'd be happy to look into it but wanted to know if there were plans for 1.0 that would make this a non-starter?
More info: https://snapcraft.io

Generally speaking I would expect you would want Flutter in userspace, not in the system directories, so it's not clear that any package manager is really suited (as opposed to just checking it out from git or using the prepackaged tarballs).

Looking at how flutter is installed it seems like a snap for this will be insanely easy. Think I might snap it as a test over the weekend.
I'm new to flutter, but I always look for snap packages before I install any other way. Snaps are so very convenient. With that said I know that android studio and flutter has some synergy (Android Studio does have a snap package already). Depending on how flutter and android studio works together, it may be necessary to create services and/or plugs to connect the two snaps, because of the confinement model.

It seems there is a snap here: https://github.com/kenvandine/flutter-snap

Looks like this was done. https://snapcraft.io/flutter


Losing requests from source when duplicating Document

Describe the bug
When duplicating a Document in Designer, the requests in the target (new) Document are duplicated (as reported on #2080) but the source (existing) Document looses the requests. On the source Document, you need to navigate to Design, then back to Debug to get them back. However, this causes the Environment to reset (as reported on #2228).
To Reproduce
Reproduction steps have been confirmed on OpenAPI 2.0 and OpenAPI 3.0.

Create a Document (e.g. based on the PetStore example)
Change the Environment (see Image 1)
Click on Debug and observe the available requests (see Image 2)
Navigate back to Documents and duplicate the Document created in step 1
Navigate to the new Document created in step 4 then click Debug. Observe the duplicated requests (see Image 3)
Navigate back to the source Document (created in step 1) and observe the reset Environment (see Image 4) and the missing requests (see Image 5)

Expected behavior
Duplicating a Document leaves the source in tact (including the Environment). The new Document does not contain two sets of requests.
Screenshots
Image 1

Image 2

Image 3

Image 4

Image 5

Desktop (please complete the following information):

OS: Windows 10 v2004 Build 19041.572
Installation Method: Designer installer downloaded from website
App Version 2020.4.2

Additional context
No other context to add.

Hi @feganmeister
Can you try our newest release located here: https://github.com/Kong/insomnia/releases and hopefully it addresses your issue.

closing due to lack of inactivity


Errors in rendering markdown

This is the readme from a friend of mine, her username is baubau09. Apparently the page cannot render this properly and resulted in this bug as you see. The odd thing is that I am using the same format for my shield badges and it can render it

See also:
#12 md formatting

@LaansDole

It should work, but check it again now!

But I don't know why the other one isn't rendered in the About-Me field.

@SchBenedikt I did some comparisons and found out that we have not added a condition for badge that has color hash code (#) which ended up like that bug

I think the error no longer exists.


Add v1.17 to k8s version list

@lego963
Using empty string instead of the missing field for the latest version is easier


Prevent recursion (and hence crash) in inspector resource assignment.

Fixes #43177

Will this also fix recursion crashes when chaining e.g. Textures in a way that recurses? E.g.:

AtlasTexture A.tres
CanvasTexture C.tres
Assign A.tres as diffuse_texture of C.tres
Assign C.tres as atlas or A.tres

@akien-mga yeah it should work for all types of resources

Thanks!


Fix api key issue

Small changes related to api_key issue while running scripts 160 and 165

This has been resolved by removing the provider block from the module


PBS Video Downloads fail with "HTTP Error 403: Forbidden (caused by HTTPError())"

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

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

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

Before submitting an issue make sure you have:

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

What is the purpose of your issue?

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

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'https://ga.video.cdn.pbs.org/videos/vietnam-war-not-edited/bfb1653c-5601-415b-b50a-86c0dd37be4c/2000008004/hd-16x9-mezzanine-1080p/dgw6tua9_vwar0103_explicit-16x9-1080p-1080p-6500k.mp4', u'-o', u'The-Vietnam-War-03', u'--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.09.15
[debug] Python version 2.7.10 - Darwin-15.6.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
[generic] dgw6tua9_vwar0103_explicit-16x9-1080p-1080p-6500k: Requesting header
WARNING: Could not send HEAD request to https://ga.video.cdn.pbs.org/videos/vietnam-war-not-edited/bfb1653c-5601-415b-b50a-86c0dd37be4c/2000008004/hd-16x9-mezzanine-1080p/dgw6tua9_vwar0103_explicit-16x9-1080p-1080p-6500k.mp4: HTTP Error 403: Forbidden
[generic] dgw6tua9_vwar0103_explicit-16x9-1080p-1080p-6500k: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 503, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2177, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 437, in open
    response = meth(req, response)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 550, in http_response
    'http', request, response, code, msg, hdrs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 475, in error
    return self._call_chain(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 409, in _call_chain
    result = func(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 558, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

You must use original URL.


Does the TrafficSplit spec allow for identification of a default route for unmatched traffic?

Scenario:

Requests with Header X get split 90/10 between svc-v1 and svc-v2.
Requests without Header X all go to svc-v1.

Currently we can achieve this with our implementation using a config like this:
  matches:
  - kind: HTTPRouteGroup
    name: header-x
  service: svc-v1  # this would be the default svc if the header is not matched
  backends:
  - service: svc-v1  # we still want to send some traffic to the default svc if the header matches
    weight: 90
  - service: svc-v2
    weight: 10

However, per the spec, trafficsplits cannot be self-referential. We couldn't figure out a way to achieve this with the current spec definition. Is there any guidance on how to implement this while remaining SMI conformant? Or might this end up being a feature request?

Per @nicholasjackson raise a PR as we could add this.

Let's add this to the discussion in the next community call.

Just an update that I still have this on my list of tasks to do, but have not been able to get around to it yet.


Required libraries

I have installed the required libraries as per the links in the readme file.
I still have errors with:
#include <bluefruit.h>
#include <Adafruit_LittleFS.h>
#include <InternalFileSystem.h>
Are these file names correct?

Looks like you'll need to install Arduino BSP for Adafruit Bluefruit nRF52 series first. I have updated the readme accordingly. Let me know if this works for you!

Thanks for the help, clicked your new link, downloaded as zip. installed library as zip in arduino:
"Specified folder/zip file does not contain a valid library"

All sorted. I had the wrong Seeed board selected.
Breadboard version working very well. Thanks!

Yaay!


Your paste site has been dead for over 3 months...

As stated, the link you have in the vanillafix.cfg file, as well as posted on your Curseforge page has been dead for at least 3 months now. Whenever someone tries to use the Get Report Link the website returns an Error 504 on cloudflare stating that the host is where the error is happening:

Is there any plans to either resolve this issue any time soon, or should we modpack authors and server owners start manually changing the preset link in the config to something that is actually working?

Related to (possible duplicate of) #262

I wish people could like change the site to something like Pastebin

Related to (possible duplicate of) #262

It is related to, however now instead of a 502 error, it is kicking a 504.
A 502 error means that the gateway itself is bad, or has bad information. A 504 means the gateway is there, and or the information is good, but for some reason it is taking too long to respond and deliver the information. These are two different problems and require different steps to resolve them. This is why I started a new thread. While they are related, they are also different.

The exact HTTP error seems to depend on not only the time of access, but also the location where you tried to access the website from, since I'm getting HTTP 502 (instead of 504) accessing the same website.

Resolved. Went into the server and did some asking around. Thanks to the user Snowie the site: https://paste.dimdev.org is back-up again! Also all previously uploaded crashes are accessable again! Not the onces between now and when the site went down.

The domain is parked now


Add Object and Context Initializers and Mutators.

This commit adds SQL statements for initializing and mutating objects and
contexts. The code is by no means complete; I'm merely shoving something
out to provide an example of the patterns. Object and context accessors
come next. Naturally these will evolve as we go on.

Looks good to me, but @gambogi should check it out, since it is a rather large commit.

Also looks good to me; merging


TypeError: Cannot read property 'node' of undefined

Bug report
I received a few of these errors in Sentry. I don't know how to reproduce them but I wanted to report them just in case.
I have a pie chart that uses the legend, so I'm guessing the error occurs there. The pie chart has been online for months and this is the first time this happened.
Feel free to close if this is not enough information.
Codepen
TypeError: Cannot read property 'node' of undefined
  at pathMouseDown(./node_modules/apexcharts/dist/apexcharts.common.js:1617:29)
  at toggleDataSeries(./node_modules/apexcharts/dist/apexcharts.common.js:16364:20)
  at apply(./node_modules/apexcharts/dist/apexcharts.common.js:16211:14)
  at HTMLDivElement.r(./node_modules/@sentry/browser/esm/helpers.js:71:23)

Extra info from Sentry:
[{currentTarget: div > div#apexchartspffwvyaj.apexcharts-canvas.apexchartspffwvyaj.dark, isTrusted: [Circular ~], target: div.apexcharts-legend-series > span.apexcharts-legend-text, type: click}]

Chrome 81
"apexcharts": "^3.7.1",
"vue-apexcharts": "^1.3.5",

I can trace the error, but I also cannot reproduce it.
If you can reproduce in a codepen or identify when exactly it occurs, I will take a closer look.


Anyone has issue on init render?

hi all, I am trying to render dynamically by passing down breakPointCols props.
here is the prop obj
{
default: 5,
1000: 4,
900: 3,
640: 2,
}
When I am under 640px device, and I click refresh, it will first render the default 5 cols then it becomes 2. Anyone knows how to solve this? (message: I dont want to use another state to calculate if this mount is finished or not, if anyone could provide a faster solution for this :)) )
I am running on Next.JS project. Thank you very much!

yeah, I have searched a way for this, but seems I have to use another state to control of it, since it is about nextjs hydration thing.
leave this message here, in case anyone having same issue from me


chore(ci): Upgrade opentelemetry-collector to last v0.x

Release Note
chore(ci): Upgrade opentelemetry-collector to last v0.x

Do we need to specify this somewhere in the documentation by any chance?

No, it's only a testing matter. The camel component is the one defining the compatibility with the OpenTelemetry protocol.
We have some reference to the opentelemetry-collector in camel-k-example but there is no fixed version declared.


Add pylint checker for connection_verify hardcoded settings

Related to #35355
Add a custom pylint checker to ensure connection_verify is not hardcoded to a boolean value.

/azp run python - servicebus - tests

API change check
API changes are not detected in this pull request.

Thanks for looking into this @vs4vijay!
Though our custom pylint rules actually live in this repo here:
https://github.com/Azure/azure-sdk-tools/tree/main/tools/pylint-extensions/azure-pylint-guidelines-checker

Thanks @vs4vijay - I'm going to close this for now. Feel free to resubmit this or any additional pylint updates to the repo I linked to above.
We really appreciate the contribution!


File Structure Design

Feature Request
0Auth
├─ ...
└─ packages
        └─ client
        │       └─ src : client implementation for 0Auth
        │       └─ package.json
        └─ server
        │       └─ src : server implementation for 0Auth
        │       └─ package.json
        └─ message
                └─ src : message for client & server for 0Auth
                └─ package.json

All implementations should be written by function, not by class.
client

host setting

set server's url

signup feature

register user information

verify feature

using verify feature from message

update feature

update user information

server

secret key & public key setting

manage server's pk and sk

signing feature

The signging functions will be implemented separately by each message types.

signature verfication feature

using verify feature from message

request feature

request user's signed data

message

Type setting

enum Type : Privacy, Trusted and Local

signature and message vector setting

using SHA256 Hash and AES encryption from cryptojs library
using EDDSA(ed25519) Signature from elliptic library

verify feature

The verification functions will be implemented separately by each message types.

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Functions to export from @0auth/client.

aysnc function register_info(url: string, properties: Property[]): Promise<> {
  // TODO: Not implemented yet.
}

A function that sends the client's information to the server and receives the server's signature when the value is correct.
But, I am not sure if this function must be implemented.

Functions to export from @0auth/server.
All function names and type names can be changed. Currently they are named arbitrarily.

function auth_privacy(properties: Property[], secret: SecretKey): Signature {
  // TODO: Not implemented yet.
}
function auth_package(properties: Property[], secret: SecretKey): Signature {
  // TODO: Not implemented yet.
}

Functions that authenticate the client's value on the server.
auth_privacy creates a Hash Tree and signs the root Hash value. Therefore, signature verification is possible even if some values are not revealed.
auth_package is used to sign a bundle hash when the values are used together.

function verify_privacy(properties: Property[], sign: Signature, public: PublicKey): boolean {
  // TODO: Not implemented yet.
}
function verify_package(properties: Property[], sign: Signature, public: PublicKey): boolean {
  // TODO: Not implemented yet.
}

Functions to check if the value is signed.
After generating the Hash value for each authentication method, verify the signature.

Types to export from @0auth/client.
All function names and type names can be changed. Currently they are named arbitrarily.

enum PropertyType {
  Row,
  Hash,
}

Type used to distinguish between the regular value and the hash value.

type Property = {
  typ: PropertyType;
  val: string
}

A type for client information.

enum AuthType {
  Privacy,
  Package,
  Local,
}

A type that checks which authentication method the client's information uses.

type Signature = {
  typ: AuthType;
  val: string
}

A type for server's signature.

@anthonyminyungi Please review the above.
If there is no problem, I will make the above skeleton.

Reviews and questions.

Let's use the name of functions and variables to camelCase. Because it has better readability and used broadly on the Javascript ecosystem.

The key named typ on Property and Signature is intended? I think the shortened words are not good for readability. Let's change'em type and value.

The types are only declared in @0auth/client?

For others, LGTM.

@anthonyminyungi

I will write it as 'camelCase'.

It is my mistake. When the variable name is written as type, there is a problem of overlapping with the type keyword, but this problem does not exist in the type internal value.

I wrote it as a client by mistake, but it is a message.


Libretro.RetroArch version 1.10.3

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I've tested on my environment before I submit this PR and it was worked fine. Now it's failed but I can't see the reason or error code. It needs Directx9.0,(it work without it but some issues can happen) and I added. Is there anything else I need to do in this state?

@wingetbot run

In unattended (silent) mode, there's a prompt that shows it needs DirectX which requires user interaction which is why it fails.
This issue is being tracked at libretro/RetroArch#13778.

I was expecting that if I add dependency in the yaml, it will install dx9 before installation start. I'll try to find another way, I belive zip installation is still not implemented(?)

I was expecting that if I add dependency in the yaml, it will install dx9 before installation start.

Support for WindowsFeatures hasn't been added yet. It's listed in the schema because it'll be coming soon in the future.

I belive zip installation is still not implemented(?)

Not at the moment but it'll be coming soon.

I was expecting that if I add dependency in the yaml, it will install dx9 before installation start.

Support for WindowsFeatures hasn't been added yet. It's listed in the schema because it'll be coming soon in the future.

I belive zip installation is still not implemented(?)

Not at the moment but it'll be coming soon.

So we will revisit the packaging nsi script end of this month meanwhile can we not close this PR? I also wonder if there is any roadmap that show us when these features will be added or not. Is it good idea to wait WindowsFeatures get implement so we don't deal with packaging on our end?


Use MKL blas get error results !

I build caffe2 with mkl blas, but when I run a test net, I get a different result with use nnpack lib or GPU model. And run this test net some times, the all result is different. Is this a bug ?
I use this way build:
mkdir build
cd build
cmake .. -DBLAS="MKL"
make
make install

If I use one thread, the result is right!

Ah! We did hit this issue in the past, but I think the most recent MKL fixed the issue. Could you provide a minimal reproducing example (or point us to the test that is creating failures)?

I download the deploy.prototxt and bvlc_alexnet.caffemodel.
wget https://raw.githubusercontent.com/BVLC/caffe/master/models/bvlc_alexnet/deploy.prototxt
wget http://dl.caffe.berkeleyvision.org/bvlc_alexnet.caffemodel
translation：
python caffe_translator.py  deploy.prototxt  bvlc_alexnet.caffemodel --init_net init_net.pb --predict_net predict_net.pb.
alxNet.zip

Thank you ! I want to know how to solve this issue ?  If I need update MKL lib or update caffe2 ?

With the most recent MKL and caffe2, I get error randomly.
Sometimes it just passes, sometimes it get huge error.
When the error is not "huge", it's ranging between 5%-11% (rel err) for conv_op_test, reported under contrib/nnpack/conv_op_test.cc:171.
Our test platform is CentOS 7 on Skylake


Specify more specific text for progress around save participants

It is nothing related to actual saving (i.e. writing to disk) is happening 'under' this progress.
Also, it is confusing to see 'Saving ' as a notification, with Cancel button as the only option - maybe it would be better to use the window progress instead, and use an ellipsis at the end of the message?

This PR fixes #

Thanks for the PR but not thanks. We would not change this. For now closing this.
fyi @jrieken


Optimize for database index use case

Hi, I just want this community to know that I am adapting this repository to optimize for database index use.
In the database world, it is almost always the case that only separator keys are stored in internal nodes, the data itself is stored separately, i.e., not directly mapped to keys as values. The rationale is that separator keys (or guiding keys) should in most cases fit into memory, even when the dataset is as large as a few TBs. See Graefe Modern B-Tree Techniques, chap. 2.1.
The data itself is stored in pages (e.g. 4KB), i.e. units that are optimized for disk IO (buffered read and write). It also allows partial access, as we cannot deserialize a few TB all at once. The data pages will support container operations with the order of magnitude of binary search.
The repository is here. It is still under development. Advice is appreciated. However, as this is part of a larger project, for which I am under deadlines, cooperation at this stage may not be viable.
I should also add that I learned a lot about modern C++ features from your repository. Thank you! Feel free to close this issue, as I understand this is not an active development on your end.

This project was originally planned as a general-purpose STL-like container, so I am not aiming for particular purpose optimized for DB. However, I'm very happy to hear that the development is being continued for useful purpose. Thank you!


Memory leak in OrderedMemoryAwareThreadPoolExecutor if channel close event does not go through it

OrderedMemoryAwareThreadPoolExecutor uses Channel instances as ChildExecutor keys (see OrderedMemoryAwareThreadPoolExecutor.getChildExecutorKey(ChannelEvent)).
Entries are removed from the childExecutors map in OrderedMemoryAwareThreadPoolExecutor only if a ChannelStateEvent is processed and the channel is closed (see OrderedMemoryAwareThreadPoolExecutor.getChildExecutor(ChannelEvent)). This effectively means that if such event does not go through the executor, channels will be leaked. Even though the map is weak, the parent class contains another map (channelCounters) which is not weak and prevents the instances from being garbage collected.
Ironically, the javadoc of OrderedMemoryAwareThreadPoolExecutor states that:

Please be very careful of memory leak of the child executor map. You must call removeChildExecutor(Object) when the life cycle of the key ends (e.g. all connections from the same IP were closed.)

Actually, if the Channel is used as key the key life cycle ends not when the ChannelStateEvent is processed by the executor, but  when the channel is closed. So I would expect that entries are removed from the map via a channel close listener - similarly to what is done in OrderedDownstreamThreadPoolExecutor.
In our case we use a separate OrderedMemoryAwareThreadPoolExecutor to handle rare events, the processing of which requires extensive intervals where the treads simply wait, hence we don't want to block worker threads. This executor does not recevieve any ChannelStateEvents - they rather go through the regular worker executor so we end up with this memory leak.

Netty 3 is EOL please upgrade


fix: autocompleted notification type

This fix was required due to the change of the NotificationNote component.
Type "info" was changed from blue to grey, type "default" got blue from the previous.


refactor: upgrade sveltekit to 1.0.0-next.292

Also upgrade adapter-static to 1.0.0-next.29
Set pre-render default to false

Signed-off-by: ankitm123<EMAIL_ADDRESS>

Release notes: https://github.com/sveltejs/kit/releases/tag/%40sveltejs%2Fkit%401.0.0-next.292


Previous Video not pausing if video container is kept in Pagensnaphelper i vertical scroll

Previous video is only pausing when I scroll twice. ie. if video is playing i position 0 and if i scroll to position 1 it is not pausing, however it is pausing if I scroll to position 2.
Can it be done?

Yes. Up to you how.


Add a setting to disable grayed-out streams in left sidebar

Right now we have a confusing feature where if a stream has not activity in your last N messages, we gray it out and move it to the bottom of the sidebar.  Discussion here:
https://chat.zulip.org/#narrow/stream/92-learning/topic/Gray.20stream/near/698011
We should add a setting to make this optional.  If somebody doesn't explicitly set the value, we can make the setting determined by the number of streams and raise the threshold.
Ultimately this feature is driven by stream_data.is_active, so the amount of JS code to handle I-don't-want-this-set is probably minimal.
This requires knowledge of how to set up settings and some familiarity with the left sidebar code.

@zulipbot claim

What would be suitable place to add the settings option?
Can we add a checkbox in the display settings option for user?

Good question. It can go
gear menu -> settings -> display settings, under "Show counts for unread messages".
We can call it: "Sort recently active streams to top of list"
(and we'll eventually link to a help doc that explains it)

I was also thinking of the same.

I am working on this issue. But have been busy with my exams. Will be able to complete the issue as major work is done.

@zulipbot claim

I have already made a PR for review.


[0.3.2] Replace start timestamp with number of observations

Proposed changes
Replace the start timestamp in the configuration with the number of observations we want for the timeseries.
We now have to set the number of data points we wish to download in the past (starting from the configured end, using the configured interval).
This is more intuitive than setting the exact start timestamp in UNIX, resulting in better DevX.
Fixes
n/a
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[x] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have locally run services that could be impacted and they do not present failures derived from my changes

Further comments
n/a

Looks good. Some failing e2e.

Previous run failed because of #1168.


make the stream returned from Map::fetch() Unpin

This should be a backwards-compatible change that makes Map::fetch() usable as intended. Using BoxStream would be a breaking change because it requires the stream to be Send which we would have to add to F and F::Output.
TODO: regression test, might just do as a code example
closes #244

Thanks @abonander , I'm going to merge this now as the doc test for this can be added after. Something I want to get locked in when we put out 0.4 is non-ignored examples for the full core API ( all the fetch variants on Query, etc. )


How to generate GraphQL schema across multiple OpenApi3 specs?

Can oasgraph generate one GraphQL schema across multiple OpenAPI3 specs?  I have several OpenAPI3 compliant services with links that reference each other.  It'd be amazing to use GraphQL to query across them.

@JHibbard This feature is not yet supported by OASGraph, but falls neatly into its intended capabilities. One challenge will be to avoid collision between GraphQL type names. Another consideration is whether OASGraph should be passed a list of OAS, or whether a single OAS that defines links using operationRefs could be the starting point.

I would also be really interested in seeing this functionality!  I have several microservices which each have their own OpenAPI spec and would like to put a GraphQL gateway in front of them.  Being able to supply all the oas specs and have it produce one unified GraphQL schema would be fantastic.

@amnuts We are currently fixing some issues for a new minor version (which will come out soon), and then want to look into this one!
We will likely approach this in different stages, i.e., initially throw errors if there is collusion between the type names used in the OAS, and eventually mitigate these problems.

That's great to hear @ErikWittern, thanks!  Right now we're kind of working around by converting each of the OAS files in turn with oasgraph and then using graphql-tools to merge the schemas together in the end.  But I feel that's probably going to limit us by not allowing the 'links:' from one spec to another - not sure, yet.
But thank you so much for this project!  It was amazing to be able to use an OAS file and have it converted automatically to GraphQL including all of the resolvers and mutations, which meant we only needed to concentrate on building a good foundation spec.  Brilliant!

@JHibbard @amnuts Yesterday I added support for multiple OASs! I am still working on documentation and other improvements but if you would like to give it try, please do!
It would also be nice just to have some early feedback so I know what to do next because as of now, we only have a basic version with limited collision handling (if operationIds or securitySchemes are the same, OASGraph will simply pick from one OAS and provide an error message. To mitigate this, you may need to use the new operationIdFieldNames option).

Now, instead of providing a Swagger or an OAS, you can also provide an array of a mix of Swaggers and OASs like in the following:
OasGraph.createGraphQlSchema([oas1, oas2, oas3])

I have currently configured the example GraphQL server (packages/oasgraph/test/example_gql_server.js) to use this feature. Because the file will likely change over time, I also created a gist with the same code.
To run it, you just need to run:
cd packages/oasgraph/test/
node example_gql_server.js

And go to http://localhost:3000/graphql
And do make queries against it, you will also need to run the example API servers:
# in a second tab
cd packages/oasgraph/test/
node example_api_server.js

# in a third tab
cd packages/oasgraph/test/
node example_api3_server.js

You can see some example queries here.

Upcoming work:

Documentation
CLI support
Links across OASs

This is really awesome @Alan-Cha, thanks!  I'll be able to give this a go next week, so will see how it goes then.  I'm looking forward to trying it out!

This is addressed by the above commits and PRs.


04-create-dax-calculations-in-power-bi-desktop.md

English:
Image and diagrams components should be localized
Current Translation:
Screenshots, image and diagrams components should be localized
Suggested Translation:
Image and diagrams components should be localized
Description:
Screenshots are not localized

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

The locfile status shown InLocalization, waiting for checkedIn status. ICMS QUERY

Locfile now located at https://github.com/MicrosoftLearning/PL-300-Microsoft-Power-BI-Data-Analyst.fr-fr/blob/OLPRODLOC/Instructions/04-create-dax-calculations-in-power-bi-desktop.md
Images displaying ok - can be closed


Notebook

Added functionality to allow AutoFeedback to be used within a notebook environment. Tests must be defined within a code cell in the notebook, and executed after the students' code.

Codecov Report

Merging #12 (4d70dfe) into master (fc5933b) will decrease coverage by 0.89%.
The diff coverage is 78.04%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
- Coverage   88.87%   87.98%   -0.89%
==========================================
  Files          11       11
  Lines         710      724      +14
==========================================
+ Hits          631      637       +6
- Misses         79       87       +8

Impacted Files
Coverage Δ

AutoFeedback/plotchecks.py
72.13% <50.00%> (-2.44%)
:arrow_down:

AutoFeedback/utils.py
90.32% <86.95%> (-9.68%)
:arrow_down:

AutoFeedback/funcchecks.py
94.84% <100.00%> (-0.44%)
:arrow_down:

AutoFeedback/varchecks.py
80.32% <100.00%> (-2.29%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Compatibility with 6.2.4

Hello,
I'm not able to install the current version with version 6.2.4. Is there a new version soon ?
Best regards

Hello,
The new 6.2.4 release is available