Update service-bus-create-namespace-portal.md

@nohaelanwer : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Fix missing value in upload model executions - add else clause to for if loops

Overview
See details in #385
Update type - breaking / non-breaking

[x] Minor bug fix
[ ] Documentation improvements
[ ] Quality of Life improvements
[ ] New features (non-breaking change)
[ ] New features (breaking change)
[ ] Other (non-breaking change)
[ ] Other (breaking change)

What does this solve?
Resolves #385
Outstanding questions

What databases have you tested with?

[ ] Snowflake
[ ] Google BigQuery
[ ] Databricks
[ ] Spark
[ ] Postgres
[ ] N/A

Thanks for all your work on this!


MapOverlay equivalent in quartz 100.0?

In the previous ArcGIS Runtime SDK, MapOverlays were provided. The main benefit of MapOverlays was the ability to paint anything directly over the Map. Is there any equivalent of onPaint(Graphics g) method in the quartz release.
I am new to JavaFX. if painting over a MapView is possible in JavaFX directly without any GIS support, Kindly let me know.

@pulink The MapOverlay is available in 10.2 version of the ArcGIS Runtime SDK for Java.  It is not currently available in v100.0.0 as of yet so there is no equivalent of onPaint(Graphics g) in 100.0.  I suggest you take this question to GeoNet forum for ArcGIS Java.  Alternatively, you can make a suggestion to ArcGIS Ideas.

@doneill Thanks.

I have created a Idea on ArcGis Ideas. Anybody, who feels the requirement of MapOverlay, can upvote it here.


Improving latency when using language model

Hello,
I am attempting to use an RNN language model but am seeing a doubling of our latency as a result. I wanted to examine anyway for me to reduce latency and came across the lm_num_threads option which is not exposed to the python and go libraries (lm_scale and lm are).
However, when examining the codebase I can't see the use of the lm_num_threads variable so unsure of its relevance. Would you recommend exposing and trying to increase threads in lm config? Do you have any other recommendations to achieve this?
Thank you in advance.

I created a custom build and exposed the lm_num_threads variable. What I found is that an increase in threads actually increases single call latency.


Added availability zone determination in AzureClient constructor

Added availability zone determination in constructor of AzureClient to avoid the need of explicitly determining it.

Pull Request Test Coverage Report for Build 25

0 of 12 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 31.827%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/clients/AzureClient.py
0
12
0.0%

Totals

Change from base Build 15:
-0.2%

Covered Lines:
565

Relevant Lines:
1742

💛  - Coveralls


Success callback for Model set function

I'm using a jQuery dialog to show a form to modify a temporary model.
So if set is success I can close the dialog, and if there's some error in validation I can show the errors without closing the dialog.
Thanks.

There's now a sync event that fires whenever a model has been successfully synced with the server -- hopefully that handles the case that you're looking for here...

I'm using set because it's not server-related, so this doesn't solve the problem :(
I'm wondering for a pluggable system for persistent save systems. Just an idea, add save plugin  localStorage, AJAX, WebSockets, custom.... You may add more than one, so save will trigger a save event and every plugin will do the magic.
What do you think? Thanks!


samples: create conversation, create analysis, export data to BigQuery

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #10  🦕
Fixes #16
Fixes #17
Fixes #19

FYI the export code sample has a try catch loop to catch an LRO error. However this error shouldn't be thrown. I filed an issue here https://github.com/googleapis/python-api-core/issues/257.


Using merge on a single VCF

Dear Fritz,
If one were to use the merge function on a single vcf would it compare and contrast within a single SV itself much the lsort function in SV-tools? If not, if one then split the single VCF file into two smaller files and ran merge on them would it work in a similar fashion?
Many thanks for your help

If you use it on a single vcf file svs will be collapse if they are close to each other but nothing else will happen.
If you split the file and do the same a similar thing will happen.
I hope that answers it.
Cheers
Fritz


Keep names in go-sdl2 and upstream SDL the same where possible

I've noticed that e.g. GetAxis() was renamed to Axis(). Please don't forget that go-sdl2 is not its own library but a wrapper around SDL. Having a 1:1 mapping between upstream SDL names and go-sdl2 names is WAY MORE IMPORTANT than conforming with the OPINIONS about Go function naming of a few old men.
Besides, the change broke client code for no good reason.

Absolutely not, this is a wrapper around SDL but should conform to community best practices. If you start making exceptions like that you get inconsistent code all over the place. That is the exact reason the Go team has put so much effort into documenting and tooling Go best practices.

Best practices for a wrapper is to keep the original names.  I also use SDL in C and in Python. The function is called SDL_JoystickGetAxis() in C. It's pygame.joystick.Joystick.get_axis() in Python. And it used to be sdl.Joystick.GetAxis() in Go.
THAT is best practice.
Keeping a name once people have started using your library. THAT is best practice, too.

The developers of this library are being very patient with you but you are coming in here without constructive criticism and strongly held incorrect opinions and making it hard for the developers to maintain this library. When you write sigh or puke you are taking energy away from the developers. Your attitude is toxic. Please take a moment and consider the impact of your actions.
As for code style, one of the main principals of writing go is its consistent code style. It makes any repo maintainable by any Go developer. It is for the better of the community. I appreciate that things are done differently in other communities, and I implement them in other communities technologies, but when you are actively participating in an open source community you normally follow their guidelines.

I understand the issue but it was vaguely decided to use Go's guidelines in this library and there is still a lot to do in this regard. And I don't think that it's handled in a good way. Package is still consider unstable with all the frequent changes to it in the master branch.
If we are actually following the guidelines then we should state what they are in the readme. Knowing what naming convention is used helps with finding the proper function or constant names.
All the breaking changes since v0.2 are kept here:
https://github.com/veandco/go-sdl2/blob/master/BREAKING.md

I really am sorry that a lot of troubles seems to have been caused by the API changes. The original developer did want to keep it simple in terms of naming (such as using SDL2 function names as they are without SDL_). However, we think that this API change will create a much better experience for Go developers using the library in the future as we're conforming to the Go coding guidelines.
We're also going to use a working versioning system so in the future so you will be able to use the major version that has worked for you while we're make breaking changes on the next major version. Of course, we will try to keep the breaking changes less and less the more time passes by. If we do make them, it should be for either ergonomic or functional reasons :relaxed:


update

update version

Coverage remained the same at 100.0% when pulling 84c231040a21ceb41d0aa2c0acd8d116da3c898b on pre-build into c3f412892c7d8cb14a8c78c3b5b61c9aae95c88c on master.


Ensure boolean Macro arguments are parsed as boolean. Resolves #441

See #441. I have rewritten Macro#extract to handle false and true arguments explicitly.

FYI, I've had to drop support for Ruby 2.6 and JRuby 9.3 (which is Ruby 2.6) to get the tests passing, as this PR utilizes the case .. in syntax (pattern matching) introduced in Ruby 2.7. JRuby 9.4 has been out since 2022-11-23 -- not terribly long, do we want to keep support for 9.3?


Can't change defaultList

Hi, this is such a great workflow (thank you for making it!) but I have one problem. I've tried to no avail to change the defaultList variable in the arg/vars node of the workflow. I put in next to the variable the name of the list I want to use "Next," and it still keeps saving the reminder to the default list I have in the Reminders app (which I need to keep as the in-app default for independent reasons). Is there a special format that I should be using? I also tried Reminders.Next, but that didn't help. Thanks!

As an update, I realized that my problem is more specific. I can change the default list if the list I want is outside of a reminders list group. But the target list I want IS in a group and I need to leave it there. Is there any way to change the defaultList variable to a list within a list group? Or does anyone know what the name would be for a grouped list? E.g., if the group is Group and the targeted list within it is ListOne, would the name be something like Group->ListOne or Group.ListOne? I have tried these and they don't work--but just to give a sense of what I'm looking for. Thanks again.

Thanks for this. I think this is a somewhat broader problem than just defaultList. I have a grocery list called "Safeway" under a group called "shopping," and something like "r corn @ Safeway" says, "No list called Safeway was found in Reminders."  I use this one enough that I'm going to ungroup it, but I imagine that others will run into this.  Also, I can confirm that ungrouping does solve it for me.


refactor(feature): <FeeRow/> component

Try out this version of the Hiro Wallet - download extension builds.

This change uses Formik's useField and requires like 3 more props to be passed, a small price to pay for greater reuse, readability.
@fbwoolf can you please QA this one 🙏🏼

Looks good, just curious if we need to pass fieldName, won't it always be 'fee'?

I dunno. Will it?
Passing it as a prop with the name prevents us from making that assumption.

Checked out this branch locally and I have an empty fee-field folder under components. Not sure where that came from?

I ran thru everything locally and it seems like all is behaving correctly.


Gitter Chat

Gitter is a nice way for developers to talk about a certain project. There's currently a gitter chat for hydra (built on fosite) but not one yet for fosite. I'm opening this issue to discuss the possibility of having a gitter chat for fosite.

Great idea!


initial Dfe.Testing.Pages documentation README

Moved to https://github.com/DFE-Digital/pageobjects-testing-infrastructure


Docker Hub is missing the -0 suffix on the first image for a tag

https://hub.docker.com/r/openmicroscopy/omero-web-standalone/tags
https://github.com/ome/omero-web-docker/blob/1271c72718409da94c134d025a82f9fb7e09b237/.github/workflows/main.yml#L58-L60
only creates higher-level tags, so if this repository is tagged with 5.14.0 instead of 5.14.0-0 the Docker Hub tags 5.14.0 and 5.14 are created, and 5 is updated.
If the image is later rebuilt and tagged as 5.14.0-1 Docker Hub tags 5.14.0, 5.14 and 5 are updated, but since there was never a 5.14.0-0 tag the original 5.14.0 image is lost.
This also applies to https://hub.docker.com/r/openmicroscopy/omero-server/tags

Thanks for the info.
Are you working on that issue in the action?

No, I think this is best handled when the GitHub *-docker repository is tagged. Extrapolating a release tag of 5.14.0 to 5.14.0-0 is doable, but since rebuilds of this repository will need a -1, -2, ... etc tag anyway it seems more consistent to start with a -0 tag.


CC button default height isn't consistent with other buttons

Follow up on #74

https://media-chrome-err0093s8-mux.vercel.app/examples/standalone-controls.html

