Consider refactoring Snapshot type bound.

This interface is geared toward small file-based snapshots. However, not all snapshots can
be easily represented as a file. Probably a more generic interface will be needed to address
also other needs.
https://github.com/datafuselabs/openraft/blob/46644c8409c7dff627c991cc65849507ad5265b3/openraft/src/storage.rs#L147-L159

I'd like to understand this a bit better. I could perhaps submit a PR to suggesting an implementation. My snapshots are actually large and not stored as files. The interface works afaik, but I do experience issues like #235 that may be related to these snapshot assumptions.

The API forces user to implement a snapshot in a byte-stream way.
IMHO, a snapshot would be better to be a stream of a user-defined data type.
And incremental snapshot replication may be useful.

@fredfortier We also have a relatively complex snapshot that is not file based. Basically, it is a set of pages to transfer with some meta information regarding page location (since the pages may cross-reference themselves). This needs to be materialized at the remote side.
Although it is surely possible to access such a snapshot using a byte stream, it's quite complex to provide a reliable working random-access implementation to it. It would be simpler to handle it in a different way, with records with associated metadata.
I even though of sending a small snapshot info (~2KB) to the follower via the existing snapshot interface and then actively request data belonging to this snapshot from the leader as needed. I still don't have a concrete idea how to cut the API the best to handle all possible use cases.

@drmingdrmer would it make sense to make Snapshot's bounds be futures_core::Stream Send + Unpin + 'static instead of AsyncRead + AsyncSeek + Send + Unpin + 'static for RaftSnapshotBuilder? I was going through looking how to implement the snapshoting in my project and ran into an issue with the current trait bounds. Mainly that AsyncSeek isn't really an option for me. And looking at the snapshot code i'm not really sure why it's needed. Any insight there would be great!

@drmingdrmer would it make sense to make Snapshot's bounds be futures_core::Stream + Send + Unpin + 'static instead of AsyncRead + AsyncSeek + Send + Unpin + 'static for RaftSnapshotBuilder?

Agree. a Stream of user-defined data chunks would be better. And it can just be a stream of Vec<u8> to simulate an AsyncRead.

I was going through looking how to implement the snapshoting in my project and ran into an issue with the current trait bounds. Mainly that AsyncSeek isn't really an option for me. And looking at the snapshot code i'm not really sure why it's needed. Any insight there would be great!

I guess it is a historical reason: the API is derived from etcd-raft in which a snapshot is just a byte stream.

@drmingdrmer would you mind checking if the approach I took here https://github.com/zach-schoenberger/openraft/tree/snapshot_stream looks good to you? If so I'll finish out updating the tests and examples

@drmingdrmer would you mind checking if the approach I took here https://github.com/zach-schoenberger/openraft/tree/snapshot_stream looks good to you? If so I'll finish out updating the tests and examples

Thank you for your suggestion.
Can you make it a draft PR so that everyone will be able to see the changes clearly online?

noting here the pr: https://github.com/datafuselabs/openraft/pull/600


[WEB-SDK] - Add Payment Method with Dynamic Field Rendering - CimbVa

The Hyperswitch SDK is a versatile tool built to streamline payment integration for developers. It simplifies the process of adding multiple payment methods, allowing you to quickly implement support for various gateways without the need to manually integrate each one. With Hyperswitch, you can efficiently manage payments across platforms, reducing complexity and boosting development speed.
Getting started

Get familiar with ReScript.
Check the README.md for project structure and setup instructions.
To setup locally, follow these steps

#  Clone repository
cd hyperswitch-web
yarn install
yarn run server     # This starts the mock server
yarn run re:start   # In another terminal

# Start Demo App in another terminal
cd hyperswitch-react-demo-app
yarn install
yarn run start        # This starts the demo app

Description:
We are looking to implement a new payment method in our WebSDK that supports dynamic field rendering. The goal is to allow the backend to define the fields required for the payment method and have the frontend WebSDK render these fields dynamically based on the response.
This feature should be flexible enough to handle different payment methods, each potentially requiring different types of input fields (e.g., text fields, dropdowns, checkboxes, etc.).

Requirements:

Payment Method Integration Details:
In this section, we provide details on the payment call response and the confirm call payload for the newly integrated payment method. Ensure you have configured the payment method at Hyperswitch Dashboard correctly. The backend already includes support for this payment method, so no additional configuration is needed.
Payment Method List Response for CimbVa -
{
  "payment_method": "bank_transfer",
  "payment_method_types": [
    {
      "payment_method_type": "cimb_va",
      "payment_experience": null,
      "card_networks": null,
      "bank_names": null,
      "bank_debits": null,
      "bank_transfers": {
        "eligible_connectors": ["adyen"]
      },
      "required_fields": {
        "billing.email": {
          "required_field": "payment_method_data.billing.email",
          "display_name": "email",
          "field_type": "user_email_address",
          "value": null
        },
        "billing.address.first_name": {
          "required_field": "payment_method_data.billing.address.first_name",
          "display_name": "card_holder_name",
          "field_type": "user_full_name",
          "value": null
        },
        "billing.address.last_name": {
          "required_field": "payment_method_data.billing.address.last_name",
          "display_name": "card_holder_name",
          "field_type": "user_full_name",
          "value": null
        }
      },
      "surcharge_details": null,
      "pm_auth_connector": null
    }
  ]
}

Confirm call Payload should look like:
curl --location 'http://localhost:8080/payments' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--header 'api-key: dev_n9y9SATrxsd8LxVDN7hkyEJcnl1CLdIii8eynRLWfj84Q6BGGS5hXhhljMT6AK1A' \
--data-raw '{
"amount": 1000,
"currency": "IDR",
"confirm": true,
"capture_method": "automatic",
"capture_on": "2022-09-10T10:11:12Z",
"amount_to_capture": 1000,
"phone_country_code": "+1",
"description": "Its my first payment request",
"authentication_type": "no_three_ds",
"return_url": "https://duck.com",
"payment_method": "bank_transfer",
"payment_method_type": "cimb_va",
"payment_method_data": {
"bank_transfer": {
"cimb_va_bank_transfer": {}
},
"billing": {
"address": {
"first_name": "Akshaya"
},
"email"<EMAIL_ADDRESS>}
}
}'

Dynamic Field Rendering:

Based on the backend response, the WebSDK should dynamically generate the necessary input fields.
Each field should be properly validated (e.g., required, length, pattern matching) based on the backend specifications.
Support different types of fields such as:

Text fields (e.g., for name, card number)
Select dropdowns (e.g., for country or card type)
Checkbox fields (e.g., for terms and conditions)

User Inputs:

Once the user fills in the fields, the WebSDK should package the input data and send it back to the backend for payment processing.

Contribution Guidelines:

Fork the repository and create a new branch for your work.
Ensure the WebSDK follows best practices for API integration and field rendering.
Write clean, well-documented code with clear commit messages.
Add unit tests to ensure the dynamic field rendering works as expected.
Make sure to follow our coding standards and contribution guidelines.

Helpful Resources:

Link to WebSDK documentation: WebSDK Docs
Link to the API endpoint specifications: API Docs

Submission Process:

Ask the maintainers for assignment of the issue, you can request for assignment by commenting on the issue itself.
Once assigned, submit a pull request (PR).
Maintainers will review and provide feedback, if any.
Maintainers can unassign issues due to inactivity, read more here
For this issue, please submit a PR on @juspay/hyperswitch-web repo, and link it to the issue.

Refer here for Terms and conditions for the contest.
If you have any questions or need help getting started, feel free to ask in the comments!

Hello @gorakhnathy7 @sakksham7 ,
I would like to take this issue as hacktoberfest contributer, please assign it to me.

[x] I followed the README for guidelines on file structure and formatting.
[x] I ensured adherence to API integration and field rendering best practices.
[x] I will write clean, well-documented code with clear commit messages.
[x] I complying with coding standards and contribution guidelines.

Thank You

@gorakhnathy7 I would like to work on it, but there are multiple issues related to dynamic field rendering

does every payment provider needs its own dynamic field renderer?
in the given backend response there is no info about field type validation required other then required?

does every payment provider needs its own dynamic field renderer?

Yes we have to do that for every payment method.

in the given backend response there is no info about field type validation required other then required?

You can ask for the validations here in the comment section but majorly some basic checks will be required and will be enough.


Fix flaky ScalingThreadPoolTests.testScalingThreadPoolConfiguration test

Description
The threadpool size in the main source code and the expected size in test code were different causing flaky test. This PR fixes the expected size in the test code.
Related Issues
Resolves #7530
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/15299/
CommitID: 1600abf
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

Flay tests - https://build.ci.opensearch.org/job/gradle-check/15299/#showFailuresLink

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/15301/
CommitID: 4d26694
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

Unrelated continuous build failures for all PR builds happening and on main branch - #7540

@ashking94 sadly manual backport is needed to 2.x, could you please take it? thank you


Fix for Ink Response Shape Differs from Parent Material in ButtonStyleButton

The shape the Inkwell widget takes differs from its parent Material widget leading to ink splashes that do not accurately follow the border. This fixes the problem for my use case at least...

https://github.com/flutter/flutter/issues/91844

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Gold has detected about 3 new digest(s) on patchset 2.
View them at https://flutter-gold.skia.org/cl/github/91849

@MateusAmin Thanks for the PR. Without a test I'm afraid we can't land the change, but I'll update the issue and ask if anyone would like to pick up the fix. Cheers!


Update README.md to reflect changes to keybindings in 1.0

Did you check docs and existing issues?

[X] I have read all the vscode-neovim docs
[X] I have searched the existing issues of vscode-neovim
[X] I can reproduce the issue with an empty init.vim/init.lua and no other vscode extensions (when applicable)

Neovim version (nvim -v)
v0.9.0
Operating system/version
Windows 10
Describe the bug
Recently, I noticed that keyboard shortcuts I had disabled for nvim no longer work. For example Ctrl-F for search I re-checked the README.md and it still mentions the old way of defining shortcuts. After digging in to the changelogs, I found that there have been some breaking changes in 1.0, but that should be highlighted much better as it will catch many developers by surprise.
Steps To Reproduce
.
Expected Behavior
.

keyboard shortcuts I had disabled for nvim no longer work. For example Ctrl-F for search

how did you disable them? give exact example code.

I re-checked the README.md and it still mentions the old way of defining shortcuts. After digging in to the changelogs, I found that there have been some breaking changes in 1.0,

I assume you're referring to https://github.com/vscode-neovim/vscode-neovim#adding-keybindings ?
what breaking changes specifically?

when expression of keybindings is changed. But I think this impact is very small, just delete it yourself. And I'm sure you didn't read the readme carefully

From the README:

To disable an existing shortcut, for example C-a, add to your keybindings.json:
{
"command": "-vscode-neovim.send",
"key": "ctrl+a"
}

