nbf + clockTolerance cause issues

https://github.com/auth0/node-jsonwebtoken/blob/73c4a5ac45d557ef0dc1c43fcae6b7fc9bc3f19e/verify.js#L116
The clockTolerance may be wrong when used together with nbf.
When the token has just been issued if added with clockTolerance it'll always fail I guess.
I'm pretty sure it should extend exp (expiration) time and shorten nbf (not before) time.
I can submit a PR if someone agrees.

Please consider this timeline as the normal condition valid jwt
----------------------------------------------------
nbf 	 	      valid usage			 exp

What the clockTolerance does it to increase the valid usage lifetime.
I'm my understanding it would be more assertive to increase to both ends of this timeframe.
What the system currently does is dislocate the valid usage.
	----------------------------------------------------
nbf+tolerance 	 	valid usage          exp+tolerance

This is moving the valid usage to the right - it does not increase it. That works for expiration. But if a token is used right after creation then nbf will fail. Say nbf=1000 and at the same time we validate it: (payload.nbf > 1000+tolerance) == false
I have this scenario currently when my frontend regenerates the token on failure, to sequentially try the same api endpoint again. node-jsonwebtoken is failing due to the clockTolerance.
It seems to me a clockTolerance implementation should increase both ends of the valid time, or at minimum just increase the expiration.
------------------------------------------------------------
nbf-tolerance 	 	valid usage		 exp+tolerance

I think there must be some confusion, as it already does exactly what you are looking for, at least from understanding.
Assuming the payload, {"exp": 1100, "nbf": 1000} with the current time being 1000, and a clock tolerance of 10 .
The nbf is verified using:
if (payload.nbf > clockTimestamp + (options.clockTolerance || 0)) {
    return done(new NotBeforeError('jwt not active', new Date(payload.nbf * 1000)));
}

would result in:
if (1000 > 1000 + 10)) {
    return done(new NotBeforeError('jwt not active', new Date(payload.nbf * 1000)));
}

This would be false, the error will not be returned, and the token would be valid.
I think there must be some other issue with the tokens.

I have this scenario currently when my frontend regenerates the token on failure, to sequentially try the same api endpoint again. node-jsonwebtoken is failing due to the clockTolerance.

Let me write the cases I can think of:
A) Clock timestamp => Frontend: 995 and API: 1000. Clock Tolerance: 10
Check: nbf=995 > clockTimestamp=1000 + clockTolerance=10 => false, all good
Since the verifier is "in the future" from the signer point of view there will be never a fail here.
B) Clock timestamp => Frontend: 1000 and API: 995. Clock Tolerance: 10
Check: nbf=1000 > clockTimestamp=995 + clockTolerance=10 => false, all good
Since the verifier is "in the past" from the signer point of view here the tolerance matters, it needs to be well defined.
B.2) Clock timestamp => Frontend: 1000 and API: 985. Clock Tolerance: 10
Check: nbf=1000 > clockTimestamp=985 + clockTolerance=10 => true, failure.
Clock tolerance should be greater.
C) Clock timestamp => Frontend: 1000 and API: 1000. Clock Tolerance: 10
Check: nbf=1000 > clockTimestamp=1000 + clockTolerance=10 => false, all good
Can you think in any other? Did I miss anything?

I agree with @MitMaro as after reviewing the code again I could figure the logic was fine.
Still I could only get rid of the issues with tokens failing on another server (right after its creation)  disabling the nbf completelly
jwt({ignoreNotBefore: true});


CDPCP-9522 - unit tests for Terraform Provider for CDP

@enis could you please take a look into it?


Rendering difference in Impeller/Skia with an image mesh

Is there an existing issue for this?

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

Steps to reproduce
Rive app animations that have meshes applied to them are not rendering on Impeller, but rendering on Skia.
Flutter v3.10.0
Setting --no-enable-impeller works.
This applies for any Rive animation that has a mesh applied. Here is the one used in the sample/video: viking.riv.zip
Steps:

See video showing the difference on iOS vs macOS. Disabling Impeller fixes the issue on iOS.
Run the attached code example on iOS to reproduce

Expected results
Impeller and Skia to look the same :smile
Actual results
They're not the same...
Code sample
Code sample
[bug.zip](https://github.com/flutter/flutter/files/11577013/bug.zip)

Screenshots or Video

https://github.com/flutter/flutter/assets/13705472/cadcd3ed-844c-4e14-89c6-bee2c0d1ad7e

Logs
Logs
flutter run

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.0, on macOS 13.3.1 22E772610a darwin-arm64, locale en-NL)
    • Flutter version 3.10.0 on channel stable at /Users/gordon/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 84a1e904f4 (2 weeks ago), 2023-05-09 07:41:44 -0700
    • Engine revision d44b5a94c9
    • Dart version 3.0.0
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/gordon/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • ANDROID_HOME = /Users/gordon/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.78.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] Connected device (3 available)
    • iPhone 14 Pro (mobile) • 7587DA78-60C7-41CC-B9DA-2D479D1A2E65 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 13.3.1 22E772610a darwin-arm64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 113.0.5672.126

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

• No issues found!

Adding video for clarity
https://github.com/flutter/flutter/assets/13705472/cadcd3ed-844c-4e14-89c6-bee2c0d1ad7e

Hi @HayesGordon. Thanks for filing this issue. I am unable to reproduce this on the latest stable channel, you can update your flutter version to verify this, but this issue occurs on the latest master and beta channels using Impeller and works fine with skia.

Stable
Master
Beta

❌
✅
✅

✅ - Reproducible
❌ - Not-reproducible
  flutter doctor -v
[✓] Flutter (Channel master, 3.11.0-2.0.pre.55, on macOS 13.3 22E252 darwin-arm64, locale en-NG)
    • Flutter version 3.11.0-2.0.pre.55 on channel master at /Users/dammya/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 835b892d7f (3 weeks ago), 2023-05-07 01:30:22 +0530
    • Engine revision 23f730efbf
    • Dart version 3.1.0 (build 3.1.0-83.0.dev)
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/dammya/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /Users/dammya/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.78.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] VS Code (version 1.78.0)
    • VS Code at /Users/dammya/.Trash/Visual Studio Code 20.07.05.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • sdk gphone arm64 (mobile) • emulator-5554                        • android-arm64  • Android 11 (API 30) (emulator)
    • iPhone 14 Pro (mobile)    • DCF2031A-7EA9-42FF-8CD4-D4584F3E7A6D • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)           • macos                                • darwin-arm64   • macOS 13.3 22E252 darwin-arm64
    • Chrome (web)              • chrome                               • web-javascript • Google Chrome 113.0.5672.126

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

• No issues found!

[✓] Flutter (Channel stable, 3.10.2, on macOS 13.3 22E252 darwin-arm64, locale
    en-NG)
    • Flutter version 3.10.2 on channel stable at /Users/dammya/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9cd3d0d9ff (5 days ago), 2023-05-23 20:57:28 -0700
    • Engine revision 90fa3ae28f
    • Dart version 3.0.2
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version
    32.1.0-rc1)
    • Android SDK at /Users/dammya/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /Users/dammya/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.78.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] VS Code (version 1.78.0)
    • VS Code at /Users/dammya/.Trash/Visual Studio Code 20.07.05.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (4 available)
    • sdk gphone arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 11 (API 30) (emulator)
    • iPhone 14 Pro (mobile)    • DCF2031A-7EA9-42FF-8CD4-D4584F3E7A6D • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)           • macos                                •
      darwin-arm64   • macOS 13.3 22E252 darwin-arm64
    • Chrome (web)              • chrome                               •
      web-javascript • Google Chrome 113.0.5672.126

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

• No issues found!

This works as expected for me at b5b2f4175f . Is this a device specific issue, @dam-ease or @HayesGordon can you confirm if this works for you?

This animation was fixed by https://github.com/flutter/engine/commit/d880a0c0732327307efcf4b70a2dd8ffc93b8fdf


Adapt bootargs file to support btrfs snapshotter

Requires https://github.com/rancher/elemental-toolkit/pull/1957 to merged first/together


Where Function with Greater Than (or equal) Operator Not Working

I'm trying to filter a JSON array of values returned from the GitHub API where "size" > 0; however, the where XYZ "gt" 0 function is not working and returns an empty array every time.  Neither does where XYZ "ge" 1 work. (XYZ is a placeholder).  Maybe there's a formatting issue I'm missing, but I've tried numerous different variations.
The full line is:
{{ $repos := getJSON "https://" (getenv "GITHUB_USER") ":" (getenv "GITHUB_TOKEN")  "@api.github.com/orgs/XYZ/repos?per_page=100" }}
{{ $sources := where (where (where $repos "size" "ge" 1) "fork" false) "archived" false }}

