Added Intersect Task

In this task, a set of shapes is given in simple language, and the goal is to determine the number of intersection points between the shapes.
An example input prompt is:
There is a circle centered at (0.3, 1.2) with radius 1.0.
There is a circle centered at (-0.0, -0.2) with radius 3.0.
There is a circle centered at (-1.6, -1.9) with radius 4.0.
How many intersection points are there?

The correct answer is 4, as can be seen in the following visualization:

This task was designed to test the geometric reasoning abilities of language models; it purposely does not involve complicated or ambiguous language---the wording is simple and follows a template pattern. This is a multiple-choice task, where the choices are the integers {0, 1, ..., 40} (the examples have been filtered such that the maximum number of intersections is always less than or equal to 40).

@googlebot I signed it!

@asteroidhouse Thank you for submitting your task! We are in the process of pairing submissions with task reviewers and will be in touch soon.

@asteroidhouse We are now requesting that task authors please include an explicit section in the README.md file called "data source", listing exactly where you got your data from (or explicitly state that you made it up yourself). Many thanks!

@shtoshni92 are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

@jeknov are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

Hi, I am one of the reviewers assigned to review this task.
Thank you for submitting this interesting and challenging task.
Feedback according to the review criteria:
Correctness: Good.
Formatting: Good.
Specificity and thoroughness: This task is testing the ability of a language model to understand spatial relationships by creating internal "visualizations". Additionally, the ability of the model to generalize to different shapes and numbers of intersections is tested by multiple versions of the task.
Difficulty: Good
Not solvable by memorizing the Internet: Good
Novelty: Good.
Justification: The section "Behavior of existing language models on task" is missed in the Readme. Please test the task on existing language models to provide the baseline scores or motivate otherwise how we should interpret performance on the task.
Size: Around 1k examples per task, good. Please add the number of examples in each task to the Readme.
Compute resources: Feasible.

Thank you for your review! We updated the README to include the number of examples in each subtask, added a data source section, and added a section on the behavior of existing language models, where we report the performance of GPT-2 on a subset of our dataset.

Hi,
I'm one of the reviewers and pretty happy with the current status of the task. @asteroidhouse has addressed all the concerns raised by @jeknov, the other reviewer. The literature review is reasonably thorough as well though adding references to work on math word problems such as Learning to Solve Arithmetic Word Problems with Verb Categorization would help the literature survey. Since these are minor issues, I'll just add my approval.

@chiafullo Accept

Thank you for the review! We added the reference you suggested for math word problems.

@chiafullo Accept