This does not work anymore. Your screenshot is from much further down in the README.

Yes the readme must be fixed, removing a default binding is now a settings option instead of keybinding entry.


fixing incorrect GA docs. supported versions were backwards. Data API…

… is newest. Reporting API is oldest.

Jira issue(s): PROOF-XXX

Changes:

fixing incorrect GA docs. supported versions were backwards. Data API is newest.


13 remove all in status

Codecov Report

Merging #15 (f9cf279) into main (4e05fc9) will increase coverage by 1.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
+ Coverage   94.57%   95.76%   +1.18%
==========================================
  Files           4        4
  Lines         295      307      +12
==========================================
+ Hits          279      294      +15
+ Misses         16       13       -3

Impacted Files
Coverage Δ

src/player_components.rs
94.66% <100.00%> (-0.14%)
:arrow_down:

src/table.rs
95.96% <100.00%> (+3.37%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


SQL Issue

The Script installer runs fine, but it cant connect to SQL.
No Connection to the Database.
SQLSTATE[HY000] [1045] Access denied for user<EMAIL_ADDRESS>(using password: YES) (Connection: mysql, SQL: select * from information_schema.tables where table_schema = hdtaag and table_name = migrations and table_type = 'BAS
E TABLE')

I'll test it later.

I didn't have the problem you mentioned and everything installed fine.


Made the tests run, but nut pass

You need to do npm install -g elm-test and then run elm-test from the project root


Use a diff styling when comparing previous versions

Wow, excellent work on this!
One thing I'd love to see added at some point is a diff style when comparing older versions of files. This would make it a bit easier to tell what specifically changed between the two periods of time. Something similar to mupchrch/split-diff?

That's a great idea.  I was looking for a good example of diff display.  Thanks

That's a great idea.  I was looking for a good example of diff display.  Thanks

Thanks again for the suggestion! I assimilated split-diff for the revision view display.

Holy cow this is so ridiculously cool! Thanks for working this in. Installing the update now :smile:


fix:usr: Show useful error when failing to open epic

... or iteration.

Published in v2.6.1

@andjosh I think the commit did not make it in for some reason. I am looking at 2.6.1 and I am not seeing the guards:
// club-story.js L293
                                    if (program.open)
                                        openURL(stories_1.default.storyURL(story));
                                    if (program.openEpic)
                                        openURL(stories_1.default.buildURL('epic', story.epic_id));
                                    if (program.openIteration)
                                        openURL(stories_1.default.buildURL('iteration', story.iteration_id));
                                    if (program.openProject)
                                        openURL(stories_1.default.buildURL('project', story.project_id));

Huh... I'll try tagging it again!

Republished in v2.6.2

I hate to tell you this but 2.6.2 has no build directory... :\

😅 okay - published in 2.6.3

Note to self: the /build is currently in the .gitignore but is not built during npm publishing of the module. So we currently need to run npm run build before npm publish....

I can confirm it worked and the code works as expected. :)
On Thu, 30 Jul 2020 at 13:56, Josh Beckman<EMAIL_ADDRESS>wrote:

Note to self: the /build is currently in the .gitignore but is not built
during npm publishing of the module. So we currently need to run npm run
build before npm publish....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/andjosh/clubhouse-cli/pull/199#issuecomment-666691822,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAP7KPI2NVCGI2OAG5PC7TR6HM6LANCNFSM4PMXNV4A
.


No longer works with "api" GitHub Enterprise subdomain

As of #1165, go-github no longer works if the api GitHub Enterprise subdomain is used, which broke our use of it. The /api/v3 suffix should only be added if the domain does not start with api.
$ curl -u <user>:<token> https://api.github.maxmind.com/api/v3/repos/maxmind/mm_website
{
  "message": "Not Found",
  "documentation_url": "https://developer.github.com/enterprise/2.20/v3"
}

$ curl -u <user>:<token> https://api.github.maxmind.com/repos/maxmind/mm_website
{
  <expected data>
}

$ curl -u <user>:<token> https://github.maxmind.com/api/v3/repos/maxmind/mm_website
{
  <expected data>
}

Hmmm... I'm not sure I understand your example...  Why are you using https://api.github.maxmind.com when the third example is actually working properly (which this repo supports) - https://github.maxmind.com ?

The api subdomain is a feature of GitHub Enterprise and worked fine with go-github through version v28. In #1165, go-github started appending /api/v3 to the end of Enterprise base URLs that didn't already have the suffix. This broke any existing users that were using the api subdomain.
We noticed this when upgrading from v28 to v32. The cause of the breakage was not immediately obvious. It was not listed as one of the breaking changes in the release notes and, from the PR, I don't think the intent was to break this particular use.
We did end up removing the api subdomain to work around this issue. However, the change in #1165 could be tweaked slightly to only add the suffix to base URLs that don't already have the suffix and are not using the api subdomain. This would prevent other users from running into this.

OK, I apologize for the breakage, as I was unaware of the api subdomain.
Yes, please feel free to write a PR to fix this for other users, and thank you for alerting us to this breakage.
Or if you would like other volunteers to address this issue, that is fine too, and I can open it up for others to contribute.  It's your call.

I can work on this one.

Hi @gmlewis ,
Is this issue still something we would like to fix? If yes, I can have a PR ready quickly.

Yes, please, @chughpiyush. Please make sure to fully unit test the various scenarios described above.


Don't install during build step

This option set the target build location. This caused the build to output the binary to the final install directory, breaking some build system assumptions.
Result seems to work just fine without it.

Hm.  So I think that step exists so that Tangerine can run after building it.  Without that step, nothing appears in linux/build/Release/bin/, which is where I'm used to running it from while iterating?  As Tangerine is written right now, it needs to be able to find the shaders directory relative to the executable, and probably also the models folder for the init model.  I'm unsure what the best way to handle this is, but I'm open to suggestions.


Add section to README about project naming history

This is an attempt to address the concerns raised in https://github.com/thoughtbot/factory_girl/issues/921 and https://github.com/thoughtbot/factory_girl/issues/849 without a full project rename.

Duplicate of https://github.com/thoughtbot/factory_girl/pull/942?
How about a link from the README to a NAME.md?

I saw #942 immediately after creating this, and send my regards to that PR.
Is your suggestion to move out the whole explanation to a new file so that we reduce the text being added to README, but make sure we link (earlyish in the file) to that naming explanation from the README?

Is your suggestion to move out the whole explanation to a new file so that we reduce the text being added to README, but make sure we link (earlyish in the file) to that naming explanation from the README?

Yup

Made a new file, reduced some verbiage, squashed, probably acceptable for a first pass thing which can be expanded later.

Thanks @mjankowski; merged as 903d4325feb9c373863601cdab3fca4b6118fca7


Fix multiple bugs with the disabled option

I contributed the disabled option some time ago (1000hz/bootstrap-validator/pull/46).
I discovered that now there are two problems with this option:

focussing the erroneous input element does not work.
the form is submitted, even though it contains validation errors.

Both errors stem from the same source, since in both functions (focusError and onSubmit) it is determined if the form contains errors by calling this.$btn.hasClass('disabled'), where $btn is the submit button. However, when the disabled option is set, the submit button is never disabled.
This pull request fixes these two bugs by using other ways to check if the form contains validation errors.

@1000hz Any comments on that?

Looks good. Just make sure your if () statements have a space before the opening parenthesis and I'll merge. Thanks!

Done.

Thanks for this. I'm surprised nobody reported the form submit issue sooner. 😬

No problem. Thanks for merging so quickly.
When do you plan to do the next release? That would make it easier for me to deploy the fix.

I'll try to do it some time tomorrow if I don't forget. It's late right now and I'm tired. If I haven't released it in a couple days, feel free to ping me in this thread.


[Windows] Always use ANGLE in ARM builds.

There's no native OpenGL drivers on Windows for ARM (only software compatibility layer), so ANGLE is a more viable option for compatibility renderer.
Fixes https://github.com/godotengine/godot/issues/85961

nice work! as an WOA user, i have already made a minimal project on a Windows on ARM laptop, then test in using opengl3 ANGLE and opengl API 4.1, and then... here are the results.
PS: i use three GPUParticle2D controllers to run. Running in vsync=off, screen_res=(2560,1600,fullscreen), max_fps here sets 0 (the project what i upload sets 60.)
opengl API 4.1: (about 72±6 fps, max_usage=66%, having a opengl->d3d12 translation layer)

opengl3 ANGLE: (about 379±13 fps, sometimes more than 400 fps, max_usage=100%)

72 fps vs. 379 fps, oh my... that's an awesome enhancement!!
PS: i advise to turn off the vsync (might look unsmoothly when enabled) and set max fps as screen refresh rate (e.g. 59, 60, 144, etc.) when choose ANGLE, this test is just for fun.
of course, the more single CPU frequency, the "more" fps (vsync=off), so cpu within low base/turbo frequency is not friendly. then actually watching on the performance of GPU is also worthy. Mine is 3.0 GHz and finally goes about 2.8 GHz because of 100% GPU usage. (but that's still awesome!! thanks to this rendering method, WOA device can really create an amazing game in different genres as well.)
and of course i will only recommend on gpu graphic which needs translation layer, that's because...
eeeh... we can view these two pics (look at fps). here's running on a x86_64 laptop within ATI Radeon HD 7700M.
this is opengl3 ANGLE, the single CPU frequency goes maximum, the GPU-3D usage is less than 50%. con seems more than pro.

and this, is opengl API 4.1. (tears of joy)

so if have a gpu which has enough performance and needn't translation layer (more important), needn't to use ANGLE. :D

Thanks!

Cherry-picked for 4.2.2.


BAEL-16656

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please


Removing shortcuts from a new tab

It seems it is impossible to hide those annoying shortcuts from a new tab in Ungoogled Chromium (which is not an issue in Google Chrome). I don't like it when my browser monitors most recent or frequently visited pages. Why would you remove the option to disable it?
I know the issue regarding this topic has already been created (#980) but it hasn't been solved in another way than it was suggested to use 3rd party extensions which as i think is not a real solution.
My browser version is 80.0.3987.149.

Why would you remove the option to disable it?

I don't recall removing any options like this.

I don't like it when my browser monitors most recent or frequently visited pages.

You could always browse in incognito mode.

it hasn't been solved in another way than it was suggested to use 3rd party extensions which as i think is not a real solution

I don't know what criteria you use to define a "real solution", but there's literally no JavaScript needed to change a new tab page. It's just a manifest key.
If that's not simple enough for you, you can also set an Enterprise Policy key NewTabPageLocation.
I think adding more way to configure options that are already natively supported is not a good use of our development time.

The solution that worked for me is opening chrome://flags and setting #custom-ntp option to Enabled with value about:blank.

chrome://flags/#disable-top-sites flag enabled should do its task [just hoping my recall from earlier experiments is valid].

chrome://flags/#custom-ntp option to enabled with value about:blank

does also work [if one does not want to use other new tab page functions].


Badges for completed repos?

What does the community think about README badges for completed repos. Say:

The color is #783578 which comes from the main README logo:

I think it looks awesome 👍

Yep, looks great.

Added badges to many completed repos.


castle tower

does this include the interior spaces for the dragon raid?

you are looking from above at the tower
On Mon, Jun 22, 2015 at 10:59 AM, GallantShow<EMAIL_ADDRESS>wrote:

does this include the interior spaces for the dragon raid?
—
Reply to this email directly or view it on GitHub
https://github.com/starlightgraphics/STORY/issues/40#issuecomment-114141419
.

--
John roper
President and CEO of Starlight Graphics Studio
Boston, MA USA
http://starlightgraphics.tuxfamily.org

scene 4, shot 3


Unclear documentation for Authorization Policy/Authentication

I've attempted to follow this page: https://docs.microsoft.com/en-us/aspnet/core/security/authorization/policies#using-a-func-to-fulfill-a-policy
I performed the following steps:

I created a new Web API project dotnet new webapi
On the Get method of the values controller I added an attribute: [Authorize(Policy = "AlwaysDeny")]
In Startup.cs I added the following to ConfigureServices:

      services.AddAuthorization(options =>
      {
        options.AddPolicy("AlwaysDeny", policy =>
        {
          policy.RequireAssertion(context => false);
        });
      });

When I get http://localhost:5000/api/values I would have expected a 401 response
I instead get a 500 the following exception:  System.InvalidOperationException: No authenticationScheme was specified, and there was no DefaultChallengeScheme found.
The documentation does not indicate any requirement for authentication. I do not want to use JWT token or Cookies.

You still need to add authentication so the authorization system has something to check against. It has no way of knowing  that something compiles to always reject.

Authentication and authorization should be somewhat separate concerns only ASP.Net chooses to conflate them. Given this design choice, it should be made clearer in the documentation that Authentication is a pre-requisite for authorization in the documentation.


Consider adding blending to Cartesian planning

Instead of having to stop exactly at the intermediate points, there are common approaches to blending Cartesian paths.
This is one resource I like: https://www.diag.uniroma1.it/~deluca/rob1_en/14_TrajectoryPlanningCartesian.pdf
Whether just blending on position is enough, or we also need to find ways to do the same with rotation e.g., here remains TBD.

Hi @sea-bass,
I learned the math in slides 6-13 from the resource you have shared above.
I also understood the implementation of examples/cartesian_path.py and src/pyroboplan/planning/cartesian_planner.py.
I have a few questions before attempting to implement blending:

When using trapezoidal scaling, how to evaluate v1 and v2 which are constant speeds on the two linear paths to be concatenated?
My guess is that I will have to evaluate the velocity using the trapezoidal trajectory at the points where blending starts and ends which is defined by d1 and d2 from via point.
Three approaches are mentioned for blending in the slides above: assign transition time, assign distance from via point, and impose acceleration. Can I use any one of these techniques for the implementation?

I think there should be a maximum velocity magnitude v assigned throughout. So at the start of the trajectory, the robot accelerates from 0 to v, at the end it decelerates, and at intermediate waypoints you want v1 = v2 = v, magnitude-wise, but their directions may be different.
As for how to parameterize, I think the best bet is to set a distance from the via point and then let the timing be dictated by max accelerations. This can be referred to as a "blending radius".
But before all this... @eholum you said you were potentially looking at this same task?

Not yet! I had started writing a Cartesian interpolator but I won't get back to it for at least 2 weeks. Happy to help review anything, though, @muhidabid.

I thought nobody was looking into this issue. If you have started working on it @eholum, please continue!
I was planning on working on it at a slow speed anyway since I have my university enrollment in a few days and I am preparing to travel.
I will look into other issues no problem


Implemented leex tokenizer

Supports everything from examples in Readme with one exception being that all string literals have to be quoted with single quotes.

🍻


Dashboard app QoL updates

QoL redesign work is beginning for the Dashboard UI pages to ensure this UI looks great until we circle back to it.
This encompasses several UX/UI updates, including:

Improved tooltip descriptions
Improved/responsive components + app pages
Spec matching adjustments

View/comment on this work in the Dashboards UI Redesign canvas

#514 opened for statistics section improvements.

#515 also submitted tonight.

Content construction for tooltips is occurring in Dashboard Visualization Descriptions. I am assigning description writers now for this text content. Thanks!

Status update:

Content work continues in Dashboard Visualization Descriptions

@andreasbueckle will take a pass before EOB Friday, then pass it back to @katyb
@katyb will resolve all comments and provide final text for most descriptions, then pass to me
I will edit descriptions in yaml in x-atlas-consortia/hra-dashboard-data

Note: Today I made quick edits in yaml as they came in.
Ready for dev & high priority:

Vega-Lite styles: #515

Status update

Awaiting resolved comments & completed content from @katyb in Dashboard Visualization Descriptions
Product team has been crushing: Most cosmetic updates have been pushed to develop. Looking forward to seeing them in main!

Status update
@katyb needs @bherr2 to take a pass at Dashboard Visualization Descriptions then send it back to her. I can edit the markdown when she approves of Bruce's edits.

Repurposing this issue to a feature request issue, as there has been no movement on this for a while.


Handle popup closing for LVL layers and Leaflet popups

This seems to do the trick!

I pulled this into a bug-fix branch until I figure out the new issue with funky icons. I don't want to tag a new version until I figure out that issue.
It's definitely time to tie LVL to a specific Leaflet version. I can't keep up with changes in master.


start counter when component show on screen

Counter is starting when component is loading. so if i am using it below in website then i am not able to see that counter animation so is there any way i can start counter when that counter is on screen?


Blending models

Hi,
How do we do blending of two models trained on different datasets?There are starting points for Tensorflow implementations. Can someone suggest the code in PyTorch?

Hi, I have reimplemented the blending function, it may help.
See https://gist.github.com/onion-liu/7969bce0d4aeb60455f7d35bb0d8fd4a


[nrf noup] [wip] Tweak needed for shared Kconfig doc

See https://github.com/NordicPlayground/fw-nrfconnect-nrf/pull/1258. The genrest.py is the same as in https://github.com/zephyrproject-rtos/zephyr/pull/19519.

Found the following issues, please fix and resubmit:
Gitlint issues
1: T5 Title contains the word 'wip' (case-insensitive): "[wip] Tweak needed for shared Kconfig doc"
1: UC3 Title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "[wip] Tweak needed for shared Kconfig doc"
Identity/Emails issues
16917d366f8188c87c6dcbd558be38f7806bf8ab: author email (Ulf Magnusson<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.

shortlog should start with [nrf noup] or [nrf toup] -- I know it's WIP, but it needs a sauce tag before merging into master when it's ready.

shortlog should start with [nrf noup] or [nrf toup] -- I know it's WIP, but it needs a sauce tag before merging into master when it's ready.

Added [nrf noup]. Guessing that means it shouldn't be merged into the main Zephyr repo.

Added [nrf noup]. Guessing that means it shouldn't be merged into the main Zephyr repo.

The meaning of each tag is covered here:
https://developer.nordicsemi.com/nRF_Connect_SDK/doc/latest/nrf/ug_dev_model.html#oss-repositories-downstream-project-history
If you're using noup, it means "this can't go upstream".
Though if we can find an upstream-friendly version it would be toup. That would be nice as downstream patches are tech debt.

Some checks failed. Please fix and resubmit.
Gitlint issues
1: T5 Title contains the word 'wip' (case-insensitive): "[nrf noup] [wip] Tweak needed for shared Kconfig doc"
1: UC3 Title does not follow [subsystem]: [subject] (and should not start with literal subsys:): "[nrf noup] [wip] Tweak needed for shared Kconfig doc"
pylint (internal test error)
pylint not found. Check that the directory it's in is listed in the PATH environment variable.

Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.

Zephyr history was rewritten in preparation for the next release. Please rebase this PR.

Zephyr history was rewritten in preparation for the next release. Please rebase this PR.

Rebased now.

rebasing master should fix the QEMU errors on the CI.

NordicBuilder:
rebasing master should fix the QEMU errors on the CI.

Wow, that's a smart bot ;)

@ulfalizer if you want this to have any effect, you need to send a PR updating west.yml like https://github.com/NordicPlayground/fw-nrfconnect-nrf/pull/1379; is there one?

@ulfalizer : This has been reverted and must be re-uploaded.


[5.0.0-rc005] Paket fails to update project file when framework restriction is used

Attached repro contains Hello World project targeting .NET Framework 4.6.1. Project references exactly one package Microsoft.CodeAnalysis.
Paket.dependencies file contains nuget Microsoft.CodeAnalysis framework: net46
"paket install" command does not change project file whatsoever, which is very much unexpected - it should. If I remove framework: net46 filter from paket.dependencies, then project file IS updated, but because the package targets many different profiles ALL of them get injected to the project file and it becomes huge. Please investigate!
PaketRepro.zip

framework: net461
content: none
redirects: off

source https://www.nuget.org/api/v2/

nuget Microsoft.CodeAnalysis framework: net46

framework:  on a package will further restrict the package (in addition to the global restrictions of the group). But (==net461) && (==net46) is the empty set. This means the package will never be installed.
This is actually explained in the documentation with a warning, but as the last published version is a stable one (paket 4) the docs do not contain this warning at the moment.

So this is a change of behavior from Paket 4, because for Paket 4 it does work, right?

Paket 4 doesn't have a well defined behavior for this at all.

Ok, makes sense. Closing.


Inconsistent class headers: qiskit.transpiler.passes

Environment

Qiskit version: 1.2.0
Python version: --
Operating system: --

What is happening?
Inconsistent class headers generated for qiskit.transpiler.passes. Some classes have explicit definitions in the headers and some do not.
For eg: BasicSwap

Whereas AQCSynthesisPlugin

It would be helpful if we generate the class headers in the docs as per the intended structure for clarity and consistency, preferably as explicit as possible.
How can we reproduce the issue?
Check out the class headers generated for each of the individual passes here: https://docs.quantum.ibm.com/api/qiskit/transpiler_passes
TranspilationPass and AnalysisPass based classes seem to have less explicit headers
What should happen?
It would be helpful if we generate the class headers in the docs as per the intended structure for clarity and consistency, preferably as explicit as possible.
Any suggestions?
No response

I am not sure I understand the issue.

AQCSynthesisPlugin is not a transpiler pass, but a synthesis plugin available for synthesizing UnitaryGate objects.


CD - prunning does not work issue

For some reason sudo docker stop $(sudo docker ps -a -q) failed to stop all containers. Command sudo docker ps -a -q yielded 0 containers. In this fix I used another command sudo docker-compose down that does actually the same.

Codecov Report

Merging #12 (53b7bc2) into dev (34984be) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #12   +/-   ##
=======================================
  Coverage   95.12%   95.12%
=======================================
  Files           4        4
  Lines          41       41
=======================================
  Hits           39       39
  Misses          2        2

Continue to review full report at Codecov.

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


jenkins_plugin resource is updated on every converge

I think this is ultimately a Jenkins problem, but I'm putting it in here while I research it. I'm guessing others using this cookbook will encounter it.
On every chef run (jenkins cookbook v2.3.1, chef 12.4.1, jenkins v1.609.3 LTS), I'm seeing the same plugin install actions firing (not unique to this plugin, happens with 10-15 of them):
  * jenkins_plugin[cvs] action install
  Recipe: <Dynamically Defined Resource>
    * remote_file[/var/chef/cache/cvs-2.12.plugin] action create (up to date)
    * remote_file[/var/chef/cache/jenkins-cli.jar] action create (up to date)
    - Upgrade jenkins_plugin[cvs] from 2.11 to 2.12

I've tried restarting jenkins in between runs, but it seems like Jenkins is actually never unpacking the new plugin:
# sha256sum /var/chef/cache/cvs-2.12.plugin
6e6dfd35e8501bf5f84a9d43d210db61165ce51a606327fc81f2efc5208478ba  /var/chef/cache/cvs-2.12.plugin

# unzip /var/chef/cache/cvs-2.12.plugin
# grep -i plugin-version META-INF/MANIFEST.MF
Plugin-Version: 2.12

# sha256sum /var/lib/jenkins/plugins/cvs.jpi
6e6dfd35e8501bf5f84a9d43d210db61165ce51a606327fc81f2efc5208478ba  cvs.jpi

# grep -i plugin-version /var/lib/jenkins/plugins/cvs/META-INF/MANIFEST.MF
Plugin-Version: 2.11

I'm thinking this a jenkins bug since I can reproduce it without Chef, just running the CLI commands by hand. I'm just wondering if, when a workaround is found, we should add to the cookbook too.

I ended up finding JENKINS-12514, which describes the scenario where an .hpi of a plugin was shipped with Jenkins itself, and that version constantly persists, even when .jpi upgraded plugins are installed.
I tested if this was true for my example using these commands:
# mv /var/lib/jenkins/war/WEB-INF/plugins/cvs.hpi /tmp/nonchef-cvs.hpi
# service jenkins restart
ok: run: jenkins: (pid 14918) 1s
# grep -i plugin-version /var/lib/jenkins/plugins/cvs/META-INF/MANIFEST.MF
Plugin-Version: 2.12

As you can see, now the plugin has been actually upgraded. Should we add a workaround that, if an .hpi for the plugin exists in the war file, we remove it, so that upgrades to plugins actually work? Or should this be a workaround in individual implementation cookbooks?
I'm leaning towards the latter, but wanted to document here.

I'm experiencing this problem as well. Every Chef runs says it's updating the plugin (that comes with Jenkins), but it never gets updated.
Excerpt from the Chef run (I'll look at the cvs plugin in this instance):
...
==> default:   Recipe: <Dynamically Defined Resource>
==> default:     * remote_file[/var/chef/cache/cvs-2.12.plugin] action create
==> default:  (up to date)
==> default:     - Upgrade jenkins_plugin[cvs] from 2.11 to 2.12
==> default: Recipe: dbl_orchestration::chef_ci
==> default:   * jenkins_plugin[javadoc] action install
==> default:
...
==> default:   * service[jenkins] action restart
==> default: [2015-10-29T18:40:32+00:00] INFO: service[jenkins] restarted
==> default:
==> default:     - restart service service[jenkins]
==> default:
==> default: [2015-10-29T18:40:32+00:00] INFO: Chef Run complete in 84.065463877 seconds

But then examining the files, it looks like the recipe isn't even updating the plugin files that Jenkins uses:
# unzip -p /var/chef/cache/cvs-2.12.plugin META-INF/MANIFEST.MF | grep Plugin-Version
Plugin-Version: 2.12

# unzip -p /var/lib/jenkins/plugins/cvs.jpi META-INF/MANIFEST.MF | grep Plugin-Version
Plugin-Version: 2.11

# grep Plugin-Version /var/lib/jenkins/plugins/cvs/META-INF/MANIFEST.MF
Plugin-Version: 2.11

According to JENKINS-12514, at least the .jpi file should be updated. I have no .hpi files in /var/lib/jenkins/plugins.
BTW, I'm using Jenkins LTS 1.625.1.
Any ideas?

Jenkins resets all plugins that are bundled to their bundled version on startup, unless a PLUGIN.hpi.pinned marker file exists. Is that what you're seeing here?

There was no .pinned marker file when I investigated this. Is this something Jenkins should create?

Yes, when you update the plugin through any of the means provided by Jenkins. When you mess with the file system, you are expected to know what you are doing.
https://wiki.jenkins-ci.org/display/JENKINS/Pinned+Plugins

@daniel-beck For the record, I believe we are using means provided by Jenkins. We're not messing with the filesystem -- we're calling install-plugin using the Jenkins CLI and passing it a path -- https://github.com/chef-cookbooks/jenkins/blob/master/libraries/plugin.rb#L316. Perhaps that's something that the Jenkins CLI doesn't handle correctly?

@martinb3 Thanks for the clarification. In this case, this is indeed a problem. I'll take a look.

@martinb3 This is a bug in the install-plugin CLI command that is being tracked as JENKINS-29726.

Thanks for the follow up @daniel-beck -- cheers! :)

Closing this one out as it looks like it was resolved upstream for 2.x


Generating molecules

Hello, thank you for your model, I am doing research work and I would like to know how to generate only valid SMILES strings using your model, could you provide this code, if possible, I will be very grateful for your answer!

I meant inference mode, forgot to say


[OpenCL] CMake wrapper needs correction

Issue: CMake's findOpenCL does not correctly take care fof release/debug libraries. As such the wrapper already installed by vcpkg needs additional logic to take care that the target consists of both release and debug information. The same need to be applied to the _LIBRARIES variable.
Is your feature request related to a problem? Please describe.
Incomplete target information in INTERFACE_LINK_LIBRARIES in the targets generated by ITK
Proposed solution
Add the Information to the _LIBRARIES variable via selectLibraryconfigurations and also add the info to the OpenCL imported target
Describe alternatives you've considered
there is not alternativ to make it correctly work.

Also needs to link OneCoreUAP.lib on Windows 10 in static builds

Also needs to link OneCoreUAP.lib on Windows 10 in static builds

Fixed by https://github.com/microsoft/vcpkg/pull/31637.

CMake's findOpenCL does not correctly take care of release/debug libraries. As such the wrapper already installed by vcpkg needs additional logic to take care that the target consists of both release and debug information. The same need to be applied to the _LIBRARIES variable.

Mitigation: #40042 makes the port install upstream's CMake config.


Wrong value generated for is_plain on certain cases

The generated method is_plain is not taking into account the alignment padding of the compiler.
See eProsima/Fast-DDS#2777 for an example.
The solution would be to generate something similar to the following:
    eProsima_user_DllExport inline bool is_plain() const override
    {
        return offsetof(<type_name>, <last_field>) + sizeof(<last_field>) == <type_name>::getMaxCdrSerializedSize();
    }

@MiguelCompany , can you explain how to use the proposed is_plain() function to detect a misaligned message? I would like to check:

if I require padding in my message
if my padding was done correctly

eProsima/Fast-DDS-Gen#169 should fix this issue. Closing.


tevent: 0.9.35 -> 0.9.36

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 0.9.36 with grep in /nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36
found 0.9.36 in filename of file in /nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36

cc @wkennington for review

Success on x86_64-darwin (full log)
Attempted: tevent
Partial log (click to expand)
* symlink /nix/store/p988c10c2a6bvc8apqymmdrdlx4wjciv-tevent-0.9.36/lib/libtevent.dylib (-> libtevent.0.9.36.dylib)
* installing bin/default/-tevent.inst.so as /nix/store/p988c10c2a6bvc8apqymmdrdlx4wjciv-tevent-0.9.36/lib/python2.7/site-packages/_tevent.so
* installing tevent.h as /nix/store/p988c10c2a6bvc8apqymmdrdlx4wjciv-tevent-0.9.36/include/tevent.h
Waf: Leaving directory `/private/tmp/nix-build-tevent-0.9.36.drv-0/tevent-0.9.36/bin'
'install' finished successfully (0.146s)
post-installation fixup
strip is /nix/store/4sdh09gmvl15cy0zb6i7mbvxh5syz206-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/p988c10c2a6bvc8apqymmdrdlx4wjciv-tevent-0.9.36/lib
patching script interpreter paths in /nix/store/p988c10c2a6bvc8apqymmdrdlx4wjciv-tevent-0.9.36
/nix/store/p988c10c2a6bvc8apqymmdrdlx4wjciv-tevent-0.9.36

Success on x86_64-linux (full log)
Attempted: tevent
Partial log (click to expand)
'install' finished successfully (0.428s)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36
shrinking /nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36/lib/python2.7/site-packages/_tevent.so
shrinking /nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36/lib/libtevent.so.0.9.36
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36/lib
patching script interpreter paths in /nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36
checking for references to /build in /nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36...
/nix/store/6bqs8b2x2q1f8lxnkh11n1272agnaj5y-tevent-0.9.36

Success on aarch64-linux (full log)
Attempted: tevent
Partial log (click to expand)
'install' finished successfully (0.524s)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/sgnq1kar6mx7jkrxvyzvihyy1c3cpc29-tevent-0.9.36
shrinking /nix/store/sgnq1kar6mx7jkrxvyzvihyy1c3cpc29-tevent-0.9.36/lib/libtevent.so.0.9.36
shrinking /nix/store/sgnq1kar6mx7jkrxvyzvihyy1c3cpc29-tevent-0.9.36/lib/python2.7/site-packages/_tevent.so
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/sgnq1kar6mx7jkrxvyzvihyy1c3cpc29-tevent-0.9.36/lib
patching script interpreter paths in /nix/store/sgnq1kar6mx7jkrxvyzvihyy1c3cpc29-tevent-0.9.36
checking for references to /build in /nix/store/sgnq1kar6mx7jkrxvyzvihyy1c3cpc29-tevent-0.9.36...
/nix/store/sgnq1kar6mx7jkrxvyzvihyy1c3cpc29-tevent-0.9.36


Cannot pass inline JSON parameters to az deployment group create

Describe the bug
Passing parameters as inline JSON to az deployment group create following the examples in the documentation does not seem to work in bash (running on Windows in WSL - similar errors are also observed in cmd, but for clarity have not been included in this issue).
To Reproduce
Using the following template file:
{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "managedIdentityName": {
            "type": "string"
        }
    },
    "resources": [
        {
            "apiVersion": "2018-11-30",
            "name": "[parameters('managedIdentityName')]",
            "type": "Microsoft.ManagedIdentity/userAssignedIdentities",
            "location": "[resourceGroup().location]",
            "properties": {}
        }
    ]
}

The documentation examples suggest that parameters can be passed inline as follows:
az deployment group create --resource-group testrg --template-file "template.json" --parameters '{ \"managedIdentityName\": \"test-identity\" }'

However this gives the following error:
Unable to parse parameter: { \"managedIdentityName\": \"test-identity\" }

Issue #15808 suggests that this should instead be
az deployment group create --resource-group testrg --template-file "template.json" --parameters '{ "managedIdentityName": "test-identity" }'

However this gives the following error:
{"error":{"code":"InvalidRequestContent","message":"The request content was invalid and could not be deserialized: 'Error converting value \"test-identity\" to type 'Azure.Deployments.Templates.Definitions.DeploymentParameterDefinition'. Path 'properties.parameters.managedIdentityName', line 1, position 69.'."}}

Expected behavior
It should be possible to pass JSON deployment parameters inline.
Environment summary
Ubuntu 18.04.5 LTS
Linux-4.4.0-19041-Microsoft-x86_64-with-debian-buster-sid
GNU bash, version 4.4.20(1)-release (x86_64-pc-linux-gnu)
Python (Linux) 3.6.10 (default, Apr 19 2021, 07:29:42)
[GCC 7.5.0]
azure-cli                         2.22.1
core                              2.22.1
telemetry                          1.0.6
Note that the Azure CLI tested is installed in WSL using apt.
Additional context
None.

ARM

@yonzhan @jsntcy Was this fixed?

accidentally close it, reopen now

This format does seem to work:
az deployment group create --resource-group testrg --template-file "template.json" --parameters '{"managedIdentityName":{"value":"test-identity"}}'

This format does seem to work:
az deployment group create --resource-group testrg --template-file "template.json" --parameters '{"managedIdentityName":{"value":"test-identity"}}'

It doesn't for me.


Add target option to tmhUtilities->entify to specify link target

In my code I would like processed links to open in a new window.  I added a $linkTarget member variable to the tmhUtilties class which, when specified, inserts target="$this->linkTarget" into links generated by the entify method.

This is an interesting idea. For backwards compatibility i've made a wrapper function for the original entify function, and made a new, scalable function for adding more parameters in the future. It'll be in the next version of tmhOAuth that's released.

Just pushed 0.62 with this in it live.

Excellent, thanks for implementing!
On 3/1/2012 3:22 PM, Matt Harris wrote:

Just pushed 0.62 with this in it live.

Reply to this email directly or view it on GitHub:
https://github.com/themattharris/tmhOAuth/pull/28#issuecomment-4269426

--
David Roda
4Walls LLC
561-249-9348


Add a basic github actions script

This will run rustfmt, clippy, and the tests for stable & nightly.
I believe you can turn these on but still allow failures. I would suggest having at least the rustfmt job be a hard failure; it's nice to have all patches correctly formatted.
I've turned of 'deny warnings' for clippy, so it should only complain about particularly egregious things?
Because this isn't my repo, I'm not sure if it will just run this action now, when I push? Hopefully you can just press a button to run it...

Let’s just do it now, see what it runs and tweak later if needed. Thanks for this; I generally have my editor run rustfmt on save and highlight clippy suggestions, but it’s good to enforce this on the codebase.


PhantomJS hangs on invisible file input

Which version of PhantomJS are you using?
2.1.1

What steps will reproduce the problem?

Have a webpage, with JS driven tabs. (Sonata Admin in my case)

Have a file upload field on the normally hidden tab.

Try to use the WebDriver interface to test above form, namely to upload some file using that field.

Which operating system are you using?
Ubuntu Trusty

Did you use binary PhantomJS or did you compile it from source?
I did use npm install phantomjs-prebuilt

Please provide any additional information below.
This seems to happen when using Behat with the MinkExtension, using it's Selenium2Driver to directly connect to phantomjs --webdriver=4444. The reason I think this is an actual issue with PhantomJS itself is, that I don't seem to see any response from PhantomJS when this happens.

Runnig `phantomjs --webdriver=4444 --debug=true, with the field hidden:
URI /wd/hub/session/36ed4620-952f-11e6-ab32-6b46ae132603/element/:wdc:1476794079369/value
2016-10-18T13:35:27 [DEBUG] HTTP Request - Method POST
2016-10-18T13:35:27 [DEBUG] HTTP Request - HTTP Version 1.1
2016-10-18T13:35:27 [DEBUG] HTTP Request - Query String
2016-10-18T13:35:27 [DEBUG] HTTP Request - Receiving Header "Host" = "localhost:4444"
2016-10-18T13:35:27 [DEBUG] HTTP Request - Receiving Header "Content-Type" = "application/json;charset=UTF-8"
2016-10-18T13:35:27 [DEBUG] HTTP Request - Receiving Header "Accept" = "application/json;charset=UTF-8"
2016-10-18T13:35:27 [DEBUG] HTTP Request - Receiving Header "Content-Length" = "100"
2016-10-18T13:35:27 [DEBUG] HTTP Request - Method POST/PUT
2016-10-18T13:35:27 [DEBUG] HTTP Request - Content Body: {"value":["\/full\/path\/to\/file.png"]}
2016-10-18T13:35:27 [DEBUG] WebPage - evaluateJavaScript "..."
2016-10-18T13:35:27 [DEBUG] WebPage - evaluateJavaScript result QVariant(QVariantMap, QMap(("status", QVariant(double, 0))("value", QVariant(QString, "INPUT"))))
2016-10-18T13:35:27 [DEBUG] WebPage - evaluateJavaScript "..."
2016-10-18T13:35:27 [DEBUG] WebPage - evaluateJavaScript result QVariant(QString, "{\"status\":0,\"value\":\"file\"}")
2016-10-18T13:35:27 [DEBUG] WebPage - evaluateJavaScript "..."
2016-10-18T13:35:27 [DEBUG] WebPage - evaluateJavaScript result QVariant(QString, "{\"status\":11,\"value\":{\"message\":\"Element is not currently visible and may not be manipulated\"}}")

But when the thing is not hidden, I actually see this, instead of the error message:
...
2016-10-18T13:34:56 [DEBUG] WebPage - evaluateJavaScript result QVariant(QString, "{\"status\":0,\"value\":\"file_input_id\"}")
2016-10-18T13:34:56 [DEBUG] HTTP Response - Status Code 200 OK
2016-10-18T13:34:56 [DEBUG] HTTP Response - Sending Header "Cache" = "no-cache"
2016-10-18T13:34:56 [DEBUG] HTTP Response - Sending Header "Content-Length" = "103"
2016-10-18T13:34:56 [DEBUG] HTTP Response - Sending Header "Content-Type" = "application/json;charset=UTF-8"

And it seems to work.
Which makes me believe, PhantomJS fails to actually generate a HTTP response, and send the error message to the client.

@fancsali does the file input have the html 5 multiple attribute?

Hey @jesg, thanks for pointing that out.
To be honest, I totally forgot, to mention that. It does not, that's the curious part. I've read the thread/ticket about that before submitting this, and it was part of the reason I ended up doing so.

@fancsali selenium send_keys/click actions are unable to interact with elements that are not visible.

Yes, I am aware of that. Although normally, I get an error response from phantomJS, which is then picket up by PHP and behat.
In this case, however, phantomJS just hangs. I can see the error message in the phantomJS debug log, but it never generates an actual response.

ok.  do you have a minimal test i can use to reproduce the issue?


Update HTML docs

I forgot to run make doc with the last PR after changing the Testo module interface slightly.
PR checklist:

[x] Purpose of the code is evident to future readers
[x] Tests are included or a PR comment includes a reproducible test plan
[x] Documentation is up-to-date
[x] A changelog entry was added to CHANGES.md for any user-facing change

Check out CONTRIBUTING.md for more details.

Can we see the doc somewhere? (I mean the rendered html).

yes.
😀


Invoke other project?

I followed each step without any problem.  After the last step 11, the actions on google simulator console open up with Right project name.  Input the "Talk to my test app", the response was the last project "Silly Name Maker".
Any idea why the project name is right but invoke other project?   Any setup I mess?

I set up app name in the Actions on Google. the app can be invoke now.


Fix bug: panicked while add single song from search to empty playlist

Panic output:
panicked at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/alloc/src/vec/mod.rs:1562:21:
insertion index (is 1) should be <= len (is 0)

Thank you a lot for the contribution. Always appreciate help on the project.


issue-1725: Make it possible to specify the license types of NPM test artifacts

Pull Request Description
This pull request closes #1725
Acceptance Test

[ ] Building the code with mvn clean install -Dintegration.tests still works.
[ ] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[ ] The tests in the strongbox-web-integration-tests still run properly.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[X] No

Does this pull request require other pull requests to be merged first?

[ ] Yes, please see #...
[X] No

Does this require an update of the documentation?

[ ] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[X] No

I created a new pull request

Closing in favor of #1757 :)


Update SQLite 3.47+

Update SQLite to 3.47 for more detail look at https://www.sqlite.org/changes.html
Including following cherrypick fix related to blob_io performance
Avoid loading the entire record into memory for an sqlite3_preupdate_old() call that retrieves an IPK value.

@mergifyio backport release/4.10.x

I've confirmed that the cherrypick fix to solve poor performance when writing small chunks of a large blob with sessions enabled works.

/azp run imodel-native


Fix buffer overflow (CWE-120)

Relevant issue number(s) if applicable
H1 #724239
License
I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Good find, went with a different fix, btw lastGroup[buflen] = 0 would overflow


add readme instructions for debian installation of cli [fixes #11]

the develop branch already exists. you can continue tracking this PR here -> https://github.com/pivotal-cf/pivnet-cli/pull/15


Follow latest API changes

This PR builds on top of the auto-generated #89 , and attempts to fix what is broken by the API changes.
Since I haven't investigated everything deeply, some review would be very welcome.

Updated to cover API usage in the recently merged offline indicator PR.

I implemented all the feedback, rebased to latest master, fixed a few more things, and added a simplification (around the Token type).


⚠ Samsung has degraded performance

In 5abc62e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 503
Response time: 11943 ms

Resolved: Samsung performance has improved in cf82808.


Feature.modifying exec permissions 735

@richbrowne
What issues does this address?
Fixes #735
What's this change do?

This change will add executable permissions on these critical files

./etc/init.d/f5-openstack-agent.ini
./etc/init.d/f5-oslbaasv2-agent
./f5_openstack_agent/lbaasv2/drivers/bigip/agent.py

Permissions are now 755 - rwxr-xr-x

Where should the reviewer start?
Just simply look at the permissions items
Any background context?
This is being checked into Liberty

Hi @richbrowne, I've got the .ini out of there... sorry for the additional noise...

Looks good.


Props not updating inside

Hi @ollija, I have several 'cell's inside this sortable grid. Some cells have props connected to a remote API. Apparently the cells that are inside the sortable_grid don't update and those which are outside do update.
My main question is: Do props inside  are updatable?
<SortableGrid>
 <View style={styles.defaultCell} key={key_item}>
  <Text style={styles.cellTitle}>Availability</Text>
  <Text style={styles.cellValue}>{this.props.activity}%</Text>. //This doesn't update
 </View>
 </SortableGrid>
 <View style={styles.defaultCell} key={key_item}>
  <Text style={styles.cellTitle}>Availability</Text>
  <Text style={styles.cellValue}>{this.props.activity}%</Text>. //This does get updated
 </View>

Try install dependency from github repository, not from npm.

@krhovsky that worked for me

npm i react-native-sortable-grid@https://github.com/ollija/react-native-sortable-grid --save

This still does not appear to be working for me. Trying to dynamically change the itemsPerRow with a button that changes local state. Are we sure this is still fixed?

Tente instalar a dependência do repositório do github, não do npm.

thank you very much

npm i react-native-sortable-grid@https://github.com/ollija/react-native-sortable-grid --save
solved the problem, but resulted in another, the movement behaves differently than expected

Isso ainda não parece estar funcionando para mim. Tentando alterar dinamicamente o itemsPerRow com um botão que altera o estado local. Temos certeza de que isso ainda está corrigido?
I have the same problem, managed to solve?

This still does not appear to be working for me. Trying to dynamically change the itemsPerRow with a button that changes local state. Are we sure this is still fixed?

I have the same problem, managed to solve?


Support Node Engine 20

Is your feature request related to a problem? Please describe.
In my Node 20 project, I want to install Lens SDK, but I got an error.
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "^18.15.0". Got "20.9.0"

Describe the solution you'd like
Allow node 20 version in engines config
https://github.com/lens-protocol/lens-sdk/blob/d3a85d1dde3f214da2ad8f29c9126f384ab9b8a6/packages/client/package.json#L99

The @lens-protocol/client does not have any hard dependency on Node 18. It just that when the project started Node 20 was not the stable release yet. Happy to relax this.
Given the error it seems your local setup has engine-strict checking set:

Unless the user has set the engine-strict config flag, this field is advisory only and will only produce warnings when your package is installed as a dependency.

To unblock you in the short term you can relax such constraint and the error will become an advisory warning.
It would be lovely if you could help us by reporting any issue in using it with Node 20.

Change for this will be part of next SDK release.

@daoleno this is now released<EMAIL_ADDRESS>Please let me know it any issue.


Add invalidCommand emit

This PR adds the ability to run a custom handler when a user tries using a command but that command name is not found.
Should close #27

Needs a JSDoc comment for the added event, but not a big deal. Will look at with the other stuff this weekend, thanks for the contribution!

Resolved conflicts with other PRs, looks good for 2.0


Utils for debugging

When hacking on cornelis, I spend a lot of time looking at messages sent by Agda and nvim to reverse engineer their protocols. Right now in my PR #98 I added a debug function to dump arbitrary things into a file.

Does the nvim API have anything for logging? notify doesn't seem like it's supposed to receive more than a couple of lines at a time.
Does debug in #98 look good enough to keep or do you have other preferences about what the Debug should look like?

nvim-hs as called by cornelis dumps a file /tmp/cornelis.log corresponding somewhat to the messages being sent between the two. Which is helpful for debugging nvim-hs, but I don't think it generalizes to other use cases.
Keep it!


[UI] Add Sidebar Peek for Notification Center

Notes for Reviewers
This PR:

[x] Adds sidebar peek for notification center
[x] Increases the peek transform for the context switcher
[x] Fixes bug for undefined notification count badge (notification tooltip shows undefined events when event.length is undefined on Meshery startup)

This PR fixes #6683
https://user-images.githubusercontent.com/73700530/208294148-d67d4e09-359f-4fec-9220-68d9f4995452.mov
before:

after:

Signed commits

[ ] Yes, I signed my commits.

@partik03 you might want to review.
@theBeginner86 thanks for this. We'll have to make the k8s context switcher peek down only as far as until the first K8s context would start to show. Otherwise, users will think that the menu is buggy (that it should be fully displayed, but gets stuck halfway).

In a similar way, so too should the Notification Center only peek outward up to about halfway through the bell icon inside the sidebar.

Thank you for taking care of the undefined bug, too. This is great, @theBeginner86

Update:
https://user-images.githubusercontent.com/73700530/208321708-6532d487-9a62-4da2-94a8-bff8c9776ce4.mov

Nice!


Cleanup the environment variables

Follow the same pattern as main artemis (HOST/PORT)

Resolved by #1


Add hint or warning when importing a package that has the same name as the current file

Hi,
I was working through the online tutorial, and when playing with the random example, I stumbled upon a weird compiler issue when trying to run the file through elm-reactor
I found multiple modules named 'Random'.

Module 'Main' is trying to import it.

Modules with that name were found in the following locations:

    directory ././Random.elm
    package elm-lang/core

I realized the file I was trying to run was named random.elm, which seems to cause a conflict when if the file contains the line import Random.
It took me a few minutes to understand what was happening (mostly because I'm an absolute beginner and don't know much about elm), but based on the excellent compiler error that elm uses, I thought that this could be something that could be improved.
It seems like if one of the import statements matches the current file, a warning (or an error) could be issued.
We could also maybe make the line directory ././Random.elm a little bit more explicit to illustrate that it references the current file.
Those are just my current thoughts, there could obviously be better options.
I would be happy to work on those improvements (if they are deemed valid), but I'm not exactly sure how the compiler really works. All I found so far is this file, that seems to be involved in the process.
It could be a good reason for me to start learning some haskell!
PS: Really loving elm so far, thank you so much for everything!
Using elm 0.17.0 on mac OS

I got this exact problem while working on https://guide.elm-lang.org/architecture/effects/random.html. A note should be added to this page in particular to signal to new Elm developers this potential name collision issue.


Update omnicorp build to handle pubchem/chebi better.

   Omnicorp knows about pubchem ids, but I guess because the names are different in pubchem there are lots of cases where we have results in omnicorp for chebi but not for pubchem.
Originally, I was thinking that omnicorp overlay should look in the equivalent identifiers on the input graph and query the cache/postgres for those identifiers as well.
But I think that's wrong - first, you only get back counts, so if you get results for 2 equivalent identifiers, there's no good way to combine them or decide between them.   Second, it makes a lot of (probably repeated) double querying.  Now I think we should resolve this upstream when we build the omnicorp database and cache.   All that we need to do is normalize identifiers where we still have the actual pubmed ids so that we can combine things.
The downside of this approach is that it will tie the cache to the normalization and biolink prefix ordering.

This needs to be fixed.  Looking at results from our benchmarks (particularly furosemide vs edema) we are not getting any furosemide omnicorp results b/c of this issue.  In particular, it has 2 PUBCHEM.COMPOUND ids.  The one in omnicorp and the one that we are querying over.


Update documentation on revised hash functionality.

Supplement for #292
I keep this separate, so corrections don't hold up #292.

I rebased this and fixed a couple of typos 7ae40cb3527e982e6fd35d48f71d085b7d87495f
Thanks!


Exec tachikoma update<PHONE_NUMBER>3331

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 67.647% when pulling ad483a9848ead286da99e6f43b425d769025019e on pazjacket:tachikoma/update-20170709163331 into ec251e61adc687ae0589e5283f1773ece07521c0 on sanemat:master.


server restart v1

use EPICS to trigger a PV

GUI button to be restored, mechanism to be poking this PV (value unimportant)

use a kafka triggering system underneath

eventually to replace the PV when we have a Kafka-compatible conda environment for the LSDC GUI and server
channel name: {beamline.lower}.lsdc.serverRestartSignal ?

on server host, have a service that monitors the PV

restart server using normal mechanism when the message is received
ownership - may be problematic. register script to be root?
consideration - what if it is not running already? who would it start it as?

more straightforward if owned by beamline staff - just start a new one as them

feedback: naming of scripts is not scientist-friendly. some issues with the documentation, including the precise PVs involved (when testing things manually)
but it does work!


一会不操作 就断开连接了，必须重新打开才行

System information:

Operating system (distribution) and version     ubuntu18
DBeaver version      21.1.5
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

org.jkiss.dbeaver.model.sql.DBSQLException: SQL 错误: Statement cancelled due to client request
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:133)
at org.jkiss.dbeaver.model.impl.jdbc.struct.JDBCTable.readData(JDBCTable.java:190)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.lambda$0(ResultSetJobDataRead.java:118)
at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:171)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.run(ResultSetJobDataRead.java:116)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetViewer$ResultSetDataPumpJob.run(ResultSetViewer.java:4711)
at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
Caused by: com.mysql.cj.jdbc.exceptions.MySQLStatementCancelledException: Statement cancelled due to client request
at com.mysql.cj.jdbc.exceptions.SQLExceptionsMapping.translateException(SQLExceptionsMapping.java:110)
at com.mysql.cj.jdbc.StatementImpl.checkCancelTimeout(StatementImpl.java:2191)
at com.mysql.cj.protocol.a.NativeProtocol.sendQueryPacket(NativeProtocol.java:1028)
at com.mysql.cj.protocol.a.NativeProtocol.sendQueryString(NativeProtocol.java:895)
at com.mysql.cj.NativeSession.execSQL(NativeSession.java:1111)
at com.mysql.cj.jdbc.StatementImpl.executeInternal(StatementImpl.java:724)
at com.mysql.cj.jdbc.StatementImpl.execute(StatementImpl.java:648)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.execute(JDBCStatementImpl.java:330)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:130)
... 7 more

Hello @tomuGo,
Please use English. Can you provide steps to reproduce your issue?

Hi all, the headline means:
DBeaver cannot establish a permanent connection, the session will be broken off within minutes if user don't input any more command.

Hello @TimZhang13579,
Thanks for clarification. But we still need steps to reproduce the issue, otherwise we can't do nothing.

Hi all, the headline means:
DBeaver cannot establish a permanent connection, the session will be broken off within minutes if user don't input any more command.

I just translate Chinese to English, I am not the owner of the question. Thanks.

Hello  @ShadelessFox
The problem recurrence steps are as follows:
1.Open a database connection session and let it stand for half an hour or an hour
2. If you try to enter a command to execute in this session, an error will be reported(Similar to the above error report)
3. At this point, you must try the connection again and open it again
The session will not automatically attempt to reconnect
I often encounter this problem

@Jackxiaojiahuo you can change keep-alive value in connection properties to non-null. If it will be set e.g. to 30, than keepalive signal will be sent every 30 seconds

Thanks, I got it


Fix issue with error messages when a file upload fails

When one file being uploaded failed, we'll cancel other uploads to stop the deploy straight away.
However, we were overriding the original error message with a generic error saying:
Canceled because upload has already failed
This makes sure we don't overwrite the shared error message

LGTM


How to convert Lora to safetensors

After training the Lora I got .bin files how to convert them to safetensors. Also I found that hugging face repo https://huggingface.co/comfyanonymous/flux_RealismLora_converted_comfyui with lora converted to comfyui, is that mean not converted Lora will not work?

convert the .bin output to .safetensors using this huggingface space:
https://huggingface.co/spaces/safetensors/convert
then, use this script to make it compatible with comfy (and everything else):
https://huggingface.co/comfyanonymous/flux_RealismLora_converted_comfyui/blob/main/convert.py

Thanks


Unable to create scheduled task from main menu

Issue description
Since I rebuild my webserver on Windows Server 2019 I started with the 2.0 Alpha version of WACS
 [INFO] A simple Windows ACMEv2 client (WACS)
 [INFO] Software version 200.0.6940.36971 (RELEASE)
 [INFO] IIS version 10.0
 [INFO] ACME server https://acme-v02.api.letsencrypt.org/

I'm able to create certificates and bind them to IIS but when I try to start
 T: (Re)create scheduled task

I get this error:
 [EROR] ComponentNotRegisteredException: The requested service 'PKISharp.WACS.Services.TaskSchedulerService' has not been registered. To avoid this exception, either register a component to provide the service, check for service registration using IsRegistered(), or use the ResolveOptional() method to resolve an optional dependency.

There is an scheduled task created so it not high priority 😄
Client version:  200.0.6940.36971 (RELEASE)
OS Name: Microsoft Windows Server 2019 Standard
OS Version: 10.0.17763 N/A Build 17763
Relevant part of log file:
For bug reports, please reproduce the bug with the --verbose parameter and copy the relevant part of the log file.
Where is the log file stored?

Thanks for helping with the testing, especially interesting to get some done on Windows 2019 since that's not in my own fleet yet. This was a dependency injection bug though, so nothing specific to 2019. I will be fixed in the next build.

Let me know if there is anything you want to test with 2019 😄


Catching all exceptions to avoid app crash

Steps to Reproduce
Some parts of the framework seems to throw exceptions that we can't catch resulting in the application crash.
My personal experience is trying to parse a Json response instead of the body of it and having all my application to crash. It bypass try catch and even FlutterError.onError = (FlutterErrorDetails details) ...
No exceptions, no stack trace, just 'connection to device lost' and application exit.
I'd like more documentation on how to handle those case locally and a global catch somewhere to be sure my app doesn't crash on case that I can not handle.
Thank you !

Is there any example code you can share?

_preloadVotes() async {
      var jsonCategories = await get('vote/categories');
// jsonCategories is not json but response, .length doesn't exist -> FlutterError.onError catch nothing, app crash
      print("Loaded ${jsonCategories.length} categories");
  }

Been updated flutter to master from alpha yesterday and today I can locally catch the error with try catch, I lost some hours on it yesterday because debug show me it bypassed. But I'll be glad to catch any non locally catch exceptions with onError.


Created SCP wrapper

Handled all four cases for different sources and destinations. Tested using a SSH server on docker .
Key steps implemented when SCP is used to transfer files:

Retrieving Source Inode Information

Extracted the src_inode_version and src_inode_metadata from the source.

Uploading Files

Uploaded the necessary files from the source to the destination.
Retrieving Destination Inode Information
Extracted the dest_inode_version and dest_inode_metadata from the source.

run get_prov_upstream function

Obtaining Remote Home Directory

Determined the remote home directory for the destination.

Directory Creation on Destination

Created the necessary directories on the destination

Uploading Source Inode Version and Process Id

Transferred the src_inode_version and process_id from the source to the destination.

Random Process ID Generation

Generated a random_pid to uniquely reference the scp process involved.

Creating SCP Process JSON

Created a JSON file representing the SCP process

Linking SCP Process ID to Destination Inode Version

Added a reference to the SCP process ID in the destination's inode version metadata.

Copying SCP Process Information to Destination

Transferred the SCP process information to a designated file on the destination.

Contains the use of function present in https://github.com/charmoniumQ/PROBE/pull/44

For command :  python -m probe_py.manual.cli scp tasks.org shofiya@localhost:home/
The provenance files on ssh server


What are the differences in the claims between id and access tokens?

I think it would be beneficial to document that the same set of claims come back with id and access tokens except for the scp claim.
Reference: Can Azure B2C claims exchange be used for Access token?

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

ID: b05bb7df-98fc-1342-640d-02063f6d8a11
Version Independent ID: f6ac357f-f846-83c5-f19f-91e1bc0954e3
Content: Token reference - Azure AD B2C | Microsoft Docs
Content Source: articles/active-directory-b2c/active-directory-b2c-reference-tokens.md
Service: active-directory-b2c
GitHub Login: @parakhj
Microsoft Alias: parakhj

@spottedmahn Thank you for the valuable feedback,we are investigating the issue.

@spottedmahn hope you're doing well! I've created a work item on myself to go update this article and include the note that you mentioned. Thanks for the suggestion! For now, I'll close this GitHub issue. The documentation should be updated by Build :P

#please-close

hey @parakhj - you too!!
Thanks!!


Administration not all helpers are exposed

PHP Version: 8.3 (doesnt matter)   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Expected behaviour:
All 8 js helpers are exposed to Shopware 6 global object (Shopware.Helper):

Actual behaviour:
Only 5 out of helpers are exposed to Shopware 6 global object:

How to reproduce:
Open any admin page and type Shopware.Helper in the developer's console.

The problem that in my plugin I have to copy the contents of another helper, which is a bit annoying to maintain.
(In my case I need to utilize the retry.helper.js helper from my extension).
I'm able to create PR, if there is no valid reason why those 3 helpers are not exposed.

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39448
[created from NEXT-39448, comment 510356]

[public]

Status Change: Open

This issue is ready for implementation.
[created from NEXT-39448, comment 511529]

[public]

Status Change: In Progress

Work on this issue has started. It is now being implemented.
[created from NEXT-39448, comment 511552]

[public]

Status Change: In Review

Work on this ticket is complete and is now pending review.
[created from NEXT-39448, comment 511601]

[public]

FixVersion Change: <IP_ADDRESS>

This issue will be / was solved in Shopware version <IP_ADDRESS>.
[created from NEXT-39448, comment 513615]

[public]
The RefreshTokenHelper, RetryHelper and UploadTaskHelper are private, so we are not including it in the public helpers.
[created from NEXT-39448, comment 515053]


F-String does not work

I wrote a counter web app, but it has two counter.
Here is the code I wrote:
def index():
    return pc.center(
        pc.grid(
            pc.grid_item(pc.heading(f"{TestStateA.counter} {TestStateB.counter}"), row_span=1, col_span=3, text_align="center"),
            pc.grid_item(row_span=1, col_span=3),
            pc.grid_item(pc.flex(pc.button("Increase", on_click=TestStateA.clicker), align="center", justify="right"), row_span=1, col_span=1),
            pc.grid_item(row_span=1, col_span=1),
            pc.grid_item(pc.flex(pc.button("Increase", on_click=TestStateB.clicker), align="center", justify="left"), row_span=1, col_span=1),
            template_columns="1fr 40px 1fr",
            template_rows="1fr 50px 1fr",
            width="fit-content",
            height="fit-content",
        ),
        width="100vw",
        height="100vh"
    )

Basically I put heading in top of grid, and the content of the heading is formatted with f-string.
And the result is:

Then, I rewrote f-string to plus sign like this
pc.heading(TestStateA.counter + ' ' + TestStateB.counter)

And it worked perfectly how I desired..

I think it's reproduced when you put f-string into content.
I'm not sure but when I wrote similar code with text and span component, same happened.
Specifics (please complete the following information):

Python Version: 3.11.1
Pynecone Version: 0.1.14
OS: Windows 10
Browser (Optional): Chrome

Maybe it is related to how we process the view function code, we compile them to React components, which is write in Javascript.

Yeah @FHU-yezi is right but I think we can try and support this in the future as we do with other var operations with strings.

Most of f-string work exept for @sserve-kr mentioned one.
I think this related to accesibility of state var.

shows value

Pure python object.
class State(pc.State):
    text: str = "default"

def index():
    return pc.vstack(
        pc.text(f"{datetime.now()}"),
    )

Default value of state var.
class State(pc.State):
    text: str = f"{datetime.now()}"

def index():
    return pc.vstack(
        pc.text(State.text),
    )

Set variable with function.
class State(pc.State):
    text: str = "this is default"

    def add_text(self, value):
        self.text = f"the input is {value}"

def index():
    return pc.vstack(
        pc.text(State.text),
        pc.input(on_blur=State.add_text)
    )

Backend var.
class State(pc.State):
    text: str = "the default value"
    _text: str = "the inside default"

def index():
    return pc.vstack(
        pc.text(f"{State._text}")
    )

shows variable name
State var.

class State(pc.State):
    text: str = "this is default"

def index():
    return pc.vstack(
        pc.text(f"{State.text}"),
    )

Could something like this be a solution?
def to_template_literal(s: str) -> str:
    """
    Takes a formatted Python f-string with references to Pynecone Vars
    and turns it into a fully functional Javascript template literal
    with working references to those same state variables.

    Most of the heavy lifting has already been done by the
    Var class, transforming the Python code inside the f-string's
    format blocks into valid and equivalent (ish) Javascript.

    Because Python's f-string syntax and Javascript's template literal
    syntax are so similar, all we have to do is add the $ in front
    of the bits we want to be formatted.
    """
    s = re.sub(r"([^{]){([^{])", r"\1${\2", s)
    return s

Or, if the regex solution is too naive/simple in some cases:
def to_template_literal(s: str) -> str:
    def process(expr: ast.expr):
        if isinstance(expr, ast.Constant):
            return expr.value
        elif isinstance(expr, ast.FormattedValue):
            return "$" + ast.unparse(expr)
        raise ValueError()  # Not sure if arriving here is actually possible. Just here for safety

    # Turn the string into an f-string and parse it with the built-in ast,
    # simply adding
    f_str = 'f"' + s.replace('"', "'") + '"'
    f_str_ast = ast.parse(f_str, mode="eval")
    str_parts = f_str_ast.body.values

    # We don't need to surround with '`' because pc already does this
    # with all strings that get compiled to JS.
    out = "".join(map(process, str_parts))
    return out

Example:
class SubState(pc.State):
    strs: list[str] = ["test"]

class State(pc.State):
    colors: dict[str, str] = {"one": "red", "two": "blue"}
    array: list[SubState] = [SubState()]
    color_key = "one"

    def toggle_color(self):
        self.color_key = "two" if self.color_key == "one" else "one"

def index():
    s = f'{1} and {State.color_key} and {State.colors[State.color_key]} and {State.array[0].strs[0]}'
    s_templated = to_template_literal(s)
    return pc.vstack(
        pc.text(s, color=State.colors[State.color_key]),
        pc.text(s_templated, color=State.colors[State.color_key]),
        pc.button("Toggle color", on_click=State.toggle_color)
    )

On load:

After pressing button:

Fixed #701


Fix chart lines loading

When creating a new dataset and selecting lines chart, the chart would stay blocked in loading mode. This was due to the fetching attribute of the OperationResult staying to true when a query was refetched while it was already in cache. Updating urql fixes the problem 🎉
Fixes: https://github.com/visualize-admin/visualization-tool/issues/211
Two minor commits that I was doing at the time of the fix:

docs: Rename dataset search to dataset browse
refactor: Extract and use set instead of array to check for inclusivity

Ah, I don't like automerge 😂 @ptbrowne can you make sure to check the imports on main?

They are imported, and we now have an eslint rule for unused imports ;)

My point was, they should not be imported? Maybe I saw wrong 😅

Oups sorry, I meant "they are used" (I checked). The eslint rule would error if they were not used ;)