This works fine when I just filter boolean values.
{{ $repos := getJSON "https://" (getenv "GITHUB_USER") ":" (getenv "GITHUB_TOKEN")  "@api.github.com/orgs/XYZ/repos?per_page=100" }}
{{ $sources := where (where $repos "fork" false) "archived" false }}

Does where XYZ "gt" 0.0 work?

I just tried xyz "gt" 0.0 and xyz "ge" 1.0 and neither worked.  What is weird is that having and if {{ if ge $repo.size 1 }} within a range works fine, but the where clause doesn't work.


Samurai broken when picking the '_______ Combo' features (ex. Kasha Combo, Gekko Combo ect)

When choosing to have the last attack in a combo replaced with its chain eg, Yukikaze, Kasha, Gekko, Oka, and Mangetsu, they no longer work. This happened to me ONLY after reaching level 92 and unlocking the gyofu  new first step in the combos but is not working even if synced down below that level once reaching level 92. I suspect gyofu is interfering somehow and gyofucking our day up.

Already fixed in #112.

ah then its not pushed i guess? its still broken for me

Correct, that PR is not merged yet, and after that it waits for a release to be made.


ENH: Add the ability to specify k in NearestNDInterpolator

This commit allows you to specify k nearest neighbors in NearestNDInterpolator. This enables you to allow for averaging over the top closest neighbors rather than just the most similar.  It also allows you to specify the averaging method, either uniform or distance.
Reference issue
addresses gh-21138
What does this implement/fix?
allows you to specify k in NearestNDInterpolator

What @j-bowhay said in #21138 : this sounds almost like an IDW, so let's consider this PR together with  #2022 and https://github.com/scipy/scipy/pull/19665

while it's true that with the distance weights it is IDW, it could still be useful having the ability to specify k points to average rather than just 1. I can continue the discussion in gh-2022

Currently, NearestNDInterpolator does something simple and straightforward: for each input point it selects a nearest-neighbor value. The only additional options are those from KD-tree.
IMO it's better to keep it as is, and add any additional heuristics to a separate object. The proposal in https://github.com/scipy/scipy/pull/19665 and the discussion in https://github.com/scipy/scipy/issues/2022 looks like a natural candidate.

I would agree with @ev-br I would also like to see this kept separate. If you would be interested in reviving https://github.com/scipy/scipy/pull/19665 by address the review comment that wold be appreciated.

No worries then, feel free to close this pull request in that case! Thanks for looking into this.

It appears we are all in agreement that a generalization of this PR is best done in a separate class, as a version of IDW or some such.
Thank you @PolarBean and I'm very much looking forward to your involvement in https://github.com/scipy/scipy/pull/19665 or its successor (if you feel like, it's perfectly OK to take over a stalled PR, address review comments and resubmit). Thank Jake for weighting in, too.


Unable to view multiple regions when combining different graph types into combination graph without hovering on legend

Describe the bug
When defining a line graph and paired result graph onto a single graph, I have defined multiple regions to be displayed on the graph. When I do this, the graph shows no regions at all.
I can see the regions themselves are defined and work if I was to hover on the legend for the relevant graph.
Link to CodeSandbox
https://codesandbox.io/s/combgraph-region-issue-sy0lk
Expected behavior
I require the regions to always show whether you are hovering on a legend or not. I would understand if the regions would still have to match between the 2 graphs in order to show to avoid confusion for users.
Screenshots
Issue:

Desktop (please complete the following information):

MacOS Catalina
Chrome and IE 11 and above

Additional context
This is important for allowing us to draw trending lines between the values of a paired result so that our users can see the trend of Blood Pressure of a patient to catch patient deterioration as soon as possible. Without it, they cannot see the trend as easily and may miss patient deteriorating resulting in patient safety issue.

Is it possible to get a timeline for this being looked at please. We need to have a rough date so that we can communicate this on the project

In Carbon-graphs, hiding regions(and displaying them when hovering on legend item) in graph when there is more than one content, is not a bug. In that case(when there is more than one content in graph), regions provided should satisfy certain conditions(depends on graph type) in order to display them on graph without hovering on legend item.
For Paired Result Graph:
When there is more than one content in paired result graph  it should satisfy certain conditions to display regions on graph according to code in carbon.

we will check if paired data is proper i.e region for each key(high, mid, low) in value should be there.
it will check if all regions are same or not. (display if regions are same)
Based on these conditions we are displaying regions in paired result graph.

cc:  @mraplumb @Veera81 @sdadn

@AshishMotanamGurunadham I did discuss this with you and you told me to raise this as an issue. If you check my sandbox example you will see that the regions between the paired result graph and line graph are identical. It was agreed that this was an issue. We discussed this back on the 22nd June 2020 where I showed you that when you define more than one region on a combination graph, it doesn't matter whether they are the same or not, the regions are hidden till you hover on the legend for the relevant graph. I understand regions should be the same just in this case they are so should be shown.

@AshishMotanamGurunadham my fault. I should have read it better. I agree with what you have said for when to display the regions. The logic makes sense and works for me.

@davidhunt88 at this time we are not planning to make changes to the region logic when there are multiple datasets selected on the graph.  Based on our internal discussion, we feel there is too great of a chance that the resulting graph could reflect misleading information if we were to allow regions associated to multiple datasets to be displayed at the same time.
Based on the example you provided, are you using a combination of graphs to draw a line between the data points in the "high" results of your paired result graph or was that merely a coincidence for the sake of an example.  If that is what you are intending, we could look at an enhancement to provide the line as part of the paired result graph when only one of the results is selected (e.g. High) instead of having to use a combination graph to do that.  This would also allow the regions to continue to be displayed.

@mraplumb Having the ability to draw a line between values on the paired result graph is exactly what I need and what I was after originally. I was suggested to combine a paired result and line graph by @AshishMotanamGurunadham in order to get round this lack of functionality. If you could add the ability to draw a line between the values of a paired result while maintaining the paired result view that would solve this. As an example, you would define to display high, low and mid values and then draw a line between the values as configured so I could say draw a line between the high values or the low values while displaying everything.
Here are two examples of how we would like it to work from our existing MPage:

@davidhunt88 after further discussion, we would be comfortable with an enhancement to add the "trend" line linking all the data points in a paired result graph when only one of the points is toggled on, via the legend, in the paired result (e.g. high, median, low).

When multiple data sets of the paired result are displayed, Carbon would behave as it does today and not display the trend lines.
Based on all previous research into this graph type, we would not want to draw the trend lines across the data points for more than one set of data points in a paired result graph at the same time.

@mraplumb so are you saying that you would not have a default trend line draw between the high values when the paired result has all values enabled? The reason I ask is that paired result is most important for Blood Pressure values where by you would have Systolic and Diastolic values logged together with a Mean Pressure on occasion. If you were to implement your solution then, unlike the line graph, there would be no way to draw a trend line between values for Blood pressures while still showing the full paired result.
This could be a problem for clients adopting advanced graphing as well as us using the Carbon library as we need the trend lines between the paired results. If you go ahead with your proposed solution then it will not resolve this issue that I have raised.
Is it possible to understand why there is a hesitation to add a trend line to the paired results without having it appear only when one of the results is toggled on via the legend? If trend lines where added as I mentioned then this would align it to the other graph types like the line graph and would give a consistent implementation approach.

@davidhunt88 your understanding is correct.  Our UX team has provided this guidance based on research that has been done around each graph type and established design standards.  There is a big concern with showing that much information in a graph face up (paired results, trend lines for each set of values and reference ranges for each set of values) which can lead to confusion for end users.

@mraplumb thank you for explaining it. I was hoping we could have something where you would have to pick one of the paired result values.
As an example, if I specify that high is the main result then only defined regions for the high values will show and a trend line would be drawn for the high results. If I defined more regions then the graph would go blank in terms of regions and you would have to hover on the legend to see the regions for that legend on the graph.
This would then align it to the current solution for combination graphs and would remove any confusion in the graphs with what is being displayed. Is this solution a better one? If it would help, could you let me know who I could discuss this with on the UX team and maybe a call might help better explain this.
Thank you for all your help with this


Update compliance database schema

This adds a namespace field to the parent policy table so that the root policy namespace and name is recorded instead of the replicated policy name which includes the root policy namespace in it. This is more intuitive since the user interacts with the root policy and not often with the replicated policy.
This also moves the parent policy foreign key to the compliance_events table. This is better as a point in time record of the parent policy associated with the policy that generated the compliance event. It does make filtering more efficient later to see any compliance event associated with a parent policy. It also allows less data duplication on the policies table.
As part of this refactoring, goku was removed in favor of a minimal translation layer of database queries to structs and vice versa.

@JustinKuli the latest commit makes the SQL queries static. Let me know what you think of this approach!

@JustinKuli alright, all the complicated "cool" stuff has been removed from the PR now. :laughing: I kept them in the commit history in case we decided to go back that route.

/unhold
Thanks (and sorry) for all the back-and-forth, but I think this looks really great now