I think this should be part of an update to use "height" as a primary sizing dimension for buttons as opposed to "width". While there may be some edge cases, this should be resolved by simply updating media-chrome-button from:
svg, img, ::slotted(svg), ::slotted(img) {
  width: var(--media-button-icon-width, 24px);
  height: var(--media-button-icon-height);
  /* ... */
}

to:
svg, img, ::slotted(svg), ::slotted(img) {
  width: var(--media-button-icon-width);
  height: var(--media-button-icon-height, 24px);
  /* ... */
}

This should also resolve cases we've encountered in real-world use cases where custom icons required additional styles to avoid weird resizing when e.g. toggling between play and pause state.

Ah I get what you mean now. Primary dimension is height for the icon specifically, so icons from different sets can be dropped in and expand the button horizontally if needed but not vertically. That makes sense to me. Let's do it. I think implied in that is we're documenting that you should use icon height to resize your buttons, and probably that's it.


Expands in command line mode

Currently this plugin expands symbols in command line mode too, is there any way to disable this? Thanks

Not currently, no. It only runs when you execute the command though, is it running automatically when you don't want it to or something?


CSS applies class not style

"test".Dump(css: "color:pink")
adds color:pink as a class
instead of
as a style attribute

Closing. This is expected behavior.


API controlled goToPage functionality

This PR creates a tabContentProps with goToPage, ownPosition and tabs and merge it with the SceneComponent children props.
The main benefit is the possibility to navigate programatically instead of relying only on the TabBarComponent 🤓 🖥
There's an example attached showing the new feature. Find it at examples/StepperBar.
Updated README and fixed eslint issues are included too 👌

@skv-headless 🙏

@skv-headless 😢

@sospedra sorry that I didn't answer before. I don't think that API that you provided is easier or more intuitive.

Maybe not. But it enables another dimension: working inside the tab. Did you check the example?

I've just added to docs https://github.com/skv-headless/react-native-scrollable-tab-view#changing-between-tabs. Is it the same thing what you want to do?

I don't think is the same. In your example you're adding a new Tab which includes a button to move to the 0 position; and that's not programatically because the trigger action cames from pressing a button.
With my PR you can actually manage the whole navigation inside any Tab. There are strongly different functionalities. Here's a gif of the example I added to the PR.

I can make a more clear documentation if you feel like is hard to understand ;)

everything is possible to do with current implementation. I guess I need to write more docs for that.

Try this

Take the ref of the ScrollableTabView like this

 <ScrollableTabView
          initialPage={0}

          ref='tabView'
</ScrollableTabView>

componentDidMount() {
    let self = this;

    console.log(this.refs);
    setTimeout(() => {
      // self.setState({ index: 1 });
      self.refs.tabView.goToPage(1);
      // alert(self.state.index);
    }, 300);
  }

This PR is very outdated, hence if this changes are still needed feel free to rebase them on top of master and open a new PR. From now on there will be faster feedback on issues and pull requests.


How can I automatically import data from an existing card for simulation?

How can I automatically import data from an existing card for simulation?

Can you send me the code file for this part? Thanks!!
email<EMAIL_ADDRESS>

No need, it's been solved


Demo example test case

This adds the example loosely based on http_parse_basic as a test case with correct and incorrect examples. Only 3/8 of the tests pass due to the short timeout. SystemTests now also runs all boogie files present in the test directory instead of just the output file, to support overriding generated definitions.
Those examples are also setup to use a recursive Makefile to lift them.
There's also some more example specs included for the different memcpy implementations I've tried that didn't work, a version of this can be converted to a test case eventually too.

How long a timeout is needed for these tests to pass?
I like having the makefiles to lift these new tests, and it would be good to generalise that to all the other tests, but it could probably be improved so you don't need to put a makefile in every variation folder? Just one in the parent where you define which variations we want for that example? There's also a bunch of stuff relating to the Boogie in the makefiles which doesn't seem like right place for that.

How long a timeout is needed for these tests to pass?

With --boogie-use-lambda-stores it only takes two seconds, without its e.g. 2.24 minutes for incorrect/malloc_memcpy_strlen_memset_free/clang_O2 on my laptop.

I like having the makefiles to lift these new tests, and it would be good to generalise that to all the other tests, but it could probably be improved so you don't need to put a makefile in every variation folder? Just one in the parent where you define which variations we want for that example? There's also a bunch of stuff relating to the Boogie in the makefiles which doesn't seem like right place for that.

I have an idea to clean up the makefiles, the boogie stuff is convenient for testing things more quickly than modifying the test code.

This seems to lift all the clang variants to x86 binaries instead of armv8

You need to update the new test cases for the clang variation, and I'm still not quite sure what the intention of the new test cases are? Is it to be run with --boogie-use-lambda-stores flag? If so, it should probably be in its own test suite that uses that flag? Are we expecting any of it to pass right now? In the near future?
All the makefile stuff seems good now.

You need to update the new test cases for the clang variation, and I'm still not quite sure what the intention of the new test cases are? Is it to be run with --boogie-use-lambda-stores flag? If so, it should probably be in its own test suite that uses that flag? Are we expecting any of it to pass right now? In the near future?

Its not about use-lambda-stores, this is an example we've used a few times in demos so i want to make sure it keeps working. Everything should pass, however 3 time out with the 10 sec vc time limit, these all pass with a 2 minute time-limit.
All of the clang tests are failing at type-check because of a mistranslation, mov w0, wzr is getting translated as Var("R0", Imm(64)),  Int(0, 32)) and then basil translates that directly to R0, Gamma_R0 := 0bv32, true;. This should be fixed eventually probably. Also, 4 of these are the 'incorrect' cases passing spuriously, so we should also fix the test cases so it actually looks for the assertion failed message rather than just not containing "verified".

Its not about use-lambda-stores, this is an example we've used a few times in demos so i want to make sure it keeps working. Everything should pass, however 3 time out with the 10 sec vc time limit, these all pass with a 1 minute time-limit.

We probably don't need all the variants then, perhaps just one? Even then, it might make sense to put in its own test suite if it needs a longer time limit? It doesn't make sense to keep it in the test suite as it currently is if it will always time out.

All of the clang tests are failing at type-check because of a mistranslation, mov w0, wzr is getting translated as Var("R0", Imm(64)),  Int(0, 32))

I don't get this when I lift it, it seems like you're using an old version of BAP/the ASLi plugin?

I don't get this when I lift it, it seems like you're using an old version of BAP/the ASLi plugin?

Seems like it

We probably don't need all the variants then, perhaps just one? Even then, it might make sense to put in its own test suite if it needs a longer time limit? It doesn't make sense to keep it in the test suite as it currently is if it will always time out.

Any thoughts on this bit?

I do want to make sure a future change doesn't make the other variants break, and if we find a way to improve the encoding so it is easier to verify then it would be nice to have the tests around to see if there is a perf improvement; but for that purpose it would probably be better to create a new test suite, I just don't have time to do that at the moment. Currently on my machine the -O2 variants verify in under 10s so we could keep those. Ideally I want to increase the timeout and run the tests in parallel so they complete in a reasonable amount of time.
                                           verifyTime

solver                 compiler
axioms                 clang                   20.695
                       clang_O2                 9.043
                       clang_no_plt_no_pic     20.557
                       clang_pic               19.829
                       gcc                     11.360
                       gcc_O2                   5.328
                       gcc_no_plt_no_pic       10.835
                       gcc_pic                 18.564
theory                 clang                   48.496
                       clang_O2                10.189
                       clang_no_plt_no_pic     49.063
                       clang_pic              120.582
                       gcc                     12.110
                       gcc_O2                   7.862
                       gcc_no_plt_no_pic       12.148
                       gcc_pic                 19.841
theory-non-extensional clang                   48.716
                       clang_O2                10.183
                       clang_no_plt_no_pic     47.989
                       clang_pic              120.600
                       gcc                     12.218
                       gcc_O2                   7.920
                       gcc_no_plt_no_pic       11.914
                       gcc_pic                 20.004

I agree with all that, that makes sense to do eventually.


No way to specify program entry point, ELF hardcodes at 0x40000

D'oh: https://github.com/windelbouwman/ppci-mirror/blob/master/ppci/format/elf/file.py#L219
This in turn leads to crazy workarounds like:
// No idea how to specify entrypoint to ppci linker, so far, entry point is
// at the beginning of the segment, and functions appear in the segment in
// the order of appearance in the source file, even if "apperance" is merely
// a prototype.

If not for inline asm, added just yesterday, it wouldn't be possible to workaround that at all (in "single source file" model, I mean).

So, please let's make sure that this gets implemented properly. This option is the option to ld:
       -e entry
       --entry=entry
           Use entry as the explicit symbol for beginning execution of your
           program, rather than the default entry point.  If there is no
           symbol named entry, the linker will try to parse entry as a number,
           and use that as the entry address (the number will be interpreted
           in base 10; you may use a leading 0x for base 16, or a leading 0
           for base 8).

Then apparently ld will need to store it in .oj file format. Likely, in resolved form (sumbol -> address). And then tools like objcopy --output-format elf will use it to fill in actual headers.

Initial steps are done here: c078ba4310dec64600ddb78a35e79fead16cd43e
TODO: add the --entry flag to ld.

Command line argument implemented here: 0043061b09872c1ac2ab98d7b9e9ba4b10473f80

Command line argument implemented here: 0043061

Thanks! Confirming it works as expected. Closing.


403 Forbidden error on MAC

I use the grav's .htaccess,I get 403 error [You don't have permission to access /opensource/php/grav-admin/ on this server.],but when i change the .htaccess to my .htaccess,I get it successfully!
my .htaccess:[
<IfModule mod_rewrite.c>
  Options +FollowSymlinks -Multiviews
  RewriteEngine On

  RewriteCond %{REQUEST_FILENAME} !-d
  RewriteCond %{REQUEST_FILENAME} !-f
# RewriteRule ^(.*)$ index.php/$1 [QSA,PT,L]
  RewriteRule ^(.*)$ index.php [L,E=PATH_INFO:$1]
</IfModule>

]
is this something wrong with the grav's .htaccess?

Well i'm not sure where that .htaccess comes from, but ours looks like this:
https://github.com/getgrav/grav/blob/develop/.htaccess
I think you are working with an .htaccess from your hosting company?