Ah nevermind … just a quick note: you don't need to use qs.stringify, as you can just pass the query as object to router.push(). Just highlighting this because this would add another package to the bundle that (hopefully) is not used elsewhere and duplicates functionality built into next.js.


Enhancement Request: Route prefix

Right now, the routes defined always start from the root. Would it be possible to add an option/parameter to set a custom prefix, which is then prefixed to every route (or every route in a specific routes directory)? (For example: /api)

I'm looking for this feature too. not sure if this module is still actively maintained by the author.

I guess not :'(

@johannlo I guess not :'(


Panic on fastanvil::Region::remove_chunk for a non-existing chunk

Message:  called `Result::unwrap()` on an `Err` value: 0
Location: cargo\registry\src\index.crates.io-6f17d22bba15001f\fastanvil-0.28.0\src\region.rs:329

I'm guessing it should be checking if the location is 0, 0 already and then just returning.

Hi there @Nemo157. I'm currently on vacation/holiday so won't be getting around to looking at this for a few weeks. Sorry you ran into issues!

Don't worry, these crates have been such a help setting up some tooling for a server. I'll probably submit a PR soon to fix it whenever you get time to review it.


String-Arrays are ignored/deleted

General information
Stringlate Version 0.14 from FDroid
LOS14 on Lenovo P2
Description
I have tested it with my own project. (https://github.com/TheAkki/Synctool)
In my translation file i have a string array. For example:
 <string-array name="ConnectionTypes">
        <item>Lokal</item>
        <item>OwnCloud</item>
        <item>FTP</item>
</string-array>

What I do:

Update the strings.
Do translation
Send to Git as pull request

When i want to merge the pull request the string array is always deleted.

I too get this bug.


[Possible feature request] Get alternative schemes from same picture file

I really enjoy Pywal first and foremost! Is it currently possible to get alternate colour schemes from the same photo? Some photos I've tried out are not using the most, IMO, exciting colours from a given photo so would love a function like this. Maybe a function to "randomize" what colour goes where (background, bold text, and so on).
Thanks!

pywal now has 5 total color backends so I think it's ok to close this.


cherry-pick perf(topn): a series of topn refactoring and optimization to release-1.10

https://github.com/risingwavelabs/risingwave/pull/17892
https://github.com/risingwavelabs/risingwave/pull/17947
https://github.com/risingwavelabs/risingwave/pull/17895
https://github.com/risingwavelabs/risingwave/pull/19451
https://github.com/risingwavelabs/risingwave/pull/19386

Resolves #19513

Why closed? 🤔

Why closed? 🤔

Too many unrelated things need to be cherry-picked. I finally thought it not worth doing it.


Remove timestamps from contracts

this has been a journey, but after discussion with @NoahSaso i'd like to suggest that we remove timestamps used only in the frontend from our contracts. in particular, this would mean removing the last_updated field from proposals in cw-proposal-single and cw-proposal-multiple.
as a guiding philosophy: we should keep our on-chain data as simple as possible. it is much easier to fix a bug in an indexer than to fix a bug in our contracts. in the case of timestamps, we can get that data pretty much effortlessly with an indexer and we already have a couple going and doing so.
yay! we get to remove code!

Not sure how I feel about this. Honestly, I think timestamps are essential data... currently, we only use them in the frontend, but I can imagine a scenario where they might be used in another contract.
It's not too much data to store. If we are worried about state we should work to store proposal text on something like IPFS. Yes, we can get this data from indexers but I like not having to rely on indexers to heavily if need be.

This is resolved right?


Provider mangakatana

Proposal: Add mangakatana provider.
Tried (Version 1.3.15):
manga-py --cbz --zero-fill --rename-pages --destination Manga/ http://mangakatana.com/manga/a-story-about-treating-a-female-knight-who-has-never-been-treated-as-a-woman-as-a-woman.20169

Got:
    raise AttributeError('Provider not found')

Yes, this provider is not listed:
https://yuru-yuri.github.io/manga-dl/
Thank you, I will try to add support for the site.

Works with 1.3.16 :+1:


contacts don't seem to work.

Hi there.
We used this module for pushbullet integration in balloob/home-assistant
Works awesome, keep up the good work!
Noticed that the contact don't seem to work. Result always is an empty list [] (yes, valid contacts in account).
Perhaps im doing something wrong, if so i'd appreciate the feedback :)
from pushbullet import PushBullet
pb = PushBullet('yourkeygoeshere')
pb.contacts

PB integrated in HASS (how we use your module, and a little shameless plug ;) )
https://github.com/balloob/home-assistant/blob/dev/homeassistant/components/notify/pushbullet.py