[Distributed] Did not find tokenizer at {tokenizer_path}

🐛 Describe the bug
torchrun --nproc-per-node 8 dist_run.py

known configs: ['13B', '30B', '34B', '70B', '7B', 'CodeLlama-7b-Python-hf', 'Mistral-7B', 'stories110M', 'stories15M', 'stories42M', 'Meta-Llama-3-70B', 'Meta-Llama-3-8B', 'Meta-Llama-3.1-70B-Tune', 'Meta-Llama-3.1-70B', 'Meta-Llama-3.1-8B-Tune', 'Meta-Llama-3.1-8B']
09-14 15:41:32.092 - dist_run:137 - Chat Model Config: TransformerArgs(block_size=2048, vocab_size=32000, n_layers=32, n_heads=32, dim=4096, hidden_dim=11008, n_local_heads=32, head_dim=128, rope_base=10000, norm_eps=1e-05, multiple_of=256, ffn_dim_multiplier=None, use_tiktoken=False, max_seq_length=8192, rope_scaling=None, n_stages=1, stage_idx=0)
[rank0]: Traceback (most recent call last):
[rank0]:   File "/home/kw2501/local/torchchat/dist_run.py", line 277, in <module>
[rank0]:     main()
[rank0]:   File "/home/kw2501/local/torchchat/dist_run.py", line 139, in main
[rank0]:     tokenizer = _build_chat_tokenizer()
[rank0]:   File "/home/kw2501/local/torchchat/dist_run.py", line 94, in _build_chat_tokenizer
[rank0]:     tokenizer_args = TokenizerArgs.from_args(args)
[rank0]:   File "/home/kw2501/local/torchchat/torchchat/cli/builder.py", line 269, in from_args
[rank0]:     raise RuntimeError(f"did not find tokenizer at {tokenizer_path}")
[rank0]: RuntimeError: did not find tokenizer at /home/kw2501/.torchchat/model-cache/meta-llama/Meta-Llama-3-8B-Instruct/tokenizer.model

Cc: @lessw2020
Versions
main branch

this highlights a current weakness - atm we are using the tokenizer downloaded by chat, but the safetensor weights by HF.
Thus, to get setup for a model we have to run:
1 - python3 torchchat.py download llama3  to get the tokenizer
2 - model = AutoModelForCausalLM.from_pretrained("meta-llama/Meta-Llama-3-8B-Instruct") to get the safetensor weights.

we can hook into the torchchat download.py to download the tokenizer, and then pull the safetensor weights via an HF call as well.


VASTAds Constructor fix for new Chrome

As discussed in issue #19 this fixes the getElementsByTagNameNS call with root.namespaceURI to call it with "*" .

Could you please split this into two PRs -- one for each feature? I'm happy to merge both.

I remember only sending one PR, the second one was just a commit, don't know why it popped up in this PR.
I'm not sure how to resend as 2 separate PRs, advice?

It's because you made this change after already having committed your AdParameter fix. PRs include all commits that differ from master. Assuming origin points to your fork, and upstream points to this repo (see git remote -v), to create separate PRs, try:
git checkout -b pr1 upstream/master
git cherry-pick 3142292e65642b9c469e13b460ce7d6c950d319d
git push -u origin pr1
git branch -D pr1
git checkout -b p2 upstream/master
git cherry-pick 933d3fa0d49302114159c88d0c410cee8773894c
git push -u origin pr2
git branch -D pr2

You should now have two new branches in your fork: pr1 and pr2. Create a PR for each one, and they should only include the appropriate commit.


Make method chaining possible

Using the RenderManager would be more convenient if every "writing" method returns the RenderManager itself.

 You should sign our Contributor License Agreement in order to get your pull request merged.

Thanks @seventyeight!
Completely valid to me. This makes all the methods consistent.
Two remarks about your change

Adopt the other return statement (see here)
Add a JSDoc "@ returns" comment to the method (like here)

Will the changes be ported to SAPUI5? Do we need the same for the .render() method?

IMHO we shouldn't enable method chaining for the render() method. That method is not part of the API given to control renderers. And I think the method chaining is mainly beneficial for such renderer  implementations.
In contrast, render() is a top level entry point for rendering. The option to communicate information about the rendering process in future via a return value seems more beneficial to me than saving one reference to the render manager.
In principle, the same even applies to renderControl(), but there consistency of the render manager API represents a value of its own, so I'm fine with it.
Reg. SAPUI5: yes the change will become part of SAPUI5. In general, any change to OpenUI5 hasn't to be ported to SAPUI5. SAPUI5 just embeds OpenUI5. There are only a few rare cases where we overlay OpenUI5 sources with a SAPUI5 specific variant (e.g. sap-ui-core.js is self-contained in SAPUI5 while it isn't in OpenUI5).

@seventyeight sorry for the delay...
Looks good :+1:
In order to merge it, could you please

adopt the commit message, e.g. [FIX] is missing, see here
squash the two commits into one, see here (3.)