The amount of tasks we received this round is tremendous.  With so many tasks to review we have decided to extend the review period to Tuesday, June 29.
Reviewers: if the submitter has made revisions please be sure to notify me with a formal ”accept” or ”does not meet criteria” by Tuesday, June 29  (if you haven't already done so).
this is an automated message

Your task has been accepted in the initial review stage.  The next stage of this process is for us to assign your task a meta-reviewer for final review and merge.  An assigned meta-reviewer will follow-up by commenting on the PR should it need further revisions.

LGTM!


Wrong state value of peer when it starts to build on Mac

When a user creates peer through Control Center, Peer state shows error message "Peer is broken" while PeerOS is being built.
Expected Behavior

CC should aware user that PeerOS is being built and it takes some time.
Current Behavior

Peer is broken means that Peer is not gonna work anymore and it will lead to the misunderstanding.

Didn't face this problem

Need to reproduce on Mac.

it has been fixed

Fixed in Master.

Done in Prod.


Question: Change lines' width of line-chart ?

Version
latest
Question
Is there some way to customize the line-chart's lines width ? The current line is too wide.

Duplicate with #113


put The Deck in an iframe

Did not do the iframe — the Deck's links don't have a target on them, so they were opening inside the iframe, and there's no simple way to defeat that.
Instead I used the Postscribe (https://github.com/krux/postscribe)  library to replace the document.write statement.

I'm closing this, but we need to keep an eye on the ads — I saw some odd behavior on dev (the ads loading the wrong place), but haven't seen it since.


RunAndGun Multiplayer Compatibility

Synced the Run And Gun gizmo, tested with a weapon, no desync

Alright, thanks.


Scala Stream Collector: add support for opt-out cookies

http://www.allaboutcookies.org/manage-cookies/opt-out-cookies.html
Piwik supports this:
http://piwik.org/docs/privacy/#step-3-include-a-web-analytics-opt-out-feature-on-your-site-using-an-iframe
I'm not sure if opt-out cookies are a good way to give end users control over what is tracked per-se, but I like the idea of making it easier for Snowplow users to give their users the option to opt out of cookie tracking.

Focused this on SSC given we are migrating away from Clojure Collector

Interesting. As the collectors are stateless, this would have to be a new cookie that registers that the nuid must not be set.

@BenFradet can you estimate the work involved here?

done as doNotTrackCookie, closing


[Fixes #79] NodePattern: Add support for multiple terms unions

This builds on  #109 and adds { multiple term | another multiple term branch }

Merged 🎉 . To be released tomorrow.


Without renaming the module, installation fails given the module name & URL mismatches.
PS C:\Users\user> go install github.com/olafhartong/DefenderHarvester@latest
go: downloading github.com/olafhartong/DefenderHarvester v0.9.8
go: github.com/olafhartong/DefenderHarvester@latest: version constraints conflict:
<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: defenderharvester
                but was required as: github.com/olafhartong/DefenderHarvester

thanks Maxime! :)


use parent_module() to pre-populate logging messages

There's more than a few instances where modules using log modules are setting the name of the module within the logging message. An example might look like:
module at(val) {
   log_error_if(val > 0, "at(): specified value val must be less than or equal to 0");
}

Including messages with the module name is incredibly helpful when tracking down message source. Consistently adding it to logging messages is a chore, made even more if logging calls are copy-pasted across modules.
parent_module() is a built-in that can rectify this: calling parent_module(1) yields the name of the module that just called the active executing module, and that name can be automatically inserted into the logging message if present. It's not a guarantee that there IS a parent module however - the logging call may come from the top level file executing - so that'll have to be checked. parent_module() may not work within a function, so that'll have to be checked also.

Commit for this issue now provides callers throughout logging.scad and those callers - parent_modules - appear within the console at the appropriate logging levels. Example:
//include <logging.scad>
include <z:/home/jong/src/openscad_logging/logging.scad>
LOG_LEVEL = LOG_DEBUG;

log_debug("from top level");

function a_true() = let(
    _ = log_debug("from within a func a")
    )
    true;

module a(){
    bool = a_true();
    log_debug("in module a()");
}

module b() {
    _ = log_debug("from within a func b ");
    log_error("yikes");
    a();
}

a();

b();

...yields:
Saved backup file: C:/Users/jong/Documents/OpenSCAD/backups/unsaved-backup-mrspnVCM.scad
Compiling design (CSG Tree generation)...
ECHO: "DEBUG: from top level"
ECHO: "DEBUG: a(): from within a func a"
ECHO: "DEBUG: a(): in module a()"
ECHO: "DEBUG: b(): from within a func b "
ECHO: "ERROR: b(): yikes"
ECHO: "DEBUG: a(): from within a func a"
ECHO: "DEBUG: a(): in module a()"
Compiling design (CSG Products generation)...

This behavior is only reliable at naming parent modules. Naming and capturing parent functions is as yet unimplemented, because I do not know how to get a stack trace into a non-log-emitted form; nothing I've seen so far indicates that such a value is even possible within OpenSCAD.


Implement helpers from pkg/runtime

Signed-off-by: Somtochi Onyekwere<EMAIL_ADDRESS>

Superseded by #575


fix creating url from route

It looks like there is some kind of mismatch when populating URLComponents such that scheme is never set and then you never get any url.
This PR makes it obvious by extracting the logic to create a url from a route and having a unit test with the example from the README.
Maybe it should be components.scheme = "freetime" but I'm not sure what needs to happen to the other components. I'm hoping this PR can help even though it only points out a problem.

I'm a little confused about the status of this PR.
Please let me know if there is something you need changed.

Thank you!! I’ll get Travis up and running
Sent with GitHawk


issue #2243 Improving azure cluster stability

#2243
Signed-off-by: Artem Glazychev<EMAIL_ADDRESS>
Description

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Merging...


Color temperature and brightness time based automations

Looking to finally replace LuxVR with OyasumiVR but am missing the ability to set color temperature automations and also have both color temperature and brightness change at different times. These are a pretty big necessity for me because otherwise I will not get naturally tired and fall asleep in VR. Also I'm pretty sure that OyasumiVR overrides LuxVR's brightness settings anyways.

This seems like a nice feature to add for sure, I wasn't aware of LuxVR, but being able to automate it in a similar way for vr sleeping purposes seems like a useful thing to have.

Also I'm pretty sure that OyasumiVR overrides LuxVR's brightness settings anyways.
I wouldn't quite say "override". If you have an Index, OyasumiVR mainly controls display brightness (the setting itself in SteamVR), the LCD backlight. Below 20%, or on any other headset, OyasumiVR does image brightness.  The brightness is controlled by how opaque that plane is. I'm pretty sure this is how other brightness overlays like OVRAS work.

Color temperature though, is a bit of a different issue. I'm just not quite sure how LuxVR does it. To do proper color temperature accurately, you'd somehow have to something like modify the existing image data and override what's being shown to the user. I am genuinely not sure how to do that.
Maybe a way to accomplish this would be to use a plane in front of the eyes, much like the brightness overlay, have it colored somewhere on a scale from yellow to blue, and have its opacity be determined by how close you are to neutral (less transparent on the extreme ends, fully transparent in the "middle"). Maybe this is how LuxVR does it, however I wouldn't call this accurate. You wouldn't be adjusting the color temperature of the image really, just slapping on a semitransparent color, and you absolutely wouldn't be able to really tie this to any meaningful/accurate number in kelvin.
Maybe this would be sufficient, maybe not, I'll at least have a closer peek at LuxVR, see if I can figure out how it does its thing. In the meantime, if you or anyone else has any idea of how to achieve this through OpenVR, feel free to drop a comment!

Heres an image of the UI in OVR Advanced Settings that allows you to change the RGB values. As you can see, they are not all at 100%, so I think that this may be the same thing LuxVR uses to change the color temperature.

I've tried looking at LuxVR's source code, but as it seems to be a Unity project I barely have an idea what's going on there.
I've also looked a bit at OVRAS's source how their color adjustment works, and that indeed seems to be what I expected: An overlay (similar to the brightness overlay) right in front of the user's eyes, with a transparent color on it: Not much adjusting the image, but more just overlaying a solid color.
I'm currently abroad without access to VR, but when I'm home I'll see if I can get similar results with OVRAS as with LuxVR. If the same effect can be achieved with both (indicating LuxVR probably does a similar thing), I'd say this is quite doable.

Ah I see now indeed, for Index/Vive/Vive Pro it's apparently possible to adjust the gain per color channel. Interesting considering the Index only has an LCD with a single backlight zone.
I'd be interested to look into implementing the color-overlay method, as that's something I could see myself using. Whether I'll end up spending the time on implementing hardware control for the Index, at this point probably not, but I'll try to set it up in a way that adding support for other methods can be done more easily.

Color/RGB gain does apply to other devices, such as Bigscreen Beyond, maybe it is acting on the image/compositor and not in hardware. I am not sure what "overlay" option is meant for, actually, the result is just bad when trying to use it

Hmm, ok that does open up more options. Makes me wonder why OVRAS documents it as only supported on certain hardware.
Maybe it's just not effective on wireless solutions (e.g. virtualdesktop/steam link/oculus link), but I just don't see why that would be the case if it's done in software.
I'll have a play around with it when I get back home next week.

I'll be closing this issue as CCT control has been implemented on the current beta.
For now this is only through SteamVR's color gain settings, so only native SteamVR headsets like the Vive (Pro), Valve Index, and Bigscreen beyond are known to be working. I might consider adding a fallback with a color overlay for unsupported headsets in the future.


git: Don't depend curl-config when cross-compiling.

Motivation for this change
Fixes building with pkgsStatic, where otherwise curl linker flags
would not be correctly determined.
Better approach than in #61527.
Things done

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

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

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/proposal-a-dedicated-cross-compiling-team/3824/2

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/proposal-a-dedicated-cross-compiling-team/3824/1

I tried to build it with the following: (perl support disabled because of #66741)
nix-build -E 'with import <nixpkgs> { crossSystem = { system = "armv7l-linux"; }; system = builtins.currentSystem; }; git.override { perlSupport = false; }'

It got me the following:
/build/.attr-0: line 3: /nix/store/wlr38bsv3n6ipxi27mcln2ajriwfwag3-lndir-1.0.3-armv7l-unknown-linux-gnueabihf/bin/lndir: No such file or directory
builder for '/nix/store/qm9fpy3dc2ky4mc7xqqkdmbsv6c50gj9-curl-libs-and-headers.drv' failed with exit code 127

I think that it's a bug somewhere in the symlinkJoin function.

I wanted to look more into this today, but got a different dependency failing:
# running Build.PL --prefix /nix/store/v9wxq82azcpabd475y8lnw6509hagss9-perl5.30.0-Module-Build-0.4229-x86_64-unknown-linux-musl --installdirs site --config perl=/nix/store/0psxy3wyw9020yr2zmsrlrijrs7sgb7l-perl-5.30.0-x86_64-unknown-linux-musl-dev/bin/perl --fullperl "/nix/store/939gkqx4mbpf5df0s4vx57ii3n79ppm9-perl-5.30.0/bin/perl"
Checking prerequisites...
  test_requires:
    !  Couldn't find a $VERSION in prerequisite File::Temp

Checking optional features...
inc_bundling_support....disabled
  requires:
    ! inc::latest is not installed

ERRORS/WARNINGS FOUND IN PREREQUISITES.  You may wish to install the versions
of the modules indicated above before proceeding with this installation

Attempt to reload Scalar/Util.pm aborted.
Compilation failed in require at /nix/store/aj4caa48nx3ql3mp710sbnm726n4yymr-perl-5.30.0-x86_64-unknown-linux-musl/lib/perl5/5.30.0/x86_64-linux/Data/Dumper.pm line 304.
Couldn't run Build.PL: No such file or directory at lib/Module/Build/Compat.pm line 341.
builder for '/nix/store/29s2b0cav595kpag1855v466cd4rgbf1-perl5.30.0-Module-Build-0.4229-x86_64-unknown-linux-musl.drv' failed with exit code 2

Any idea what that is?

@nh2 Can you please resolve the merge conflict?

@nh2 Can you please resolve the merge conflict?

I'll have to re-test this thoroughly then and also look in detail into the feedback given above; converting back to draft PR for now.

pkgsStatic.git almost builds but fails at t2082-parallel-checkout-attributes.sh so closing this


Fall back language

Updated load function to use $filename in place of $this->langFilePath.
Assign merged arrays back to $config in init function.
Return merged array $a in array_merge function.

Sorry I will try again with only the changes I have made showing in the diff.


Quote and Narration Layout Edits: Desktop

This pull request solves a couple issues that were raised through FSP review with some new organization to the desktop version.
One concern is how difficult it was to notice and/or pay attention to the narration box when it was across the bottom of the screen. Another concern is that because the quote text was loaded as a d3 text element underneath another svg layer, the links didn't work to be able to open the links in the desktop version. (note, this is a moot point in mobile because the quotes are in the bottom nav bar, which is clickable.) Also, during preliminary accessibility checks, I noticed that the d3 text wasn't getting picked up as tabable in the keyboard navigation, this also solves that problem. Finally, several people noted that the images seem randomly scattered on the timeline, rather than placed consistently.
Solution:
Extract the quotes in desktop from the svg and add them as their own div on top of the chart. In addition, put the drought image, title, and narration text all in their own section (implemented with css flex grid).

Additional benefit: Everything is now tied to the major drought events so that the scroll triggers are simpler and the image, quote, title, red inset map highlights, and narration are all synced up. This reduced the overall complexity of several components (e.g., droughtNarrations_desktop.js)

Landing view edits:

The arrow placement for the map here is off.
the map would look better off the chart  on the righthand side now that you have more organization.
"How to read the timeline" only includes some of the information on how to read the timeline, and some of it isn't within view. I like the effect of it being revealed as you scroll, but suggest moving the color legend so you see it when the page first loads
The photo above "how to read the timeline" is visually dominant. Suggest dropping for the first load. What if that leading text were all on the right side? that would reduce the number of different pieces to see
I really like the quote box, but think it would look better on the righthand side below the other text. That way we see more of the chart. You couldn't put the quote and the map both on the right side like I'm suggesting, but should do one.

Text behind chart:

A few other minor edits to address before merging:

numbering in the references needs to be removed.
change link and visited link colors. They stand out a lot and don't fit the theme of the page. Could use default colors or adjust to page style

text and photo alignment on right panel - make subtitle flush with image?

Thanks for the review, Cee. I'll incorporate your edits except for the adding more things to the right part until we can hear from @hcorson-dosch-usgs with some updated screenshots. The issue is that one of the biggest reasons for the change in layout with some elements on the right, was that on Hayley's and Mandie's laptop screen, the narration text was out of space (old screenshot here):

I have been designing this new layout to make sure there's enough room on the right for the longest narration, which is the Turn-of-the-Century drought. This is why the quote and inset map are on the chart, rather than on the right. Hayley can you pull this branch and then take a quick screenshot of the Turn of Century Drought (longest narration) and 1950s drought (long-ish quote) on your laptop screen so we can see how this is all fitting? (I can't quite mimic the size and resolution on my Mac, I can only get close)

Here are screenshots from my laptop screen - looks like there's quite a bit of variation in text sizing between platforms at the moment.

Firefox
Edge
Chrome

For the initial view, I think it could look nice to have the title and first paragraph extend fully across the page, and to place the 'How to read the timeline' level with the start of the timeline:

As is, the initial view remains really busy on desktop -- feels like there is too much to look at.

Agree with Hayley on the landing view.

I didn't realize the font sizes would be so different by browser, why is that? Is it predictable enough that we can add logic to make them the same?

I didn't realize the font sizes would be so different by browser, why is that? Is it predictable enough that we can add logic to make them the same?

I goofed and didn't realize that my Firefox browser was zoomed out, so the difference is not as stark as it appeared. I apologize for that. I've updated the screenshots, above.

I don't fully understand the variation from browser to browser, e.g., why Chrome text is bigger than Edge and Firefox in the updated screenshots. Browsers do have default font sizes -- the default font size in all of my browsers is 16px, which seems to be the standard. I took a quick look at our font-sizing specifications, which is a bit of mish-mash w/ holdovers from former sites of ours. In DroughtHistory.vue all but the yAxisText is set with px values for desktop. We're using em font-size values for the mobile-specific css at the moment. In App.vue there's a mix of px and em sizing.

So what to do?
We could explore adding browser specific css, but I think a better first step would be to improve how we size fonts throughout the page.
I've been thinking we should try to move away from em b/c the sizing is pretty inconsistent. From what I understand they are relative to the parent element, so vary depending on how deep in the page structure we are:
.
What to use to set font sizes seems historically to be a bit controversial among web developers. See this discussion. The second recent answer from this past January makes me think we should go w/ rem, which is relative to the root element only. (Though there is some discussion there about using em instead of rem to determine media queries, b/c of Safari ---- right now we use px). This article and this post (has helpful CodePens) support using rem - a quick scan of google search results seems like rem is recommended in most more recent articles/posts. It seems like the right choice for accessibility reasons, too, so I think we should give rem font sizing a go here.
For easier rem scaling, it sounds like one approach is to use % to scale the root font size to 10px, so that you can easily map any rem value mentally to its pixel value (this is from Stack Overflow):

a common suggestion is to make the apparent size of rem equal to 10px. The magic number for that is 10/16 which is 0.625, or 62.5%.
:root {
font-size: 62.5%;
}
If you're using the default font-size, I'm 10px tall.
The problem now is that your default font-size for the rest of the page is set way too small, but there's a simple fix for that: Set a font-size on body using rem:
:root {
font-size: 62.5%;
}
body {
font-size: 1.6rem;
}
I'm the default font-size
It's important to note, with this adjustment in place, the apparent value of rem is 10px which means any value you > might have written in px can be converted directly to rem by bumping a decimal place.
padding: 20px;
turns into
padding: 2rem;

Of course the other complexity is that some text is added directly to the <html> and some is added to the <svg> with D3... I'm not sure how that plays out with sizing, and if any <svg> scaling settings mess w/ the css font sizing...

I like this 10 px rem approach. It's always been an issue with different scaling when text is added directly to the svg, so that's going to need to be addressed regardless.

Ok team, I think I incorporated all the above feedback. Major thanks to Hayley for sorting out the text sizing information!!
One of the biggest changes I made, after a lot of deliberation about how to keep the focus on the data in the timeline and to reduce clutter, was to remove the drought-specific images from the site. This allows there to be a lot more room for text, the inset map can be on the right, and I think that the value of the photos (especially on mobile) was getting lost as it was.
Landing page (Firefox):

Landing page (Safari):

1970s drought (Firefox):

Took regional descriptions out of here, put them as alt text

Requesting re-review from @cnell-usgs, and it might also be nice to see how this is working on small laptop from @hcorson-dosch-usgs. Although, Mandie is presenting the website to OCAP tomorrow, so getting it merged and pushed to beta before that would be great.

I'll pull the changes and take some screen shots. From your screenshots, the changes look great!

Overall I think this is a big improvement -- much cleaner, and I don't miss the images.
See screenshots and notes for a few issues.

Firefox
Edge
Chrome

Landing page

On Edge, the drought buttons wrap to a second line
Could we either hide the first quote until scroll, or drop it? It blocks most of the timeline from view on a laptop-sized screen

Drought periods

I think the main issue I see is with text sizing -- the drought descriptions go off the page for each drought.
The quote for each drought also needs to be a bit higher up the page, as the longer quotes get cut off.

Regional section

In the regional section, you might trying setting a max width for the grid, at least on laptop sizes, to bring the violin charts closer to the polar chart.

Authors

Could we link Scott's chart (from twitter) when it is referenced?

Another option for the drought descriptions at right (rather than dropping the font size) would be to split each in two and have it switch to the second part halfway through the drought

Also - LOVE having the map in the upper right. Feels really natural there and looks great

Another option for the drought descriptions at right (rather than dropping the font size) would be to split each in two and have it switch to the second part halfway through the drought

This is how I handled this issue in mobile, and I think it works well. I'll set up an issue now and incorporate that into the next version.


Grant your VM access to a Secret stored in a Key Vault

Hi ,
To grant VM to access the secrets key , we need to add Get + List in Key Management also for successful secrets retrieve , need to update the documentation to include the permissions for Key Management.

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

ID: b6ae37a8-0a93-e077-d15f-50d2d0d6aa99
Version Independent ID: 41e1b28a-704d-431c-5dc8-9cbf063f9cc3
Content: Use a Windows VM system-assigned managed identity to access Azure Key Vault
Content Source: articles/active-directory/managed-identities-azure-resources/tutorial-windows-vm-access-nonaad.md
Service: active-directory
Sub-service: msi
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@kgopi05 Thanks for the feedback! We are currently investigating and will update you shortly.

@kgopi05 Can you please elaborate a little bit more? You need access to the keys only if you are retrieving them in the VM. Were you getting any error trying to get secrets when you did not give the permissions to the keys?
Can you provide the steps you followed so that I can reproduce this?

Hi,
I have provisioned the VM with System Managed identity and try to access the Key Valuts secrets
as per the document , however , i'm got Access denied message , Raised the Microsoft case #<PHONE_NUMBER>00540 and as per the Engineer advise along with secrets management permission i have given Key (Get+List) permissions and then able to retrieve the secrets .

@kgopi05 Thanks for the info. I am working internally to validate this and will get back to you.

@kgopi05 I worked with the engineer on that case. We confirmed that permissions to the keys are not required for retrieveing the secrets.
You will recieve an email about this. I will go ahead and close this thread. If you ahve any further questions regarding this, feel free to tag me.


Added pipe implementation

The program can be run only with the combination of --help(optional), --pipe  and --file (in any order), any other combinations will result exit error. Named pipe (FIFO) is used in this program
Run the program:

./environment_setup.sh to set up bazel environment
./bazel-bin/src/send --pipe <pipe name> --file <file name>
./bazel-bin/src/receive --pipe <pipe name> --file <file name>

Run test:

./build_and_test.sh to build the programs and run all tests

P.S  "test2" is the test file used in this program, queue implementation is commented out for another PR.

For unit testing, one separate test folder "testReceive" is added for running parallel test for send and receive methods

I think you should click on Resolve conversation when you change your code with a comment or agreed with the reviewer.
There still is 2 comments from me that you didn't change anything or respond.

Can you tell me which comments you are referring to?

Outdated

I had a comment on them. Sometimes, when there are too many comments you have to click on an expand button to see them.


Missing jobs.<job_id>.defaults.run.working-directory

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_iddefaultsrun
What part(s) of the article would you like to see updated?
There should be a jobs.<job_id>.defaults.run.working-directory
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_iddefaultsrun
Says:

Use jobs.<job_id>.defaults.run to provide default shell and working-directory to all run steps in the job. This keyword can reference several contexts. For more information, see "Contexts."

That should be visible in the ToC on the right, and have proper paragraphs. I shouldn't have to read the entire document ot discover this.
Additional information
It should explain how it interacts with previous levels.
There's a workflow that uses a global:
defaults:
  run:
    working-directory: ruby

I have no idea how to reset the working directory to the default in one job.

Refiled as #31346


fix: remove defaultValue from controller checkbox

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.2-alpha.20a67b.0@codecademy/gamut-labs@19.16.14-alpha.20a67b.0@codecademy/gamut@35.4.2-alpha.20a67b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6100173bd1c5ff1bf52c9afe--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.2-alpha.a6bc7f.0@codecademy/gamut-labs@19.16.14-alpha.a6bc7f.0@codecademy/gamut@35.4.2-alpha.a6bc7f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6100483a5074ef6042745a53--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.2-alpha.c8aa57.0@codecademy/gamut-labs@19.16.14-alpha.c8aa57.0@codecademy/gamut@35.4.2-alpha.c8aa57.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6100517cdc94cf0092efd543--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.2-alpha.d915c9.0@codecademy/gamut-labs@19.16.14-alpha.d915c9.0@codecademy/gamut@35.4.2-alpha.d915c9.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://61005eecab039e190d5e65a5--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.2-alpha.4c533d.0@codecademy/gamut-labs@19.16.14-alpha.4c533d.0@codecademy/gamut@35.4.2-alpha.4c533d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://610067761063881f5e1f6f56--gamut-preview.netlify.appDeploy Logs


Make auto choose future tech/civic

When player plays long game he researched everything on tech/civic tree. At this point is only possible to choose "Future technology/civic".
Couldn't be good that CQUI does this for you: automatically choose next future tech/civic so you have less clicking, you only see small message that future tech/civic is chosen?

Science and Culture do get very annoying at the end of the game and they should have put a solution for that in the base game, but they also should have done a lot more for the UI overall. I would certainly appreciate taking that little nightmare away, but just keeping CQUI working after all these patches is probably more work than the contributors want to do.

@Azurency Does this still need to be done?


[FLINK-21030][runtime] Add global failover in case of a stop-with-savepoint failure

What is the purpose of the change
In FLINK-21030, we experienced a bug when calling stop-with-savepoint on an embarrassingly parallel job where one pipeline failed causing the job ending up in an inconsistent state. The solution was to trigger a global failover in such a case.
Brief change log
We couldn't rely on the ExecutionGraph's terminationFuture to complete. The solution was to monitor the terminationFutures of all current executions, instead. We cause a failure if at least one of them switches to a non-FINISHED state indicating that something went wrong while stopping the job.
Verifying this change
Two tests were added:

testing the behavior if the failure happens during savepoint creation. No explicit restart needs to be triggered. The restart is handled by the CheckpointFailureManager.
testing the behavior if the failure happens after creating the savepoint. In this case, a job failover will be triggered that brings all tasks back into RUNNING state.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: yes
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@pnowojski @rkhachatryan Could one of you review d9330d5 to get an approval for this code change from @AHeise ?

Could one of you review d9330d5

d9330d5 looks weakly good to me. Weakly as from my experience it's easy to miss some corner cases in the cancelation/cleanup code and I wouldn't be surprised that it would cause some test failures.

CC @rmetzger

I followed @tillrohrmann proposal and introduced StopWithSavepointOperations. Its implementation StopWithSavepointsContext implements the stop-with-savepoint using states which makes the code easier to read. Additionally, StopWithSavepointContextTest replaces the DefaultSchedulerTest stop-with-savepoint test cases.

The following state diagram reflect the most recent code changes to support the code review.

Adapting the StopWithSavepointOperations interface in 46f8c9e helped reducing the complexity of the state graph:

Nice :-)

Thanks for looking into the changes, @tillrohrmann. I addressed your comments and replied to some of them. Please have a look.

I think I have given bad advice on the savepoint clean up @XComp. I think that the CheckpointCoordinator will use the savepoint for recovery once it has been completed. Hence, we actually cannot delete the savepoint if a failure happens after the successful creation of the savepoint. Sorry for misguiding you here.
The only solution I can think of atm is that we could log the path to the savepoint so that the user can manually clean it up after the job has been terminated. I think we can only properly solve this problem after FLIP-47 has been implemented.

I squashed and rebased the branch adding another cleanup hotfix.
@AHeise Could you double-check your change 5bf7da7. I had to resolve some minor conflicts due to @pnowojski fix 19ceee05

I rebased once more due to UnalignedCheckpointCompatibilityITCase failing: The test was fixed on master by 6d2a05a.

I squashed and rebased the branch adding another cleanup hotfix.
@AHeise Could you double-check your change c249642. I had to resolve some minor conflicts due to @pnowojski fix 19ceee05

LGTM!

Thanks @tillrohrmann for the remarks. I applied your comments and created PR #15034 (release-1.12) and PR #15035 (release-1.11).

Build #290 contains two failures:

FileBufferReaderITCase.testSequentialReading is covered by FLINK-21416
I created FLINK-21515 to cover SourceStreamTaskTest.testStopWithSavepointShouldNotInterruptTheSource

The test failures seem to be unrelated. Also the e2e have passed for 93304d6. Merging it now.


C62241: Fixing hyperlink syntax

Hello, @rmcmurray,
Localization team has reported source content issue that causes localized version to have broken/different format compared to en-us version.
Description: missing brackets () in source file prevent hyperlink from being created.
Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR  then share your PR number so we can confirm and close this PR.
Many thanks in advance.

#sign-off

Thanks so much @rmcmurray , although it seems it wasn't automatically merged with the sign-off. Could you please Merge it manually on your side or ask some manager to merge it?
Regards.
CC @mbaldwin

Thanks so much @rmcmurray , although it seems it wasn't automatically merged with the sign-off. Could you please Merge it manually on your side or ask some manager to merge it?
Regards.
CC @mbaldwin @arob98 @JennieHubbard @Alisha-Acharya

Hi @rmcmurray , It seems this wasn't automatically merged with the sign-off. Could you please Merge it manually on your side or ask some manager to merge it?
Regards.
CC @mbaldwin @arob98 @JennieHubbard @Alisha-Acharya

@Taojunshen can you merge this?

@Taojunshen can you merge this?

@Taojunshen can you merge this?


wizard: Running a snapshot silently fails

One-liner
Wizard Snapshot fails to execute a snapshot but on the browser it says "Snapshot uploaded!".
Context & details
Creating the snapshot .py and .dvc files goes well, but when clicking on the button to execute the snapshot, on the browser I see "Snapshot uploaded!", however the snapshot was not properly executed. When I look at the terminal, it shows an error (for an example error, see this other issue: https://github.com/owid/etl/issues/1752).

hi @larsyencken,
In https://github.com/owid/etl/pull/1755 I improved the message after running the snapshot step.
However, the error is not caught from the web app at the moment, and the user still needs to check the terminal (that's what the updated message says).
I can look into improving this.

Improvement shipped in

https://github.com/owid/etl/pull/1809

Should be fixed now


endpoint play_media, doesn't work with existing

there is an existing playlist called "wellenrauschen"
the endpoint is working for a "Play Artists" or a specific album
I try to play it with api.play_media(self.zone_id, ['Playlists', 'wellenrauschen], action="Play Now")
API returns ERROR roonapi: Could not find media path element 'wellenrauschen' in []

The roon api media path isn't necessarily what you'd guess from using the app.
I tried to add really good error messages to deal with this.
Start with something like
roonapi.play_media(
    output_id, [""]
)

You should get an error like
2022-02-06 23:42:35,977 ERROR  roonapi -- Could not find media path element '' in ['Library', 'Playlists', 'My Live Radio', 'Genres', 'Qobuz', 'Settings']

This will give you the options at that point in the path, which you can hopefully use to build up the command you need.

I think I've used the API correctly and did exactly as you recommended. So my path was ['Playlists', ] But I was not able to grab the media information. The Python API replied with "no media found for". The error message can be found here: https://github.com/pavoni/pyroon/blob/e157ca11d63204682f74a81f9a6dc86d851ed2e1/roonapi/roonapi.py#L572

The error in your first message ERROR roonapi: Could not find media path element 'wellenrauschen' in [] means that the code was looking for wellenrauschen at a point where there was nothing at that point in the hierarchy. It also implied that it had found 'Playlists' ok.
The error you link in your second message is different - in that case it's found the media to play - but is looking for a specific command - and failing to find it.
Because this is all dynamic - I'm afraid you're going to need to investigate to try and find what's happening.
Perhaps try the command I suggested - and let me know the exact message.


Fully migrate the codebase to this repo from Yasumu's repo

Currently the functional runtime lives inside Yasumu's repo itself. We need to fully migrate the js runtime code to this repo.  What I mean is being able to add stuff needed from outside. Yasumu may need some extra implementation details on the other side.


Allow specifying config and cache directory via environment variables

This allows users to specify the location of the configuration and cache directory via the RESTISH_CONFIG_DIR and RESTISH_CACHE_DIR environment variables. As an example, this aids in a use case where we want to ship different API configurations for different products, let them manage the configuration, and invoke restish as needed.
If not specified, the existing default paths are used.

Thanks! :+1:


Bad text recognition

Describe the bug
Does not correctly understand the text of the Ubuntu server console.

Where is the bug

Grab Frame

To Reproduce
Steps to reproduce the behavior:
Grab Frame on the image attached
Expected behavior
That all characters are recognized correctly as there is no video interference that distorts the text or creates ambiguity about the characters.
Screenshots
Where did you get Text Grab?

GitHub releases

Exe

Desktop (please complete the following information):

OS: Windows 10 Pro
Version: 22H2 (19045.4529)
Text Grab Version: 4.5 (bed6db1)

Additional context
If I OCR the attached image (I opened it with Microsoft Paint of Windows 10 Pro) with zoom at 100%, I get the following text:
[ CIK ] Reached target System Power Off.
[ 1745.141662] sgstemd-shutdown [1] : Haiting for process: java

If I OCR the attached image (I opened it with Microsoft Paint of Windows 10 Pro) with zoom at 200%, I get the following text:
[ OK ] Reached target System Power off.
[ 1745.141662] sgstemd-shutdau-lr-l [1] : Hai ting far pracess: java

The correct text is:
[ OK ] Reached target System Power Off.
[ 1745.141662] systemd-shutdown [1]:  Waiting for process: java

With the comand: Get-WindowsCapability -Online | Where-Object { $_.Name -Like 'Language.OCR*' }, I have these languages installed:
Name  : Language.OCR~~~en-US~<IP_ADDRESS>
State : Installed

Name  : Language.OCR~~~it-IT~<IP_ADDRESS>
State : Installed

Ubuntu console font parameters:
FONTFACE="Fixed"
FONTSIZE="8x16"

This is a bug with the underlying OCR models being used. I am eager to use updated models when they become available to 3rd party developers, but currently the best way to improve would be to use the Text Grab settings to install Tesseract and use those models.

OK, thanks, I'll follow your advice 😃


Add status updates and logs fetching from github actions

The app submitters must be able to see within the warehouse what happened to the last requested build and if that failed, then inspect the logs

Added fetching the job link once the job is started and showing that to the user in the success message.


Creation of $secpassw for ACR Service Principal fails after migration to MicrosoftGraph API

The below line is used to generate a secure password which is passed to New-AzADServicePrincipal
$secpassw = New-Object Microsoft.Azure.Commands.ActiveDirectory.PSADPasswordCredential -Property @{ StartDate=Get-Date; EndDate=Get-Date -Year 2024; Password=$password}
When  attempting to do this with version 7.1 of the Az Modules, you receive the following error:

New-Object: Cannot find type [Microsoft.Azure.Commands.ActiveDirectory.PSADPasswordCredential]: verify that the assembly containing this type is loaded

Which according to this issue: https://github.com/Azure/azure-powershell/issues/16671, relates to the shift to MicrosoftGraph API with newer Az PowerShell modules.
The sample needs updating to reflect this and an alternate approach

In addition, any references to ApplicationId are now likely to be AppId instead.

@fillip1983 Which sample is this for?

@fillip1983 Which sample is this for?

Apologies. It’s for the creation of a Service Principal for use with an ACR:
https://github.com/Azure/azure-docs-powershell-samples/blob/master/container-registry/service-principal-create/service-principal-create.ps1


Can not import dataset when cvat is behind cloudflare tunnel proxy

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce

Export dataset from roboflow
Convert dataset from roboflow_coco to coco 1.0 using datamuro
Zip the dataset
Navigate to your cvat instance proxied by Cloudflare tunnel.
Project > Actions > Import Dataset
Select COCO 1.0 as format and try to upload the zip file
Try to upload stuff_train.json

Expected Behavior
I expect to be able to import datasets into the project
Possible Solution
No response
Context
Trying to import dataset and I get an error. Here are some things ive done to my cvat env already:

Applied the CSRF_TRUSTED_ORIGINS fix (for POST on django admin page)
Set CVAT_HOST=cvat.<REDACTED>.com
Set traefik '--entryPoints.web.forwardedHeaders.trustedIPs' to all of cloudflares IP ranges and my local host's IP that is running cloudflared (fixes not being able to POST aka create tasks)

This error does look exactly like the error I was having when trying to create a task, but there is not a Content-Mismatch warning
Error
Error: tus: failed to upload chunk at offset 0, caused by [object ProgressEvent], originated from request (method: PATCH, url: http://cvat.<REDACTED>.com/api/projects/1/dataset/d85df7ff-29b6-40dc-a5ad-2c0bf296d5a2, response code: n/a, response text: n/a, request id: n/a).

My site is https only and tus seems to be requesting http://.
How can I fix this?
Environment
- Latest cvat docker images (tags: latest and dev) both tags have same issue.
- Docker version: Docker version 24.0.6, build ed223bc
- Debian Bullseye (Proxmox)
- `docker compose logs` have no erros I can see.

cvat_server                    | INFO:     <IP_ADDRESS>:0 - "POST /api/projects/1/dataset?org=&use_default_location=true&format=COCO+1.0&filename=cvat.zip&conv_mask_to_poly=true HTTP/1.0" 202 Accepted
cvat_server                    |
cvat_server                    | 2023-10-13 00:04:11,575 DEBG 'uvicorn-0' stdout output:
cvat_server                    | INFO:     <IP_ADDRESS>:0 - "POST /api/projects/1/dataset/ HTTP/1.0" 201 Created
cvat_server                    |

There is a weird warning in traefik DEBUG logs for cvat_server and cvat_ui:
traefik                        | time="2023-10-13T02:02:16Z" level=warning msg="Could not find network named 'cvat' for container '/cvat_ui'! Maybe you're missing the project's prefix in the label? Defaulting to first available network." providerName=docker container=cvat-ui-cvat-ffcdb0fe823bfd37e399ab2b68c2fcddd89e28447f87bb268f417b41e429ab2d serviceName=cvat-ui
-------------------------------------------------

traefik                        | time="2023-10-13T02:02:16Z" level=warning msg="Could not find network named 'cvat' for container '/cvat_server'! Maybe you're missing the project's prefix in the label? Defaulting to first available network." serviceName=cvat providerName=docker container=cvat-server-cvat-18b2d93dc8c774858312dc9ce0257f0e574dc0b483dda959b10872dc886e35ac

There is a cvat_cvat network.
╰─❯ docker network ls
NETWORK ID     NAME                  DRIVER    SCOPE
32463f9aaca3   bridge                bridge    local
e626d6bee2b5   cloudflared_default   bridge    local
e663802c6bf7   cvat_cvat             bridge    local
4105980eab99   host                  host      local
4ea78961b37d   none                  null      local

Both cvat_ui and cvat_server have
# cvat_server
    networks:
      cvat:
        aliases:
          - cvat-server
# cvat_ui
    networks:
      - cvat

More weird actions: I was getting the same error when creating a task but all of a sudden, It allows me to create a task, but it sits at 0%.

I am also noticing some weirdness with tags on images. I issued a docker compose pull and now I am seeing latest and dev tags.
cvat/ui                           dev               c0b16e084d1f   45 hours ago    145MB
cvat/server                       dev               d2b9351cedf0   46 hours ago    2.77GB
traefik                           latest            cc365cbb0397   3 days ago      151MB
cvat/ui                           <none>            d99892aa9005   4 days ago      145MB
cvat/server                       <none>            8c2b8b050d3a   4 days ago      2.77GB
postgres                          15-alpine         9f959158ce82   10 days ago     237MB
cvat/server                       latest            ed304b59905c   11 days ago     3.14GB
cvat/ui                           <none>            5358101b1b13   3 weeks ago     144MB
cvat/server                       <none>            e685cf17c422   3 weeks ago     2.78GB
postgres                          <none>            ab8fb914369e   2 months ago    237MB
traefik                           v2.9              d1e26b5f8193   6 months ago    138MB
clickhouse/clickhouse-server      22.3-alpine       d4088d71d1dc   6 months ago    773MB
eqalpha/keydb                     x86_64_v6.3.2     b9c88d8f9d06   8 months ago    129MB
grafana/grafana-oss               9.3.6             f925787d0f79   8 months ago    317MB
timberio/vector                   0.26.0-alpine     d8ecc9831523   10 months ago   122MB
openpolicyagent/opa               0.45.0-rootless   8723f2dc306a   12 months ago   84.3MB

Just updating, its been almost 24 hours and the task is still sitting at 0% importing.

Hi, sorry to bug again but can anyone possibly take a look at this? I can't create any tasks or import any datasets so, I can't use cvat.
I really don't want to use roboflow. I would prefer to run cvat but, it has been unbelievably unreliable. I've spent more time debugging and fixing cvat than I have working on my projects.
Any advice or pointers would be appreciated.

I nuked the install and managed to get things working again.


Where is default admin password?

Gitea version (or commit ref):
Git version:
Operating system:
Database (use [x]):

[ X] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[X ] Not relevant

Log gist:

Description
Hello, after installation into docker by compose i still don't understand how to login into admin panel. Google didn't help me find default admin password on your web site.
Screenshots

Just use the password for the first account you made

@zloidemon There is no default password during the install process you would be prompted to create a user, otherwise the first user registered to Gitea becomes the admin.

@techknowlogick thank you. Found at bottom on the install page. I seem better add checking that form before installing with modal window to avoid installation.

Hey. But if I have set DISABLE_REGISTRATION=true from docker environment, how can I login as admin? I can't create a new user. And localhost:3000/install page is not available (404) to access.

Use the command line tool gitea admin create-user. Ensure you have set the config values correctly in particular the -c option for the app.ini.
See https://docs.gitea.io/en-us/command-line/

@zeripath Thanks. Yeah, for now, I did this to initialized manually after creating docker successfully. But it still need do some extra "hack in" work, which cannot left everything in a single yaml configuration.
In docker-compose, there is still no post-run/postUp/onrun feature to execute custom action after container is ready. I think that is a much better way to initialize. See docker-compose issue#1510 and docker-compose issue#1809 for more discussions.
or let gitea add a specific process to handle this DISABLE_REGISTRATION=TRUE case.


NavView Hamburger Button Clipped OnLoad

Describe the bug

NavigationView hamburger button is clipped OnLoad. Repros in "NavigationView Stretch Test" page and "NavigationView Animation Test" page.
Steps to reproduce the bug

Steps to reproduce the behavior:

Go to NavigationView Stretch Test page or NavigationView Animation Test page of MUXCTestApp
Hover over Hamburger button

Expected behavior

Hamburger button backplate should not be clipped
Screenshots

Version Info

NuGet package version:
MUX 2.6

Windows app type:

UWP
Win32

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

October 2020 Update (19042)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop

Xbox

Surface Hub

IoT

Additional context

I've seen this, too.

Closing as it no longer repros - fixed somewhere along the way.


Typo correction

Corrected end of line 25 to read "restart code package".

@RJM76 : Thanks for your contribution! The author, @motanv, has been notified to review your proposed change.

@RJM76 Thanks for the fix.  #sign-off

@rwike77: I'm sorry - only the author of this article, @motanv, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


[Backport 2.x] [Remote Store] Make translog transfer timeout configurable using dynamic setting

Backport https://github.com/opensearch-project/OpenSearch/commit/b7396e1cbaf5d8d10ad2917afbaab54f1d4e2816 from https://github.com/opensearch-project/OpenSearch/pull/12704

This had to be manually backported due to conflicts in CHANGELOG.md.


Evaluation code

I'm trying to reproduce your evaluation on GSO dataset but the results are different in your paper. Could you please provide the evaluation code on GSO dataset?

The difference may arise from the variations in input and output views during testing. We will release the code later.


comment blocks misformatted

a comment block like this

turns into this

and the compiler complains about an unclosed comment

it would be cool if those comment blocks stay the same

this behaviour seems to be weird as well

^ actually doesn't seem to be restricted to comments

Strange! I'll look into this. Thank you for the examples.

Fixed the first example in version 0.2.3.
If you're still running into that alternating indentation issue, would you mind sending a copy/pasteable code snippet so I can fully recreate this on my end?

I think it's easier if I just provide access to the repo, sent you an invite.
https://github.com/flowerpowerdao/punks-nft-canister/tree/ac9c898d81cb86f399bf98df57b6393e4425ec9d


Support non-fungible tokens

While non-fungible tokens (NFTs) can be supported by creating a new fungible contract with a single unit, this would be an extremely wasteful workaround. Should support NFTs as native resources.
Rather than having an amount like fungible resources, non-fungible resources should have some sort of identifier nonce, which when combined with the NFT contract ID indexes a unique NFT. In other words: nft id = hash(nft contract id ++ nonce). This prevents resource ID collisions in the balance tree.

Native non-fungible tokens would add non-trivial protocol complexity, and ledger-based NFT contracts work just fine.


About reset messages in parallel nodes

Hi @altacch
About the Reset Messages for the parallel node. I think the running children need to be halted whenever the parallel node returns success or failure.
Consider the example Parallel(T1, T2, T3, N=3). As soon as one child T fails, the execution of the other children is no longer needed.
What do you think?

Sounds right, it would be the analogue of a "short-circuit" evaluation strategy for the usual Boolean operators. It may be nontrivial to implement it in Coq, though...

Reset support for parallel nodes has been added (following the above logic). Of course, some testing is probably needed...


Add NewHandler function to handle handler creation

Sorry, I did not notice this before but this PR may be redundant.
Even without a "NewHandler" function, one can easily create a handler from a func (payload) by currently doing
handler := eventbus.EventHandlerFunc(func (payload) {
    // handle stuff
})

I remember doing this because it is similar to http.Handler and http.HandlerFunc in the net/http package.
Although I am not considering renaming EventHandler to simply Handler and EventHandlerFunc to HandlerFunc


No tables get created at staging

Before submitting your issue, please make sure that you've checked the checkboxes below.

[ 2.4.x ] I am running the latest release version of Cachet.
[7+ ] I am running at least PHP 5.5.9. You can check this by running php -v.
[x ] I have ran rm -rf bootstrap/cache/*.

In a previous environment I was having no issues deploying Cachet, tables were created in the database and all. Now, with a new MariaDB service, it seems tables do not get created at all (as if migration never happened), but at staging I see no obvious error logs of what could have caused this. Would you have any suggestion regarding where to look?

this should be relative to your new mariadb, as you said it is the only which change, maybe you should check roles on your mariadb


ServiceBusReceiver raises ServiceBusError instead of SessionLockLostError when trying to complete a message with an expired lock

Package Name:  azure-servicebus
Package Version: 7.8.1
Operating System: Linux
Python Version: 3.7.9

Describe the bug
ServiceBusReceiver raises ServiceBusError instead of SessionLockLostError when trying to complete a message with an expired lock
To Reproduce
Steps to reproduce the behavior:

put message on the servicebus
collect in with peek-lock mode
wait until the lock expires
try to complete the message
catch exception raise

Expected behavior
raise SessionLockLostError according to the documentation.
The bug is clearly visible here.
should be:
if not self._session and message._lock_expired:
            raise SessionLockLostError(
                message="The lock on the message lock has expired.",
                error=message.auto_renew_error,
            )

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context

azure.servicebus.exceptions.ServiceBusError

The lock on the message lock has expired.

  File "/usr/local/lib/python3.8/site-packages/algotools/service/service_classes/message_service_abstract.py", line 82, in do_service
    self.complete_message()

  File "/usr/local/lib/python3.8/site-packages/algotools/service/service_classes/message_service_abstract.py", line 312, in complete_message
    self.message_manager.complete_message(self.message)

  File "/usr/local/lib/python3.8/site-packages/algotools/service/messages_manager/azure_servicebus_msg_receiver.py", line 66, in complete_message
    self.receiver.complete_message(message)

  File "/usr/local/lib/python3.8/site-packages/azure/servicebus/_servicebus_receiver.py", line 853, in complete_message
    self._settle_message_with_retry(message, MESSAGE_COMPLETE)

  File "/usr/local/lib/python3.8/site-packages/azure/servicebus/_servicebus_receiver.py", line 490, in _settle_message_with_retry
    raise ServiceBusError(

Thanks @IdoWSC . Ill take a look in to it, there is a comment right above that line where the generic error is being thrown due to a gap in features in the underlying protocol. Once I have a better understanding of whats happening Ill update this thread.


Trouble disabling request-body-object

I think that version 0.52 disabled overriding Spectral rules in .validaterc file. I followed this workaround to disable request-body-object and saw
Warning:  'spectral' rules can only be configured through a valid Spectral config file.
Spectral rule settings in the .validaterc file are ignored.

So I read the documentation and created .spectral.yaml file in the same directory as .validaterc. The content was:
extends: @ibm-cloud/openapi-ruleset
rules:
  request-body-object: off

However, it didn't do the trick and I still hit request-body-object rule errors. Ditto with --ruleset .spectral.yaml. I wonder if I am missing something or there is a bug.

I found that the following Spectral config works:
extends: '@ibm-cloud/openapi-ruleset'
rules:
  request-body-object: off

Hence my problem was with the single quotes. I didn't have them because I copy-pasted from the documentation.
My issue transforms to the linter not reporting YAML validation errors in .spectral.yaml and silently ignoring the config. Version 0.51.3 reports them.

@vmarkovtsev Documentation is now fixed, thanks to your PR. As for YAML validation errors, we no longer do the parsing but let Spectral handle it. We should be printing any Spectral errors we receive if there is an issue - are you seeing otherwise?

@dpopp07 As far as I see Spectral stays silent on parsing errors.

For me using extends: '@ibm-cloud/openapi-ruleset' with  -r .spectral.yaml parameter still does not work.
[Info] Problem reading Spectral config file, using default IBM Spectral ruleset. Cause for error:
ENOENT: no such file or directory, open '/home/*/*/*/*/@ibm-cloud/openapi-ruleset'

It only works when i do somehing like this:
extends<EMAIL_ADDRESS>rules:
 some-rule: 'off'

I hit the same error as @jnnyrttn in https://github.com/IBM/openapi-validator/issues/366

@dpopp07 After installing 0.54.1 i hit the same error. I see old versions or spectral-*.  in package.json
    "@stoplight/spectral-cli": "^6.1.0",
    "@stoplight/spectral-core": "^1.6.1",

I am using:
node v17.4.0
npm 8.3.1

@jnnyrttn Thanks for checking that. The latest version of the validator (0.56.2) includes pinned Spectral dependencies in the package.json. That has solved the issue for me. Let me know if you're still experiencing the issue in that version. If you are, I wonder if old versions are being cached somewhere.

I also see this when using the latest docker image to run. I.e. run using the following:
docker run --volume `pwd`:/data jamescooke/openapi-validator:0.57.0 swagger.yaml

The following .spectral.yaml seems to be ignored:
extends: '@ibm-cloud/openapi-ruleset'
rules:
  property-case-convention: off

And the following works:
extends<EMAIL_ADDRESS>rules:
  property-case-convention: off

@jnnyrttn @DanRoseus I am seeing this in the latest version of the validator as well. I don't think that pinning dependencies made a difference (my testing environment must have been messed up somehow).
This is a bug in Spectral and I've opened an issue against their repository - https://github.com/stoplightio/spectral/issues/2072
I'll note a workaround here - if you install the @ibm-cloud/openapi-ruleset package in a local folder and run the tool from that folder, Spectral is able to find the ruleset in the file system. Not ideal but hopefully that helps in the meantime.

@vmarkovtsev @jnnyrttn @DanRoseus To keep you updated on the latest info from Spectral, it turns out that it's actually required for rulesets to live in the local file system. See this comment on the issue I opened. So, it turns out the workaround I mentioned is actually just the standard use case. So you'll need to install the @ibm-cloud/openapi-ruleset in the same directory that your Spectral config file lives in.
I will update our documentation to reflect this and will close this issue once I do. If any of y'all run into issues using the prescribed method, post here and I can re-open.

I'm having this issue as well. I can't for the life of me get a handful of rules ignored for the latest version 0.81.0..the most notable being:

property-case-convention
oas3-schema

@smridge Can you share more about your configuration? This should now be working as documented

@smridge Can you share more about your configuration? This should now be working as documented

Ah ok, I will give this another try and see if I still run into any issues. Thank you for the reference!

This has been troubling me for a while now and I tried again today with v0.96.0 but unless I use @DanRoseus's workaround, my .spectral.yaml is ignored.

Are you seeing that the .spectral.yaml file is being ignored altogether or that it's not being honored because the ruleset it's pointing to can't be found?
The location of the ruleset in the filesystem relative to the tool does matter

Good question, I don't have the answer to that.
I'm using the docker version of the tool and run it like this:
docker run --volume "$(PWD)":/data jamescooke/openapi-validator:0.96.0 -v -s -r .spectral.yaml ./src/api.json
My guess is that it's the latter but I'm not sure how to debug it - the docker logs don't show any errors.

Oh okay, I see. It looks like it is a matter of not finding the file, since the docker image probably doesn't include the ruleset as a separate package. I know it's a little hacky but I think you can workaround this by pointing the config file directly at the ruleset dependency. Something like extends<EMAIL_ADDRESS>

Yep, that's what I'm doing. I just thought I'd comment to show this is still an issue (even though the ticket is closed) so others can find the solution more easily.

I appreciate it. I'm leaving it closed because it seems the issue is not with this project but with the Docker image, which is community-supported.
@jamescooke if you're still maintaining the Docker image, perhaps the image should include the ruleset package as a dependency so the config file can specify extends: '@ibm-cloud/openapi-ruleset' without any trouble.

@dpopp07 Yes, I'm still maintaining the Docker image, but I'm no longer using the validator on a daily basis at work.
I've opened https://github.com/jamescooke/openapi-validator-docker/issues/200
There are specific questions on this image issue - TL;DR:

What's the smallest failing test that can be built into the GitHub Actions system to ensure that the ruleset has been installed correctly?
Does the docker image build system need to track the version / tag of openapi-ruleset ?

All answers / support very welcome - please add them to https://github.com/jamescooke/openapi-validator-docker/issues/200


Bugfix: Combline planar grads for TFR with mean, not RMS

What does this implement/fix?
When plotting the topography of a TFR, planar gradiometers were automatically being combined using the RMS of both sensors. However, since TFRs are not sensitive to the direction of a dipole, this behavior is unexpected. Furthermore, this makes it impossible to see "negative" power when comparing the difference between two conditions. After computing power / TFRs, planar gradiometers should be combined by taking the mean, not the RMS.
This commit changes topoplots to combine planar grads using mean, not RMS, for TFR and PSD.
Additional information
This page explains a similar analysis in Fieldtrip: http://www.fieldtriptoolbox.org/example/combineplanar_pipelineorder/

thx @gbrookshire
can you update https://github.com/mne-tools/mne-python/blob/master/doc/changes/latest.inc
and https://github.com/mne-tools/mne-python/blob/master/doc/changes/names.inc to document the bug fix and get the credit for your fix? thanks

Hey @agramfort
Done -- Let me know if there's anything else you need!

Thanks @gbrookshire


Updatee family view

More information, more easily available, with the pedigree. Very early draft

Show pedigree rows to start

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

Project coverage is 77.15%. Comparing base (8ecf5ce) to head (67085d7).
Report is 3 commits behind head on dev.

Files
Patch %
Lines

test/data/generate_data.py
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #771      +/-   ##
==========================================
- Coverage   77.16%   77.15%   -0.01%
==========================================
  Files         157      157
  Lines       12948    12950       +2
==========================================
+ Hits         9991     9992       +1
- Misses       2957     2958       +1

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

Family view:

Participant View:

Analysis View:

Thanks @dancoates, I've:

applied your suggestions
added a quick route for an analysis page,
updated a few places to use a link instead of an anchor. Seems to work okay from my local testing!
Used word-breaks to force a wrapping to make the views appear better (including replacing <pre> with a <span style="font-family: monospace">)


rename "beakerx" to "python"?

i am pretty sure that this directory: https://github.com/twosigma/beakerx/tree/master/beakerx
should be named "py" or "python".  what about the one inside of it:
https://github.com/twosigma/beakerx/tree/master/beakerx/beakerx
?  right now when i clone this repository it gets named "beakerx" by default and 3 in row is too confusing for sure.

I thought about this and believe the current model is actually right, but on a technicality.
It is typical for a Python repo to share the name of the module we'd use in code. This ends up being something like a directory called jamesx and then a subdirectory also called jamesx. The repo name can be anything, so this is a convention.
In our case, we are packaging the usual form inside another directory, like jamesx/jamesx/jamesx.
If I break that apart and number them, like jamesx (1) / jamesx (2) / jamesx (3), I believe 1 and 3 should stay the same. 1 is the repo name and 3 is the module that gets imported in code. 2 is then arbitrarily changeable. python or py would both be suitable, but I am not sure what the convention is. Having a directory above the directory where "setup.py" lives doesn't come up too often.
I wonder if this issue will go away after we break the kernels out into their own packages?

oh you are right, creating new packages for each kernel changes this anyway, and we already have a PR to do that so no sense changing this now.  let's just focus on getting that PR right: https://github.com/twosigma/beakerx/pull/6333


[✨ FEATURE]: Toast messages for info error and warnings.

Feature Description 📝
Currently we are using window alert for any type of error or information. This not aligns with a modern web application. We can use vue toastify for this error or info alerts. Which will make the website more user friendly.
Motivation 🌟
Due to alerts the interaction with the website stop. We have to close it before doing anything. Also it is not a good practice to show alerts for error or info messages. Hence vue toastify can be of good choice.
Expected Behavior 🤔
We can see the demo of vue toastify here https://www.npmjs.com/package/vue3-toastify. This can be used all over the website where alerts are used.
Additional Information ℹ
If you thing that there can be better and more intuitive solution. Please let me know.

@jvJUCA Please assing me this issue.

@jvJUCA Please assing me this issue.

OK assigned, this update in user alerts will be such a addition to a better user experience.


[GSsoC2023] Adding a login page and using a better hashing algorithum

Is your feature request related to a problem? Please describe.
A new login page and better hashing algorithun to avoid data leakes as this is a finiancial app we should also use a stronger encryption while storing data.
Please assign this issue to me if possible.
Cheers
Describe the solution you'd like

better encryption.
better hashing
Memory management
Optimization

Describe alternatives you've considered

MD-5 hashing
256 bit encryption
Addition of export feature

Hi, can you please describe what the new login page looks like? what new features should it include?

I will design a new login page in figma and send it to you asap.

And for encryption I will code a Cypher and send it to you by tmmrw evening.
Sorry for this delay I have a class test worth 5 credit the day after tomorrow  😅
So I want some time.

Will it be possible @SagarSharma2809 ??

Hey @Supriyo-I, I believe the login page already exists.
but yes, all the transaction data is still publically visible on Explorers like Etherscan, I would appreciate it if you could come up with an approach to encrypt/decrypt that.
how about an approach where we encrypt the data with the user's public key and decrypt it with the user's private key?

Yes I will surely do that as soon as as possible
Thanks for giving me this opportunity

Go Ahead @Supriyo-I

can u assign me this issue

Hi @J0SAL ,
Sorry for the delay, the problem is taking some time.The delay is caused by my sem ending exams from 4th of June.
I will try my best to complete it by next week.
Sorry for this inconvenience.
Thanks 😊


Android keyboard animations isn't smooth and snaps to the position when It's in an Edge to Edge SystemUI mode

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create an empty page with scaffold
Place a Column with Expanded Container and at the bottom BottomAppBar with a textfield inside
Click on the textfield and watch it snap to position

Expected results
The keyboard pops out and the text field stays on top of it at all times
Actual results
The keyboard pops out and the textfield follows a little behind and then pops in the position. It only happens when it's in a Edge to edge systemui mode done with SystemChrome.
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    SystemChrome.setEnabledSystemUIMode(SystemUiMode.edgeToEdge);
    SystemChrome.setSystemUIOverlayStyle(
      const SystemUiOverlayStyle(
        systemNavigationBarColor: Colors.transparent,
        statusBarColor: Colors.transparent,
        systemStatusBarContrastEnforced: false,
        systemNavigationBarContrastEnforced: false,
      ),
    );

    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('HomeView'),
      ),
      body: Column(
        children: [
          Expanded(child: Container()),
          BottomAppBar(
            color: Colors.transparent,
            child: Container(
              child: const TextField(
                decoration: InputDecoration(border: OutlineInputBorder(), label: Text("Съобщение")),
              ),
            ),
          ),
        ],
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
This is the result that is happening when in Edge to edge
https://github.com/flutter/flutter/assets/51166786/11d18540-a88d-4d4a-9ce0-ac34a62aea29
This is the result that should be happening in Edge to edge but it does not
https://github.com/flutter/flutter/assets/51166786/216b25f1-3c68-48ad-b306-8f9f8fd92daf

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.0, on Microsoft Windows [Version 10.0.22631.2048], locale en-US)
    • Flutter version 3.10.0 on channel stable at D:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 84a1e904f4 (3 months ago), 2023-05-09 07:41:44 -0700
    • Engine revision d44b5a94c9
    • Dart version 3.0.0
    • DevTools version 2.23.1

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at C:\Users\kaloy\AppData\Local\Android\Sdk
    • Platform android-34, build-tools 33.0.1
    • ANDROID_HOME = C:\Users\kaloy\AppData\Local\Android\Sdk
    • Java binary at:
      C:\Users\kaloy\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\223.8836.35.2231.10320515\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.5.5)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.5.33627.172
    • Windows 10 SDK version 10.0.22621.0

[✓] Android Studio (version 2022.2)
    • Android Studio at C:\Users\kaloy\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\222.4459.24.2221.10121639
    • Flutter plugin version 74.0.2
    • Dart plugin version 222.4582
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[✓] Android Studio (version 2022.3)
    • Android Studio at C:\Users\kaloy\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\223.8836.35.2231.10320515
    • Flutter plugin version 74.0.2
    • Dart plugin version 222.4582
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1)
    • IntelliJ at C:\Users\kaloy\AppData\Local\JetBrains\Toolbox\apps\IDEA-U\ch-1\231.8109.175
    • Flutter plugin version 73.0.4
    • Dart plugin version 231.8109.91

[✓] VS Code (version 1.80.1)
    • VS Code at C:\Users\kaloy\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.62.0

[✓] Connected device (4 available)
    • Pixel 7 Pro (mobile) • 34031FDH30014X • android-arm64  • Android 14 (API 34)
    • Windows (desktop)    • windows        • windows-x64    • Microsoft Windows [Version 10.0.22631.2048]
    • Chrome (web)         • chrome         • web-javascript • Google Chrome 113.0.5672.127
    • Edge (web)           • edge           • web-javascript • Microsoft Edge 114.0.1823.82

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Seems to be fixed already: https://github.com/flutter/engine/pull/39391
I think we just need to wait for 3.11 release

@vishnumad oh, ok then. That is nice!
Thanks for redirecting me to this issue!


[rtl] Fix RF read enables for illegal instruction/fetch error

The read enables should only be asserted where an actual RF read will
occur. Where there is an illegal instruction or a fetch error the raw
decoder signals might still be asserted but should be squashed before
they become the true enable signals.

Ah yes, had missed that as I was just running VCS DV which doesn't check for such things.
It's due to how we're generating the illegal CSR signal. It's factoring in the enable (indicating the instruction is clear to execute) which it doesn't need to do. I had actually spotted this a couple of weeks ago when playing around with FPGA timing had made a note to fix it then totally forgotten about it! (I'll make sure I'll do a github issue next time I see something similar and don't fix immediately).

Requesting new review as this now has a second change for the illegal CSR logic.


Tracking issue: Forest executable re-organization

Issue summary
It's becoming increasingly painful to keep all commands in forest-cli. Pain points include commands that aren't just RPC calls to the forest-daemon and wallet commands that demand much higher scrutiny — a deeper discussion happened here.
Let's split forest-cli into three parts:

forest-cli: This executable is a human-friendly wrapper around the Forest RPC.
forest-tool: This executable handles tasks not involving the forest-daemon (such as downloading snapshots).
forest-wallet: This executable is a human-friendly wrapper around the Forest RPC. It's kept separate from forest-cli because it deals with sensitive data and the attack surface should be kept small. This will be the only executable that may access the store of private keys.

Notes:

Moving the wallet handling out of the daemon will break compatibility with Lotus. However, we can add a separate compatibility layer between the Lotus-miner and the Forest-daemon. This won't be a concern for quite a while, though.

Sub-tasks:

[ ] Move non-rpc commands to forest-tool

[ ] forest-tool fetch-params
[ ] forest-tool config dump
[ ] forest-tool snapshot fetch
[ ] forest-tool snapshot validate
[ ] forest-tool archive info
[ ] forest-tool db clean, side-note: Maybe this command should be renamed to db destroy.
[ ] forest-tool db stats

[ ] Print a helpful error message if old commands are used with forest-cli.
[ ] Remove unused command line flags from forest-cli.
[ ] Change the default RPC port from 1234 to 2345. We used 1234 to be compatible with Lotus, but we'll handle compatibility separately.
[ ] Move wallet commands to forest-wallet

[ ] forest-wallet send
[ ] forest-wallet new
[ ] forest-wallet balance
[ ] forest-wallet default
[ ] forest-wallet export
[ ] forest-wallet has
[ ] forest-wallet import
[ ] forest-wallet list
[ ] forest-wallet set-default
[ ] forest-wallet sign
[ ] forest-wallet verify

Other information and links
https://github.com/ChainSafe/forest/discussions/2274

Alternative proposal: a single forest binary front for these various commands, and better segregation in the code:

get rid of the Config psuedo global - subcommands request what they need through clap, with defaults provided through a config file

The configuration is already a known issue and there were attempts to make it right, unfortunately, it got buried at the bottom of the backlog.
As for one binary, it was the case for Forest some time ago. We decided to split. While splitting further the forest-cli is not exactly a must.
Having a separate forest-wallet is important from a security point of view. It is a critical piece of code that should be easily auditable, and the more clear the boundaries between forest-wallet and forest the better. Right now, we're not doing very well on this front. See https://github.com/ChainSafe/forest/issues/1517

Done.


Client connect interface

Codecov Report

Merging #121 (22ce230) into master (33b1757) will increase coverage by 0.43%.
The diff coverage is 44.30%.

@@            Coverage Diff             @@
##           master     #121      +/-   ##
==========================================
+ Coverage   43.15%   43.58%   +0.43%
==========================================
  Files          20       20
  Lines        1483     1489       +6
==========================================
+ Hits          640      649       +9
+ Misses        755      752       -3
  Partials       88       88

Flag
Coverage Δ

unittests
43.58% <44.30%> (+0.43%)
:arrow_up:

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

Impacted Files
Coverage Δ

executor/executor.go
42.54% <43.05%> (+3.09%)
:arrow_up:

executor/client.go
50.00% <57.14%> (+6.75%)
:arrow_up:

executor/systemload.go
68.08% <0.00%> (+3.19%)
:arrow_up:

Continue to review full report at Codecov.

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


Add testing matrix for multiple Python versions

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/61
Originally created by @aaronsteers on 2021-03-25 22:35:33

Currently the SDK is only tested in CI using Python 3.6. We should also have tests running for Python 3.7 (see #58) and Python 3.8. We should be able to use the matrix functionality for this: https://docs.gitlab.com/ee/ci/yaml/#parallel-matrix-jobs

View 4 previous comments from the original issue on GitLab


issue fixed Easy Issue 3 (Python) #1520

Pull Request Type
What kind of change does this PR introduce?

Solved an issue/task

What is the current behavior?
No code to print "Hello, Python!".
Issue Number: https://github.com/fork-commit-merge/fork-commit-merge/issues/1520
What is the new behavior?
Implementing the python code to print "Hello, Python!" to console

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Re-enable log plots

Using criterion 0.2.1 the plots are shown on linear scale. Previous related conversations (#106, #107) have shown that there is interest but the implementation was buggy/not sufficient. The plots are now in linear scale since 20c369dfafe9316e19e34a5cd3aff42cfa472314.
I'm attempting to benchmark multiple algorithms which have different scaling, for example with O(N^2), O(N), O(N log N) and O(sqrt(N)).
I can't easily compare those scaling on a linear scale as the O(N^2) obviously dominates all others.
Could an option to get a logarithmic scale be added back?
Thanks a lot for this amazing crate! 👍

Here's how it looks like:

Urg... silly me, it's already supported! :D
See http://japaric.github.io/criterion.rs/criterion/struct.PlotConfiguration.html
Here's an example:

Hey, I'm glad you solved your problem. Thanks for trying Criterion.rs!


Custom configure core/init.lua

Feature description
Hi, could you add a way to customize the contents of core/init.lua?
The gui_config() inside it overrides the font configurations in neovide's config.toml.
Additional information
No response

This will cause an error because gui_config() is called without any arguments.

This will cause an error because gui_config() is called without any arguments.
Perhaps a condition to check if the table is empty could be added to gui_config(), like:
local gui_config = function()
    if next(settings.gui_config) then
        vim.api.nvim_set_option_value(
            "guifont",
            settings.gui_config.font_name .. ":h" .. settings.gui_config.font_size,
            {}
        )
    end
end

U'r right. This code snippet can be a PR.


feat(@angular/cli): add option to preserve symlinks for module resolution

Similar to the NodeJS option of the same name.
This provides additional support for linked packages and peer dependency usage scenarios.

LGTM, thanks!

This is wonderful, but looks like this option is still undocumented?
https://github.com/angular/angular-cli/search?utf8=✓&q=preserve-symlinks&type=

whar can I do for solved it?


Add a PyInstaller build for unsupported distros

This adds configuration to build the CLI as a PyInstaller executable along with the necessary supporting code. The PyInstaller executable would be an option for users of distributions whose package manager we don't have support for.
Since there's no automatic update mechanism available, this also implements a new self-update command that pulls the latest version of the CLI and replaces the current version with that one.
Unfortunately, this version of the CLI does not ship with its own version of Docker Compose. Packaging Compose into PyInstaller isn't straightforward, and it can't be run as a module because the Python interpreter is embedded in the binary. Instead, the expectation is that the user installs Compose with whatever mechanism they prefer.
See #13, which this PR does not fix on its own.
To-do Before Merge

[x] Figure out where the executables should be uploaded to
[x] Update the self-update command to pull from a real URL
[x] Manual testing

[x] Ubuntu 18.04
[x] Ubuntu 20.04
[x] CentOS 7

Manual tests looking good!


Development

Added in basic requirements,

Add php 8 compatibity to tests
Add github actions for CI
Added initial tests
Added license file

Thank you.

Thank you.


[BUG] Code snippets don't appear on mobile

[BUG] Code snippets don't appear on mobile
Load https://chalarangelo.github.io/30-seconds-of-code/ on Chrome Android, or in the mobile emulation view on Chrome (on desktop)
Scroll down
Notice that the code snippets do not appear:

That should be a network error i guess. Have you tried refreshing the page ?

Yeah, several times. It works fine in Chrome on my laptop, but breaks as soon as I switch to mobile emulation in the dev tools.

Cannot confirm on Chrome, Firefox, Chrome Mobile Emulator, Firefox Mobile emulator, Chrome(Mobile), Samsung Browser(Mobile). What version of Chrome and Chrome Mobile are you using?

Deleted previous comment, have reliably reproduced the error on all browsers.
Can confirm on certain functions code does not appear, and on some functions code will appear after scrolling the code box. It seems to be sporadic which ones don't want to work and which do, and consistent for some functions.

Apparently, adding transform: scale(1) to the code elements fixes the bug in Chrome (probably different render handling, not sure?). Can others having the problem confirm that - I wasn't able to reproduce the bug in Safari or FF?

@Daniel15 Thanks for sharing the issue! We will fix that :wink:

This is most likely a bug with my CSS framework (I'm using the latest alpha, so this might be something I have overlooked). I'll see what I can do. @taltmann42's suggestion sounds promising. I'll test and see if it works.

@taltmann42 I can confirm. Happens on my android in Chrome. We will fix it.


Add Socratic 32

Topics and links can be pushed directly to this branch, or added as comments/review.

Misc / General Stuff

First Mobile Native Hardware Wallet, https://twenty-two.xyz/

UX/Design

Bitcoin Symbol Campaign, https://twitter.com/Nexo/status/1767566233384186180

Lightning/LDK

Unleashing Liquidity on the Lightning Network with "lightning-liquidity", https://lightningdevkit.org/blog/unleashing-liquidity-on-the-lightning-network-with-lightning-liquidity/

Preview looks good, ready for (squash) merge I think?


Untaint request in failover BBE

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #20379 into release-stage will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           release-stage   #20379   +/-   ##
==============================================
  Coverage          15.21%   15.21%
==============================================
  Files                 50       50
  Lines               1341     1341
  Branches             208      208
==============================================
  Hits                 204      204
  Misses              1121     1121
  Partials              16       16

Continue to review full report at Codecov.

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


[T3A3][W15-A2] Gng Jia Hui

Some comments added. Feel free to add more comment if you have any questions. Once you are done, you may close the PR after reading comments.


Some Question About the Ablation Exps

Thanks for your great work!
I understand u stress the unsupervised learning of the mask and I just read your code to make sure u successfully learn the mask in an unsupervised manner. But I wonder that we all get the occlusion maps as the supervision to train the MaskFlownet-S. Simply adding an EPE or a cross-entropy loss may guide the MaskFlownet-S to learn a better attention mask. I understand it will take a long time to generate all of the mask maps in these datasets. It is indeed a problem.
Here are some questions about the demonstration of middle results:

Did u do the comparison of the supervised and unsupervised learning of mask and their final influence on the result?
The mask seems to be right on the foreground-background case, but actually, we don't really care about the background flow which is far from the foreground. Do u get more visualization of the mask on the objects which are moving close to each other like the third and seventh row in Figure 12?

Thanks for your attention.

Hi Lidong, thanks for your interest and your insightful questions!
For question 1, we haven't done any experiment on the comparison. A problem is that the mask used to filter useless information (shaded area) after warping might not be the same with the ground truth occlusion map at each level. Please feel free to draw comparison between our method and others' and we are looking forward to hearing what you discover!
For question 2, we have already released the weights to produce the visualization, so you can use them to produce more! What we think is that, object-object case has smaller size and larger relative motion, so it might be harder than fore-background case, but the mechanisms behind them are the same- background can be seen just as an object with relatively small motion.

@simon1727 Thanks for your answer.
For question 1, the reason I talked it is that I used to conduct some exps on occlusion years ago. I did use the occlusion map(ground truth) to guide the refinement, it gave a great promotion. But the promotion becomes not that obvious when I switch the occlusion map to the supervised learned one. Although my result is lost now, I still remember how hard it is to learn the occlusion map.
It is a great discovery of your work to show that the unsupervised learning of attention maps might be more proper to guide the refinement. An unsupervised activation map from the learned feature map in the last few layers indeed shows the attention for the final activation. Thanks again for your work.
For question 2, what I am interested in is actually the big motion cases. Object-object cases are always having bigger relative motion. I may be able to generate some cases on crowded scenes if I can access the data later. The problem might be changed if the motion is too big as we all know both Flownet and PWC is hard to cover a low-frequency video. I might need to conduct more exps to see the result.
Thanks again for your great work and patience.


Memory Leak

Hey guys!
I am running a couple Dash P2Pool servers but it appears that the servers will display a too high memory usage and needs to be rebooted around 4-6 hours to start the pool again. Is this an issue that can be fixed?Currently running the pool on debian.

Hi,
I'm not sure what too high memory usage is, or why you have to reboot the servers. Do you have any other information, metrics?


Format set axioms

Use display math mode instead of inline mode for set axioms
Remove trailing whitespace

I didn't update the formatting for the axiom of infinity because I don't know what's the project's preferred environment for multiline math, but that one is also typeset incorrectly.

Sorry this took so long


Add g:loaded_miniyank

It'd be nice to have the above variable defined so when testing I could disable the plugin and not have to also manually comment lines such as:
map p <Plug>(miniyank-autoput)
I'd like to be able to do this basically: (in ~/.vim/after/plugin/miniyank.vim)
if exists('g:loaded_miniyank')
   map p <Plug>(miniyank-autoput)
endif

There is a generic way to do this for plugins with <Plug> mappings: Use the condition if mapcheck("<Plug>(miniyank-autoput)", "n") != ""

That works for me. Thanks