in the pushbullet api docs changelog its says:
"Removed Contacts calls as the official apps no longer user Contacts. These have been replaced with the Chat objects."
https://docs.pushbullet.com/v12/#changelog

Hi!
@benbuf you are right! We need to support chat api.
@tomduijf now I re-test the contacts and looks good with pushbulet.py and also with curl check. Do see any on pushbullet web https://www.pushbullet.com/#settings/people ?
For example (I change some personal data):
>>> from pushbullet import PushBullet
>>> pb = PushBullet("token")
>>> pb.contacts
[Contact('User Name' <user.name@example.com>)]

curl -u token: -X GET https://api.pushbullet.com/v2/contacts
{"accounts":[],"blocks":[],"channels":[],"chats":[],"clients":[],"contacts":[{"active":true,"iden":"aaabbbcccdddeeefffgggh","created":1.430499126449944e+09,"modified":1.430499126459729e+09,"name":"User Name","email":"user.name@example.com","email_normalized":"username@example.com","image_url":"https://lh5.googleusercontent.com/-aaabbbbcccc/AAAAAAAAAAI/AAAAAAAAAAA/aaabbbcccdd/photo.jpg","status":"user"}],"devices":[],"grants":[],"pushes":[],"profiles":[],"subscriptions":[],"texts":[]}