I mean that : yours .htaccess(https://github.com/getgrav/grav/blob/develop/.htaccess) is not work in my mac ,the error is 403

And i use anoher .htaccess form [http://git.oschina.net/rainfer/YFCMF/blob/master/.htaccess?dir=0&filepath=.htaccess&oid=23ba96fef1481b830b3f12ac052e1a1e92e8c27a&sha=d2d960ff59d3d76a575321712ffffbc7d0e10f11],the grav is work successfully.
so i guess that there are something wrong with the [https://github.com/getgrav/grav/blob/develop/.htaccess] in mac

Well your the first one to report it.  We all use Macs here, either with MAMP, native Apache, and Brew Apache.  All work fine.  What kind of server setup do you have that's causing problems?

I use the native Apache

I run the bin/grav install on my mac,and it work!

So the problem was you downloaded from GitHub and the vendor/ folder was not having the Composer dependencies?

Closing as was solved


reference to Audubon Core in links on AC page

The home page https://www.tdwg.org/standards/ac/ for Audiovisual Core has links to content under Parts of the Standard that reference "Audubon Core" rather than rebranded "Audiovisual Core":

Audiovisual Core main vocabulary http://rs.tdwg.org/ac.htm
variant controlled vocabulary http://rs.tdwg.org/acvariant/
format controlled vocabulary http://rs.tdwg.org/format/
subtype controlled vocabulary http://rs.tdwg.org/acsubtype/

@gkampmeier It took me three tries but I think (hope?) I have caught all of the instances of "Audubon".


[api-docs] Update applications link on oauth2 docs

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/a0b0355cad339621d35599ca873f4972f2d73f15

This commit will not effect our lib's code. No changes required.


ISPN-10691 Jackson 2.10.0

https://issues.jboss.org/browse/ISPN-10691

Failures are unrelated

Thanks @tristantarrant


alt-p swallowed by PiP extension

When using remotedesktop.google.com (in a window, on ChromeOS) Alt-P is captured by the extension, instead of being sent through to the remote host.
I asked someone on the Chrome Remote Desktop team, and they said:
It uses the _execute_browser_action feature of extensions, which swallows keypresses unilaterally (as can be seen by using Keyboard Event Viewer with the extension active). It could be rewritten to use a content script and handle the key only if there are relevant elements on the page, but I don't know if the owner would be interested in doing that (it would still affect CRD, but we could set the disablePictureInPicture attribute on our video tag to avoid it).
Having some solution for not capturing Alt-P would be nice.

Thanks for the feedback @rspier!
I gave a quick try to your proposal (see https://github.com/GoogleChromeLabs/picture-in-picture-chrome-extension/compare/swallow) and found some issues:

A content script would have to be injected in all frames that users visit. It seems like a performance issue to me.
We would have to create a custom options page so that user can change the keyboard shortcut if desired as they can do today by visiting chrome://extensions/shortcuts
This issue is not specific to this extension. Every chrome extension on the web store using _execute_browser_action may have some unexpected behaviour on remotedesktop.google.com. How about filing a bug to chrome extension team to not capture browser action keyboard events and ping<EMAIL_ADDRESS>
Note that we already filter out videos with disablePictureInPicture attribute today: https://github.com/GoogleChromeLabs/picture-in-picture-chrome-extension/blob/66f0392c2bb5d12c992ad97705747cf259e27240/src/script.js#L18

Hi @beaufortfrancois -
Thanks for the response.
Your reminder that chrome://extensions/shortcuts exists provides a good workaround.  I just sent #30 to document that option.
I agree, the content script has some potential disadvantages.    Not capturing browser action keyboard events at the browser level might lead to confusion as the event might go both to the extension and the underlying frame -- that seems worse.
I'll follow up with the CRD team regarding disablePictureInPicture and the other ideas.
I'll also

I've merged https://github.com/GoogleChromeLabs/picture-in-picture-chrome-extension/pull/30

I emailed Devlin to get their take on this.  Thanks!

Perfect. Thanks!


update history version check

history.getCurrentLocation  was removed on history v4.0.0-2 so I thought this check could be updated.
I also took a look to the check on v3, it seems that it will catch correctly(besides wording) histories created with history versions >=4.0.0-2 but no versions 4.0.0 and 4.0.0-1, I could also make a PR for that if you think is ok.

Thanks!


Set width for slick slides

Hello!
Thank you for this great plugin.
I would like the option to add width or max-width for slick slides...
http://jsfiddle.net/0j3eadon/
In my example, I want the slider to be fullscreen as follow, but I want the slide to have a width so I don't have to add custom
 on each title. I know you can add margins to make it work, but because I am using a fullscreen type of style, I can't add hardcoded margins to make it work. I also cannot wrap my slide inside a container, because I want to be able to drag my slide out of the screen.
Thank you!
Stephen

Here is exactly what I would need...I added
 to make it work though

Okay just figured out a solution :
Add div that wraps everything inside your slide
max-width on that


How can I prevent login_hint to be added to URL when loginRedirect() is called?

Library

[x]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
When my users want to switch to another account, I want to redirect them to the general AAD login page where they can enter their email and password.
The problem is that when i call loginRedirect(), msal.js always appends a login_hint that has the currently logged-in account.
This logic happens in ServerRequestParameters.addHintParameters.
Is there a way to make msal js not provide the login hint?
A possible workaround is to use select_account instead of login in prompt, but I'm already using Me control for account switching from within my app (without needing to redirect), so I'm interested in redirecting to user/password entry rather than a "pick an account" UX.

You can try calling logout() before calling loginRedirect(), maybe with some logic to detect if there is a user currently logged in.

Thanks, Is there any other option? logging out is kind of a last resort since it affects other browser tabs / applications

Also I'm trying to implement the workaround (select_account), but the issue i'm encountering is that I'm not able to login with a personal account (only work or school).
here's the url i'm being redirected to:
https://login.microsoftonline.com/common/oauth2/v2.0/authorize?response_type=id_token&scope=https%3A%2F%2Fhelp.kusto.windows.net%2F.default%20openid%20profile&client_id=f9818e52-50bd-463e-8932-a1650bd3fad2&redirect_uri=https%3A%2F%2Fdataexplorer.azure.com%2F&state=eyJpZCI6IjM1NTI0MjJhLTcwZTUtNDQ3Ni04Yjk2LWJhOWU1YWY0ODc2YSIsInRzIjoxNTkzNTg4MzE1LCJtZXRob2QiOiJyZWRpcmVjdEludGVyYWN0aW9uIn0%3D&nonce=e178522d-d6ec-4d49-9397-b21b9e7b6763&client_info=1&x-client-SKU=MSAL.JS&x-client-Ver=1.3.1&login_hint=user%40ronylkusto.onmicrosoft.com&prompt=select_account&client-request-id=917f6c48-df7b-4d19-882c-c013bfa6b75c&response_mode=fragment
What should I change in MSAL configuration to allow for personal accounts? I was under the impression that common endpoint is for both personal and organizational accounts.

OK i think i can use onRedirectNavigate to remove the login hint and redirece myself.
Is that a good solution?

Sorry for the delay over the long weekend. Yes--onRedirectNavigate allows you to control the redirect yourself, so your solution should definitely be possible. More info on that in #1691


ace: 6.5.11 -> 7.0.1

Motivation for this change
Things done

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

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

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

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

2 packages built:

ace
yarp

Result of nixpkgs-review pr 120130 at 4fb499f4 run on x86_64-linux 1

2 packages built successfully:

ace
yarp

1 suggestion:

warning: patch-phase
patchPhase should not be overridden, use postPatch instead.
Near pkgs/development/libraries/ace/default.nix:21:3:
   |
21 |   patchPhase = ''substituteInPlace ./MPC/prj_install.pl \
   |   ^


Blower response time is too slow

John Batteh: We definitely need to nail down the blower inertia value.  Currently it is set at 0.69s but with this sort of time constant, I think the response will be pretty slow.  We'll set up the latest test to try and achieve 20 breaths/min and see how the system responds.
Just thinking about it quickly- there will definitely be a problem and we don't even need to run a sim.  For 20 bpm and IE=1/2, then the intake event is 1s long.  So if the time constant is 0.69s, your speed response alone will make it difficult to hit your PIP target.  On your target hardware, are you able to run something like the breathing parameters above and hit the PIP target?  If so, that is interesting and we need to re-evaluate the blower inertia.  It is critical.

@neelfirst with the pinch valve implemented, is this ticket ready to close?

I know in practice the pinch valve is done and is our primary solution for improving blower response time. So this ticket can be closed.
However I don't see enough of a paper trail in GH issues that declare the pinch valve fully implemented. In particular -
#655 should have at least a pointer to the final design and
#629 should have a pointer to the completed test data, for that design.
The Slack discussion for the pinch valve failure modes should be in #655 and tagged for future FMEA work. (did this)

Decision is generally made, pending completeness of test data.  Closure pending closure of the two tickets called out above.

Life leader pinch valve has proven valve to work for over 4 million cycles (12 continuous weeks of operation). Tracking of testing continues in #629
Mechanical design of pinch valve superseded by #941
We consider the general question closed, and specific response scenarios will be addressed in detailed integration testing with a more mature design.

Life leader pinch valve has proven valve to work for over 4 million cycles (12 continuous weeks of operation). Tracking of testing continues in #629
Mechanical design of pinch valve superseded by #941
We consider the general question closed, and specific response scenarios will be addressed in detailed integration testing with a more mature design.


Icons from EXE/DLL as Custom Profile Icon Source

Description of the new feature/enhancement
I was experimenting with adding the Python 3.7.3 shell to Windows Terminal and was trying to figure out a way to add its icon with extracting it from python.exe. Unless there is some way I am unaware of to use an icon embedded in an EXE or DLL as a profile's icon, this appears to be a feature that is missing. I would expect to be able to use the EXE or DLL's path as part of the profile's icon path, with an integer specifying the index after a comma, similar how autorun.inf files specify an icon to use.
Is this already possible to do? If not, is this something that should be added?

/dup #1504. Thanks for filing! It's definitely something we'd like to do.


Bugs encountered w/ allin database

Searchbar = 'chaoss'
For the Package Versions visualization:
dash.exceptions.LongCallbackError: An error occurred inside a long callback: Value of 'names' is not the name of a column in 'data_frame'. Expected one of ['dep_age', 'count'] but received: index To use the index, pass it in directly as `df.index`. Traceback (most recent call last): File "/opt/app-root/lib64/python3.9/site-packages/dash/long_callback/managers/celery_manager.py", line 151, in run user_callback_output = fn(*maybe_progress, *user_callback_args) File "/opt/app-root/src/pages/repo_overview/visualizations/package_version.py", line 117, in package_version_graph fig = px.pie(df, names="index", values="dep_age", color_discrete_sequence=color_seq) File "/opt/app-root/lib64/python3.9/site-packages/plotly/express/_chart_types.py", line 1482, in pie return make_figure( File "/opt/app-root/lib64/python3.9/site-packages/plotly/express/_core.py", line 2076, in make_figure args = build_dataframe(args, constructor) File "/opt/app-root/lib64/python3.9/site-packages/plotly/express/_core.py", line 1485, in build_dataframe df_output, wide_id_vars = process_args_into_dataframe( File "/opt/app-root/lib64/python3.9/site-packages/plotly/express/_core.py", line 1210, in process_args_into_dataframe raise ValueError(err_msg) ValueError: Value of 'names' is not the name of a column in 'data_frame'. Expected one of ['dep_age', 'count'] but received: index To use the index, pass it in directly as `df.index`.
For the Contributor File Heatmap, for grimoirelab-sortinghat:
dash.exceptions.LongCallbackError: An error occurred inside a long callback: 'NoneType' object has no attribute 'count' Traceback (most recent call last): File "/opt/app-root/lib64/python3.9/site-packages/dash/long_callback/managers/celery_manager.py", line 151, in run user_callback_output = fn(*maybe_progress, *user_callback_args) File "/opt/app-root/src/pages/codebase/visualizations/cntrb_file_heatmap.py", line 246, in cntrb_file_heatmap_graph df = process_data(df_file, df_actions, df_file_cntbs, directory, bot_switch) File "/opt/app-root/src/pages/codebase/visualizations/cntrb_file_heatmap.py", line 347, in process_data level = directory.count("/") AttributeError: 'NoneType' object has no attribute 'count'

For Reviewer File Heatmap, for oss-aspen/8Knot:
dash.exceptions.LongCallbackError: An error occurred inside a long callback: 'NoneType' object has no attribute 'count' Traceback (most recent call last): File "/opt/app-root/lib64/python3.9/site-packages/dash/long_callback/managers/celery_manager.py", line 151, in run user_callback_output = fn(*maybe_progress, *user_callback_args) File "/opt/app-root/src/pages/codebase/visualizations/reviewer_file_heatmap.py", line 246, in reviewer_file_heatmap_graph df = process_data(df_file, df_actions, df_file_cntbs, directory, bot_switch) File "/opt/app-root/src/pages/codebase/visualizations/reviewer_file_heatmap.py", line 319, in process_data df_dynamic_directory = cntrb_per_directory_value(directory, df_file) File "/opt/app-root/src/pages/codebase/visualizations/reviewer_file_heatmap.py", line 424, in cntrb_per_directory_value level = directory.count("/") AttributeError: 'NoneType' object has no attribute 'count'

dash.exceptions.LongCallbackError: An error occurred inside a long callback: Value of 'names' is not the name of a column in 'data_frame'. Expected one of ['dep_age', 'count'] but received: index To use the index, pass it in directly as df.index. Traceback (most recent call last): File "/opt/app-root/lib64/python3.9/site-packages/dash/long_callback/managers/celery_manager.py", line 151, in run user_callback_output = fn(*maybe_progress, *user_callback_args) File "/opt/app-root/src/pages/repo_overview/visualizations/package_version.py", line 117, in package_version_graph fig = px.pie(df, names="index", values="dep_age", color_discrete_sequence=color_seq) File "/opt/app-root/lib64/python3.9/site-packages/plotly/express/_chart_types.py", line 1482, in pie return make_figure( File "/opt/app-root/lib64/python3.9/site-packages/plotly/express/_core.py", line 2076, in make_figure args = build_dataframe(args, constructor) File "/opt/app-root/lib64/python3.9/site-packages/plotly/express/_core.py", line 1485, in build_dataframe df_output, wide_id_vars = process_args_into_dataframe( File "/opt/app-root/lib64/python3.9/site-packages/plotly/express/_core.py", line 1210, in process_args_into_dataframe raise ValueError(err_msg) ValueError: Value of 'names' is not the name of a column in 'data_frame'. Expected one of ['dep_age', 'count'] but received: index To use the index, pass it in directly as df.index.
^^ error is the same for padres, confusing because requirements.txt should be pinned...

@cdolfi assigned to you just so you see it, not for you to fix it

@JamesKunstle : I am getting the same issue at https://ai.chaoss.io
The issue looks to me like its pythonic :
app-server-1       | ValueError: Value of 'names' is not the name of a column in 'data_frame'. Expected one of ['dep_age', 'count'] but received: index
app-server-1       |  To use the index, pass it in directly as `df.index`.
worker-callback-3  | [2024-03-12 23:54:16,254: INFO/MainProcess] Task long_callback_10bbaf4b32a7bdb8429745d161fb00e5e27b0dc0[9de34a60-8baf-4e5d-92d6-515bf1fe42aa] received

Package Version Updates comes up empty, which you probably already noticed. Pretty sure this is a Pandas thing in 8Knot.

(I also verified the same issue running locally against both ai.chaoss.io and projectbadge.chaoss.io where the smaller database lives. )

@sgoggins Yeah I have the same conclusion. This is a bug on our end. I was curious if there was an obvious table problem.


docs: hexo-generator-tag v1, hexo-renderer-ejs v1 & hexo-renderer-stylus v1

Check List
Please read and check followings before submitting a PR.

[x] Others (Update, fix, translation, etc...)

WIP, I would like to include hexo-renderer-stylus v1.1


How to Create a Contact Form in WordPress

Proposed title of the article
How to Create a Contact Form in WordPress
Introduction paragraph (2-3 paragraphs):
A website's contact form is an essential component. It serves as a link between the customer and the provider. It's simple to make a contact form for the WordPress platform. To create a successful contact form for a WordPress site, you only need to follow a few simple steps.
In core PHP, we've seen far too many different types of contact form examples. The concept of writing the process flow is the same regardless of platform, framework, or CMS. So, if you know how a contact form works, you'll have no trouble understanding how the WordPress version works.
In this edge, we'll create a custom WordPress template for this contact form's user interface. Then, using the admin panel, map the template for the contact page.
Key takeaways:
1.Create a WordPress environment
2.Create a new contact page using WordPress admin
3.Create child-theme for the active theme
4.How to create a contact template
5.Send AJAX request using wp_ajax hooks
6.WordPress contact form output
References:
N/A

Good afternoon and thank you for submitting your topic suggestion. @Robbiewanjau
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.


EXPLAIN ANALYZE (DEBUG) docs

Fixes #6923.
Fixes #7030.
Fixes #5988.
Fixes #7135.
Fixes #7136.
Will also fix forthcoming release note issues on:

https://github.com/cockroachdb/cockroach/pull/46790

PR includes the following changes:

Updated EXPLAIN ANALYZE syntax diagram.
Made minor edits to 19.2, 20.1 EXPLAIN ANALYZE docs for clarity.
Updated 20.1 EXPLAIN ANALYZE page to include DEBUG option docs.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/204676a9555e9afb4ed4abda9d51614119a7fb21/
Edited pages:

v19.2/explain-analyze.md
v20.1/explain-analyze.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/aeaabdbebe1c0560179342ada2d620a32be218e9/
Edited pages:

v19.2/explain-analyze.md
v20.1/explain-analyze.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3257a12fd9a2341fc3b5c0aa5f37dc32a04a7a1f/
Edited pages:

v19.2/explain-analyze.md
v20.1/explain-analyze.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d409743b7ad7f6469e006c49aed0c8cea2e497ac/
Edited pages:

v19.2/explain-analyze.md
v20.1/explain-analyze.md


Tabs Tabindex Accessibility

Current Behavior
Devtools like Axe mark tabs as a critical accesibility error since the tabindex=0 and that isn't friendly to assitive technologies according to: https://www.a11yproject.com/posts/how-to-use-the-tabindex-attribute/
Reproduction / Steps To Reproduce
Use Tabs and run some accessability tools
Anything else?
No response

I'll take this issue @endigo9740

Dang it @Hugos68 I told you not to look at our docs :P
Thanks @ryceg

Also @Hugos68 this is a duplicate of: https://github.com/skeletonlabs/skeleton/pull/1398
I'll aim to merge this tomorrow, so it'll be part of the next release.


SILDebugScopes: Don't ignore ConditionalClauseInitializerScope.

ConditionalClauseInitializerScope often create redundant scopes, however, they are needed to correctly represent the scopes in an expression such as:
if case .something(let foo), let foo = foo.
This patch changes SILGen to lower them 1:1 from ASTScopes.
rdar://110841130

@swift-ci test and merge


wox: Add Wox-beta, i.e. pre-release v1.4.1196

The extras bucket already has the latest release Wox v1.3.524, however it was released more than 3 years ago. The latest pre-release v1.4.1196 is about a year old, it includes nice improvement in search speeds, many customizations in UI since the latest release, but also for me it fixed a critical bug on Windows 10 that made app unusable after computer is being put into Sleep mode.
I suggest to add this version as a beta, since autoupdate ignores pre-releases, but the Wox's maintainers neither update the latest release nor add more pre-releases
I used the original wox.json, only changed version, url of archive, hash, and extract_dir from "lib\\net45" to "lib\\net462".

@issaclin32 Please use "Squash and Merge" to keep git log simplify 😄

@issaclin32 Please use "Squash and Merge" to keep git log simplify 😄

OK. Thanks for noticing.


fixes using repositoryLink if libraryWebsite is not set

without the fix, this would not make sense:
library.getLibraryWebsite() != null ? library.getLibraryWebsite() : library.getRepositoryLink()
;-)

@Fentin you are totally right. Thank you so much


arm64 1.2.7 docker wont starting

Here is my docker file:
version: "3.8"

services:
  neonlink:
    image: alexscifier/neonlink:latest-arm64v8
    container_name: neonlink
    volumes:
      - ./bookmarks.sqlite:/app/db/bookmarks.sqlite
    restart: unless-stopped
    ports:
      - "<IP_ADDRESS>:3333:3333"

when I try to start I got the following error:
$ docker-compose up
Creating network "neonlink_default" with the default driver
Creating neonlink ... done
Attaching to neonlink
neonlink    | exec /usr/local/bin/docker-entrypoint.sh: exec format error

when I change back to 1.2.6 everything working fine.

I tried with the latest version again, and it generates for me the same problem. I tried it with empty database but won't help


Provide better examples for migrating from v1 to v2

Feature Request
I've been using react-native-permissions v1 to access the geolocation on Android and iOS. I have some RxJS code that wraps the library for easier usage. Previously all I needed to do was to add the native permission types to configuration files for Android/iOS and then call Permissions.request("location", { type: "whenInUse" }) and check the response.
It's however unclear to me that how the old way of requesting a single permission translates to asking for each permission separately in v2. The README has an example of calling multiple permissions, but that also means that each permission would need to be handled separately.
So, it would be nice if there would be a small migration guide that would show with examples how to migrate from v1 to v2. Asking for geolocation permission would be a good example to give as it works a bit different on Android and iOS.
It would also be nice if there would be some kind of way (new method?) to preserve the old behavior of only requesting one permission for both Android and iOS.

The changes has been made because the permissions differs way too much between the platforms (Android is more granular). An easy way to achieve the same behavior could be to create a file where you choose the permission per platform:
import { Platform } from "react-native";
import { check, request, PERMISSIONS } from "react-native-permissions";

const camera = Platform.select({
  android: PERMISSIONS.ANDROID.CAMERA,
  ios: PERMISSIONS.IOS.CAMERA,
});

const contacts = Platform.select({
  android: PERMISSIONS.ANDROID.READ_CONTACTS,
  ios: PERMISSIONS.IOS.CONTACTS,
});

export default {
  checkCamera: () => check(camera),
  requestCamera: () => request(camera),
  checkContacts: () => check(contacts),
  requestContacts: () => request(contacts),
};

Thanks @zoontek, that already helps a lot and that's one example to could be added to the README. What about when using geolocation, where you can have multiple permissions on Android, e.g. PERMISSIONS.ANDROID.ACCESS_COARSE_LOCATION and
PERMISSIONS.ANDROID.ACCESS_FINE_LOCATION? I guess that that in v1 those were taken care of explicitly.
Sorry for asking all these questions, but hopefully I can find a upgrade path for my project where I don't have to rewrite my whole geolocation handling when updating to the newest version.

@kristerkari In this case, you only need to request the "highest" permission (PERMISSIONS.ANDROID.ACCESS_FINE_LOCATION), and PERMISSIONS.ANDROID.ACCESS_COARSE_LOCATION will be granted automatically.
Same if, for example, you request PERMISSIONS.ANDROID.WRITE_CONTACTS: PERMISSIONS.ANDROID.READ_CONTACTS will be granted. It's the classic Android behavior.
This project example app refresh all checks at each request so you can easily spot which permissions depends on another.

oh I did not know about that, that might be another thing that might be good to mention in the README now that the way of asking the permissions has changed.

Alright, so the migration changes for me were:
v1
import Permissions from "react-native-permissions";

Permissions.request("location", { type: "whenInUse" })

↓ ↓ ↓ ↓ ↓ ↓
"authorized"

v2
import { PERMISSIONS, request } from "react-native-permissions";

const permission = Platform.select({
  android: PERMISSIONS.ANDROID.ACCESS_FINE_LOCATION,
  ios: PERMISSIONS.IOS.LOCATION_WHEN_IN_USE
});

request(permission)

↓ ↓ ↓ ↓ ↓ ↓
"granted"

@kristerkari That's right!


Update examples&documents for orchestration-refactoring

According to #4522/#4681/#4701 that refactor orchestration module, and then we should update our examples&documents now.
We need several volunteers or contributors join us to compelete it.
Anyone is welcome to get your own mission by replying here.

I will refactor the example synchronously when dealing with this issue #4778


Error: Status is REQUEST_DENIED. Requests to this API must be over SSL

Whene I'm usign googlegeocoder with my apiKey in localhost for testing my app I get this error, but if I call url into browser (https://maps.googleapis.com/maps/api/geocode/json?sensor=false&key=****&address=) all works ok.

You need to specify https adapter when you use apiKey
for example:
require('node-geocoder').getGeocoder('google', 'https', extra);

I am going to add an example to documentation

No, I still get same error.

I think this is a file capitalization issue.
In a recent commit, a new file lib/httpadapter/httpsAdapter.js was added (note capital A) but the requires are lowercase. https://github.com/nchaulet/node-geocoder/commit/c2bb5939cdf64a0d1062de9a39305fa6170894fc

yes, there are two errors about capitalize

Sorry for the typo, I fixed it :)

I can close the issue ?


perf(setTick): just a little optimization to be at 0.00ms on iddle

hey, i just did that because i saw some guys complaining about the ~0.05ms on iddle xd, so i think its a bit better for those picky people.. ^^

thx <3

Désolé pour l'attente, je ne l'avais pas vu.


Amazing project!

Amazing!
I waited for this to happen in over 10 years now!
This is like Spotify was 2009 before it become bloated and now you brought it back! I really hate the > 2015 html5 bloated CEF client!
Thanks for building this 'time machine' I've tried to do for a decade!

Thank you for the kind words :) don't overestimate the project though, it's missing quite a few important features and it will never match the official client! I am glad you enjoy it nonetheless! I'll close the issue if you don't mind, I have a lot of issues to keep track of already ;)


Filter attributes need to both be strings to compare correctly.

While trying to implement a filter for events with :action=>:resolve I realized that the value of :action in the Sensu Event is a symbol and my filter attributes are in JSON.  The values of hash_one[key] and hash_two[key] need to be converted to a string before they can accurately be compared.
Example Filter JSON:
{
 "filters": {
   "do_not_resolve": {
     "attributes": {
       "check": {
         "do_not_resolve": true
       },
       "action": "resolve"
     },
     "negate": true
   }
 }
}

Example Sensu Event:
{
  :id=>"b024326c-ef4a-4d56-95f9-72332e5a2702",
  :client=>{
    :name=>"rt4.prod.example.com",
    :address=>"<IP_ADDRESS>",
    :subscriptions=>["base", "rt4.prod.example.com", "testing"],
    :keepalive=>{
      :thresholds=>{
        :warning=>120,
        :critical=>180
      },
      :handlers=>["default", "pagerduty"],
      :refresh=>1800
    },
    :version=>"0.13.1",
    :timestamp=>1421791596
  },
  :check=>{
    :command=>"monitor_that_pid.sh",
    :subscribers=>["testing"],
    :handlers=>["default", "pagerduty"],
    :refresh=>1800,
    :interval=>60,
    :description=>"Monitors Some PID",
    :do_not_resolve=>true,
    :name=>"monitor_some_pid",
    :issued=>1421791537,
    :executed=>1421791537,
    :duration=>0.013,
    :output=>"OK",
    :status=>0,
    :history=>["2", "2", "0", "0", "0", "0", "0", "0", "0", "0", "0", "0", "0", "0", "2", "2", "2", "2", "2", "2", "0"]
  },
  :occurrences=>6,
  :action=>:resolve
}

I'm pretty sure, at the moment, filtering on any Event :action is broken since the actions are symbolized and you can only have a string in the Filter attribute JSON.
I've tested a decent bit, but would appreciate a review or any comments/suggestions!

I don't think this will work if both values are identical hash objects.

@portertech -- I think it would work just fine in that case too.  Here is my very limited testing.
Ex:
> hash_one = {testing: 'test', inside_hash: {new_test: 'also_testing'}}
 => {:testing=>"test", :inside_hash=>{:new_test=>"also_testing"}}
> hash_two = {testing: 'test', inside_hash: {new_test: 'also_testing'}}
 => {:testing=>"test", :inside_hash=>{:new_test=>"also_testing"}}
> hash_three = {testing_failure: 'test', inside_hash: {new_test: 'testing_failure'}}
 => {:testing_failure=>"test", :inside_hash=>{:new_test=>"testing_failure"}}

> hash_one == hash_two
 => true
> hash_one == hash_three
 => false
> hash_one.to_s == hash_two.to_s
 => true
> hash_one.to_s == hash_three.to_s
 => false

Could you elaborate a bit more on why you think this will not work?

What if the hashes are not sorted? Could end up w/ different strings.

[1] pry(main)> hash_one = {testing: 'test', inside_hash: {new_test: 'also_testing'}}
=> {:testing=>"test", :inside_hash=>{:new_test=>"also_testing"}}
[2] pry(main)> hash_two = {inside_hash: {new_test: 'also_testing'}, testing: 'test'}
=> {:inside_hash=>{:new_test=>"also_testing"}, :testing=>"test"}
[3] pry(main)> hash_one == hash_two
=> true
[4] pry(main)> hash_one.to_s == hash_two.to_s
=> false

I think my example might have been misleading.  Sorry about that.
The issue is that the Sensu event['action'] is a Symbol and the filter JSON's action is a String.
Need to find a way to make sure all values in the Sensu event['action'] are of Class String before doing a comparison.
Ex with symbol:
> hash_one = {:testing=>"test", :inside_hash=>{:new_test=>:also_testing}}
 => {:testing=>"test", :inside_hash=>{:new_test=>:also_testing}}
> hash_two = {:inside_hash=>{:new_test=>"also_testing"}, :testing=>"test"}
 => {:inside_hash=>{:new_test=>"also_testing"}, :testing=>"test"}
> hash_one == hash_two
 => false

Is filtering by an action working at all? Or am I just mis-using it?

I've thrown together https://github.com/sensu/sensu/pull/858, going to cut a beta release w/ the change.

What version of Ruby are you using?

Object comparison should return true.

/opt/sensu/embedded/bin/ruby --version
ruby 2.0.0p353 (2013-11-22 revision 43784) [x86_64-linux]

Master has string comparison of filter attribute value and event attribute value. The specs include a test for event :action :+1:


Try to export pcd in Google Colab but failed

Describe the bug
Traceback (most recent call last):
  File "/usr/local/bin/ns-export", line 5, in <module>
    from scripts.exporter import entrypoint
  File "/usr/local/lib/python3.8/site-packages/scripts/exporter.py", line 20, in <module>
    from nerfstudio.exporter import texture_utils, tsdf_utils
  File "/usr/local/lib/python3.8/site-packages/nerfstudio/exporter/texture_utils.py", line 36, in <module>
    from nerfstudio.exporter.exporter_utils import Mesh
  File "/usr/local/lib/python3.8/site-packages/nerfstudio/exporter/exporter_utils.py", line 29, in <module>
    import pymeshlab
  File "/usr/local/lib/python3.8/site-packages/pymeshlab/__init__.py", line 11, in <module>
    from .pmeshlab import *
ImportError: /usr/local/lib/python3.8/site-packages/pymeshlab/lib/libmeshlab-common.so: symbol __cxa_throw_bad_array_new_length version Qt_5 not defined in file libQt5Xml.so.5 with link time reference

After training the project, I tried to export pcd by using ns-export in Google Colab I got the error message above.
Need some help. Thanks.

Hi, here is the current workaround from another issue: link


Possible to detect an open lid / removed dust container on the 1st gen Model?

Hey there.
I am running a Roborock Vacuum v1 which works great with your adapter.
Every 10 runs through my home i use
mihome-vacuum.0.control.goTo
to move him in front of the bin so i am reminded of emptying the dust container.
Is there a way to detect me opening / closing the lid or refitting the dust container in order to automatically sending him back to the charger?
So far the codes from
mihome-vacuum.0.info.state
i found are

16 Going to spot
3 Waiting
6 Back to home
8 Charging
17 Zone Cleaning
11 Spot Cleaning

Would i be possible for you to display some more states like i mentioned above?
Thanks in advance.

Afaik no Xiaomi/Roborock cleaner has a sensor for dust bin level and no sensor for the lid as well.
But you can place a xiaomi door sensor inside the dust bin and in the little hole under the lid there is enough place for the magnet. I have an own object that counts the cleaned area after each empting of dust bin. If the value reaches 240 m² iobroker sends the robot to goto coordinates and if the xiaomi motion sensor detects motion in this room, Alexa says that the dust bin should be emptied. After re-inserting the empty dust bin the door sensor is triggered and iobroker can send the robot back to docking station.

Great Idea.
Thank you very much. That's probably what i am going to do then.

that is something for the forum not for an issue


Modal is positioned at the bottom of the page

I added modal to my component:
return (

          {
            this.state.isShowingModal &&
            <ModalContainer onClose={this.handleClose.bind(this)}>
              <ModalDialog onClose={this.handleClose.bind(this)}>

Dialog Content

More Content. Anything goes here

              </ModalDialog>
            </ModalContainer>
          }

          <SearchBox />
          <LabelForm />
          {this.renderRepos()}

However, when I open it, ModalDialog gets wrong value of topOffset. In my case it gets 3232.5px which puts it at the bottom of the page.

I the code I see that this value is injected by centerComponent. Should I pass something to make it work correctly ?

Ok, I found the issue. I didn't have doctype declaration in HTML file. In this case documentElement and body actually switch, so centerComponent returns wrong value.

I see... I've been using the UnstyledFlexDialog myself - if you want to look into that. It uses flexbox instead. It's much more reliable.


UX: Provide full diff for code samples

Our code samples are super condensed and each line of them has a purpose.
For example, diff for first code sample of native_function_invocation is:
 * Example #1. Fixing with the default configuration.
   ---------- begin diff ----------
   --- Original
   +++ New
   @@ -2,9 +2,9 @@

    function baz($options)
    {
   -    if (!array_key_exists("foo", $options)) {
   +    if (!\array_key_exists("foo", $options)) {
            throw new \InvalidArgumentException();
        }

   -    return json_encode($options);
   +    return \json_encode($options);
    }

   ----------- end diff -----------

The second sample is this:
 * Example #2. Fixing with configuration: ['exclude' => ['json_encode']].
   ---------- begin diff ----------
   --- Original
   +++ New
   @@ -2,7 +2,7 @@

    function baz($options)
    {
   -    if (!array_key_exists("foo", $options)) {
   +    if (!\array_key_exists("foo", $options)) {
            throw new \InvalidArgumentException();
        }

   ----------- end diff -----------

in fact, they are using very same input code, just different configuration. But in later, we don't know that, as we don't see json_encode in diff at all.

Proposed PR would change 2nd example into:
 * Example #2. Fixing with configuration: ['exclude' => ['json_encode']].
   ---------- begin diff ----------
   --- Original
   +++ New
   @@ -1,10 +1,10 @@
    <?php

    function baz($options)
    {
   -    if (!array_key_exists("foo", $options)) {
   +    if (!\array_key_exists("foo", $options)) {
            throw new \InvalidArgumentException();
        }

        return json_encode($options);
    }

   ----------- end diff -----------

:+1: for the idea, we could even create a real full differ on sebastianbergmann/diff one day.

What would be the difference in generated output between differ in this PR and that full differ you imagine about? TBH, not sure if there is a need - for udiffer, it was important for us, we wanted to use it heavily and in the and, a lot of ppl were looking for udiffer implementation in PHP.
But full differ ? Well, there is not that much usage of it in the wild, and here, at our code samples case, it's super edge case.
Long story short - not sure if it's worth to spend time on it


About minSDK version

Yoy said that  it support Android 4.1 (API 16)+
But it throw a exception  on API19;
with:
Caused by: com.facebook.react.bridge.JSApplicationIllegalArgumentException: Attribute     selectableItemBackgroundBorderless couldn't be found in the resource list;
had depend on: com.android.support:appcompat-v7:23.3.0
May I have done something wrong?

I haven't met this error.
Please check npm>=3.x node>=4.x,
then remove node_modules file and run npm install,
try again.

env:
node:v6.4.0
npm:3.10.3
react-native-cli:1.0.0
react-native:0.31.0
It seems that something wrong with android package;
I will check my android env / build tools (windows)
and try  run-android on macos ;
Thanks!

It's plugin: react-native-scrollable-tab-view 's Bug
https://github.com/skv-headless/react-native-scrollable-tab-view/issues/397

@keanuyaoo Thanks! I will stay focused on it.


Storage Account SAS

Based on this section https://docs.microsoft.com/en-us/azure/virtual-machines/extensions/diagnostics-linux#protected-settings
the extension stored the SAS key, then when SAS expire, the extension stops working and can not send the log to the storage account?
Is there any document talking about SAS key expire, we need to update the extension or re-install the extension?
Is there a RABC role option to access storage account from the extension?

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

ID: 13249027-1524-dd0c-9d83-15a9733a3d19
Version Independent ID: bf8a2d1a-8d9b-228a-6919-16127f315d40
Content: Azure Compute - Linux Diagnostic Extension
Content Source: articles/virtual-machines/extensions/diagnostics-linux.md
Service: virtual-machines-linux
GitHub Login: @axayjo
Microsoft Alias: akjosh

@RonnieQuan
Thanks for the feedback! We are currently investigating and will get back on this.

@RonnieQuan As there are already other issues opened by you on the same query in these issues #52880 and #52774, I will close this issue and we can continue the discussion on the other issues.


Eport API: Zero length response gacks with "undefined method `strip' for nil:NilClass"

I reported #102 back in November to MC and they said that the Export API was permanently changed to return a single empty space as the response payload if there was nothing to return. #102 fixed that, but starting recently, MC has reverted back to the old behavior of returning a zero length response.
This patch to the patch should accommodate for both scenarios

Thanks @daveespo. Will merge this as soon as soon as the build is complete.

You are awesome. Thanks! :-)


net_snmp: fix CVE-2018-18065

Motivation for this change
This addresses a remote DoS attack (CVE-2018-18065).
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 nox --run "nox-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)
[ ] Fits CONTRIBUTING.md.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: net_snmp
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: net_snmp
Partial log (click to expand)
gzipping man pages under /nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/share/man/
strip is /nix/store/dxf1m7dhc4qb655bdljc1fsd74v1nag3-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/lib  /nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/bin  /nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/sbin
patching script interpreter paths in /nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3
/nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/bin/net-snmp-config: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
/nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/bin/net-snmp-create-v3-user: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
/nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/bin/mib2c-update: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3...
moving /nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/sbin/* to /nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3/bin
/nix/store/vhp1bm7bb1zjcklj7sjhyqa0yb150y4i-net-snmp-5.7.3

Success on aarch64-linux (full log)
Attempted: net_snmp
Partial log (click to expand)
gzipping man pages under /nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/share/man/
strip is /nix/store/428gs2z4b8f9byvghzlpbjwjb3a7jwxx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/lib  /nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/bin  /nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/sbin
patching script interpreter paths in /nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3
/nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/bin/net-snmp-create-v3-user: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
/nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/bin/net-snmp-config: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
/nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/bin/mib2c-update: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3...
moving /nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/sbin/* to /nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3/bin
/nix/store/39pkydwlzsgzb82ji8y4a7whq8l8xgig-net-snmp-5.7.3


Ability to link EKS-D release to Platform Version on AWS EKS service

What would you like to be added:

Some ability to link between this page that is official AWS documentation to a released version of the relevant build here.

Why is this needed:
This helps keep up with documentation to show exactly what has changed between versions and would help include these details in any tool encouraging users to adopt the most recent release/version of Kubernetes for their team!

@jrm16020 - Since your goal is to get documentation for EKS-D releases (based on your inputs in "why is this needed"), have you considered looking at the EKS-D changelog? E.g.: https://distro.eks.amazonaws.com/releases/1-27/9/CHANGELOG-v1-27-eks-9/ This should give you the information on what has changed between versions.

@vipin-mohan the changelog is great but there's no clear link between the EKS-D release and the EKS platform version made available online in the service.
v1-27-eks-9 != eks.5 for the 1.27 release branch.

Hi,
I've also stumbled to this issue by looking for something similar to the OP.
I was sure EKS was based on EKS-d, i mean why have 2 versions of the same? but the release dates don't make sense.

v1-27-eks-9 != eks.5 for the 1.27 release branch.

v1-27-eks-5 (June 09, 2023)
vs
1.27.4 | eks.5 (August 30, 2023)

There is no one-to-one mapping between EKS platform versions and EKS-D patch releases. Moreover, EKS platform versions (and similarly EKS-D patch releases) don't introduce breaking changes. I'm not clear about why you want to align an EKS-D patch version with an EKS platform version. Can you share more details?

Sure, i need to understand the changes made in both of them so that i know if it's something that can cause issues for other products being used, even if the changes aren't breaking changes they might still interfere with the things we are running or plan to run.


Custom functions in SQLite EF Core 3.*

In previous versions of EF Core (2.*) It was possible to use C# functions as a part of LINQ request which were evaluated on client side. As the client evaluation is not possible anymore in EF Core 3.*, It is also not possible to use custom functions in SQLite. I used to do something like this:
public static class DbFunctionsExtensions
{
    [DbFunction]
    public static string Unaccent(this DbFunctions _, string value)
    {
        // Code for removing accent
    }
}

and used it in such a way:
myDbContext.Test
      .Where(x => EF.Functions.Unaccent(x.Text) == "xyz")
      .ToList();

Function EF.Functions.Unaccent was evaluated on the client, which was not a problem in this case as the database (SQLite) runs in the same process as the client.
Is there any way to add such custom function to SQLite DB provider in EF Core 3.* and evaluate it on the client?

I finally found a solution. UDF seems to be a better way than using the client evaluation.
Function above could be registered as follows:
modelBuilder.HasDbFunction(typeof(DbFunctionsExtensions).GetMethod(nameof(DbFunctionsExtensions.Unaccent)))
                .HasTranslation(args =>
                    SqlFunctionExpression.Create(
                        "unaccent",
                        new[]
                        {
                            args.Skip(1).FirstOrDefault()
                        },
                        typeof(string),
                        null
                    )
                )
                .HasParameter("_").Metadata.TypeMapping = new StringTypeMapping("string");


Zero amount handling in engine.js

engine.js bails out if zero values are received
2017-06-23 02:24:44  0.28 XRP-USD   -0.4%    98467    --         0.30%             0.00 XRP   0.00 USD/home/eigilb/Prosjekt/Z/work/node_modules/numbro/numbro.js:1111
        var parts = x.toString().split('.');
                     ^

TypeError: Cannot read property 'toString' of undefined
    at multiplier (/home/eigilb/Prosjekt/Z/work/node_modules/numbro/numbro.js:1111:22)
    at /home/eigilb/Prosjekt/Z/work/node_modules/numbro/numbro.js:1127:22
    at Array.reduce (native)
    at correctionFactor (/home/eigilb/Prosjekt/Z/work/node_modules/numbro/numbro.js:1125:21)
    at cback (/home/eigilb/Prosjekt/Z/work/node_modules/numbro/numbro.js:1218:34)
    at Array.reduce (native)
    at Object.multiply (/home/eigilb/Prosjekt/Z/work/node_modules/numbro/numbro.js:1224:48)
    at writeReport (/home/eigilb/Prosjekt/Z/work/lib/engine.js:715:72)
    at Timeout._onTimeout (/home/eigilb/Prosjekt/Z/work/lib/engine.js:633:11)
    at ontimeout (timers.js:386:14)
    at tryOnTimeout (timers.js:250:5)
    at Timer.listOnTimeout (timers.js:214:5)

I think this is the same as #457, which has been resolved.


Typo in "guestbook" at https://firebase.google.com/codelabs/firebase-get-to-know-flutter#5

As you can see in the image below the word is written as "gustbook" instead of "guestbook" in the tutorial.

PTAL @ericwindmill


mdi:weather-night-partly-cloudy icon at night

Apparently, OpenWeatherMap reports partly cloudy conditions during night time as "partlycloudy", not considering the time of day.
This results in the hourly forecast graph containing the mdi:weather-partly-cloudy icon during the night, sometimes next to a clear night sky mdi:weather-night icon, looking a bit silly.
I'd like to kindly ask for a workaround, if possible, that would at backend level check for time of day and force the state and icon to be replaced with night-partly-cloudy and mdi:weather-night-partly-cloudy, giving more consistency.
//Duplicate of https://github.com/decompil3d/lovelace-hourly-weather/issues/122, please remove.

Duplicate of https://github.com/decompil3d/lovelace-hourly-weather/issues/122

Sorry for making this a mess.


Flask-JWT-Extended upgrade causes > 4.4.4 causes 'Cookie' object has no attribute 'name'

Description
Upgrading version of Flask-JWT-Extended to >= 4.5.0 introduces breaking changes that cause test cases to fail with the error 'Cookie' object has no attribute 'name'.

We need to update the API code to work with newer versions.
Steps to Reproduce
On your local machine:

[ ] In requirements-app.txt, upgrade Flask-JWT-Extended to >=4.5.2, which should result in a build with a newer version.
[ ] Build the application and test Docker images
[ ] Run all test cases

Workaround
N/A
Impact/Urgency
N/A
Expected Behavior

[ ] All Unit Test cases pass locally and in Github
[ ] requirements-app.txt and requirements.txt updated to reflect the newer version in use

QA Considerations
For QA to populate. Leave blank if QA is not applicable on this ticket.
Additional Info & Resources
https://flask-jwt-extended.readthedocs.io/en/3.0.0_release/tokens_in_cookies/

We misdiagnosed the problem.  Flask-JWT-Extended has been upgraded; it was not the cause of the AttributeError.


Create about:* protocol for content pages

about:* seems like a special scheme we can't override.
Not sure we can have URL scheme's that don't contain //, so we'd have tofino://home. We can fix URLs on the client to convert tofino:home -> tofino://home if we need. Anyways, after lots of digging, that's what I think the case is.
Also interesting to note, a handful of chrome:* URIs do work -- maybe we can proxy tofino://* calls to that.


[DRAFT] initial notes re: style guide

NB - this is a draft for discussion

addresses

#57
#60

there's some opinion expressed here.  comments appreciated.

Hello, I think this is a very good idea to:

add a  TOC in the README file.
move all the contributions guide, style  guide ... from the README to a separate md so the README will be more about presenting the content of the repo

very useful!  i think i see what you’re going for here now. i think we can dust this up accordingly

cool!
Thanks! Could we please also include the contributing.md in the mkdocs.yml as well (in add to include a link to the contributing.md in the README.md file).
Same if we create other md files (external reference, about,  deployment guide, installation, ...).


Define accepted log public keys

The Chromium implementation of CT is limited in support of public keys to the set of public keys it accepts for the Web PKI:

RSA (nominally, 2048, 3072, 4096)
ECC using NIST P-256, P-384

The Chromium implementation does not explicitly support Curve25519, although it could, and does not support other forms of EC keys.

We also need to check what key algs our compliance monitoring infrastructure supports to provide the minimal set of supportable key types.

Also, of note is Section 2.1.4 of RFC 6962 which states:
Various data structures are signed.  A log MUST use either elliptic
curve signatures using the NIST P-256 curve (Section D.1.2.3 of the
Digital Signature Standard [DSS]) or RSA signatures (RSASSA-PKCS1-
V1_5 with SHA-256, Section 8.2 of [RFC3447]) using a key of at least
2048 bits.


Advanced type hinting support

What was changed

Removed dacite
Wrote custom type-hint support for deserializing

Why?
Need to support all sorts of type hints like Optional, Union, etc
Checklist

Closes #78

Looks like that wasn't too bad

@Sushisource - It wasn't too bad, but I fear people may have custom needs and it may be hard to document in detailed form what we specifically support. Also, while the code is small, it may not be obvious to users that, for example, a union tries all union types and chooses the first one (so if the first dataclass matched I wouldn't try the second).


Updated Readme Layout and Added Content

added resources to list
new_layout_init
added reactjs cheatsheets and layout

Hi @achingachris
thanks for making this pull request.
there seems to be some merge conflicts. You can resolve them and request a review

@achingachris On the Cheatsheets table, kindly update the Link title to Description since that column has descriptions.

@achingachris On the Cheatsheets table, kindly update the Link title to Description since that column has descriptions.

On it,thanks for pointing it out

@stephanieopala Updated the Cheatsheet table


Several improvements and added some examples.

Good additions; vectorizing where possible will help keep augraphy fast enough for comfortable use, and I think we may need to start adding longer examples like these.


jsonable_encoder alters json_encoders of Pydantic objects

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from pydantic import BaseModel, SecretStr
from fastapi.encoders import jsonable_encoder

class Credentials(BaseModel):
    password: SecretStr

ENCODERS = {SecretStr: lambda v: v.get_secret_value() if v is not None else None}

creds = Credentials(password="helloworld")

a = jsonable_encoder(creds)
print(a)  # {'password': '**********'}, as expected
b = jsonable_encoder(creds, custom_encoder=ENCODERS)
print(b)  # {'password': 'helloworld'}, as expected
c = jsonable_encoder(creds)
print(c)  # gives {'password': 'helloworld'}, but should be {'password': '**********'}?

creds = Credentials(password="123456789")

d = jsonable_encoder(creds)
print(d)  # gives {'password': '123456789'}, but should be {'password': '**********'}?

Description

I'm using jsonable_encoder at multiple locations in the code, sometimes with custom_encoder and sometimes without.
My understanding is that passing custom_encoder should only affect the current call and not modify any global state.
What actually happens is that it modifies __config__.json_encoders, and as a result, public API endpoints might suddenly start returning secret values that should be filtered.

Operating System
Linux
Operating System Details
No response
FastAPI Version
0.78.0
Python Version
Python 3.9.2
Additional Context
Using Pydantic version 1.9.1.

I see that it use __config__.json_encoders and also pass to next call with encoder, so could we use encoder = encoder.copy() to avoid  modifies __config__.json_encoders?

Yes, copying encoder before modifying it fixes the issue. Since FastAPI requires at least Python 3.6 merging the dictionaries could also be done with {**a, **b}, so an alternative fix would look like this:
diff --git a/fastapi/encoders.py b/fastapi/encoders.py
index 4b7ffe31..39cc2925 100644
--- a/fastapi/encoders.py
+++ b/fastapi/encoders.py
@@ -52,7 +52,7 @@ def jsonable_encoder(
     if isinstance(obj, BaseModel):
         encoder = getattr(obj.__config__, "json_encoders", {})
         if custom_encoder:
-            encoder.update(custom_encoder)
+            encoder = {**encoder, **custom_encoder}
         obj_dict = obj.dict(
             include=include,  # type: ignore # in Pydantic
             exclude=exclude,  # type: ignore # in Pydantic

Note that I'm not sure yet if the change has any negative side effects. Would be good to know to know if there was a reason why .update() was used in the first place. :thinking:

For reference, the code in question was initially added in https://github.com/tiangolo/fastapi/pull/715. There is no explanation why the pydantic config itself is updated, so it probably was meant to be a copy in the first place.


NativeModules.VRTNodeModule.getNodeTransform is not a function

I am using getTransformAsync on a Viro3DObject, but I get the warning

TypeError: NativeModules.VRTNodeModule.getNodeTransform is not a function

I am using react-native version 0.50.3 and react-viro 2.1.0, running on a Samsung 8.

Hi @draperunner, this is a known bug on Android. This will be fixed in our next release.

Hi @draperunner,
This issue is now fixed in our Viro React v2.2.0 release:
https://docs.viromedia.com/v2.2.0/docs/releases
Let us know if you have any other issues!

Great! I am currently not using getTransformAsync, so I'll just close this issue and re-open if I have problems in the future.


Developing for DefectDojo in Visual Studio 2019

Greetings and well met. I am a developer attempting to build out defect dojo and was wondering how I might make it so I can work with it in Visual Studio? I like the way VS allows me to create virtual environments with a right click. How can I get the app to run inside VS with a Virtual Environment?

@dbrewerton This is not currently supported. Unfortunately I don't see the team getting bandwith to support this. If this is something you are interested in working on, we would gladly accept a pull request to the documentation.


Bug: clear command is easy to accidentally execute and delete all users

Behavior
clear is such an easy command to accidentally enter and entirely delete your data
Expected Behavior
Make clear harder to type, like clear all
or
Add a confirmation prompt.

Closing this because not really an issue


chore(main): release 9.8.0

:robot: I have created a release beep boop
9.8.0 (2024-11-15)
Features

adapters: allow for o1 models at some point with improved schema (601fa72)
copilot: now working with o1 models (7283611)

Bug Fixes

chat: clear references table (7c0f6fb)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v9.8.0
:sunflower:


Download own avatar with PowerShell

Something I'd like is for there to be a simple method of grabbing/downloading your own avatar that you currently have on LANCommander, and copying it to a location where it might be needed.
I have a few games that support avatars where you simply place, for example, "avatar.png" in the game files and then that avatar will show up ingame. It would be cool if there was something similar to the "Name Change" function but instead for avatars. "Avatar Change"?

There are a couple of API endpoints that could be used for this purpose:
/api/Profile/Avatar
This will return the image of the current user's avatar, but the request has to be authed
/api/Profile/{username}/Avatar
This route will return the avatar image of the specified user and can be used with no authentication.
There may be a way to do more event-driven scripts in the future, but as of right now I'd probably recommend using one of these two routes with a Before Start script.

Ah that's great, thank you. Now I'm just wondering how I would go about grabbing the current username (and not PlayerAlias) for this purpose? :)

Ah, that might not be a bad idea to include in the list of variables.


SILOptimizer: Break circular dependency with SIL library by moving canDuplicate()

Fixes a cycle introduced by https://github.com/apple/swift/pull/61051.

@swift-ci please smoke test

@swift-ci please smoke test macOS


brogue: 1.7.4 -> 1.7.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/brogue/versions.
meta.description for brogue is: '"A roguelike game"'.

Checks done (click to expand)

built on NixOS
/nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5/bin/brogue passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 1.7.5 with grep in /nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5
directory tree listing: https://gist.github.com/fd3b2ff07f1abac023ca14b7d5f5ea59
du listing: https://gist.github.com/866aa87597a7771e4c2e471340fcedb7

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/43dcb8391de5f20a4765a690d1e4d27559c3e45b.tar.gz -p brogue
ls -la /nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5

cc @svenkeidel for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: brogue
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: brogue
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: brogue
Partial log (click to expand)
gcc -O2 -march=i586 -o bin/brogue src/brogue/Architect.o src/brogue/Combat.o src/brogue/Dijkstra.o src/brogue/Globals.o src/brogue/IO.o src/brogue/Items.o src/brogue/Light.o src/brogue/Monsters.o src/brogue/Buttons.o src/brogue/Movement.o src/brogue/Recordings.o src/brogue/RogueMain.o src/brogue/Random.o src/brogue/MainMenu.o src/brogue/Grid.o src/brogue/Time.o src/brogue/PowerTables.o src/brogue/Sqrt.o src/platform/main.o src/platform/platformdependent.o src/platform/curses-platform.o src/platform/tcod-platform.o src/platform/term.o -L. -L/nix/store/w8ql441mmrmv7b2h7ygmdclj90sn52cz-libtcod-1.5.1 `/nix/store/mxb44g4l4qhwkx4syrjgfygw388jf8q7-SDL-1.2.15-dev/bin/sdl-config --cflags` `/nix/store/mxb44g4l4qhwkx4syrjgfygw388jf8q7-SDL-1.2.15-dev/bin/sdl-config --libs` -ltcod -Wl,-rpath,. -lncurses -lm -Wl,-rpath,.
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5
shrinking /nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5/bin/brogue
strip is /nix/store/dxf1m7dhc4qb655bdljc1fsd74v1nag3-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5/bin
patching script interpreter paths in /nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5
checking for references to /build in /nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5...
/nix/store/q764k00389k1nr07amyfllnxy8a8kwj2-brogue-1.7.5

I tested it and it works. :+1:


Can't use package_dir in setup.cfg in 40.0.0

Since documentation doesn't say to much about how to use package_dir in setup.cfg except:
Type names used below:
dict - list-comma where keys are separated from values by =

and
Options
Key | Type
-- | --
package_dir | dict

I tried
package_dir =
    = src

but it results in:
./setup.py install
Traceback (most recent call last):
  File "./setup.py", line 4, in <module>
    setup()
  File "/usr/lib64/python3.6/site-packages/setuptools/__init__.py", line 131, in setup
    return distutils.core.setup(**attrs)
  File "/usr/lib64/python3.6/distutils/core.py", line 121, in setup
    dist.parse_config_files()
  File "/usr/lib64/python3.6/site-packages/setuptools/dist.py", line 495, in parse_config_files
    ignore_option_errors=ignore_option_errors)
  File "/usr/lib64/python3.6/site-packages/setuptools/config.py", line 114, in parse_configuration
    meta.parse()
  File "/usr/lib64/python3.6/site-packages/setuptools/config.py", line 400, in parse
    section_parser_method(section_options)
  File "/usr/lib64/python3.6/site-packages/setuptools/config.py", line 373, in parse_section
    self[name] = value
  File "/usr/lib64/python3.6/site-packages/setuptools/config.py", line 177, in __setitem__
    value = parser(value)
  File "/usr/lib64/python3.6/site-packages/setuptools/config.py", line 466, in _parse_version
    version = self._parse_attr(value, self.package_dir)
  File "/usr/lib64/python3.6/site-packages/setuptools/config.py", line 309, in _parse_attr
    if attrs_path[0] in package_dir:
IndexError: list index out of range

@rlalik , just checked what all work correctly
from my setup.cfg:
[options]
include_package_data = True
package_dir =
= src
packages =
laziest
my files tree:

src:

laziest:
*.py files

all work correctly and packed in whl
setuptools    version 40.2.0
in doc exist:
"And such projects also need something like package_dir={'':'src'} in their setup() arguments, but that’s just a normal distutils thing.)"
and it's really described in distutils - https://docs.python.org/2/distutils/setupscript.html#listing-whole-packages
but in setuptools not provided an example of how to write it in setup.cfg, it's true, but I don't think what it's a really critical
if smbd from contributors think what it's good to have in doc - I can send PR with documentation additions


Support for Aggregate quota resources for all namespaces of a tenant

Aggregate of resource quotas for all namespaces for a tenant shall remain below Maximum aggregated resource quotas, which might associated with tenant CR in some way. Given that workloads of a tenant would share the resource of the same Kubernetes cluster with other tenants, and the tenant admin has ability to create new namespaces and associate resource quotas to it, maximum aggregated resource quotas would provide a safeguard against resource starvation of of workloads for other tenants.
Maximum aggregated resource quotas will be required to be defined for following compute and storage quotas (this is just for example, list can certainly grow):
1- maximum number of cpus across all namespaces for a tenant.
2- maximum size of memory across all namespaces for a tenant.
3- maximum size of storage for a given storage class across all namespaces for a tenant.
4- maximum size ephemeral storage across all namespaces for a tenant.

I think the Aggregate Resource Quota can be enforced at TenantAdmin Level.
Could this be enforced by using the HNC (https://github.com/kubernetes-sigs/multi-tenancy/tree/master/incubator/hnc ) ? or any other mechanism ?

HNC doesn't directly support hierarchical resource quota, though there was an IBM team working on implementing a type of RQ that supported HNC. There's no reason someone couldn't build it, it's just not a priority for our usecases right now, but we'd be happy to accept contributions.

@adrianludwin , do you mind adding reference to the code or issue here, in case it exists , thanks.

It was the first agenda item on Dec 3 2019: https://youtu.be/V0Zw82nOAEE

This issue relates to https://github.com/kubernetes-sigs/multi-tenancy/issues/664


Add worker nodes to OCP4

Add worker nodes to OCP4

@davidkirwan All look great
just need one confirmation on the link syntax part


RUM-6256: Optimize MD5 byte array to hex string conversion

What does this PR do?
This PR changes the way to convert MD5 bytes to HEX string.
Motivation
When we broke down the steps of generating image wireframe, we noticed that joinToString function takes a huge amount of time when generating the MD5 resource id:

Suspicious reasons:

String Concatenation Overhead:
joinToString internally uses StringBuilder, but string concatenation inside the lambda (using "%02x".format(it)) could be inefficient. If format() is invoked for each byte, it involves repeated function calls, which adds to overhead.

Repeated Memory Allocations:
"%02x".format(it) creates a new string for each byte, which leads to multiple temporary objects being created.

Outcome
After using the new function to convert the byte arrays, we can see the duration optimisation rate is up to 99%
Span list

joinToString
convertToHexString

Time series

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.03%. Comparing base (a6bd6a5) to head (99cd1bc).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2273      +/-   ##
===========================================
+ Coverage    69.95%   70.03%   +0.09%
===========================================
  Files          731      731
  Lines        27182    27215      +33
  Branches      4573     4580       +7
===========================================
+ Hits         19013    19060      +47
+ Misses        6885     6877       -8
+ Partials      1284     1278       -6

Files with missing lines
Coverage Δ

...ay/internal/recorder/resources/MD5HashGenerator.kt
66.67% <100.00%> (+13.33%)
:arrow_up:

... and 31 files with indirect coverage changes