Material-UI website redirect v1-5-0.material-ui.com/* to v1.material-ui.com/*

[X] This is not a v0.x issue.
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
Navigating to https://v1-5-0.material-ui.com/* should redirect to v1.material-ui.com/* (e.g. https://v1-5-0.material-ui.com/demos/buttons/ should redirect to https://v1.material-ui.com/demos/buttons/)
Current Behavior 😯
Navigating to v1-5-0.material-ui.com results in a 404 response
Context 🔦
Our internal company documentation links to the old v1-5-0.material-ui.com/* links. Ideally we would just refactor the links to point to the new urls, but all our documentation is versioned by npm package version, and going back through all the old documentation builds to refactor the urls per npm version would be quite a difficult task. Whereas, I can imagine setting up a redirect might not be quite so difficult? What's in it for Material-UI? It would mean you would make Google happy. Google loves redirects ;)

Tech
Version

Material-UI
v1.5.x

@samwalshnz Sorry about that. We should be good from an SEO point of view, we were pointing the canonical of v1 to the naked domain. We could try the following, do you want to give it a go?
diff --git a/docs/static/_redirects b/docs/static/_redirects
index d8ab1b305..754982cf1 100644
--- a/docs/static/_redirects
+++ b/docs/static/_redirects
@@ -36,6 +36,7 @@ https://material-ui.dev/* https://material-ui.com/:splat 301!
 /lab/* /components/:splat 301
 /css-in-js/* /styles/:splat 301
 /premium-themes* https://themes.material-ui.com/ 301
+https://v1-5-0.material-ui.com/* https://v1.material-ui.com/:splat 301!

 # link shortener
 /r/styles-instance-warning /getting-started/faq/#i-have-several-instances-of-styles-on-the-page 302

@samwalshnz Do you want to submit a pull request?

I'll help out, @samwalshnz. It's been a minute since I have pitched in around here. 😆

Solved after #16658 and 95cbe4fd6


RadioGroups and InputChips are not able to be disabled

Current Behavior
It is not possible to fully disable either of these components (both visually and functionally).
RadioGroup
Adding the disabled property does nothing
InputChip
Adding the disabled property does disable the input itself, but existing chips are still able to be interacted with and removed
Steps To Reproduce

Use either the RadioGroup or InputChip components
Add the disabled property
See that they are not disabled

Anything else?
I fixed a related, but simpler issue in #

I'll handle this as part of: https://github.com/skeletonlabs/skeleton/issues/782

Now resolved as part of: https://github.com/skeletonlabs/skeleton/issues/857
Please test and post feedback in that ticket please.


Demo changes

Trivial detail - the standard thus far is to use lower case letters (K6)

ah, looks like I added that to the test :)


[item] "Orbital-cannon-thing"

I imagine this would be either a consumable with a long CD (in the inventory or in the shop) , a special ability for some upgraded version of boots(or radiance :3 - "sunstrike radiance")  or a boss_item ability.
Upon activation :
*Cast range - global (or veeryy big)
*Doesn't use a lot of mana
*Casts Invoker's sunstrike , but, additionally , lights up the area from where it was fired from (revealing the caster) and leaves a medium damage DoT circle on the point of attack instead of the normal "after-sunstrike-burned-ground-texture" for 2-3 seconds.
Simple idea. If you have any questions - ask.
-VoidsKeeper

What is mean that DoT? Dfence of Troy? Dogeing or Throwing? Duche of Townmedia? Devil of Toilet? I have no idea...

@Wujekklawy  ,  it means "damage over time" (a poison affect )

@VoidsKeeper i think Devil on Toilet is better anyway xD

Actually i don't see other purpose for it than stealing enemy camps, damage per second is not the best for killing low hp heroes,  ok i must admit that it would be good in teamfights if you have stuns or roots. So maybe lower the cast range dramatically and give it more range in aoe or damage making it teamfight item instead of "let's steal enemy camps from half of the map" :)

Well , it's intended to do a variety of things (you described a few). The numbers (or ,atm, the speculations )   are mostly insignificant. I just wanted to share the idea of the mechanic ;P

We got Azazels Crossbow which is kinda similar. Not a big fan of global ranges. Closed.


Add new test helpers for Asset Manager

This looks good. I've since added another TODO for deleting any assets, which is something supported by the test helper for the Publishing API. Could you add one for that, too?
https://github.com/alphagov/content-publisher/blob/c0de7133228830da64124c51dc060cc94b30d9c3/spec/features/delete_draft_asset_manager_down_spec.rb

@benthorner Done - let me know if you're happy with that?


Version on PowerShellGallery

Hello,
thanks for the extremely useful extension!
Unfortunately, only the 2.2 version is available on the PowerShellGallery.
greetings

Yeah the owner want to make some more changes bevor Publishing the New Version..

Thanks for the answer.
Unfortunately I couldn't find the info anywhere.

Thanks for the answer.
Unfortunately I couldn't find the info anywhere.

Yea you right..
i wrote him an Email thats where I got the Info

@Pyromane see #5

@Pyromane see #5


Async requests for Together

Use async requests for Together models mpt-7b, redpajama-incite-base-3b-v1, vicuna-13b.

PTAL again, thanks.


Update subscription overrides

Issue: https://github.com/open-cluster-management/backlog/issues/15624
Signed-off-by: Jakob Gray<EMAIL_ADDRESS>

/lgtm


Circular references processor should not change user data

I understand that it's not memory-efficient, but you are working with business rules which might rely on references.
Consider opting out this behavior, or display a warning somewhere, and this behavior should be documented.
I spent 6 hours of debugging code because of the sentry event. Imagine my feelings when I figure out it's caused by raven.
Fixes https://github.com/getsentry/raven-ruby/issues/903

@estepnv thanks for the fix!
but this approach could increase memory allocation by 16% or more, which is something we should avoid if possible (you can see the detail about this analysis at https://github.com/getsentry/raven-ruby/pull/994's description). so I will close this once #994 is proved working and merged.


[Torch] Add cast to double, fix flatten conversion

This is another fix to support hummingbird project.

Cast to double was missing
Conversion of torch.flatten was wrong and not tested.

please review @siju-samuel

Thanks @masahi @leandron This PR is merged.


darwin: bump minimum supported version to 10.15

We can't realistically claim to support 10.7 or any version that Apple
no longer supports so let's bump the baseline to something more
realistic.
Refs: https://github.com/libuv/libuv/pull/482
Refs: https://github.com/libuv/libuv/pull/3405

As a point of reference, for Node.js:

branch
minimum supported macOS version

master
10.13 (but will likely be bumped soon ahead of Node.js 18.x https://github.com/nodejs/build/issues/2815)

v17.x
10.13

v16.x
10.13

v14.x
10.11

v12.x
10.11

Node.js 12 and 14 are in maintenance -- I'd be okay to take the stance we'd stop updating libuv in those if any compatibility issues arise.
We also currently test on macOS 10.14 and 10.15 in the Jenkins CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit-osx-cmake/

I believe Apple stopped supporting 10.14 last year, that's why I picked 10.15.

It would be a shame if Node.js 16.x could no longer  take libuv updates.

As long as it works, it works, right? I remember the LTS branches hardly ever took libuv updates so maybe it's academic anyway.

Generally speaking, Apple still does updates on "Latest - 1" versions. Maybe it could be phrased like such?

@saghul Can you suggest alternate wording?
I do expect people to open issues asking for clarification if we're not explicit about version numbers because, well, people are people.

I'd put macOS >= 10.15 as the PR suggests, and then in the notes add something like: We generally support the latest macOS version and the previous one.

I shortened it to "Current and previous macOS release", otherwise the table starts looking unwieldy. PTAL!


Use different count key names on unique users periods

Different periods on unique-users statistics for Performance
platform use different key names.

Jenkins failed on the branch reference name 🤦♂ Renaming...


Integrations: Update Message Buttons example

Update https://docs.mattermost.com/developer/interactive-messages.html?highlight=matterpoll#message-buttons for https://github.com/mattermost/mattermost-webapp/pull/4993.

hi @justinegeffen can you please tell me what needs to be done for this issue?

Hi @yash2189! The PR contains the history of the change and screenshots. However, as you will see in my last comment, I am waiting additional confirmation on the changes made. I have not yet received that feedback but I believe @hanzei is a great resource and may be able to guide you in terms of what needs to be updated on the relevant page.

Thanks @justinegeffen . @hanzei Can you please tell me what needs to be updated on the page?

https://github.com/mattermost/mattermost-webapp/pull/4993 did add support for different styles to message buttons. This should be reflected in a new section in https://docs.mattermost.com/developer/interactive-messages.html that explains with a screenshots the different supported styles. The PR does contain a list of all of these.
Let me know if that helps you get started.

Sure @hanzei ,thanks I'll have a read on the content provided by you and will get back once I'm done with my reading.

@hanzei I read the content , so is it just like a small section that needs to be added for Message buttons only?

Yes, there should be a new section for style of message buttons with screenshots.

@hanzei As for the screenshots do I need to setup the local environment?

Either that or you use the one from https://github.com/mattermost/mattermost-webapp/pull/4993

May I take this one?


Do not allow + show an error if user imports a Parity Signer account for a chainId other than the current chain

@axelchalon mentioned:

@parity/qr-signer returns us the address and chainId when scanning the account, so this can be done. However I think it might be easier to simply show the Signer account in the accounts list if we're connected to the right chain (we might want to show a warning at one point if we import a Signer account for a chainId other than the chain we're currently connected to -- otherwise the user might be surprised not to see their imported signer account show up in the accounts list after completing the import).

I'm able to look into this now that I've resolved https://github.com/paritytech/fether/issues/466


Support parsing DNS config from WMIC on Windows

This PR implements DNS server config autodetection on Windows by parsing WMIC output. I've looked into reading this from nslookup (works, but blocks when network is not available), ipconfig /all (difficult to parse due to localization), registry (non-trivial implementation) and wmic (which I ended up using). Afaict this command is available on all Windows versions starting with Windows XP and uses some CSV/JSON-based output format that we can parse.
The WMIC command will now be used on Windows by default by the Config class as introduced with #93. This PR includes a number of test cases for the expected WMIC output, but I would to see if anybody else can test this on their local Windows setup. Any feedback is much appreciated!
Builds on top of #92 and #93
Refs #29

See also #95 for a follow-up PR to support multiple DNS servers for a single network interface.
I've also had the chance to perform a number of measurements to see how long the WMIC command takes. A usual run takes around 0.15s, with rare (random?) spikes up to 8s on my test server that does nothing in particular. The very first run seems to take ~1.5s.
We may want to eventually look into parsing ipconfig /all which seems to fluctuate less. Note that parsing likely requires more effort as its output is localized (on some Windows versions?). PRs are much appreciated :+1:
Also, this feature currently returns a list of all DNS servers. It does currently not take "special" DNS servers into account, such as DNS server settings automatically assigned to VirtualBox network interfaces, see https://gist.github.com/clue/ed235ad8d50087d098e4de5fd370b25e#file-gistfile1-txt-L102 and https://www.virtualbox.org/manual/ch09.html#nat-adv-dns.
Practically, this feature is documented to only actually apply the first DNS server from this list (see #93 and also #6). Because phsyical network interfaces seem to always use a smaller "interface ID", this is unlikely to cause any issues for now. PRs to improve this eventually are much appreciated! :+1:


[TheYetee] zero items

This bridge has started to produce zero items.
http://rssbridge.localhost/?action=display&bridge=TheYetee&format=Html

It is slow (~10 seconds) but it still produces items for me.

Same. Produces 0 items sometimes for some reason.


ARROW-8499: [C++][Dataset] In ScannerBuilder, batch_size will not wor…

…k if projecter is not empty

Unit test possible?

Unit test possible?

Is unit test always required for a quick fix like this? I thought this may belong to the kind of changes that could be easily proved right.

As a matter of principle, functional correctness needs to be validated by tests. If you don't test then something that is working, but not tested, may stop working as the result of refactoring.

OK but as the PR is already merged, maybe a follow-up JIRA ticket is needed?

Yes, indeed. Thank you


python311Packages.langchain-core: 0.1.44 -> 0.1.46

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.langchain-core is: Building applications with LLMs through composability
meta.homepage for python311Packages.langchain-core is: https://github.com/langchain-ai/langchain/tree/master/libs/core
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.1.45 with grep in /nix/store/rhi2h455imn7qj33f5r6zlfyamqq3jfb-python3.11-langchain-core-0.1.45
found 0.1.45 in filename of file in /nix/store/rhi2h455imn7qj33f5r6zlfyamqq3jfb-python3.11-langchain-core-0.1.45

Rebuild report (if merged into master) (click to expand)

11 total rebuild path(s)

11 package rebuild(s)

First fifty rebuilds by attrpath

khoj
pentestgpt
python311Packages.langchain
python311Packages.langchain-community
python311Packages.langchain-core
python311Packages.langchain-text-splitters
python312Packages.langchain
python312Packages.langchain-community
python312Packages.langchain-core
python312Packages.langchain-text-splitters

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rhi2h455imn7qj33f5r6zlfyamqq3jfb-python3.11-langchain-core-0.1.45 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.langchain-core https://github.com/r-ryantm/nixpkgs/archive/acbe6fa038af25f02199f0f9c203145021cffec0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/acbe6fa038af25f02199f0f9c203145021cffec0#python311Packages.langchain-core

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rhi2h455imn7qj33f5r6zlfyamqq3jfb-python3.11-langchain-core-0.1.45
ls -la /nix/store/rhi2h455imn7qj33f5r6zlfyamqq3jfb-python3.11-langchain-core-0.1.45/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages marked as broken and skipped:

khoj
khoj.dist

18 packages built:

pentestgpt
pentestgpt.dist
python311Packages.langchain
python311Packages.langchain-community
python311Packages.langchain-community.dist
python311Packages.langchain-core
python311Packages.langchain-core.dist
python311Packages.langchain-text-splitters
python311Packages.langchain-text-splitters.dist
python311Packages.langchain.dist
python312Packages.langchain
python312Packages.langchain-community
python312Packages.langchain-community.dist
python312Packages.langchain-core
python312Packages.langchain-core.dist
python312Packages.langchain-text-splitters
python312Packages.langchain-text-splitters.dist
python312Packages.langchain.dist

Maintainer pings
cc @natsukium for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.langchain-core is: Building applications with LLMs through composability
meta.homepage for python311Packages.langchain-core is: https://github.com/langchain-ai/langchain/tree/master/libs/core
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.1.46 with grep in /nix/store/i48l89shipi157f59f7sidyv5hjh78dc-python3.11-langchain-core-0.1.46
found 0.1.46 in filename of file in /nix/store/i48l89shipi157f59f7sidyv5hjh78dc-python3.11-langchain-core-0.1.46

Rebuild report (if merged into master) (click to expand)

11 total rebuild path(s)

11 package rebuild(s)

First fifty rebuilds by attrpath

khoj
pentestgpt
python311Packages.langchain
python311Packages.langchain-community
python311Packages.langchain-core
python311Packages.langchain-text-splitters
python312Packages.langchain
python312Packages.langchain-community
python312Packages.langchain-core
python312Packages.langchain-text-splitters

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i48l89shipi157f59f7sidyv5hjh78dc-python3.11-langchain-core-0.1.46 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.langchain-core https://github.com/r-ryantm/nixpkgs/archive/091da93929204d083f91575bda7979cef60f2e00.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/091da93929204d083f91575bda7979cef60f2e00#python311Packages.langchain-core

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i48l89shipi157f59f7sidyv5hjh78dc-python3.11-langchain-core-0.1.46
ls -la /nix/store/i48l89shipi157f59f7sidyv5hjh78dc-python3.11-langchain-core-0.1.46/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages marked as broken and skipped:

khoj
khoj.dist

18 packages built:

pentestgpt
pentestgpt.dist
python311Packages.langchain
python311Packages.langchain-community
python311Packages.langchain-community.dist
python311Packages.langchain-core
python311Packages.langchain-core.dist
python311Packages.langchain-text-splitters
python311Packages.langchain-text-splitters.dist
python311Packages.langchain.dist
python312Packages.langchain
python312Packages.langchain-community
python312Packages.langchain-community.dist
python312Packages.langchain-core
python312Packages.langchain-core.dist
python312Packages.langchain-text-splitters
python312Packages.langchain-text-splitters.dist
python312Packages.langchain.dist

Maintainer pings
cc @natsukium for testing.


[LinalgExt] Add scale to linalg_ext.attention op

The pytorch SDPA op specifies a scaling factor for the attention operation that is currently unsupported in the lowering to TMTensor because the corresponding LinalgExt op does not include an argument for the scale.
https://github.com/openxla/iree/blob/7782a414ea473c59f6d7a882cb510690ed666c79/compiler/src/iree/compiler/Dialect/LinalgExt/IR/LinalgExtOps.td#L514
Additionally, the current default scale of 1.0 does not match the default specified by the pytorch op it lowers from:
https://github.com/llvm/torch-mlir/blob/933db87a07fb828b17b75f2f5f396a434f8f1a17/lib/Conversion/TorchToTMTensor/TorchToTMTensor.cpp#L1502
vs
https://pytorch.org/docs/stable/generated/torch.nn.functional.scaled_dot_product_attention.html
The linalg_ext op needs to specify the scale based on the pytorch op, or the lowering to TMTensor the needs to handle the scale as a separate leading elementwise multiplication.

https://github.com/openxla/iree/pull/16679 Added scaling to linalg_ext op. The support on TMTensor side and torch-mlir side needs to be plumbed now.

Closing as completed by #16679


log issue command

Would be cool if we could have a command to log an issue to github (though maybe only for certain users as I could see this being very spam heavy

Could make it a mod user privledge. There's only ~7 of them. Or if slack supports user groups could make one with that command privledge


Turn vdim ordering on by default.

one of the login nodes for dane is down which is probably why that clang job is failing.


Rename preview doesn't seem to work

Testing: microsoft/vscode#98821

😕 I cannot reproduce anymore... @Tyriar Can you give it a try?

I can't repro either, I'm guessing my notebook became corrupted somehow as https://github.com/microsoft/vscode/issues/99443 also was happening all the time back then and now isn't (after I've deleted and recreated the notebook since).

I guess, it must have been something in-flight. When I wrote the test plan it quickly tested things and while the diff is bogous no errors are thrown. Then, when you reported the issue I could reproduce the error. Now it's gone. Bit rotting and healing?

closing but I'll keep an eye open


DateTimeEnvironment impl for VerificationEnvironment is too restrictive

The following impl block breaks with custom loader types:
https://github.com/spruceid/ssi/blob/2446a459876ed673db9f2a0239d807f2e757b60a/crates/claims/core/src/verification/claims.rs#L145-L149
All other impl blocks correctly assume that loaders can be customized:
https://github.com/spruceid/ssi/blob/2446a459876ed673db9f2a0239d807f2e757b60a/crates/claims/core/src/verification/claims.rs#L151-L173

Good catch, type parameters are missing in the DateTimeEnvironment implementation.


Peak time calculation not right in TDEM_Planewave_Wholespace app

Hi,
I was using the app TDEM_Planewave_Wholespace.ipynb to explore the peak time. I set conductivity to 1 S/m. And adjust the time to see when the peak reaches the depth of 400 m.

The app shows that it takes about 0.1 s for the peak to arrive at 400 m depth.
I also used the equation $t_{max} = \frac{\mu\sigma z^2}{6}$ to calculate the peak time, which is $t_{max} = 4 \pi 10^{-7} X 1 X400^2/6=0.0335 s, which is quite differnt from what the app shows.
Could anyone please explain what was going on here? Thanks!

I thinks here the concept of peak time is bit mixed up.
To figure out tmax we need to plot Ex as a function of time.  Peak shown in the profile  does not necessarily correspond to the peak amplitude at -400 m. Does this making sense?


Some questions about GLUE evaluation process

Hi, I finally generated the model of glue, but met a new problem:
In this code:

I meet this problem:

I think it may be caused by the sparse data type of counts data:

Could you please help me? Thanks a lot.

Is this the same problem as #27? Could you check the scanpy version?

Oh, yes, it is pretty strange because I passed the previous check. I will try again later. Thanks a lot.


getPatternLength typo & fixed allocate memory for array issue

Codecov Report

Merging #62 (47e46ca) into master (0dcc25a) will increase coverage by 4.36%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
+ Coverage   75.03%   79.40%   +4.36%
==========================================
  Files          17       17
  Lines         705      704       -1
  Branches       14       14
==========================================
+ Hits          529      559      +30
+ Misses        163      133      -30
+ Partials       13       12       -1

Impacted Files
Coverage Δ

packages/epanet-js/src/Project/Project.ts
96.98% <100.00%> (+3.57%)
:arrow_up:

...ges/epanet-js/src/Project/functions/TimePattern.ts
74.28% <100.00%> (+37.14%)
:arrow_up:

...ges/epanet-js/src/Project/functions/NetworkLink.ts
80.88% <0.00%> (+17.64%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


⚠ Jenkins Plugins Artifactory has degraded performance

In c8fdd93, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1395 ms

Resolved: Jenkins Plugins Artifactory performance has improved in ed81d5a after 3 hours, 55 minutes.


Toast-Notifications do not appear while chrome is on full screen mode.

Is this a limitation of Windows Do Not Disturb mode that doesn't allow toasts to be displayed in fullscreen mode?


[FEATURE]: [PowerTranz] Use connector_request_reference_id as reference to the connector

:memo: Feature Description

We transmit our reference id from Hyperswitch to the connector, enabling merchants to subsequently locate the payment within the connector dashboard.
Presently, there is a lack of uniformity in this process, as some implementations employ payment_id while others utilize attempt_id.
This inconsistency arises from the fact that payment_id and attempt_id correspond to two distinct entities within our system.

:hammer: Possible Implementation

To mitigate this confusion, we've introduced a new field called connector_request_reference_id in our RouterData.
This id should replace them as a reference id to the connector for payment request.
One might need to have exposure to api docs of the connector for which it is being implemented.
You can check this PR for further reference https://github.com/juspay/hyperswitch/pull/2052

:bookmark: Note : All the changes needed should be contained within hyperswitch/crates/router/src/connector/
:package: Have you spent some time checking if this feature request has been raised before?

[X] I checked and didn't find a similar issue

:package: Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

:sparkles: Are you willing to submit a PR?

I'm happy to work on this issue, so kindly assign me this issue.
Thanks

Hey @tarunsinghofficial, since you're already assigned to #2332, we'll leave this open to other contributors for now. You may pick this up once your work on #2332 is completed, and if no other contributors have expressed interest on this. I hope that's fine by you.

Can I be assigned? I have submitted a PR to a different issue, so meanwhile it gets merged or rejected, let me work on this.

hey @SanchithHegde, can i work on this issue? didn't get assigned any issue in this repository yet, would get to work asap.

Hello @Himanshu-370 , Thanks for the submission. I have assigned this issue to you

Hello @Himanshu-370 , It seems like this issue have been resolved by PR #2413 . Im closing this issue and if you would like I can assign you other issue of your preference


Publish contributor guide

Publish contributor guide for maintainers of oah-bes-vm, oah-bes-vm installer, oah-installer & oah-shell.

@panickervinod @harimohanr
https://github.com/Be-Secure/oah-bes-vm/pull/35 Approved by @asa1997 @sudhirverma .


:sparkles: debug rate-limiting

What this PR does / why we need it:
Debug rate-limiting issue.
If we provision several baremetal server and the ssh-key already exists, rate-limit gets reached.

We have HCloud twice now in hcloudutil.HandleHcloudRateLimitExceeded I don't think we should do this change here

ok, I reverted this change.


fix(build): use browserslist config on all chunks

The previous behavior that set browserslit config from babel.config.js
did not work on files not provided in repo (e.g. webpack bookstrap).
The new behavior copy the appropriate .browserslistrc file to the repo
before build.
Fixes IE 11 compatibility which was broken since https://github.com/iamstevendao/vue-tel-input/pull/264, release in v5.4.0.
see dist diff (npm diff dist/vue-tel-input.umd.js) :
diff --git a/dist/vue-tel-input.umd.js b/dist/vue-tel-input.umd.js
index v5.10.0..v5.10.0 100644
--- a/dist/vue-tel-input.umd.js
+++ b/dist/vue-tel-input.umd.js
@@ -8,7 +8,7 @@
        else
                root["vue-tel-input"] = factory();
 })((typeof self !== 'undefined' ? self : this), function() {
-return /******/ (() => { // webpackBootstrap
+return /******/ (function() { // webpackBootstrap
 /******/       var __webpack_modules__ = ({

 /***/ 7679:
@@ -97,7 +97,7 @@
 /***/ }),

 /***/ 3099:
-/***/ ((module) => {
+/***/ (function(module) {

IE does not supports arrow functions so they should not be in the dist file.
This fixes also removes unused polyfill from webpack internals inside the modern build (npm diff dist-modern/vue-tel-input.umd.js)

@gnuletik Thanks for your help!!!


PATH_TO_WEIGHTS

Hello，where is the file "weights=PATH_TO_WEIGHTS", I can not find it. Kind regard!

It is the path to the pretrained weight. You can download the provided weights or adopt the trained weight.

Where can I download the weight file? I prefer to use training weights, but I'm not very clear about the composition of the "path_to_weights" file mentioned in your code. I hope you can say it in detail. Thank you very much! Kind regard!

The download instruction can be found in readme, you can download through google drive or baiduyun.

Ok, I will try it now. Thanks again for your reply patiently.

Hello, I have trained the model on SHREC'17, I have the epoch_model.pt file. I just don' know how to extract the weight form the file.

I built this folder and put the model parameter files in it, but it still doesn't seem to work.

Kind regard!

You can replace PATH_TO_WEIGHTS to the path tho the epoch_model.pt
e.g. python main.py --phase=test --weights=./epoch200_model.pt
Relevant code can be found here

I have solve this problem by modify pointlstm.yaml file, thanks, my friend! I love you !

Hi, I have solved this problem by modifying the file "pointlstm.yaml". Love you, my friend, Kind regard!

how?i encounter the same problem!Kind regard

@zeng947
The 'load weights' option is used to load pretrained model for inference or debug, you can change the weight path by change the config file or through the comand. PATH_TO_WEIGHTS means the path to the pretrained weights, rather than the string "PATH_TO_WEIGHTS".

@zeng947 The 'load weights' option is used to load pretrained model for inference or debug, you can change the weight path by change the config file or through the comand. PATH_TO_WEIGHTS means the path to the pretrained weights, rather than the string "PATH_TO_WEIGHTS".

thanks!i have solved it!


fixed spelling in validation error

Description of changes
fixed spelling error in signup validation error from "incude" to "include"
Issue Resolved

Fixes #

Good catch!


Argument of type 'QueueingSubject<{}>' is not assignable to parameter of type 'Observable'

With rxjs-websockets 1.1.0, queueing-subject 0.1.1 & typescript 2.4.2 I have the following code:
import Rx from "rxjs/Rx";
import { QueueingSubject } from "queueing-subject";
import websocketConnect, {Connection} from "rxjs-websockets";

const queuingSubject = new QueueingSubject();

// ...

const connection: Connection =  websocketConnect(
    myUrl,
    queuingSubject,
    myWebSocket
);

After upgrading from rxjs 5.4.2 to 5.4.3 I get the following type error in the webSocketConnect call:
error TS2345: Argument of type 'QueueingSubject<{}>' is not assignable to parameter of type 'Observable<any>'.
Property 'source' is protected but type 'Observable<T>' is not a class derived from 'Observable<T>'.

I cleaned the cache (I'm using yarn), deleted node_modules and reinstalled but still get this error....

You need to add a type parameter to the queueing subject.

@ohjames I experimented with various types, e.g. QueueingSubject<any>() but couldn't get rid of the error.  However, both QueuingSubject and QueueingSubject<any> are working with Typescript 2.5.0.

it works fine for me with ts 2.4.2 and  on the subject.

@ohjames is that with yarn or npm?

Why would that make a difference?

Because this isn't the only problem I had, and several others seem related to either having multiple versions of something after upgrade, or else transitive dependency resolution confusion.

suspicious face

@mikebridge @ohjames
I'm having a pretty similar problem - did you ever get this sorted?

Shouldn't this line also allow for the QueueingSubject type?
https://github.com/ohjames/rxjs-websockets/blob/master/src/index.ts#L25

@ohjames  I never figured out whether it was a yarn issue or an issue with the packages themselves, but removing everything in node_modules and re-installing fixed it for me.

Hmm no joy for me. I've tried a fresh node_modules install with both npm and yarn`.
Even using as it doesn't like it:

It's pretty weird because even if I use a real Observable<string> it still complains!

Ok, I'm using this workaround: https://stackoverflow.com/a/41661734 (the cache clean approach didn't work)
But now I'm seeing strange errors like Property 'share' does not exist on type Observable<string> when I try to share the messages observable. Although it builds ok.
Something is definitely up with this package!

@danielcrisp Something isn't up with the package, that's a rudimentary error with rxjs module system that has been falsely reported by two other people already now. You need to:
import 'rxjs/add/operator/share to get share().
As for the previous issue, this is because you have two copies of rxjs (and thus two different Observable types). Again a packaging error unrelated to rxjs-websockets. Make sure that all of your rxjs dependencies are of an exact identical version so that they can be deduplicated into a single package.

For the lacking share() operator check out the rxjs docs: https://github.com/Reactive-Extensions/RxJS/tree/master/src/modular (ctrl+F for "Batteries Included").
For the previous issue, you need to understand npm better.

I'm starting to suspect I need to make rxjs a peer and dev dependency... I've gotten really bad advice on peer dependencies in the past from people which it looks like I shouldn't have trusted.

Relying on npm dependency duplication as I had advised isn't a good idea at all... in a previous package I was told by someone I was handling dependencies wrong and they misinformed me and I never bothered to verify. I've taken the time to understand peer dependencies fully now and can see it is the right choice for this project.
Released version 4.0.0 with the fix, sorry for the confusion!

Great! Thanks James


How to manipulate attribute value

Prerequisites
Following the 5 minutes quick  start

Generate the export by running the mentioned command in docs.

Versions

PHP version: 7.1.16
Laravel version: 5.7.28
Package version: 3.1.11

Description
How to add column on runtime. Means Models has not the column and value is generated on the basis of modal fields.
Code
<?php namespace App\Exports;

use App\User;
use Maatwebsite\Excel\Concerns\FromCollection;
use Maatwebsite\Excel\Concerns\ShouldAutoSize;
use Maatwebsite\Excel\Concerns\WithHeadings;

class UsersExport implements FromCollection, ShouldAutoSize, WithHeadings {

	public function headings (): array {
		return [
			'First Name',
			'Last Name',
			'Full Name',
			'Email',
			'Product Update',
			'Questionnaire',
			'Create at',
			'Updated at',
		];
	}

	public function collection () {
		return User::all( ['first_name',
					'last_name',
					'name',
					'email',
					'is_news_letter',
					**'Here I want to add column on the basis of some conditions',**
					'created_at',
					'updated_at',
					] );
	}
}

I'd be advised to read further than only the quick start guide. The package contains a lot more features than that are used in the quick start guide. You can find the mapping logic here: https://docs.laravel-excel.com/3.1/exports/mapping.html


Bump miner commit ID for 2<IP_ADDRESS>_GA

Depends on https://github.com/NebraLtd/hm-miner/pull/7

Missing one instance of FIRMWARE_VERSION


Upgrade to the latest Solidity

Ref https://github.com/foundry-rs/foundry/issues/4916

Done: https://github.com/PaulRBerg/foundry-template/commit/fcc743888370871c06a9691f3b27f9735858fff0

Rationale for upgrading:
https://twitter.com/PaulRBerg/status/1737169159052468479


[Enhancement]: Move examples from JUnit 4 to JUnit 5

Made changes as per the previous PR
https://github.com/testcontainers/testcontainers-java/pull/6950#partial-pull-merging
Related Issue is at https://github.com/testcontainers/testcontainers-java/issues/6921

Author will not be able to continue the work. See comment


Getting CognitoID when using User Pool

So following the CognitoYourUserPoolsSample I read the docs to combine this into the cognito identity to get the cognito identity out.
I find that in the samples refresh method I can do
        let pool = AWSCognitoIdentityUserPool(forKey: "UserPool")
        let credentialsProvider = AWSCognitoCredentialsProvider(regionType: .usEast1, identityPoolId: Constants.identityPoolId, identityProviderManager:pool)
        credentialsProvider.getIdentityId().continue ({ (task) -> Any? in
            if let result = task.result {
                print("coginto id \(result)")
            }
            else {
                print(task.error?.localizedDescription)
            }

            return nil
        })

This will then trigger the AppDelegates
startPasswordAuthentication() -> AWSCognitoIdentityPasswordAuthentication
And the callback comes back to the refresh method with the cognitoID once I logged in.
However, after doing this, the self.pool?.currentUser().isSignedIn is still set to false?
And if I instead try and do the way the sample code is doing it
        self.user?.getDetails().continue(successBlock: { (task) -> Any? in
            DispatchQueue.main.async {
                self.response = task.result
                self.usernameLabel.text = self.user?.username
            }
            return nil
        })

The credentialsProvider.getIdentityId() won't work.
Question is, how do I retrieve the cognitoID with the sample code?

If signedIn is false, where are you executing that line?  It is likely that you are executing that line before you are actually signed in.  If you put self.pool?.currentUser().isSignedIn after setting the self.usernameLabel.text = self.user?.username in the refresh method, it should be YES.
"And if I instead try and do the way the sample code is doing it."  To be clear, the sample code only makes service calls to Cognito Your User Pools. Everything you see in getDetails comes from Cognito Your User Pools.  Identity ids come from Cognito Federated Identity.  If you are trying to obtain an identity id using the sample, you need to do something like you mention, it will not make any calls to Cognito Federate Identity on it's own.

@behrooziAWS ok works now when I test on the device rather than the simulator. The problem I was seeing is that on the simulator the sign in doesn't work, although it think it does. So when I then called credentialsProvider.getIdentityId() it was calling the startPasswordAuthentication() again.
Is it normal that it won't work in the simulator? I'm running Xcode-beta with iOS10 and Swift 3, I guess that might be the problem.
In any case, works on the device now. Thanks.

@behrooziAWS So after successfully login and then getting the cognitoIdentity id back with getIdentityId() I then attempt to load a DyanmoDB object with the AWSDynamoDBObjectMapper but it then says

{"__type":"com.amazon.coral.service#MissingAuthenticationTokenException","message":"Request is missing Authentication Token"}

I have set up roles to allow the Cognito Auth role to access the table.
What am I missing? Do I need to explicitly say what the auth role is?

I have not had issues running in the simulator with Xcode 7.3 and iOS 9.3.  I have not tried with the beta and iOS 10 yet.  There is no synchronization logic in the user pools SDK to only run one authentication ui at a time.  When I was testing the sample earlier, it doesn't gracefully handle displaying the authentication ui while it is already showing and will crash. If you are explicitly calling getIdentityId make sure it is done in a place where the app already has a valid user pools session, like in the success block of self.user?.getDetails().
You do not need to specify the auth role other than in your identity pool configuration in the AWS console.  How are you configuring the credentials provider for your AWSDynamoDBObjectMapper?  Are you setting the default AWSServiceConfiguration to the credentials provider that has your user pool object as the identityProviderManager?
AWSCognitoCredentialsProvider *credentialsProvider = [[AWSCognitoCredentialsProvider alloc] initWithRegionType:AWSRegionUSEast1 identityPoolId:@"IDENTITY_POOL_ID" identityProviderManager:pool];

AWSServiceConfiguration *configuration = [[AWSServiceConfiguration alloc] initWithRegion:AWSRegionUSEast1 credentialsProvider:credentialsProvider];

AWSServiceManager.defaultServiceManager.defaultServiceConfiguration = configuration;

@behrooziAWS Thanks, got it working now, seems it wasn't setting the configuration properly after auth.


Support for API keys

It looks like there's no built-in way to generate the aws_apigatewayv2_route with api_key_required set to true.
I believe the work around would be to create the routes and integrations outside of the module, which reintroduces a bit of the boilerplate that using this module does away with.

This module is in the early alpha version (v0.x.x), many things are missing and will be added when time allows it.
Meanwhile, your workaround sounds like a valid solution.

Closing this issue as it's been added here


Variables and their requires_grad flags

Why all of your variables here and here's requires_grad argumand set to False ? You set requires_grad parameter of coord_tensor variable to false and then you concatenate it with the output of cnn (defined as x),whose requires_grad arguman is  true by default I guess,  at line 78. In this case what is the requires_grad parameter of concatenated variable (output of line 78 which is also x_flat)?

Sorry, currently I cannot answer this question now because I am under military training. I promise to answer it 3 weeks later. Please wait until then

I couldn't find any requires_grad parameters set to be true or false. As far as I know, setting required_grad to be false is only required when you want to fix certain layer. RN is end-to-end learning system, so I suspect I didn't need to set any reuiqres_grad to be false. If I am wrong, please tell me.


Enable syntax highlighting in README

Great library ! 🌟
Just adding some color here 😅

Cool ! Thanks for this, learned something new


Can't download php runtime

Issue: After I started the setup-file, I got a error message, that the setup can't download the php runtime.
What I tried: Installed PHP manually but the program doesn't open (correct path proofed with "php -v" in CMD)

Should be fixed for now,  I didn't realize they're deleting the old releases as new ones get created. I'll host the file somewhere myself so it doesn't happen again.

Hmm, perhaps your country is one of those that blocks https://catbox.moe which I use as a fallback URL in case the official link gets dead (which happens from time to time).
Do you know of any file sharing server which lets you directly download files and is not blocked in your country?
I've released 1.6.7 which has a working primary link now, but it's likely it's gonna happen again in the future.

I've released v1.6.8 which contains another fallback URL that will be used as a last resort, this time to my custom server.


[Service Bus] Create required resources for running the browser tests.

Currently, the live headless browser tests in the CI for service-bus rely on the left over resources from live node tests.
cc: @mikeharder @ramya0820 @chradek

This is because we rely on @azure/arm-servicebus (which supports only AAD auth and not connection string) to delete/create the Azure resources in Service Bus
As per the latest investigations from @chradek, interactive login is the only way to get AAD auth in browser. Since we cant have interactive login in our tests, we are left with the below options:

Add azure-sb as a dev dependency and use it to delete/create resources

Pro: Uses connection string, we can stop using AAD in tests
Con: This is the very library that our new service bus library claims to replace. So, if our users find this in our dependency, it might send mixed signals

Wait until https://github.com/Azure/azure-sdk-for-js/issues/1057 is implemented,

Pro: This allows us to delete/create entities using our own library that is being tested
Con: Might take a while

Have a nodejs script to delete/create entities before running the entire karma suite. Add a suffix to the names of the entities to indicate that these are meant for browser tests

Pro: Quickest solution. Karma tests will no longer use resources that are being used by Nodejs.
Con: Entites will not be recreated before each test, but I believe we can live with this

For the short term, I would suggest to follow the 3rd approach.
Currently, our nightly test runs have been updated to run the browser tests only if the node tests pass to ensure that there is less chances of failed node tests resulting in missing resources for browser tests. Since the node tests themselves arent passing all the time, this results in getting very less coverage for browser tests.
Poposed solution:

Update tests so that the browser tests use a separate set of Azure queues, topics and subscriptions for Browser tests. Maybe add a suffix _browser to the default names that are being used.
Have a nodejs script that will delete and then create the above resources using @azure/arm-servicebus
This script should be run before the browser tests are run
Update pipeline to run the node and browser tests in parallel as they don't depend on each other anymore

Currently, our nightly test runs have been updated to run the browser tests only if the node tests pass to ensure that there is less chances of failed node tests resulting in missing resources for browser tests. Since the node tests themselves arent passing all the time, this results in getting very less coverage for browser tests.
Proposed solution:

Update tests so that the browser tests use a separate set of Azure queues, topics and subscriptions for Browser tests. Maybe add a suffix _browser to the default names that are being used.
Have a nodejs script that will delete and then create the above resources using @azure/arm-servicebus
This script should be run before the browser tests are run
Update pipeline to run the node and browser tests in parallel as they don't depend on each other anymore

@ramya-rao-a @AlexGhiondea
About the requirements on having separate entities with _browser prefix, did we mean only on SERVICE_BUS_CONNECTION_STRING mainly? Or did we want to keep that same and use different queues, topics and subscriptions?
cc @KarishmaGhiya @mikeharder

Same Service Bus account, different queues/topics/subscriptions

Also added README for users benefit as it is unclear as to when will the AAD changes come in, and if we really want to do away with parallelism of running tests (given the benefits we get in saving on time for the builds).

Since we have nightly CI test runs, running the browser tests, this issue is not really blocked by https://github.com/Azure/azure-sdk-for-js/issues/3799. Please remove the dependency

Yes, the reason I had it in as a dependency was that I was unable to run browser tests on local for this issue due to the rollup bits failing (in that particular workspace). Now that it's resolved and just seems to be one off, brittle issue.. we can consider it closed.

Completed with #3785


[Bug]: Slack intergration failing with version 4

Steps to Reproduce
I am using the --slack switch along with the 2 x SLACK environment variables to send a message to our slack channel. This is working fine in Prowler 3.14.0, however with the latest version 4.0.1 we do not recieve any messages in slack
Expected behavior
Recieve slack messages in existing slack channel
Actual Result with Screenshots or Logs
{"timestamp": "2024-04-15 12:25:02,549", "filename": "slack.py:55", "level": "ERROR", "module": "slack", "message": "AttributeError[41]: 'str' object has no attribute 'type'"}
{"timestamp": "2024-04-15 12:25:02,549", "filename": "slack.py:21", "level": "ERROR", "module": "slack", "message": "TypeError[12]: cannot unpack non-iterable NoneType object"}
How did you install Prowler?
From pip package (pip install prowler)
Environment Resource

Fargate Task
WSL 2 Terminal

OS used

Alpine
Ubuntu

Prowler version
4.0.1
Pip version
3.11
Context
No response

I am Reviewing to see if I can fix it

Hi @garym-krrv, could you try it with this branch 3769-bug-slack-integration-failing-with-version-4? I know you were trying to fix it but I realised once I saw the issue.
Thanks for using Prowler 🚀

Hey @jfagoagas thanks for looking.. Certainly.. Im on it now...

Hi @jfagoagas , I can confirm that the slack intergration works on the branch 3769-bug-slack-integration-failing-with-version-4
The messages are recieved in the slack channel.


Add extra info on oref0-setup with xdrip-js Logger

Perhaps this isn't the best way of flagging this, but I think it's important to make people aware about the separate Logger config.

This is probably unnecessary due to https://github.com/openaps/oref0/issues/1397#issuecomment-1369437755


Remove Cooler Activity Status

Awesome theme! One of my favorites! But is there a way to remove the Cooler Activity Status and just have it as default?
Thanks!

i'll add that as optional addons on 2.0

Okay cool. I didn't know there was a 2.0 coming out.


log-search: Match against path without query parameters.

The nginx logs have query parameters; display them, but do not use them for URL matching.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @alexmv!


Blank Column

Checklist

[x] I updated the card to the latest version available
[x] I cleared the cache of my browser
[x] I verified that I'm really running the lastest version in my browser console
[x] I checked if there is another issue opened with the same problem (i think)

Describe the bug
Since the last update all my graphs with "column" are not displaying any column and if i don't add "datalabels" the card will be kind of blank
Version of the card
Version: 1.8.1
To Reproduce
This is the configuration I used:
      - type: 'custom:apexcharts-card'
        graph_span: 7d
        span:
          end: day
        series:
          - color: 'rgb(235,10,104)'
            entity: sensor.tplink_1_today_energy_kwh
            type: column
            group_by:
              func: max
              duration: 1d
            show:
              datalabels: true

Screenshots

Expected behavior
Visible column.
Desktop (please complete the following information):

Browser [chrome]
Version [88.0.4324.192 (Official Build) (x86_64)]

Smartphone (please complete the following information):

Device: [Android]
Browser [mobile app]
Version [last]

Additional context
I have tried several graph config with "columns" i've found on the forum and this repo with no success at all. I have tried to delete the card, reboot, install the card back, refresh cache, etc, etc..

Thanks for your help.

I can't reproduce so far.
Multiple questions:

Can you confirm the javascript console show the right version of apexcharts-card (ie. 1.8.1)
Do you have any errors related to apexcharts-card in the javascript console?
Is it working with line or area?
Do you have a theme applied to your Home-Assistant instance
What happens if you remove the color parameter?

Thank you for your help RomRider. It's really a weird issue.

I can confirm i'm using the correct card version.

I do receive errors in the log, but only for this card i shared here (with a reduced config).

I did have a custom theme applied, i disabled it but nothing have changed.
I had already removed the color parameter during my attempts to solve this issue by my own, with no success.

Sorry to bother you with this...

Same problem here with the card. worked fine in column mode now only datalabels.

my configuration
  - type: 'custom:apexcharts-card'     experimental:       color_threshold: true     graph_span: 20d     all_series_config:       opacity: 1       type: column     span:       end: day       offset: '-1d'     header:       show: false     series:       - entity: sensor.dagelijks_verbruik_lastperiod1         group_by:           func: max           duration: 1d         offset: +1d         color_threshold:           - value: 0           - value: 12             color: '#FF6600'           - value: 18             color: red     apex_config:       plotOptions:         bar:           horizontal: false           dataLabels:             position: bottom             orientation: vertical       dataLabels:         enabled: true         enabledOnSeries: false         style:           colors:             - '#fff'           fontSize: 12px         background:           enabled: false       chart:         height: 180px       xaxis:         labels:           show: false       yaxis:         show: true         opposite: true         labels:           align: left         tooltip:           enabled: true       tooltip:         enabled: true         fixed:           enabled: true           position: topLeft           offsetY: 10           offsetX: 10         x:           show: false           format: MMM d         marker:           show: false

ok so just now i noticed that the graphs still worked on an other instance op HA but that one was not yet updated to core-2021.3.1 but i did already update the apexchart-card to 1.8.1
now after updating HA the graphs stopped working there to so looks like it is a problem coming from home assistant?

Interesting finding @jostie123, thanks for sharing. At least i know i'm not alone with this weird bug.

Seems like you are using card-mod, layout-card and stack-in-card. Would you mind just testing in a simple view with just the graph, then adding each addon until it breaks and report back?
I believe card-mod is the culprit but I'm not yet sure.

Thank you for your help RomRider, the columns now appears on a simple view when i disable layout-card:

Great, thanks! I've managed to reproduce with layout-card, thanks. I'll have a look at it, not sure yet if it can be fixed on my side or if it's on layout-card's side.

So I don't know what happens with layout-card but it refreshes the card 5 times every time it needs a refresh... So I believe the issue is on layout-card's side...

Thanks for the feedback RomRider, i will try t get rid of it aswell.

Seems like there are already some reports here: https://github.com/thomasloven/lovelace-layout-card/issues/99

Problem solved when disabling layout-card:

Many thanks to you @RomRider! I enjoy your work and your commitment for the community. Cheers!

I've managed to fix it in the end, so you can re-integrate layout-card ;)

I think i can live without it, my ui display layout survived and the loadings seems a little faster.
All Good for me, thanks for your support Rom!


LSC: Extract language server on first use

Change the build to include the LS runtime archive in the VSIX rather than extracted version, and extract on first use. Compare the version so we replace if we have a newer version bundled than what is currently extracted.

Done, but does not compare the version. We'll see what we decide on next week and create work item for that.


update flash to 1.0.1

Coverage remained the same at 94.203% when pulling 013d9ed6422dd72af7d24fcbd344ab262a42cc62 on update-vendor into f2d90d9887b94533980a27b0ae1e3566d2d882c2 on master.


Default Route (All-Traffic (0/0)) not propagating to respective attached subnet's route table

Hi,
I have setup the hub and spoke successfully.  However when attaching an existing subnet of an associated VPC to the transit gateway (using the Attach-to-tgw tag) , the route table of that subnet is not updated to point all traffic to the TGW (as set in the parameters on the hub stack creation i.e. All-traffic (0/0)
Am I missing something here?

Would appreciate if something can assist with this please.

@mariomic-maker Thanks for reaching out to us. Can you please confirm that the subnet you tagged didn't already have an existing default route in the the associated route table. By design, the solution does no overwrite existing default routes.

Closing issue. Please confirm that the subnet you tagged didn't already have an existing default route in the the associated route table, and re-open if the issue persists.


Dragons Dogma 2 - Ray Tracing completely ruins graphics (and can never be turned off again!)

 It can not be turned off ever again. Disabling them doe snot turn it off.
Please help removing this. Or I can never play the game again! :(
Upload logs and any crash dumps
No logs, no crash dumb, the game is just ugly af now and I can do nothing about it. It's completely ruind.
Identify the REFramework version
https://www.nexusmods.com/dragonsdogma2/mods/8
To Reproduce
Steps to reproduce the behavior:

Go to 'Graohics'
Click on 'Enable Ray Tracing Tweaks'
Click on 'Any Ray Trace Type from the dropdown'
Can never be turned off again (Disabling Ray Tracing Tweaks and changing the Type to Disabled DOES NOT disable them!!! It only says it's disabled but it IS NOT)

Expected behavior
That the graphics resets to normal graphics without the ugly (fake) ray tracing enabled!
 This ruins the game completely!

You can turn the option off entirely and close the menu, restart the game. It will not turn RT on afterwards.

Fixed in e55866daf4f785d2cf28a901631ff5bd5acc6fb5

You can turn the option off entirely and close the menu, restart the game. It will not turn RT on afterwards.
It stayed one time. But I disabled everything and removed the generated settings file from the folders. Now it's default again. Great you found the issue allready.


GPII-4243:  Reverting to previous version of universal container image.

This pull reverts the universal container version changed in the previous deployment pull.

In previous chats, we discussed leaving staging how it is so that the devs can see the error Kavya pointed out, then reverting.  We would then test a follow-up fixed version in a dev cloud before submitting a new deployment pull.

LGTM


oxml.section.sections cannot get the header or footer

'CT_SectPr' object has no attribute 'sectPr_lst'？

Please post support questions on StackOverflow using the python-pptx tag. Put more effort into forming a clear and complete question than you expect us to put in to answer it. This question is neither clear nor complete.


Add @offsetOf built-in function

This adds a function @offsetOf(comptime StructType: type, comptime field_name: []const u8) -> num_lit_int that returns the offset in bytes of a field from the beginning of its containing struct.

The diff looks great. Couple more things if you're up for it:

want to add the doc/langref.md entry for the new builtin function?
can you make all these commits into 1?

Will do =) I'll fix it up tomorrow


Validator should allow scalar int/float texel types for OpImageWrite

https://github.com/KhronosGroup/SPIRV-Tools/blob/e1a6f8d1c3ec4f814d6160ab46b8f397f89f8184/source/validate_image.cpp#L1072-L1077

Fixed by https://github.com/KhronosGroup/SPIRV-Tools/pull/1014


Retrieve Attendance of an event

https://www.meetup.com/meetup_api/docs/:urlname/events/:id/attendance/#list

@lazywinadmin
Hi, I am new to GitHub, and try to learn things. Sometimes I use your work and scripts, and would like to say Big Thank you for what you do! I am an admin who is trying to keep up with PSH and apply learned skills, and it seems you have a very solid experience in coding.
Sorry if put this to the wrong  issue.
THANK YOU!

Thanks @Dkerget ! 😄
Not the right place but I really appreciate your message 👍
I'm still learning too! it's an endless journey ! The more you know... the less you know
Keep learning and don't hesitate to try stuff and ask questions


Fix .equalTo, .endAt, and .startAt filters when used without key.

When used without key they contained a spurious comma, since
['equalTo', ['a', undefined]].join(':') is 'equalTo:a,' (with comma at the end) and not  'equalTo:a' which would be the result of ['equalTo', ['a']].join(':').

BTW: Possibility to do so is in the official documentation e.g.: https://firebase.google.com/docs/reference/js/firebase.database.Reference#equalTo

I think this may have been fixed in the v3 branch, could you have a look? If not could you do a PR to my fork and I'll merge it into v3

I can't test it right now, but looking at the code it does seem to be fixed there. When will you release v3 to the NPM repo?

Hopefully have an alpha version out soon

RC1 out on npm, can confirm this is working. Closing


Significant performance loss after adding support for abbreviations

Looks like 26e0202 made the regexes less performant.
10kb file is enough to see it slowly coloring the syntax as the file is opened.
Try if this works better
char|chara|charac|charact|characte|character

then the current solution
char(a(c(t(e(r)?)?)?)?)?

The regex evaluation using
char|chara|charac|charact|characte|character

instead of
char(a(c(t(e(r)?)?)?)?)?

is five times faster given the present amount of keywords. It took me 25s to color the entire procedure before and 5s after. Tested on 50+ kb procedure.


Updating status of a couple of projects

Comments

Nice work @jfcottrell on sending a PR back for the Beginning Git course. I see that you didn't quite complete all the challenges—was everything OK?
I hope you found the course fun and informative—if you weren't already using git for your code, hopefully this has inspired you to do so. Given that we've now launched the next series "Mastering Git", you'll be able to get started straight away!
You can now wear this #GitWarrior badge with pride—and Tweet your historic victory!

sam


How many channels does this support?

In the MIDI spec, a channel MIDI message (e.g. note on, note off, program changed, etc.) has a channel represented by 4 bits (i.e. 0-15). However, that's only a limitation of the MIDI messages. Since TSF uses functions as its interface, not MIDI messages, does it still have this restriction of only 16 channels? (Having searched the code for the number 16, I think the answer is "no", which is good, but it's safer to check with you!)
The reason I ask is because we're looking at trying to support >16 channels in our MIDI player. As far as I can tell, normally these are encoded by choosing which MIDI device to send the event to, with the assumption that each MIDI device handles just 16 channels. So channels 0-15 go to MIDI device 0, 16-31 go to MIDI device 1, 32-47 go to MIDI device 2, etc. You would configure this on your MIDI software. To record this concept in a Standard MIDI files, this is usually done by telling each track which MIDI device to use with a 0x21 (port name) or 0x9 (device name) meta-event.
Taken literally, this would mean spinning up multiple TSF instances, since TSF takes the role of an output MIDI device. However, if TSF can handle more than 16 channels, it would be far easier just to have a shim layer in front of TSF that simply receives a MIDI event for MIDI device D for channel C, and maps that to channel 16*D+C on TSF. This way we need just one instance of TSF, one audio stream connecting TSF to an audio device, etc.

A soundfont synthesizer is usually limited by voices not channels. Because a single SF2 instrument can be built out of multiple voices that play simultaneously that is what actually matters for the mixing, not the meta layer that is MIDI.
TSF by default allocates more voices when needed when new notes are requested to start. If desired, a limit can be set with tsf_set_max_voices (which can also help with multithreading concurrency, see #49).
Now TSF has 2 interfaces, one is to just play raw SF2 presets with tsf_note_on. The other is playing MIDI messages with the tsf_channel_* functions. The tsf_note_on is lower level and directly starts up voices. The tsf_channel_* functions take a channel number and it will (re-)allocate an array up to whatever number is requested. So if you were to call tsf_channel_set_presetindex(f, 5000, preset_index); it would just allocate an array for managing 5000 MIDI channels and that's it. So that also has no limit.
In short, you should be fine with just one instance :-)


Canvas support for nk_love_image

Added support for Canvas objects in nk_love_checkImage - allows use of Canvas objects where images are expected.

Great work, thanks for the PR.


Timeout when adding new node in flow

matheusr@vali:~$ curl -X POST -H "Authorization: Bearer ${JWT}" http://<IP_ADDRESS>:8000/flows/v1/node -H "content-type: application/json" -d '{"image": "raulnegreiros/mysql-insert","id":"mysql-insert"}'
The upstream server is timing out

log flowbroker:

[harbor-master][modem] error parsing body as json
[nodes] image raulnegreiros/mysql-insert created
[nodes] container flowbroker.3cbb345 was created
Target: 1239534d0078
[nodes] container up: flowbroker.3cbb345:1239534d0078c9f2118ac034a07b266befa4fdd2b4b69e9607c479f072be61b9

The error occurred because there was another docker running:

matheusr@vali:/opt/teste/current/docker-compose$ sudo docker network ls
[sudo] password for matheusr:
NETWORK ID          NAME                                        DRIVER              SCOPE
b386e3a6aa05        bridge                                        bridge              local
56ea55e3b105        docker-compose_default           bridge              local
fe36c3da2f8a          docker-compose_flowbroker     bridge              local
9335e3dce2dd        dojot_default                             bridge              local
02bef01c5c63         dojot_flowbroker                       bridge              local
020f43f3cffe            host                                           host                 local
1bf04062040b        none                                          null                   local
matheusr@vali:/opt/teste/current/docker-compose$

After stopping the other docker the node was added.
log flowbroker:

[harbor-master][modem] error parsing body as json
[nodes] image mmagr/kelvin:latest created
[nodes] container flowbroker.bad26c0 was created
Target: 4eed2845a9d2
[nodes] container up: flowbroker.bad26c0:4eed2845a9d23002c835f3095da235d5e7a0212d72554e33685e501e84d0fc2e
[dispatcher] remote [4eed2845a9d2] took 6ms
[dispatcher] remote [4eed2845a9d2] took 2ms
[dispatcher] remote [4eed2845a9d2] took 1ms
[nodes] container meta {"id":"sample/kelvin","name":"kelvin","module":"kelvin","version":"0.0.0"}

I'm submitting a ...

[ ]  bug report
[ ]  feature request
[x] support  request

Do you want to request a feature or report a bug?
Suport request
What is the current behavior?

Kelvin node - Virtual Machine 1
This example "How can I add a new node type to its menu?" is not working.

Another node - Virtual Machine 2
When I try to add another node, I do not get success too.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

Kelvin node - Virtual Machine 1

Another node - Virtual Machine 2

What is the expected behavior?
It should create a Kelvin and Telegram node on "Flow Manager".
Please tell us about your environment:
Virtual Machine 1
Version: 0.2.0
Docker version: 18.06.0-ce,
Operating system:  Ubuntu 16.04.4 LTS
Virtual Machine 2
Version: 0.3.0
Docker version: 18.03.1-ce
Operating system:  Ubuntu 16.04.4 LTS
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

Hi Daniel,
is it a problem to you change to the version 0.3.1? We suggest this one, due to several corrections. I did a quick test here and it seems to be working, if you find something else please let me know.
Best regards,

Hi @raulnegreiros
I  changed to the Battojutsu.1 version. It is working for me now.
Regards!

Great!
Have fun ;)

The error occurred because there was another docker running:

matheusr@vali:/opt/teste/current/docker-compose$ sudo docker network ls
[sudo] password for matheusr:
NETWORK ID          NAME                                        DRIVER              SCOPE
b386e3a6aa05        bridge                                        bridge              local
56ea55e3b105        docker-compose_default           bridge              local
fe36c3da2f8a          docker-compose_flowbroker     bridge              local
9335e3dce2dd        dojot_default                             bridge              local
02bef01c5c63         dojot_flowbroker                       bridge              local
020f43f3cffe            host                                           host                 local
1bf04062040b        none                                          null                   local
matheusr@vali:/opt/teste/current/docker-compose$

After stopping the other docker the node was added.
log flowbroker:

[harbor-master][modem] error parsing body as json
[nodes] image mmagr/kelvin:latest created
[nodes] container flowbroker.bad26c0 was created
Target: 4eed2845a9d2
[nodes] container up: flowbroker.bad26c0:4eed2845a9d23002c835f3095da235d5e7a0212d72554e33685e501e84d0fc2e
[dispatcher] remote [4eed2845a9d2] took 6ms
[dispatcher] remote [4eed2845a9d2] took 2ms
[dispatcher] remote [4eed2845a9d2] took 1ms
[nodes] container meta {"id":"sample/kelvin","name":"kelvin","module":"kelvin","version":"0.0.0"}

Hello @betefaber
Sorry, but I did not understand what kind I should stopping to this working.
Now I am using Carate version. When I try to add Kelvin Node example, I get the same problem "The upstream server is timing out"


luci-mod-network: nitpick of CNAME, MX, SRV syntax formatting

Not strictly, but it is correct and disambiguates.
When you specify a domain name without the ".", it's considered a relative domain name and is appended with the current domain name or the local domain, depending on the context. Including the "." at the end makes it an absolute domain name, ensuring that it's explicitly pointing to the root of the DNS hierarchy.
https://en.wikipedia.org/wiki/SRV_record:

target: the canonical hostname of the machine providing the service, ending in a dot.

See also:
https://en.wikipedia.org/wiki/CNAME_record
https://en.wikipedia.org/wiki/MX_record
https://en.wikipedia.org/wiki/Root_name_server


iOS 8 rotation bug built with Xcode 5

(note: an updated pull request)
The rotation issue only happens after Xcode 6 with iOS 8.
Xcode 5 can still be used without building 64-bit code for app updates, until June 1, 2015.

I've created a MBProgressHUD_Test project to address this problem.
Just build for iOS 8 devices with Xcode 5 / 6.

I modified this code in dc534dcb31a81e952b9d29de2189af3a56da91c6. I'm more inclined to not support old developer tools in the main fork.


Jobs fire while app is booting do not have a hibernate session

Grails 3.1.1 Quartz Plugin 2.0.1-2.0.7
When a job is scheduled instantly (eg resuming from a persistent store) It will fail without a Hibernate Session. Subsequent runs do have a Hibernate session -
ERROR grails.plugins.quartz.listeners.ExceptionPrinterJobListener - Exception occurred in job: Grails Job org.quartz.JobExecutionException: org.hibernate.HibernateException: No Session found for current thread

We tried to set the loadAfter, but it did not resolve the issue. https://github.com/grails-plugins/grails-quartz/commit/0e2a909936afeab9322035e1a5da84ef3700ebfd
DEBUG quartz.QuartzGrailsPlugin - Registering hibernate SessionBinderJobListener
INFO org.grails.plugins.datasource.TomcatJDBCPoolMBeanExporter - Registering beans for JMX exposure on startup
ERROR grails.plugins.quartz.listeners.ExceptionPrinterJobListener - Exception occurred in job: Grails Job

^ exception before the app starts
Grails application running at http://localhost:8080 in environment: development
DEBUG grails.plugins.quartz.listeners.SessionBinderJobListener - Persistence session is flushed.
DEBUG grails.plugins.quartz.listeners.SessionBinderJobListener - Persistence session is opened.

^ SessionBinderJobListener working as expected after app is running
I don't see this behavior in Grails 2.5.3 with Quartz Plugin 1.0.2

resolved in 2.0.10


Sub / parent skills greyed out

As a temp user, if I visit http://dev.algorea.org/en/#/skills/by-id/3000;attempId=0;path=/details or http://dev.algorea.org/en/#/skills/by-id/3001;attempId=0;path=3000/details, the sub-/parent-skills are grey out... so with link disabled, for no reason I think.

Same problem in regular chapter children lists.


XNode#toString() cannot display all information

MyBatis version
3.5.5
java version
1.8
discription
I found that this method org.apache.ibatis.parsing.XNode#toString() has a defect, it only displays nodes of type Node.ELEMENT_NODE. The reason for this problem is that the method org.apache.ibatis.parsing.XNode#getChildren will only list the child nodes of type Node.ELEMENT_NODE. Therefore, when I am debugging, I cannot see all the information.
a xml file
<mapper namespace="demo.StudentMapper">
  <select id="selectFullStudent" resultMap="fullResult" databaseId="mysql">
    select
      STUDENT.ID ID,
      STUDENT.SNO SNO,
      STUDENT.SNAME SNAME,
      STUDENT.SSEX SSEX,
      STUDENT.SBIRTHDAY SBIRTHDAY,
      STUDENT.CLASS CLASS,
      SCORE.CNO CNO,
      SCORE.DEGREE DEGREE,
      HOUSE.HOUSE_ID HOUSE_ID,
      HOUSE.HOUSE_HOLDER HOUSE_HOLDER,
      HOUSE.HOUSE_MEMBER HOUSE_MEMBER
    from STUDENT, SCORE, HOUSE
    <where>
      <if test="sex != null">
        STUDENT.SSEX = #{sex};
      </if>
      and STUDENT.SNO = SCORE.SNO and HOUSE.HOUSE_MEMBER = STUDENT.SNO
    </where>
  </select>
</mapper>

Java code
package issue;

import org.apache.ibatis.parsing.XNode;
import org.apache.ibatis.parsing.XPathParser;

import java.io.InputStream;

public class ReproduceXnode {
    public static void main(String[] args) {
        InputStream resourceAsStream = ReproduceXnode.class.getClassLoader().getResourceAsStream("issue/StudentMapper.xml");
        XPathParser xPathParser = new XPathParser(resourceAsStream);
        XNode select = xPathParser.evalNode("mapper/select");
        System.out.println(select.toString());
    }
}

Expected result
  <select id="selectFullStudent" resultMap="fullResult" databaseId="mysql">
    select
      STUDENT.ID ID,
      STUDENT.SNO SNO,
      STUDENT.SNAME SNAME,
      STUDENT.SSEX SSEX,
      STUDENT.SBIRTHDAY SBIRTHDAY,
      STUDENT.CLASS CLASS,
      SCORE.CNO CNO,
      SCORE.DEGREE DEGREE,
      HOUSE.HOUSE_ID HOUSE_ID,
      HOUSE.HOUSE_HOLDER HOUSE_HOLDER,
      HOUSE.HOUSE_MEMBER HOUSE_MEMBER
    from STUDENT, SCORE, HOUSE
    <where>
      <if test="sex != null">
        STUDENT.SSEX = #{sex};
      </if>
      and STUDENT.SNO = SCORE.SNO and HOUSE.HOUSE_MEMBER = STUDENT.SNO
    </where>
  </select>

Actual result
<select resultMap="fullResult" databaseId="mysql" id="selectFullStudent">
    <where>
        <if test="sex != null">
        STUDENT.SSEX = #{sex};
      </if>
    </where>
</select>

I'm fixed this issue.Pr


Feature/network selection

Hey! Hoped I'd get you're thoughts on the below changes.
I didn't see a way to select a network the way it specifies in you're api docs here, so I added an optional config for it. I also updated the readme with an example but indicated it was not necessary. Let me know what you think, thanks!
Also added a public licence if that works for you guys.

Thanks for the quick review!


Add support for compressor plugins

This is a follow-up to PR #584, adding support for external compressor plugins.
The first commit in this PR will match PR #584 until that is merged.
It is equivalent to PR6717 in RocksDB.
The Compressor and related classes are made part of the public API to allow development of plugins and to expose compressors in options.
Compressor plugins can be used to easily integrate new compression algorithms into Speedb. They can also implement compression techniques tailored to specific types of data. For example, if the values in a database are of numeric type (e.g., arrays of integers) with particular distributions (e.g., limited range within each block), the values could be compressed using a lightweight compression algorithm implemented as a plugin (such as frame-of-reference or delta encoding).
Options for Compressors
New options are added to support plugin compressors.
For example, compression was previously configured by compression (of type CompressionType) and compression_opts (of type CompressionOptions). This PR adds a compressor option (pointer to Compressor) to specify a compressor object (which includes type and options).
This approach was followed for the following options:

ColumnFamilyOptions: compression (compressor), bottommost_compression (bottommost_compressor)
AdvancedColumnFamilyOptions: compression_per_level (compressor_per_level), blob_compression (blob_compressor)
CompactionOptions: compression (compressor)

The existing CompressionType/CompressionOptions options are preserved for backward compatibility.
If the user doesn't specify compressors (leaving them null), the CompressionType/CompressionOptions options are used as before. Otherwise, compressors override the existing options.
A new constant kPluginCompression is defined in CompressionType for plugin compressors. The SST properties block stores information about the specific compressor in the compression_name field. This is used to instantiate a suitable compressor when opening the SST.
Option String Examples
Built-in compressor (existing options)
compression=kZSTD;compression_opts={level=1}
Built-in compressor (new options)
compressor={id=ZSTD;level=1}
Plugin compressor (new options)
compressor={id=my_compressor;my_option1=value1;my_option2=value2}
Options Object Example
Built-in compressor (existing options)
Options options;
options.compression = kZSTD;
options.compression_opts.level = 1;

Built-in compressor (new options)
Options options;
ConfigOptions config_options;
  Status s = Compressor::CreateFromString(
      config_options,
      "id=ZSTD;level=1",
      &options.compressor);

Plugin compressor (new options)
Options options;
ConfigOptions config_options;
  Status s = Compressor::CreateFromString(
      config_options,
      "id=my_compressor;my_option1=value1;my_option2=value2",
      &options.compressor);

db_bench
For db_bench, compression_type and individual compression options (such as compression_level) were left unchanged for backward compatibility.
compression_type is still used with plugin compressors to specify their name. Other compressor options can be passed using compressor_options.
Built-in compressor (existing options)
--compression_type=zstd --compression_level=1
Built-in compressor (new options)
--compression_type=zstd --compressor_options="level=1"
Plugin compressor (new options)
--compression_type=my_compressor --compressor_options="my_option1=value1;my_option2=value2"
Limitations/Future Work
Compressor plugins are currently not supported for

WAL compression: it requires adding streaming compression to the Compressor interface, as described in PR #584
Compressed secondary cache
BlobDB: the blob_compressor option allows passing options for built-in compressors, but plugins are not supported. Supporting plugins would require additional metadata to be stored with blob files.

These limitations will be addressed by future PRs.

@lucagiac81 please rebase

#397 should be able to help solve this problem.  Using it, we can set the compressor as an option and not serialize the compression+opts if the Compressor is not set.  Once your PR is merged, I suggest you open another issue to outline that issue, preferably with an associated unit test.  Then I can help work on a fix.

Code is rebased on latest main based RocksDB 8.6.7 (the previous code was based on RocksDB 8.1.1).
I kept some of the changes in separate commits for ease of review.
Fix vector option parsing

A parsing error was discovered with compressor_per_level, which is a vector of Compressor (Customizable).
The vector is serialized with an outer set of braces. For example: {{id=NoCompression; parallel_threads=1}:{id=Snappy; parallel_threads=1}}
DefaultOptionsFormatter::ToVector calls NextToken which only removes the outer braces and returns one token with all the vector elements, instead of taking one element at a time.
This commit resolves the issue by removing the outer set of brackets.

Add test for option override issue

This exposes the bug described in a a previous comment in this PR (using compression and compressor options in option string).

Separate derived compression settings

This solves the bug exposed in the previous commit.
It adds a "derived" compressor field in MutableCFOptions, which is computed from the compressor/compression options. In this way, user selection is never overwritten by default settings.
I also tried to fix the bug using PR397 (with kUseBaseAddress), but I could not cover all cases.
With that PR397, I’m able to update “compressor” and “compression” fields in MutableCFOptions when parsing either option in the string.
However, I still run into issues when parsing the option file.
I noticed that the parse order from the option file is: compression, compressor, compression_opts
When parsing compression_opts, we need to apply the options only if the user specified the "compression" option. If the user specified "compressor", the options are already included.
We could only keep "compressor" in the option file, but the file would become incompatible with previous versions of Speedb.
I have the code for this solution available in a separate branch if you'd like to review it.


merge option with existing manifest

Hi, can you provide more details.

My bad, I had forked this project and right before I pushed my changes I realized you could accomplish the same task using a custom format function. Although, the original intent was to have an option that would merge with existing JSON data. That being said I'm going to close this issue.


fix(deps): Fixing peer dependencies

:tada: This PR is included in version 7.1.17 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


switch to modules for CI

Previously, go get would download the latest version of each dependency since
v1.11 is in the transitionary modules/GOPATH state and lacks GO111MODULE=on.
Now, let's turn GO111MODULE=on for all commands and download just the go.mod
deps. (also, does so individually for v1 and v2 since they have different
go.mod files)

@jadekler, kokoro is still failing.


[NEW SHEET] Patient 13 (v1.5)

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: < Patient 13 "Édition blouse blanche" (v1.5)  >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: < Mister Frankenstein > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: < Patient 13 system  > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.

[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >

[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description
Hello to you, I offer you a new character sheet for the role-playing game Patient 13 in its latest version 1.5.
There is already a character sheet for this role-playing game but in its original version.
I first considered adapting the sheet to integrate all the updates and add functionality that streamlines the gaming experience for the user but I came to the conclusion that I had to start again on blank coding.
Not that the upgrade would have been impossible but the coding itself wasn't very familiar to me and on top of that, I wouldn't have been able to add everything I wanted to.
Thus the proposed version of the sheet takes the design of the official sheet.
I integrated darkmode as well as a fully automated roll template system which allows for success/failure feedback (as well as all the variants) and thus facilitates the progress of the game.
Many scripts (not optimized I grant you but coding being a passion, and the script worker system very recent for me, the functionality took precedence over the "beautiful, economical online coding" ^^)
Unlike my previous contributions in partnership with different publishers, this sheet is not strictly called "official" because I did not have it validated by the creator of the game. However, you will be able to see that the latter entirely covers the design and uses all assets made available by the publisher.
Thank you for reading this long text, wishing you a good day.
Evilbram

Character Sheet Info Roll20 Internal Use only.

Hello @ctotone!
Everything in the code looks great! I just want to request that you modify the README to say something like 'Inspired by the Official Character Sheet Template for "Patient 13"' so there isn't any confusion about it being official or not. Once that is updated, I can get this merged for you :) Thanks!

The modification has just been made.
I took as a basis a readme from an official sheet that I had made previously, forgetting to add the "inspired".


Current lang.error detail() function return type is different from  the spec.

Current implementation has the following signature
public isolated function detail(error<DetailType> e) returns DetailType = external
But Spec(https://ballerina.io/ballerina-spec/spec.html#lang.error) says it should be
public isolated function detail(error<DetailType> e) returns readonly & DetailType = external;
Affect version: slbeta3

Fixed in https://github.com/ballerina-platform/ballerina-lang/pull/33542


tmt clean guests|runs ... doesn't

With tmt-1.31.0-1.fc39.noarch, tmt clean doesn't seem to do anything. I see this with sessions like tmt run --until report provision --how virtual --image fedora-39 (I do this to interactively debug after a failure), but a session can also be generated with tmt try -l -v fedora-rawhide (which apparently fails to boot and times out trying to connect after two minutes). After that:
❱❱❱ virsh list --all
 Id   Name               State
----------------------------------
 2    tmt-001-YTPzyviG   running

❱❱❱ ls -ld /var/tmp/tmt/run* /var/tmp/tmt/testcloud/instances/*
drwxr-xr-x. 3 martin martin 4096 Mar 20 06:42 /var/tmp/tmt/run-001
drwxr-xr-x. 3 martin martin 4096 Mar 20 06:42 /var/tmp/tmt/testcloud/instances/tmt-001-YTPzyviG

After that, neither of these commands has any observable effect, both the libvirt VM and the run-* and instances/* remain untouched:
❱❱❱ tmt clean guests -vvdd
clean
guests

❱❱❱ tmt clean runs -vvdd
clean
runs

The output looks a bit different with -l/--last:
❱❱❱ tmt clean runs -vvddl
clean
runs
Workdir '/var/tmp/tmt/run-001' already exists.
    Removing workdir '/var/tmp/tmt/run-001'.

❱❱❱ tmt clean guests -vvddl
clean
guests
No metadata found, using the default plan.
Workdir '/var/tmp/tmt/run-001' created.
Read file '/var/tmp/tmt/run-001/run.yaml'.
Run data not found.
Workdir '/var/tmp/tmt/run-001/default/plan' created.
Create the data directory '/var/tmp/tmt/run-001/default/plan/data'.
Create the environment file '/var/tmp/tmt/run-001/default/plan/data/variables.env'.
    Workdir '/var/tmp/tmt/run-001/default/plan/discover' created.
    Read file '/var/tmp/tmt/run-001/default/plan/discover/step.yaml'.
    Step data not found.
    Read file '/var/tmp/tmt/run-001/default/plan/discover/tests.yaml'.
    Discovered tests not found.
    Workdir '/var/tmp/tmt/run-001/default/plan/provision' created.
    Read file '/var/tmp/tmt/run-001/default/plan/provision/step.yaml'.
    Step data not found.
    Read file '/var/tmp/tmt/run-001/default/plan/provision/guests.yaml'.
    Provisioned guests not found.
    Workdir '/var/tmp/tmt/run-001/default/plan/prepare' created.
    Read file '/var/tmp/tmt/run-001/default/plan/prepare/step.yaml'.
    Step data not found.
    Workdir '/var/tmp/tmt/run-001/default/plan/execute' created.
    Read file '/var/tmp/tmt/run-001/default/plan/execute/step.yaml'.
    Step data not found.
    Read file '/var/tmp/tmt/run-001/default/plan/execute/results.yaml'.
    Test results not found.
    Workdir '/var/tmp/tmt/run-001/default/plan/report' created.
    Read file '/var/tmp/tmt/run-001/default/plan/report/step.yaml'.
    Step data not found.
    Workdir '/var/tmp/tmt/run-001/default/plan/finish' created.
    Read file '/var/tmp/tmt/run-001/default/plan/finish/step.yaml'.
    Step data not found.

But same result, no actual  cleaning.

Able to reproduce this problem if I run clean runs first,but if I run clean guests first,all things work well for testcloud created guests.
lnie@fedora:~/test-tmt$ tmt clean guests -vvddl
clean
guests
Using tree '/home/lnie/test-tmt'.
Workdir '/var/tmp/tmt/run-025' already exists.
Read file '/var/tmp/tmt/run-025/run.yaml'.
Workdir '/var/tmp/tmt/run-025/plans/testcloud' already exists.
Create the data directory '/var/tmp/tmt/run-025/plans/testcloud/data'.
Create the environment file '/var/tmp/tmt/run-025/plans/testcloud/data/variables.env'.
Workdir '/var/tmp/tmt/run-025/plans/testcloud/discover' already exists.
Read file '/var/tmp/tmt/run-025/plans/testcloud/discover/step.yaml'.
Successfully loaded step data.
Module 'tmt.steps.discover.shell' already imported.
status: done
Read file '/var/tmp/tmt/run-025/plans/testcloud/discover/tests.yaml'.
Workdir '/var/tmp/tmt/run-025/plans/testcloud/provision' already exists.
Read file '/var/tmp/tmt/run-025/plans/testcloud/provision/step.yaml'.
Successfully loaded step data.
Module 'tmt.steps.provision.testcloud' already imported.
status: done
Read file '/var/tmp/tmt/run-025/plans/testcloud/provision/guests.yaml'.
Module 'tmt.steps.provision.testcloud' already imported.
Workdir '/var/tmp/tmt/run-025/plans/testcloud/prepare' already exists.
Read file '/var/tmp/tmt/run-025/plans/testcloud/prepare/step.yaml'.
Successfully loaded step data.
Module 'tmt.steps.prepare.shell' already imported.
status: done
Workdir '/var/tmp/tmt/run-025/plans/testcloud/execute' already exists.
Read file '/var/tmp/tmt/run-025/plans/testcloud/execute/step.yaml'.
Successfully loaded step data.
Module 'tmt.steps.execute.internal' already imported.
status: done
Read file '/var/tmp/tmt/run-025/plans/testcloud/execute/results.yaml'.
Workdir '/var/tmp/tmt/run-025/plans/testcloud/report' already exists.
Read file '/var/tmp/tmt/run-025/plans/testcloud/report/step.yaml'.
Successfully loaded step data.
Module 'tmt.steps.report.display' already imported.
status: done
Workdir '/var/tmp/tmt/run-025/plans/testcloud/finish' already exists.
Read file '/var/tmp/tmt/run-025/plans/testcloud/finish/step.yaml'.
Successfully loaded step data.
Module 'tmt.steps.finish.shell' already imported.
status: todo
Read file '/var/tmp/tmt/run-025/plans/testcloud/provision/step.yaml'.
Successfully loaded step data.
Module 'tmt.steps.provision.testcloud' already imported.
Read file '/var/tmp/tmt/run-025/plans/testcloud/provision/guests.yaml'.
Module 'tmt.steps.provision.testcloud' already imported.
Step is done, not touching its data.
Using the 'ProvisionTestcloud' plugin for the 'virtual' method.
step is done, not overwriting plugin data
Waking up testcloud instance 'tmt-025-stLYXxCU'.
testcloud version: 0.9.10
Provision wake up complete (already done before).
Stopping guests in run '/var/tmp/tmt/run-025' plan '/plans/testcloud'.
finish
workdir: /var/tmp/tmt/run-025/plans/testcloud/finish
        Stopping testcloud instance 'tmt-025-stLYXxCU'.
        guest: stopped
        Removing testcloud instance 'tmt-025-stLYXxCU'.
        guest: removed
        Workdir '/var/tmp/tmt/run-025/plans/testcloud/provision/default-0' already exists.
        summary: 0 tasks completed
    status: done
    Write file '/var/tmp/tmt/run-025/plans/testcloud/finish/step.yaml'.lnie@fedora:~/test-tmt$ virsh list --all

Id   Name               State
1    tmt-024-AexPXgUW   running
lnie@fedora:~/test-tmt$ tmt clean runs -vvdd --id run-025
clean
runs
Removing workdir '/var/tmp/tmt/run-025'.
lnie@fedora:~/test-tmt$ rpm -q tmt
tmt-1.31.0-1.fc39.noarch

For mrack created guest users will see:https://github.com/teemtee/tmt/issues/2796,but that's a different problem:)

Hi, @martinpitt,would please check tmt clean guests --vvddl  and then tmt clean runs -vvddl fix your problem?Thanks:)
Btw, you need to start from the beginning,ie ,tmt run --until report provision --how virtual --image fedora-39

Starting from a clean slate: Fresh reboot, /var/tmp/tmt is empty, no running VMs.
tmt try -l -v fedora-rawhide
tmt clean guests -vvddl
tmt clean runs -vvddl

That works fine indeed (tmt-1.32.1-1.fc39.noarch). So cleaning up the last VM/run works, thanks for the tip!
But if I start a second session, then -l only ever cleans run-002, run-001 stays around even if I run the clean commands a second time.
--help and the command names suggest that they would clean all runs, though?

So cleaning up the last VM/run works, thanks for the tip!

My pleasure.

But if I start a second session, then -l only ever cleans run-002,
run-001 stays around even if I run the clean commands a second time.

That's because  run-001 is NOT last run, run-001 is the second to last run:)
I understand your request: when the last run is  deleted, the second to
last run becomes the last run,and so on.
With current implement,tmt doesn't support that.
On Sun, Apr 7, 2024 at 3:29 PM Martin Pitt @.***> wrote:

Starting from a clean slate: Fresh reboot, /var/tmp/tmt is empty, no
running VMs.
tmt try -l -v fedora-rawhide
tmt clean guests -vvddl
tmt clean runs -vvddl
That works fine indeed (tmt-1.32.1-1.fc39.noarch). So cleaning up the last
VM/run works, thanks for the tip!
But if I start a second session, then -l only ever cleans run-002,
run-001 stays around even if I run the clean commands a second time.
--help and the command names suggest that they would clean all runs,
though?
—
Reply to this email directly, view it on GitHub
https://github.com/teemtee/tmt/issues/2772#issuecomment-2041349923, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKFR23BXUSFXM6TJNUG5MP3Y4DYVHAVCNFSM6AAAAABE62QY2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBRGM2DSOJSGM
.
You are receiving this because you commented.Message ID:
@.***>

I understand your request: when the last run is  deleted, the second to last run becomes the last run,and so on.

That'd be nice, but it's actually not my request. It was just an attempt to work around. The request here is that the clean command works without -l.

tmt clean runs works well here:
@.:~/tmt/tmt$ ls /var/tmp/tmt/
instances  run-001  run-002  testcloud  testcloud.lock
@.:~/tmt/tmt$ tmt clean runs -vvddl
clean
runs
Workdir '/var/tmp/tmt/run-002' already exists.
Removing workdir '/var/tmp/tmt/run-002'.
@.:~/tmt/tmt$ tmt clean runs
clean
runs
@.:~/tmt/tmt$ ls /var/tmp/tmt/
instances  testcloud  testcloud.lock
@.***:~/tmt/tmt$ rpm -q tmt
tmt-1.32.1-1.fc39.noarch
Would you please check whether there is a run.yaml file in  your
/var/tmp/tmt/run-001 dir?Thanks.
On Mon, Apr 8, 2024 at 4:37 PM Martin Pitt @.***> wrote:

I understand your request: when the last run is deleted, the second to
last run becomes the last run,and so on.
That'd be nice, but it's actually not my request. It was just an attempt
to work around. The request here is that the clean command works without
-l.
—
Reply to this email directly, view it on GitHub
https://github.com/teemtee/tmt/issues/2772#issuecomment-2042180569, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKFR23FCVQTMXRLS2ZQGZOTY4JJNFAVCNFSM6AAAAABE62QY2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBSGE4DANJWHE
.
You are receiving this because you commented.Message ID:
@.***>

Yes,that is  because tmt try doesn't create a run.yaml file,and tmt clean //guests/runs commands without --id/--last  will only clean things from the runs whose dir has a run.yaml file in it.
That is a bug of tmt, however, you could workaround that by specify the --id of the run or use --last if what you want to clean are from the last run.

@skycastlelily, do I understand correctly that this issue should now be fixed by #2889?

Yep, but  https://github.com/teemtee/tmt/pull/2882 and  https://github.com/teemtee/tmt/pull/2805 are still valid for robustness reason,gonna to rebase that two merge requests:)