@kovacsbalu haven't tested it in a while. At the time of submitting this issue it wasn't working (perhaps pb restored part of the functionality for backward compatibility).


vmap codegen, for review

Here's how the output looks:

https://github.com/pytorch/functorch/blob/main/functorch/csrc/VmapGeneratedPlumbing.h
(yes, we check in the codegen as a short-term solution because we
didn't have time to add it as a step to setup.py)

short-term / long-term plans

we've copy-pasted tools/codegen from pytorch/pytorch into functorch
In the long term, pytorch/pytorch will package its codegen somehow
(https://github.com/pytorch/pytorch/issues/73212)

Questions for reviewer:

Are we using PyTorch's codegen abstractions the right way?

Should be ready for another review. I also regenerated the plumbing and checked it in with this PR (functorch/csrc/VmapGeneratedPlumbing.h)


Can't get the program to work

When testing with websocat the request fails.
~/o/supreme [1] $ websocat ws://<IP_ADDRESS>:5000/echo
websocat: WebSocket response error: Sec-WebSocket-Accept is invalid
websocat: error running

It works when I set custom heather, which changes the order of the request headers, I don't know if that's relevant.
~/o/supreme $ websocat ws://<IP_ADDRESS>:5000/echo -H='sec-websocket-key: CHBsjysDVjJ77jY4A'
TICK
TICK
TICK
TICK

I can't get the example to the program to work with any other WebSocket client.

…which changes the order of the request headers, I don't know if that's relevant.

It shouldn’t. It’s easy to see that headers in both cases are read correctly by simply inserting “printfn” here: https://github.com/erpuno/ws/blob/master/src/6455.fs#L30.

…any other WebSocket client.

Which? At least, it should work correctly with wscat.

Thanks for the quick reply.
So it works with wscat. And I'm not sure why but not with websocat, the flutter client, websocket-client.
All the other clients work with websocat -s 8888.
Test case: client websocat
👎
It fails when trying to connect.
~/o/ws (master|✚2) [1] $ websocat ws://<IP_ADDRESS>:1900/echo
websocat: WebSocket response error: Sec-WebSocket-Accept is invalid
websocat: error running

Wireshark capture

Test case: client websocat with custom header
👍
~/o/ws (master|✚2) [1] $ websocat ws://<IP_ADDRESS>:1900/echo -H='Sec-WebSocket-Key: CHBsjysDVjJ77jY4A'
hello
hello
TICK
TICK
^C⏎

Wireshark capture

Test case: client wscat
👍
~/o/ws (master|✚2) $ wscat --connect ws://<IP_ADDRESS>:1900/echo
Connected (press CTRL+C to quit)
< TICK
> hello
< hello
< TICK
> ⏎

Wireshark capture

Test case: Flutter WebSocket Client
👎
It Fails to connect. The server never responds with HTTP 101.
IOWebSocketChannel.connect('ws://<IP_ADDRESS>:1900/echo')

Test case: Flutter WebSocket Client using the websocat server
👍
Server
websocat -s 8888

Client
IOWebSocketChannel.connect('ws://<IP_ADDRESS>:8888/')

Wireshark capture

Ultimately I'm looking to make it work with flutter.