In addition to the above, you also need to accept the contributor license agreement (it's just one or two clicks: open "Details" above where it says "Contributor License Agreement is not signed yet"). Thanks!

"I hereby agree to the Contributor License Agreement"

@seventyeight thanks :+1:
There seems to be a problem with CLA assistant matching the mail address after you've adopted it in your commit.
The agreement for your user is already stored in the tool since Feb 25, so that's fine.
Any new PR should get automatically verified.


 v_2.0.5 Setting delayed execution parameters does not take effect

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
No response
Use case
No response
Related issues
这个是我的测试，我在工作节点a设置了延迟参数 没有生效

Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I assigned this issue to you @LongNineSun because you select to submit PR


scalastyle integrated with coala!

Hi, We are coala (Language Independent Code Analysis Application). We are now using scalastyle as linting tool for our Scala support (Scala linting feature request).
You can have a chat with us anytime at https://gitter.im/coala-analyzer/coala. We will be happy to work together. We are a pretty active community so you will always find someone on gitter :)

This gets you some editor integrations and a few additional output formats you might not have yet so it might be valuable for your users and you could, if you wanted to rely on us, stop worrying too much about user interaction. That's the goal of coala basically, stripping away the awful stuff from static code analysis writing ;).

Cool. That sounds great. Anything I can do to help, just shout.


Metadata must be an object

CloudFormation Lint Version
cfn-lint 0.72.10
What operating system are you using?
Mac
Describe the bug
Cfn-lint should error on Metadata that is not an object.
Expected behavior
Fail with an error when Metadata is not an object.
Reproduction template
Resources:
    Something:
        Metadata: Some text...

Should be
Resources:
    Something:
        Metadata:
            Comment: Some text...

Error from CloudFormation:
error creating changeset: Template format error: Any Metadata member must be an object.

This will be fixed in v1.  Going to close this one.


Type inference wrong for strings prefixed with + and 0

It is interpreting +1 as numeric 1:
print(compactdata.loads("a=+1"))
->  {'a': 1}

And 01 as numeric 1:
print(compactdata.loads("a=01"))
->  {'a': 1}

These are invalid numeric JSON values. CompactData type inference should result in these being strings, like so:
print(compactdata.loads("a=+1"))
->  {'a': '+1'}
print(compactdata.loads("a=01"))
->  {'a': '01'}

Fixed in https://github.com/NUMtechnology/compactdata-python/commit/f536f3bbc54e353118a96f622cc0ea57e3743ca7.


possibly doc error?

The doc in http://jsguy.github.io/mithril.animate/#Using_it
m("div", { opacity: o.ani }, "I can fade away");
m("div", { onclick: m.trigger('opacity', 0), "I also fade away");

should have m.e(... instead?

Yep, typo snuck in, thanks for spotting 👍


[Feature]: Transpose the VS Code extension into a WebStorm Plugin

Context
There's a great extension built for VS Code but many people across or organisation are using WebStorm and it would be nice to get the same features.
Additional context, WebStorm >>> VS Code 😜
Proposal
Build a plugin for WebStorm with same functionalities as the VS Code extension.

duplicate of #799


Have to run make twice

On el6, I have to run make twice to make vmod-tcp to compile. The first run dies at
make[2]: *** No rule to make target vcc_if.lo', needed by libvmod_tcp.la'.  Stop.
Running make again works.
This problem is not visible on el7 or the fedoras.
Ingvar

Overtaken by events. This module has been merged into varnish-modules.
Build system configuration is differnt there and I believe this error is not present there.
If you can reproduce it on varnish-modules, please reopen a new issue there.


Add support for loading adapters from HuggingFace Model Hub

WIP for adding HuggingFace's model hub as a second source for loading adapters. Implementation via https://github.com/huggingface/huggingface_hub.
Example
Adapter: https://huggingface.co/calpt/adapter-bert-base-squad1
Loading from HF Hub
model = BertForSequenceClassification.from_pretrained("bert-base-uncased")
model.load_adapter("calpt/adapter-bert-base-squad1", source="hf")

Loading (the same adapter) from AdapterHub
model = BertForSequenceClassification.from_pretrained("bert-base-uncased")
model.load_adapter("squad1", config="houlsby", source="ah")  # source="ah" is default & can be omitted

Hi @osanseviero!
Looks great! Thanks for looking into this and let me know if you need anything else from our side.

Hey Calpt! Nothing else from your side at the moment until this PR is submitted. We're adding code snippets for adapter transformers which reads from adapter_config.json as discussed 🚀 🌔 . Feel free to take a look:
https://github.com/huggingface/huggingface_hub/pull/62

From the Hub side, if you add - adapter-transformers to the tags, you get a working code snippet that uses adapter_config.json. Check out https://huggingface.co/calpt/adapter-bert-base-squad1 for example.

@osanseviero Looks great! Thanks so much for your support on this! We'll add some documentation on our side and release this with the next feature update.


[BUG] "Target offscreen... Font size: Uninitiated" error is reported when running the experiment in a higher screen size resolution

Description:
"Target offscreen... Font size: Uninitiated" error is reported when running the experiment in a higher screen size resolution (4096x2160). Looks like this error is reported when the experiment window size (google chrome window) is too small with respect to the screen resolution. The screen resolution is set as 4096x2160. With this higher resolution, if the experiment window size is not big enough, it reports the error. After adjusting the window size to some higher size, able to load experiment successfully.
To Reproduce:

Set the screen resolution to 4096x2160.
Run the experiment. When the chrome browser is launched, resize the browser window size to a smaller size.
Follow the instructions and proceed with the experiment.
=> "Target offscreen... Font size: Uninitiated" error is reported just before displaying the crosshair image.
If the issue is not recreated, further resize the browser window smaller and try again.

Expected behavior:
Should be able to run the experiment without any error irrespective of the browser window size.
Screenshots:

Desktop (please complete the following information):
OS: Fedora
Browser: Google Chrome

https://trello.com/c/1LlkDGRM


need to download from ftp in OceanTransports/06_MOC.ipynb

Could we use e.g. FTPClient.jl or something else to download files automatically if needed?

Has been resolved by downloading from the Dataverse archive instead


Need help with installation

We tried to install the fastpack per the instructions mentioned in https://community.dynatrace.com/community/display/DL/Sitecore+FastPack. That is, installed the sitecore-fastpack-1.0.dtp on dynatrace server and then deployed the 3 files under Dynatrace.PatchRenderings.zip to the CD server. But the dashboard keep showing "Application Process unavailable (unexpected)" and "no measurements available for selected timeframe" for all charts under sitecore cache misses and sitecore measures (as an example).

Hi,
can you please post your questions here: https://community.dynatrace.com/community/display/DL/Sitecore+FastPack (at the buttom of the page).
Thanks a lot
Ingo


Zak trip length with Ardougne Rooftops

🐛 Bug Report - BSO
While doing Ardougne rooftop course, zak is not extending the trip by 40% like intended.  I’ve not had the problem with almost other skilling/pvm/minigame/etc. trips until trying to run Ardougne Laps.

Hotfixed? Or smoothbrain


Please document the list of NuGet packages in README.md

It's hard to discover information about which NuGet packages should be used and when. Please add a readme section which lists the main NuGet packages to use and why each one exists.
For example, when I need to run vstest.console.exe from a build script on a build server without VS installed, which is the smallest NuGet package or metapackage which I need?
I happen to know the answer to that question now, but I initially failed to find anything and I just happened to be told as part of another issue.

Please see here: 0014-Packaging.md.

On GitHub, it's more common to list these in the README (or a link therefrom) or in the wiki that is included in all projects by default. I wasn't expecting to spelunk for documentation

That looks great! It's exactly what I hoped for!
Now, can this be done for all packages published from this repo?
For example, when writing a test adapter, you'd want to reference Microsoft.TestPlatform.ObjectModel.
Or here's something I'm looking at right now. In the tests for the NUnit adapter, I want to create an ITestCaseFilterExpression from a given string to test against. The keys to that are public in Microsoft.VisualStudio.TestPlatform.Common.dll So what I'm hoping to know is, which NuGet package should I reference? Microsoft.TestPlatform.TestHost seems like the smallest package which contains it, but the lib/net45 assemblies are missing. Do you think it's possible for the NuGet package documentation to answer questions like this?

@pvlakshm , can you please update the readme, & close this


Process will crash if the OPENAI_API_KEY is not set

When using the latest code, the following error occurs:
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]Traceback (most recent call last):
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]  File "/app/tg.py", line 11, in <module>
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]    from responder import respond_prompt, respond_quota
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]  File "/app/responder.py", line 10, in <module>
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]    from utils import get_quota, prepare_save_images, save_images
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]  File "/app/utils.py", line 8, in <module>
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]    client = OpenAI()
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]  File "/usr/local/lib/python3.10/site-packages/openai/_client.py", line 93, in __init__
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]    raise OpenAIError(
2023-11-07T09:17:51Z app[148e772c06d589] nrt [info]openai.OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable
2023-11-07T09:17:52Z app[148e772c06d589] nrt [info] INFO Main child exited normally with code: 1

shit my problem

will fix it

使用最新的版本，同样遇到相似问题：
Arg parse done.
Config parse done.
OpenAI init done.
Traceback (most recent call last):
File "/root/tg_bing_dalle/tg.py", line 159, in
main()
File "/root/tg_bing_dalle/tg.py", line 55, in main
client = OpenAI()
File "/root/.venv/lib/python3.9/site-packages/openai/_client.py", line 93, in init
raise OpenAIError(
openai.OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable
去掉 -c config.toml 才正常。

拉下最新代码


Modlunky launching through Discord

Everytime I try to run Playlunky it seems to always launch through discord. I'm able to tell because I'm not able to open the steam overlay, nor does it say I'm playing Spelunky on Steam, and it also has a Discord icon next to my name when I try to play online.
The issue started when I added Spelunky to my registered games on discord. It worked fine at first because I did that during the time I was playing it, however the next time I launched Playlunky, it would only launch through Discord.
I don't think it's related to any of the mods I currently have, I think the problem may stem from the discord_game_sdk.dll file that's in my folder. When I try to launch Playlunky or vanilla Spelunky with the discord_game_sdk.dll file deleted, Playlunky gives me the error that discord_game_sdk.dll was not found, while Spelunky just won't launch without it.
I've checked that it is reading the correct Spelunky installation.

Apparently restarting my computer seemed to fix this. No clue why.


Update- security

What problem does this PR solve?
This is related to https://github.com/tikv/tikv/pull/7792
What is changed and how it works?
update security and TiKV logos
Related changes
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

No release note

So what's the difference between it and #7792?

So what's the difference between it and #7792?

I addressed the comments you made in https://github.com/tikv/tikv/pull/7792 and updated the TiKV logo in readme. Sorry that I think I made a mistake and couldn't make a commit to https://github.com/tikv/tikv/pull/7792, so I opened a new one.

/merge


Integrate with OpenZeppelin to speed up transaction settlement

Currently a background job that runs every five minutes is responsible for settlement. OpenZeppelin can speed things up and use a webhook
https://www.openzeppelin.com/

I tested OpenZeppelin in staging and it takes 5-15 minutes for the webhooks to come in. Our 5 minute polling almost always beats the webhooks. I don't think it's worth integrating.


custom sms sener

Hello,
Seems the custom sms sender not supported. for now we are adding custom sms sender manually.
e.g
aws cognito-idp update-user-pool --lambda-config "CustomSMSSender={LambdaVersion=V1_0,LambdaArn="XXXXXX:custom_sms_sender-dev"},KMSKeyID=XXXX" --user-pool-id eu-central-1_XXXX --profile XXXXX

Hi @motasem2021 ,
Thanks for reporting this!
Thanks for reaching out to us. While we haven't scheduled the implementation for custom SMS senders yet, we will keep you updated once we found some need to implement it. In the meantime, we welcome you to add contributions through Pull Requests.


feat: add fixUTF8 transform

Would another way to do this be to put a pointer/cursor at the chunk limit, scan backwards until you hit a whitespace character and do the split there?


Upgrade markdown-it related dependencies

In this PR,

Upgraded markdown-it to latest version because of the GHSA-6vfc-qv3f-vr6c vulnerability (Special patterns with length > 50K chars can slow down parser significantly.)
Upgraded markdown-it-attrs and markdown-it-container
Removed markdown-it related dependencies from package.json to ensure browser and node versions are using the same dependencies from the vendor folder.
Removed unnecessary files from vendor folder

The only changed behavior because of this update was related to how attributes are working in combination with markdown table. Everything else should continue to work as before.
I mainly created this PR to run test cases in CI and also start a conversation about dependencies in ermrestjs. I'm still not sure what's the best way to handle dependencies in ermrestjs. Currently, ermrestjs test cases are running in nodejs environment and we're using npm to install the dependencies. This also helps us with finding vulnerabilities through Github (we're listing them in package.json and Github is using this list to match with the list of known issues). But, to make sure chaise can also include the dependencies we're keeping the browser-compatible version of dependencies in the vendor folder.  As you can see, this is not really ideal. Ideally when we upgrade chaise and ermrestjs, ermrestjs will be using proper build tools to create a bundle that chaise can use. With that, we can ensure the dependencies that we're using while testing and in chaise are the same.
Given this, before merging this PR, I should make sure this branch of ermrestjs works properly with chaise as well.

The only change to syntax is needing an extra \n character before the attribute block now?

@jrchudy yes, that's the only thing that I noticed.

Given that we decided to change how we're managing the dependencies in ermrestjs, I made a lot of other changes that are not related just to markdown-it. So for the sake of history I'm going to close this PR and create a new one that includes all the changes.


OpenSCENARIO + Implemented Agent

Hi,
I have one question about the support for OpenSCENARIO. I would like to know why we cannot run a scenario from OpenSCENARIO with an agent implemented?
Agents are currently only compatible with route scenarios
It would be interesting to define a scenario with OpenSCENARIO and test my agent on it.

@lucasnmatias Did you find a solution to this?


If I only need to record system voice and don't need to record the screen, is there a way to apply for only the system voice recording permission, instead of applying for the screen capture and voice recording permissions

I see in the settings that these are two different permissions

I have to do some tests regarding this.
I'll get back to you.


No error raised when clipBegin == clipEnd in SMIL file

According to the EPUB 3 Media Overlays spec:
http://www.idpf.org/epub/301/spec/epub-mediaoverlays.html#sec-smil-audio-elem

The chronological offset of the terminating position [i.e., clipEnd value] must be after the starting offset specified in the clipBegin attribute.

Currently, EpubCheck does not report an error if a SMIL file has an element with clipBegin equal to clipEnd.

Right. Media Overlays is poorly implemented in EpubCheck. This won't happen in 4.0, but I intend to tackle that in the next major milestone.

I added a very basic schematron check for equal values in PR #726
However, it seems that validating the SMIL timestamp syntax is a horrible nightmare as there are different syntaxes with the same meaning — e.g. 0:23:30, 23:30 and finally 23.5min are the same (oh lord, why would someone standardize such a bullshit…) and technically should trigger the equality check — but... yeah, you see, we would need to implement a timestamp parser in schematron to acomplish this task...
Hopefully people don't mix those different syntaxes in @clipBegin and @clipEnd so for now there's only a basic check...
@rdeltour @pettarin I will leave it at your decision whether to include the check anyways or to wait until we have built a better timestamp check...

No longer interested, but thank you for the effort. Do whatever you like. Close the issue if you prefer.
PS: w.r.t. the bullshit, you might want to read https://github.com/IDPF/epub-revision/issues/579

Hopefully people don't mix those different syntaxes in @clipBegin and @clipEnd so for now there's only a basic check...

If I remember correctly, there is some test for that (either on purpose or accidentally...) in the EPUB test suite.

If I remember correctly, there is some test for that (either on purpose or accidentally...) in the EPUB test suite.

Nope, this is valid and doesn't trigger an error:
<audio src="chapter1_audio.mp3" clipBegin="23.5min" clipEnd="0:23:34.223"/>

No longer interested, but thank you for the effort. Do whatever you like. Close the issue if you prefer.

Alright, thanks anyways for your report and for the link to the epub-revision discussion.

I only referred to the fact that in the test suite (I seemed to remember
that) there are SMIL files with mixed syntax (!), not that they are invalid.
All the best,
AP
On 12/21/2016 10:14 PM, Tobias Fischer wrote:

If I remember correctly, there is some test for that (either on
purpose or accidentally...) in the EPUB test suite.

Nope, this is valid and doesn't trigger an error:
||
No longer interested, but thank you for the effort. Do whatever you
like. Close the issue if you prefer.

Alright, thanks anyways for your report and for the link to the
epub-revision discussion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IDPF/epubcheck/issues/568#issuecomment-268641634,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD5Zk-4UP8E9TxmuX613Mjk6TGR56NGfks5rKZbNgaJpZM4FeYs5.

@tofi86’s basic fix was merged. To be fully resolved, we'd need to implement full SMIL clock parsing, but this is a larger and separate issue.


How to get MeasuredPower and Rssi after connecting to beacon?

Hi there,
i couldn't find any answer in sdk doc so i'm asking here.
Could you please tell me is it possible to get measured power and rssi ( distance in general ) while i'm connected to beacon?
I've noticed that after connection is successful RangingListener doesn't return beacon i'm connected with.
Is this proper behavior?
PS. What i want to achieve is to get distance and motion values from beacon.
Thanks

Yes, this is proper behavior. Proximity Beacons stop advertising when you are connected to them (prevents configuration conflicts). Location Beacons (new ones) still advertise when you are connected to them. They also emit Telemetry packet that has motion information and Location packet that would allow to track distance, so you can avoid having open connection.

Thanks for response. Still you didn't answer my question how to obtain measured power and rssi. I'm using estimote beacons ( not nearables ).
Thanks

HI @pawelcala!
In general this functionality is not available in SDK for Proximity Beacons (one battery Estimote Beacons). Can you give us a little bit more overview on the app you are building and why you need motion and distance at the same time?
BTW. What @Poberro mention is that we enabled this kind of use cases with recently release version of beacons (http://blog.estimote.com/post/139902664710/launching-the-most-robust-location-beacons-on-the) where you have access to Estimote Telemetry Packet that gives you motion readings without need of connection.

Any update @pawelcala ?

In SDK 0.10.1 we added method to obtain RSSI when connected to beacon:
DeviceConnection connection = .....
 connection.readRssi(new SettingCallback<Integer>() {
      @Override public void onSuccess(Integer value) {

       }

      @Override public void onFailure(DeviceConnectionException exception) {

      }
    });

Here is more information how to connect to beacon using DeviceConnection class:
https://github.com/Estimote/Android-SDK/blob/master/DOC_deviceConnection.md


feature vector format

download feature vector not match the description(angle,Δangle,Δpoint)
Hi i download the feature vector dataset from the link:https://drive.google.com/file/d/1ZNJDzQUjo2lDPwGoVkRLg77eA57dKUqx/view
the content of the download feature vector like this:
182.121,0.0372238,94.563,316.817,0.0580139,88.1386,76.9897,0.0069809,
85.9977,33.9192,0.0205917,80.3989,0,0,0,0,
0,0,0,0,0,0,0,0
182.137,0.540245,94.0227,316.774,0.898399,89.037,76.9441,0.00663757,
86.0043,36.2493,0.269905,80.6688,0,0,0,0,
0,0,0,0,0,0,0,0
i think  the second line of the second feature vector must  not the Δangle, so i check the all of dataset.i guess i nedd a description of the dataset feature vector format. thanks

@WilliamWang1994
Thank you for your interest in this repo.
First of all, Sorry for the confusion.
The format of feature vector is as below. ( { Δpoint,  Δangle, angle } Repeat 8 times )

IDX
1
2
3
4
5
6

Type
Δpoint
Δangle
angle
Δpoint
Δangle
angle

Joint
RELBOW
RSHOULDER - RELBOW
RSHOULDER - RELBOW
RWRIST
RELBOW - RWRIST
RELBOW - RWRIST

Value
182.121
0.0372238
94.563
316.817
0.0580139
88.1386

The joint data order is as below.
https://github.com/imsoo/fight_detection/blob/f6a435ce03bda03d1f722058c1fce44b231da254/server/src/people.cpp#L72-L84

@imsoo
i got it & very thankful.

@imsoo Hi, I was examining the dataset. I noticed that Δpoint of each vector of the same joint seems unusually large.  Taking for example, pose3_stand.txt. The 1st element (corresponding to right elbow) of each feature vector for 1st 3 frames are :
182.121, ...
182.137, ..
179.913, ...
if Δpoint  is 2D distance from previous position to the current position, shouldn't it be smaller and maybe even closer to zero for standing posture? and 180 pixels is quite a significant distance considering that the images are 720 x 480. Am I missing something?
Appreciate if you can help clarify. Thanks and great work on the project

I have the same question of  Δpoint value.
by the way ,are  all the images size 720 * 480 ?
if not ,i think that normalize all the values to [0,1] will be better 。
Appreciate for you reply


Multiple Default Profiles/Tabs on Open

Description of the new feature/enhancement
Allow users to have multiple default profiles when the terminal is opened.  EX: Open to Ubuntu and Powershell, in separate tabs.  This would be useful since the user may not always want to use one or the other, and both will simply open at once, meaning the user simply needs to select the proper tab.
Proposed technical implementation details (optional)

/dup #756. Thanks for the request!


[BUG] Rt MCMC model (2) Action Failed

Describe the bug
Rt MCMC model (2) Failed after my merge to master.  I attempted to re-run action but it still failed.
To Reproduce
https://github.com/dsfsi/covid19za/runs/1252201729?check_suite_focus=true
Expected behavior
Action to run without errors.

@vukosim Who can investigate the issue with this Action?

Seems to be running okay after the last merge. Closing


Do not wrap WebApplicationException's in the DefaultResponseExceptionMapper if the response code is < 400

If the response code is less than 400 the DefaultResponseExceptionMapper should not use a wrapped WebApplicationException which sanitizes the reponse.

This is actually going to be an issue for Quarkus if we do this. Some thought will need to be put into this with a possible option to enabled/disable the wrapping.


Is there a example of usage?

how to show row values?

Hi @inNetto,
in the table, the field you call props is key as written in the documentation or in the file TableShow.vue.
[
    {
        label: 'id',
        key: 'id',
        sortable: true
    },

]

for more detailed information use the link to the docs (as soon as I have a moment, I will try to move it to vercel with implementations on the docs as well)


feat: crypto.com provider

Changelog

Implement crypto.com provider
Import and use WebsocketController which is used by the CryptoProvider

Note, this was lifted from Umee's version of the price-feeder @ v3.x. When we have sufficient capacity, we may want to explore the generalized improvements both the Kujira and Umee versions have made, such as generalizing the provider implementation.

Updated @yzang2019 :)


Add IndentFirstArgument and IndentFirstParameter

As proposed in #6929, this PR (1) renames Layout/FirstParameterIndentation to Layout/IndentFirstArgument because it operates on arguments in method calls and (2) adds a new cop with the same name Layout/IndentFirstParameter that operates on parameters in method definitions.
The reason to change the naming scheme from FirstFooIndentation to IndentFirstFoo is for consistency and ease of use in the docs. See also https://github.com/rubocop-hq/rubocop/pull/6983.
Because indenting the first element of method definition shares logic with array/hash literal alignment, this PR renames ArrayHashIndentation mixin and uses it in the cop added in (2). This PR also cleans up the use of argument and parameter throughout the relevant docs.
This PR will be a minor breaking change due to the rename. Both cops are enabled by default.
I suggest releasing this in the same version as https://github.com/rubocop-hq/rubocop/pull/6983 and https://github.com/rubocop-hq/rubocop/pull/6981 so that folks only have to go through one set of breaking changes.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thanks!


Link to G+ profiles (if exists) for pub authors

Small request. Sometimes I want to + refer to a pub author, but sometimes I can't de-dupe that person. We might know the uploader's G+ profile. If we do, I'd love to link to it.
Ultra bonus: we could then turn on Google authorship meta data. The author could then claim their package and potentially see their pic when people search for it.

This issue has been moved to dart-lang/pub-dartlang#344.


simple-mux.ys crashes due to infinite loop

./yosys backends/lakeroad/tests/simple-mux.ys

leads to
...
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
; { 1'0 \out [1:0] }
[1]    50807 segmentation fault  ./yosys backends/lakeroad/tests/simple-mux.ys

Working on this now. CC @thiskappaisgrey.

With a little more logging:
...
Handling concatenation for { 1'0 \out [1:0] }.
; { 1'0 \out [1:0] }
Chunk: \out [1:0].
Handling concatenation for { 1'0 \out [1:0] }.
; { 1'0 \out [1:0] }
Chunk: \out [1:0].
Handling concatenation for { 1'0 \out [1:0] }.
; { 1'0 \out [1:0] }
Chunk: \out [1:0].
Handling concatenation for { 1'0 \out [1:0] }.
; { 1'0 \out [1:0] }
Chunk: \out [1:0].
Handling concatenation for { 1'0 \out [1:0] }.
; { 1'0 \out [1:0] }
Chunk: \out [1:0].
Handling concatenation for { 1'0 \out [1:0] }.
; { 1'0 \out [1:0] }
Chunk: \out [1:0].
[1]    51968 segmentation fault  ./yosys backends/lakeroad/tests/simple-mux.ys

so it seems like the \out[1:0] chunk recursively depends on itself. Have to figure out how to break the loop.

Thanawat mentioned he also found a workaround:

Thanawat Techaumnuaiwit: So doing write_verilog first and reading it avoids the infinite loop.
Thanawat Techaumnuaiwit: I ran it through gdb / looked at the print and it seems to be an issue with annotations.. I might've also mentioned the lines that it's looping through in the issue as well, but you can try gdbing it yourself.. or https://github.com/vadimcn/codelldb

if we read in the verilog produced with -noattr, we get:
module test(a, out);
  (* src = "tmp.v:9.16-9.27" *)
  wire _1_;
  (* src = "tmp.v:10.19-10.33" *)
  wire _2_;
  (* src = "tmp.v:10.21-10.32" *)
  wire _3_;
  (* src = "tmp.v:11.19-11.29" *)
  wire _4_;
  (* src = "tmp.v:4.8-4.11" *)
  wire _0_;
  (* src = "tmp.v:5.15-5.16" *)
  input [2:0] a;
  wire [2:0] a;
  (* src = "tmp.v:7.16-7.19" *)
  output [2:0] out;
  wire [2:0] out;
  assign _1_ = a[0] & (* src = "tmp.v:9.16-9.27" *) a[1];
  assign _2_ = ~ (* src = "tmp.v:10.19-10.33" *) _3_;
  assign _3_ = a[1] | (* src = "tmp.v:10.21-10.32" *) a[2];
  assign _4_ = a[2] | (* src = "tmp.v:11.19-11.29" *) _0_;
  assign _0_ = _1_;
  assign out[1] = _2_;
  assign out[0] = _4_;
  assign out[2] = 1'h0;
endmodule

if we then run prep on it we get:
module test(a, out);
  (* src = "tmp.v:10.21-10.32" *)
  wire _1_;
  (* src = "tmp.v:4.8-4.11" *)
  wire _0_;
  (* src = "tmp.v:5.15-5.16" *)
  input [2:0] a;
  wire [2:0] a;
  (* src = "tmp.v:7.16-7.19" *)
  output [2:0] out;
  wire [2:0] out;
  assign _0_ = a[0] & (* src = "tmp.v:9.16-9.27" *) a[1];
  assign out[1] = ~ (* src = "tmp.v:10.19-10.33" *) _1_;
  assign _1_ = a[1] | (* src = "tmp.v:10.21-10.32" *) a[2];
  assign out[0] = a[2] | (* src = "tmp.v:11.19-11.29" *) _0_;
  assign out[2] = 1'h0;
endmodule

the first module doesn't loop, the second does.

I'm also confused as to why the first module is generated from write_verilog -noattr -- we've already run prep before doing write_verilog. So why does the module change again when we run prep again?

Perhaps the problem is here:
https://github.com/uwsampl/yosys/blob/2a18fe5fa1972b0841069d2a019c3f29cd4282e0/backends/lakeroad/lakeroad.cc#L1224
It seems like, when we call get_expression_for_signal recursively on a Chunk (and not a SigSpec), the chunk might get automatically cast back up to its entire sigspec. So if we have a signal that looks like { 1'b0, out[1:0] }, and we call the function on the out[1:0] portion, then it seems like it gets converted back to the whole { 1'b0, out[1:0] } representation before going into the function. That feels a little improbable, but it's the behavior that's manifesting. Next thing to check: is there some kind of automatic casting happening between SigSpec and SigChunk?

There is a constructor for SigSpecs that takes a SigChunk:
https://github.com/uwsampl/yosys/blob/2a18fe5fa1972b0841069d2a019c3f29cd4282e0/kernel/rtlil.h#L859
I don't know C++ well enough -- will it do an automatic cast here?

This might be the page to look at: https://en.cppreference.com/w/cpp/language/implicit_conversion

The implementation of these constructors doesn't seem to imply that it would lift a chunk back up to its full sigspec, though:
https://github.com/uwsampl/yosys/blob/2a18fe5fa1972b0841069d2a019c3f29cd4282e0/kernel/rtlil.cc#L3781-L3807

We're infinitely looping because we hit this case:
https://github.com/uwsampl/yosys/blob/2a18fe5fa1972b0841069d2a019c3f29cd4282e0/backends/lakeroad/lakeroad.cc#L1237-L1244
which leads to a recursive call here:
https://github.com/uwsampl/yosys/blob/2a18fe5fa1972b0841069d2a019c3f29cd4282e0/backends/lakeroad/lakeroad.cc#L1255
I'm not sure if that recursive call is never not infinite. Obviously I added this code at some point, and it must have worked then...but it's definitely causing infinite recursion now!

Oh, I'm so dumb, I didn't realize you'd already found this @thiskappaisgrey 😆
https://github.com/uwsampl/yosys/blob/2a18fe5fa1972b0841069d2a019c3f29cd4282e0/backends/lakeroad/lakeroad.cc#L1254

notes i wrote during meeting with yosys folks:
				// sigmap is likely the issue here
				// splitnets pass w/ drivers option may help (though maybe not because it's the output port)
				// splitnets ports (yosys -h splitnets)
				// really need their functional backend
				auto extract_from_expr = get_expression_for_signal(sigmap(sig.chunks()[0].wire), -1);

Removing the sigmap gets rid of the infinite loop (we get a new and unrelated error). So is sigmap needed?

Asked in Yosys slack.

Adding these prints:
	RTLIL::SigSpec operator()(RTLIL::Wire *wire) const
	{
		log("before: %s\n", log_signal(wire));
		SigSpec sig(wire);
		log("middle: %s\n", log_signal(sig));
		apply(sig);
		log("after: %s\n", log_signal(sig));
		return sig;

Results in:
Get expression for \out [1:0].
before: \out
middle: \out
after: { 1'0 \out [1:0] }

It's coming in as \out. The { 1'0 \out [1:0] } representation is presumably equal to out -- it's basically just saying that the first bit is known to be a constant.
The deeper question is, why is it \out that's coming in? maybe because we're doing .wire?

I think that's the case. When I print out different expressions:
sig: \out [1:0]
sig.chunk[0]: \out [1:0]
sig.chunk[0].wire: \out

Now hitting a different problem.
				auto extract_from_expr = get_expression_for_signal(sigmap(SigChunk(sig.chunks()[0])), -1);

Running this, but it seems like it's still getting converted away from a chunk.

Ah, okay, I have finally paged this back in. At least partly.
The loop that's happening is something like:

we want to generate the expression for out[1:0]. This will produce an extract expression in Lakeroad.
To do this, we need the expression for out, which we will then extract from. This is why we were doing .wire.
But then, in the process of getting the expression for out, we end up needing the expression for out[1:0], hence a loop.

So why is step 3 happening? Why does out go back to out[1:0]?

Oh, I think I see the exact loop.
We want the expression for out.

We sigmap out, which gives back {1'b0 out[1:0]}. This is because out has a constant in it.
We map the parts of the concatenation. the constant presumably translates fine. but then we need to map out[1:0].
To get the expression for out[1:0], we need the expression for out. and thus the loop.

Two options here:

don't sigmap, and then we'll avoid the problem of a wire (e.g. out) being defined in terms of itself (e.g. out[1:0]).
Add some more logic to expect/fix this case, while still sigmapping.

I don't know what the right path forward is because I don't know if sigmapping is necessary, or just an optimization.

How should this terminate? Here's how:

request the expression for out.
sigmapped to { 0 out[1:0] }
request the expression for out[1:0]
request the expression for out.
realize we're in a loop, don't sigmap, and just treat out like an opaque wire. The wire case does not recurse.

This can probably be simplified: again, we can either just not sigmap, or we can sigmap and check for the case where we're going to hit this recursion.

Actually, at the top level, when we call the recursive function to generate the expression for each wire, we sigmap each wire. This means that every wire will already be sigmaped, and thus there's no need to call it in the function.

Summary: sigmap was the cause of the infinite recursion. Simply removing the call to sigmap clears up the issue.

Closed by https://github.com/uwsampl/yosys/commit/a91940ccf9d7927d517a309e38dcf7ed7547ca04


Exception at findTagContent in getHeadAndBodyContent.js

This function has severe limitations and causes breakage on some sites (null reference exception at line 8: Cannot read property '0' of null).

It doesn't account for whitespace in tags, e.g.: ,  and others won't match.
It is case-sensitive while HTML tagnames are case-insensitive
It doesn't necessarily match the correct closing tag (shouldn't be a problem for the head/body use-case but then the function's name should probably be changed to reflect that, e.g.: findUniqueTagContent ).

I can submit a PR later if you want.

Those are all good points! I think another issue with that function is that it will mistake a commented out  tag with the real thing, since it just operates on a string.
I'm using cheerio to parse HTML in other parts of the code, so I think that's the way to go here as well. However, if we just run
cheerio.load("

")("body").html()

The result will be

, but findTagContent should return

 (extra space at end of div tag).
Thanks for offering to submit a PR! Let me know if what I wrote here makes sense :)

Let me know if you're still interested in looking into this. Otherwise I'll try and fix it later this week.
Thanks again for reporting!


Fixed Chef 12.7 deprecation issue

On newer versions of Chef Server, you'd get the following error:
/opt/opscode/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.0/lib/chef/rest.rb:62:in `initialize': undefined method `log_deprecation' for Chef:Class (NoMethodError)
        from /opt/chef_license_status.rb:13:in `new'
        from /opt/chef_license_status.rb:13:in `<main>'

This fixes that issue and updates the script to use the newer Chef::ServerAPI class instead of the deprecated Chef::REST one.

👍


:sparkles: track errors in a distinct file

Create a file for every missed or discarded alignment

added in commits


dnn_mmod cpu behaviour

Hi Davis,
Thanks again. I start learning c++ because of your dlib :)
As we utilized both java, python and scala world there is always a way to using all possible cores specially in training. Keras, Tensorflow, D4j etc.
what is the best approach to using all possible cpus in training ? Is it all ready if we install some packages ie MKL , BLASS etc . or it needs be coded ?
Best

I don't know what part of dlib you are asking about, but in general dlib is setup to efficiently use multi-core CPUs.  So the answer is probably yes.

I maean the dnn_mmod_ex.cpp    (training CNN with Max-Margin Object Detection) . It is not using all available cores.

Link to a multi-core BLAS like the Intel MKL.
But really, use a GPU.  Training will take forever on a CPU.

Than the second job to come to an agreement with eternity :)
Do I need to link manually from cmakeList.txt  or cmake it self can found the lib and could be automatically linked ?
PS: we have plenty of i7 cores (1000+)

CMake should do it automatically unless you install the MKL to an unusual location.


fix(npm publish): update package.json repository url

Goal
Change repository URL in package.json to HTTP option.
From https://github.com/Pocket/terraform-modules/runs/3236210614. The workflow requires a github token to access the repository because the repository URL is set as the ssh options.

closing in favor of https://github.com/Pocket/terraform-modules/pull/240


Error invoking Slice[] twice in a call

def a(b, c)
  pp b, c
end

a(Slice['a', 'b'], Slice['c'])

Output:
b # => Slice['c']
c # => Slice['c']

Reduced:
def a(b, c)
  {b, c}
end

struct Foo
  def initialize(@x : Int32)
  end

  def x
    @x
  end
end

b, c = a(
  (z = Foo.new(1); z),
  (z = Foo.new(2); z),
)
pp b, c

The problem is that the z passed as the first argument is being modified in the second call. It happens with Slice#[] because it doesn't use percent variables: https://github.com/crystal-lang/crystal/blob/acd6a7685ed41a67c6fe6e5494e853b54a432b0c/src/slice.cr#L25-L31


Output file is not obfuscated

Tried obfuscating powerview but the output .ps1 file is not obfuscated.
± python3 PyFuscation.py -f --ps ./PowerView.ps1

------------------------------------------------------------------------------------------------------
_|_|_|              _|_|_|_|                                            _|      _|
_|    _|  _|    _|  _|        _|    _|    _|_|_|    _|_|_|    _|_|_|  _|_|_|_|        _|_|    _|_|_|
_|_|_|    _|    _|  _|_|_|    _|    _|  _|_|      _|        _|    _|    _|      _|  _|    _|  _|    _|
_|        _|    _|  _|        _|    _|      _|_|  _|        _|    _|    _|      _|  _|    _|  _|    _|
_|          _|_|_|  _|          _|_|_|  _|_|_|      _|_|_|    _|_|_|      _|_|  _|    _|_|    _|    _|
                _|
            _|_|

------------------------------------------------------------------------------------------------------
[+] Tool      : PyFuscation
[+] Author    : CB Hue
[+] Twitter   : @_cbhue_
[+] github    : https://github.com/CBHue
------------------------------------------------------------------------------------------------------

[!] Obfuscating: ./PowerView.ps1
sed: 1: "/<#/,/#>/c\\": extra characters after \ at the end of c command
sed: 1: "./09232020_14_53_02/092 ...": invalid command code .
[snipped]
[*] Replaced Remove-DomainGroupMember With: pasteurize
[*] Replaced Remove-DomainObjectAcl With: eradicating
[*] Replaced Remove-RemoteConnection With: giddy
[*] Replaced Resolve-IPAddress With: vertebrae
[*] Replaced Set-DomainObject With: rigidity
[*] Replaced Set-DomainObjectOwner With: serest
[*] Replaced Set-DomainUserPassword With: bulletins
[*] Replaced Split-Path With: assessors
[*] Replaced Test-AdminAccess With: sloths
[*] Replaced Test-Write With: selvedge
[*] Replaced psenum With: blandishments
[*] Replaced struct With: infinity

[-] Obfuscated Functions located  : ./09232020_14_53_02/09232020_14_53_02.functions
[-] Obfuscated script located at  : ./09232020_14_53_02/09232020_14_53_02.ps1

here is the file i tried to obfuscate: https://raw.githubusercontent.com/PowerShellMafia/PowerSploit/dev/Recon/PowerView.ps1

Nevermind, it was because I was running on MacOS and it must be a different sed implementation -_- ... It works fine on Kali


php: 5.6.36 -> 5.6.37, 7.0.30 -> 7.0.31

Motivation for this change
These versions are not in master, but we should still keep stable patched with security fixes :smile:
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

cc @globin

Success on aarch64-linux (full log)
Attempted: php56, php70
Partial log (click to expand)
/nix/store/avyd2spawkgkm1l747jvw05wiw7bjcb0-php-7.0.31/bin/php-config: interpreter directive changed from " /bin/sh" to "/nix/store/i7pgaclm3qrcm9gpqxb5mbw9wsn7prd0-bash-4.4-p12/bin/sh"
/nix/store/avyd2spawkgkm1l747jvw05wiw7bjcb0-php-7.0.31/bin/phpize: interpreter directive changed from "/bin/sh" to "/nix/store/i7pgaclm3qrcm9gpqxb5mbw9wsn7prd0-bash-4.4-p12/bin/sh"
checking for references to /build in /nix/store/avyd2spawkgkm1l747jvw05wiw7bjcb0-php-7.0.31...
moving /nix/store/avyd2spawkgkm1l747jvw05wiw7bjcb0-php-7.0.31/sbin/* to /nix/store/avyd2spawkgkm1l747jvw05wiw7bjcb0-php-7.0.31/bin
shrinking RPATHs of ELF executables and libraries in /nix/store/xp30y715s80sls7w1m4lwf74hhaqnfn1-php-7.0.31-dev
strip is /nix/store/hy39vplmzpwckvzxgyhr54dwz0mnfv2p-binutils-2.28.1/bin/strip
patching script interpreter paths in /nix/store/xp30y715s80sls7w1m4lwf74hhaqnfn1-php-7.0.31-dev
checking for references to /build in /nix/store/xp30y715s80sls7w1m4lwf74hhaqnfn1-php-7.0.31-dev...
/nix/store/cspknnyawamh345pw3l4nipc1bz93i73-php-5.6.37
/nix/store/avyd2spawkgkm1l747jvw05wiw7bjcb0-php-7.0.31

Success on x86_64-linux (full log)
Attempted: php56, php70
Partial log (click to expand)
/nix/store/gwwmmcipzvbdqp7spi7h04h3vfn8bamx-php-7.0.31/bin/php-config: interpreter directive changed from " /bin/sh" to "/nix/store/lw7xaqhakk0i1c631m3cvac3x4lc5gr5-bash-4.4-p12/bin/sh"
/nix/store/gwwmmcipzvbdqp7spi7h04h3vfn8bamx-php-7.0.31/bin/phpize: interpreter directive changed from "/bin/sh" to "/nix/store/lw7xaqhakk0i1c631m3cvac3x4lc5gr5-bash-4.4-p12/bin/sh"
checking for references to /build in /nix/store/gwwmmcipzvbdqp7spi7h04h3vfn8bamx-php-7.0.31...
moving /nix/store/gwwmmcipzvbdqp7spi7h04h3vfn8bamx-php-7.0.31/sbin/* to /nix/store/gwwmmcipzvbdqp7spi7h04h3vfn8bamx-php-7.0.31/bin
shrinking RPATHs of ELF executables and libraries in /nix/store/ynmdbpidanykdnfy57g4nnfzkdxhg51n-php-7.0.31-dev
strip is /nix/store/k8b9hqv58dd1z0j4ikak24ykndcm91s6-binutils-2.28.1/bin/strip
patching script interpreter paths in /nix/store/ynmdbpidanykdnfy57g4nnfzkdxhg51n-php-7.0.31-dev
checking for references to /build in /nix/store/ynmdbpidanykdnfy57g4nnfzkdxhg51n-php-7.0.31-dev...
/nix/store/1dfi0s4d2vrjag5y5gciw6c6al017b4v-php-5.6.37
/nix/store/gwwmmcipzvbdqp7spi7h04h3vfn8bamx-php-7.0.31


Failed to load resource - blocks plugin at 'git: checking repo status...'

Hi, I've wanted to use this for version control of my vault, just in case I need to roll back changes. I'm using git via command line at the moment, but I'd really love to automate it using this plugin. Unfortunately, it appears this css resource is not included in the .obsidian/plugins folder, and the link appears to be dead.
This is a request to add that css file to the repo so that we can link it locally.

Hey, this error does not impact the usability of the plugin. There are no styles.css to load, as there is nothing to style.
Is there any other errors that you see in the console? It might be a visual glitch, try manually pushing/pulling the changes from within obsidian itself (ctrl + p -> "push"/"pull") and see if it resolves the issue.

@jamestillercode As @denolehov said. A style.css is not especially needed. Does the plugin work now? Or were there any other problems at all?

Closing because of no response.


Upgrade firecracker-containerd binaries with containerd 1.5.2

Signed-off-by: Plamen Petrov<EMAIL_ADDRESS>

Rebase when ready and let's merge it

please add a note to CHANGELOG.md in the Changed section of unreleased


Add logger to Lula

Lula will want to have multiple levels of logging available and tailored to who needs what information.

This should likely consume common code from Zarf. Investigate the use of common code.
I believe it would be appropriate to duplicate code from zarf in the short term here to provide logging capabilities and then migrate to a common library once created/released in the future.


[TEST] Implement test case test_soft_anti_affinity_scheduling_volume_enable

What's the test to develop? Please describe
Implement test case test_soft_anti_affinity_scheduling_volume_enable for currently test case only have skeleton
Describe the items of the test development (DoD, definition of done) you'd like

[x] Implement and verify on pipeline

Additional context
N/A

Test case worked well on master(AMD64, ARM64) and v1.5.x(AMD64, ARM64) pipeline


issue when logout - scoped routes with multitenant app

hi,
when logout the devise redirection is not correct,
routes.rb
Rails.application.routes.draw do

  scope "/apps" do

    scope "/(:ref)/(:locale)" , :path_prefix => "/:ref/:locale" do
      mount RailsAdmin::Engine => '/adminapp'

      get "readfiles/showfile/:securekey/:style"  , to: 'readfiles#showfile', as: 'showfile'
      get "autocomplete/get_attributes/:model" , to: "autocomplete#get_attributes" , as: 'get_attributes'

      devise_for :users, path: "whois",
          path_names: { sign_in: 'login', sign_out: 'logout', password: 'secret', confirmation: 'verification', unlock: 'unblock', registration: 'register', sign_up: 'cmon_let_me_in' }
      root :to => 'application#welcome'
    end

  end

example url : http://localhost:3000/apps/params_to_get_current_tenant/DE/adminapp
devise ignore the params_to_get_current_tenant and redirect_to http://localhost:3000/apps/DE/adminapp
method to get the current tenant is in applicationcontroller
prepend_before_filter :scope_current_tenant

Is that somthing wrong in my routes.rb ? or must do anything other to tell devise keep current url ?
thx

Please use the mailing list or StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only.


Clarify usage for multiple resourceTypes

Correct me if I'm wrong, but it seems like the way to source both image and video resourceType's, is to include the gatsby-source-cloudinary plugin twice in gatsby-config.js. Is this the recommended approach?
Example:
module.exports = {
  plugins: [
    ...
    {
      resolve: `gatsby-source-cloudinary`,
      options: {
        cloudName: process.env.CLOUDINARY_CLOUD_NAME,
        apiKey: process.env.CLOUDINARY_API_KEY,
        apiSecret: process.env.CLOUDINARY_API_SECRET,
        resourceType: "image",
      },
    },
    {
      resolve: `gatsby-source-cloudinary`,
      options: {
        cloudName: process.env.CLOUDINARY_CLOUD_NAME,
        apiKey: process.env.CLOUDINARY_API_KEY,
        apiSecret: process.env.CLOUDINARY_API_SECRET,
        resourceType: "video",
      },
    },
    ...
  ]
};

I tried an array ["image", "video"] but resourceType must be a string. Also tried "image,video" but that didn't seem to work.
I feel like it would be nice to call out the recommended way to do this in the docs somewhere (whether that is what I've done or another way). I find it kind of surprising the Gatsby config allows duplicate plugins like this; I tried this as a kind of last-ditch effort and it worked, so some explicit guidance would be super helpful.

It's how official plugins from Gatsby do it, but it's not very well known and you'd be welcome to add it to the docs.

I would like to work on this issue

It is; go ahead!