validate password complexity in the front end

Details

We were calling the authenticate command even when the password was too short, I have added a password complexity check. Now the authenticate command won't be called unless the user enters a valid password.
Fixed Issues

$ https://github.com/Expensify/App/issues/8794
Tests | QA Steps

Go to login page on web or desktop
Add your email or phone number
Open the network tab on the browser or desktop app
Enter a 4-5 character long password
Verify you see an error and verify that you don't see any new request in the network tab

[x] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (PR Author) Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there’s a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained “why” the code was doing something instead of only explaining “what” the code was doing.
[x] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named “index.js”. All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[ ] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose and it is

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn’t already exist
[ ] The style can’t be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.

PR Reviewer Checklist

[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there’s a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained “why” the code was doing something instead of only explaining “what” the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named “index.js”. All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn’t already exist
[ ] The style can’t be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.

Screenshots

Web
https://user-images.githubusercontent.com/83179295/166814644-7414d52a-fee1-4697-a0d1-e752e3042315.mov

Mobile Web

Desktop

https://user-images.githubusercontent.com/83179295/166814637-858e477d-b71b-4f19-8233-61b763fba856.mov
iOS

Android

#### PR Reviewer Checklist

- [x] I verified the PR has a small number of commits behind `main`
- [x] I verified the correct issue is linked in the `### Fixed Issues` section above
- [x] I verified testing steps are clear and they cover the changes made in this PR
    - [ ] I verified the testing environment is mentioned in the test steps
- [ ] I verified testing steps cover success & fail scenarios (if applicable)
- [ ] I checked that screenshots or videos are included for tests on [all platforms](https://github.com/Expensify/App/blob/main/CONTRIBUTING.md#make-sure-you-can-test-on-all-platforms)
- [ ] I verified tests pass on **all platforms** & I tested again on:
    - [ ] iOS / native
    - [ ] Android / native
    - [ ] iOS / Safari
    - [ ] Android / Chrome
    - [ ] MacOS / Chrome
    - [ ] MacOS / Desktop
- [ ] I verified there are no console errors related to changes in this PR
- [ ] I verified proper code patterns were followed (see [Reviewing the code](https://github.com/Expensify/App/blob/main/PR_REVIEW_GUIDELINES.md#reviewing-the-code))
    - [ ] I verified comments were added when the code was not self explanatory
    - [ ] I verified any copy / text shown in the product was added in all `src/languages/*` files (if applicable)
    - [ ] I verified proper naming convention for platform-specific files was followed (if applicable)
    - [ ] I verified [style guidelines](https://github.com/Expensify/App/blob/main/STYLING.md) were followed
    - [ ] I verified the JSDocs style guidelines (in [`STYLE.md`](https://github.com/Expensify/App/blob/main/STYLE.md#jsdocs)) were followed
- [ ] I verified that this PR follows the guidelines as stated in the [Review Guidelines](https://github.com/Expensify/App/blob/main/PR_REVIEW_GUIDELINES.md)
- [ ] I verified other components are not impacted by changes in this PR (i.e. if the PR modifies a shared library or component like `Avatar`, I verified the components using `Avatar` are working as expected)
- [ ] I verified the UI performance was not affected (the performance is the same than `main` branch)
- [ ] If a new component is created I verified that a similar component doesn't exist in the codebase

PR Review. @AndrewGable Good to merge.
🎀 👀 🎀 C+ reviewed

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @AndrewGable in version: 1.1.58-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by @AndrewGable in version: 1.1.60-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Fixed typo in in-line comments.

LGTM, thanks

May you please get the same fix into more-teapots and choreographer-30fps too?  thank you so much!


Use u32 key in HEIGHT_TO_HASH table

This is a marginal improvement, and since it will require a database rebuild, should be bundled with another change that requires a rebuild.
We currently use a u64 height in the HEIGHT_TO_HASH table, but a u32 would be more than enough.

3 megs who cares


commit

Current dependencies on/for this PR:

main

PR #16

PR #17   👈

This comment was auto-generated by Graphite.


Enable Plan Upgrades

Summary

Checklist

[x] Keep pull requests small so they can be easily reviewed.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking,
as they show up in the changelog
[x] Update the documentation.
[x] Update tests.

With the new requirements I needed to completely restructure the upgrade validity check. I might have gone a little overboard with how generic the solution is.
We can now allow any subset of plan size changes with a white-list approach:
OSB_PLAN_UPDATE_SIZE_RULES="xsmall>small|medium>large"

This would only allow plan changes from xsmall to small and from medium to large.  I will definitely document this thoroughly, but maybe its generally a bad idea, too generic, and too confusing?


closes #3

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@acdeb3b). Click here to learn what that means.
 Consider uploading reports for the commit 8fd93e8 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   59.77%
  Complexity        ?       23
=========================================
  Files             ?        9
  Lines             ?       87
  Branches          ?        4
=========================================
  Hits              ?       52
  Misses            ?       32
  Partials          ?        3

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@rultor merge

@l3r8yj OK, I'll try to merge now. You can check the progress of the merge here


Localization

Localization for Pangolin Desktop.
It works, but I'd say it's still WIP because it currently has only two languages and the language can only be changed in the Quicksettings  and not everything is localized yet. I'll try to localize everything - done.
If you want to localize for your language you can already create a json file for your language based on en.json.
Thanks to @Horus125 for the help.

[x] Implement localization API
[x] Create Localization json for en and de
[x] Localize complete desktop
[x] Localize date format
[x] Remember selected language

Wow, nice


zipalign fastlane finished with errors

I'm getting the following issue when using zipaling command:
zipalign(apk_path: "#{lane_context[SharedValues::SIGNED_APK_PATH]}")
--------------------------------ERROR------------------------------------
Found no similar issues. To create a new issue, please visit: https://github.com/fastlane/fastlane/issues/new Run fastlane envto append the fastlane environment to your issue /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/helper/sh_helper.rb:36:inpopen': [!] No such file or directory - zipalign (Errno::ENOENT)
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/helper/sh_helper.rb:36:in sh_control_output' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/helper/sh_helper.rb:10:in sh'
from /Users/mariano/Documents/Android_Projects/AppFree/fastlane/actions/zipalign.rb:23:in run' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/runner.rb:243:in block (2 levels) in execute_action'
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/actions/actions_helper.rb:50:in execute_action' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/runner.rb:221:in block in execute_action'
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/runner.rb:217:in chdir' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/runner.rb:217:in execute_action'
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/runner.rb:139:in trigger_action_by_name' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/fast_file.rb:146:in method_missing'
from Fastfile:42:in block in parsing_binding' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/lane.rb:33:in call'
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/runner.rb:49:in block in execute' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/runner.rb:45:in chdir'
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/runner.rb:45:in execute' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/lane_manager.rb:52:in cruise_lane'
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/command_line_handler.rb:30:in handle' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/commands_generator.rb:105:in block (2 levels) in run'
from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.1/lib/commander/command.rb:178:in call' 	from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.1/lib/commander/command.rb:153:in run'
from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.1/lib/commander/runner.rb:444:in run_active_command' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:38:in run!'
from /usr/local/lib/ruby/gems/2.3.0/gems/commander-4.4.1/lib/commander/delegates.rb:15:in run!' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/commands_generator.rb:302:in run'
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/commands_generator.rb:42:in start' 	from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/fastlane/lib/fastlane/cli_tools_distributor.rb:65:in take_off'
from /usr/local/lib/ruby/gems/2.3.0/gems/fastlane-2.19.1/bin/fastlane:15:in <top (required)>' 	from /usr/local/bin/fastlane:22:in load'
from /usr/local/bin/fastlane:22:in <main>'

Environment issue, if anyone have the same issue, follow this and you'll be ok:
http://stackoverflow.com/a/30912162


[#4946] Sharing status shouldn't be modifiable during metadata review

Resolves #4958
Resolves #4946

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[x] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Create a resource and add metadata to make public
Click the PUBLISH button and accept in the modals to put the res in "review_pending"
See that, once in "review pending" the resource can no longer be made discoverable/private

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4967

Good point that it would help to consolidate all of these "hoops" that a resource must "jump through" into a centralized location


chore: upgrade charts

updated chart jx3/jx-slack from 0.0.60 to 0.0.62

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign hferentschik
You can assign the PR to them by writing /assign @hferentschik in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Multivariate student density error

sorry this was a mistake !

In the future, if you could please leave your (unedited) comment. I agree it was a mistake, but it's useful to keep for future users who may make the same mistake. Thanks!


add yarn.lock

I'm often cleaning my node_modules and reinstall it with yarn, and i notice that it always creates a new yarn.lock file. I think we should decide wether we want yarn.lock on the root of this project or not.
If we want to ignore it, i'll close this and do https://github.com/viankakrisna/create-react-app/tree/ignoreyarnlock instead.

This has been brought up before, I'm pretty sure the reasoning we haven't done it is that a yarn.lock isn't very portable cross-platform (because of optional deps, e.g. fsevents).
If we do anything, it'll probably be to ignore the lock files. It doesn't do us any good, we want to know when things break and they have no effect on the end-user when they install, so I think this will be a no go for now.

Ok then, opened another PR for adding it to .gitignore. I've searched the repo and haven't found anything mentioning yarn.lock on the root of this project. Is there any particular issue / pr that brought this up?

This has been brought up before, I'm pretty sure the reasoning we haven't done it is that a yarn.lock isn't very portable cross-platform (because of optional deps, e.g. fsevents).

Seems like this has been fixed?
https://github.com/yarnpkg/yarn/pull/2811

See my comment in https://github.com/facebookincubator/create-react-app/pull/2014#issuecomment-300811661—we intentionally don't want to either add or ignore it in this repo.


Selenide tests failed after upgrading to 6.17.2 from 5.12.2 which throws NoSuchMethodError

My existing tests failed when Selenide was upgraded to 6.17.2 from 5.12.2
I have a sample Selenide project which is working fine with  Selenide 5.12.2
I just wanted to upgrade to the latest version and try new features
It throws java.lang.NoSuchMethodError: 'org.openqa.selenium.WebDriver$Timeouts org.openqa.selenium.WebDriver$Timeouts.pageLoadTimeout(java.time.Duration)' after the browser gets launched

Selenide Version: 6.17.2
Chrome: 116
Browser Driver Version: 116
Selenium Version: 4.11

@LidiyaBabu I think you have incorrect versions of dependencies. Just run gradlew dependencies or mvn dependency:tree and check which versions of selenium-*.jar you have.
See https://selenide.org/2020/11/17/why-proxy-does-not-work-in-selenoid/ and https://www.javaadvent.com/2020/12/how-to-debug-dependency-conflicts-in-maven-and-gradle.html


Create process to allow editors to update the confidence level of documents

I dont know if this counts as a duplicate of this issue. If so i'll just close it and we can continue using the other one for the overall confidence level thing.

Closing this as duplicate of #107. Changed title of that one to make it more clear.


[ionic 3.3.0] Error: Failed to install apk to device

I'm encountering an error when running :  ionic cordova run android

ionic cordova run android
build dev started
...
BUILD SUCCESSFUL
Built the following apk(s):
C:/.../platforms/android/build/outputs/apk/android-debug.apk
ANDROID_HOME=D:\Users\SBS\AppData\Local\Android\sdk
JAVA_HOME=C:\Program Files\Java\jdk1.8.0_121
No target specified, deploying to device '96cac6f8'.
Skipping build...
Built the following apk(s):
C:/.../platforms/android/build/outputs/apk/android-debug.apk
Using apk: C:/.../platforms/android/build/outputs/apk/android-debug.apk
Package name: io.ionic.starter
Error: Failed to install apk to device: [  1%] /data/local/tmp/android-debug.apk
[  2%] /data/local/tmp/android-debug.apk
...
[100%] /data/local/tmp/android-debug.apk
C:/.../platforms/android/build/outputs/apk/android-debug.apk: 1 file pushed. 2.6 MB/s (5156463 bytes in 1.865s)
pkg: /data/local/tmp/android-debug.apk
Failure [INSTALL_FAILED_CONTAINER_ERROR]
[ERROR] Cordova encountered an error.
You may get more insight by running the Cordova command above directly.
[ERROR] An error occurred while running cordova run android (exit code 1).

Ionic info:
global packages:
ionic/cli-utils      : 1.3.0
Cordova CLI      : 6.5.0
Ionic CLI            : 3.3.0
local packages:
ionic/app-scripts                      : 1.3.7
ionic/cli-plugin-cordova          : 1.3.0
ionic/cli-plugin-ionic-angular  : 1.3.0
cordova platforms                       : android 6.1.2
Ionic Framework                          : ionic-angular 3.3.0
System:
Node       : v6.10.1
OS           : windows 10
I also upgraded Cordova CLI to 7.0.1 but I still have the same issue

Did you get its solution? I am sticking with the same issue, How you resolved Please share.

Hi, all
Uninstall  app in your device & check again  ...

I've got the same issue!

@wcjord.. This usually occurs when we change our development environment and try to install the new application among the old one in our device, for winning this error, you have to uninstall all the existing ionic application in your device and install the new app again, this will surely work.
Thanks

Thanks @Supriya412 works for me too.

Hi, all
Uninstall app in your device & check again ...

Thank you friend!


Array optimizations don't elide some getter calls

Previously all no-op indexing operations were fully removed in the
optimization passes. This was fine for array-like's that follow numpy's
semantics, but not all objects fully follow these semantics.
Since chunks always should follow numpy slicing conventions, the only
thing we need to worry about is original backend storage (in calls to
from_array). We fix the optimization passes to never fully remove
slices if the indexing operation originated in from_array.
Fixes #2823.

Planning on merging tonight if no comment.


K3S Release Documentation Update

Proposed Changes
This PR contains complementary documentation regarding the release process.

Types of Changes
Documentation

Verification

Testing
Manual verification of these steps occurred during the last release.

Linked Issues

User-Facing Change

Further Comments

Some variables have brackets and some don't. Please make them consistent.

Closing in favor of a new pr.


Feature/snyk

@st3v3nhunt this looks good, but there's issues with the snyk check, so can't merge as yet.


Youtube extractor fails with a TypeError

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS>. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
Poland
Description
Looks like Youtube changed their signature checks again? They did so in such a way that the yt_dlp js interpreter breaks. I run yt_dlp under a debugger; it turns out that index is 42.0 (a float) instead of an int which range() expects. Also I'm not sure about the reproducability of this bug.
Verbose log
[debug] Command-line config: ['https://www.youtube.com/watch?v=WSSnD8vnJ7Q', '-f', '298+521', '-Uv']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8 (No ANSI), err utf-8, pref UTF-8
[debug] yt-dlp version 2<IP_ADDRESS>
[debug] Python version 3.8.10 (CPython 64bit) - Linux-5.4.0-80-generic-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4
[debug] Optional libraries: Cryptodome, keyring, mutagen, sqlite, websockets
[debug] Proxy map: {}
yt-dlp is up to date (2<IP_ADDRESS>)
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=WSSnD8vnJ7Q
[youtube] WSSnD8vnJ7Q: Downloading webpage
[youtube] WSSnD8vnJ7Q: Downloading android player API JSON
[debug] Loading youtube-nsig.f1ca6900 from cache
WARNING: [youtube] nsig extraction failed: You may experience throttling for some formats
n = cu3wyu6LQn2hse ; player = https://www.youtube.com/s/player/f1ca6900/player_ias.vflset/en_US/base.js
WSSnD8vnJ7Q: Traceback (most recent call last):
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/extractor/youtube.py", line 1863, in _decrypt_nsig
    self._player_cache[sig_id] = func(s)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/extractor/youtube.py", line 1890, in <lambda>
    return lambda s: jsi.extract_function_from_code(*func_code)([s])
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 536, in resf
    ret, should_abort = self.interpret_statement(stmt, var_stack)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 145, in interpret_statement
    v = self.interpret_expression(expr, local_vars, allow_recursion)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 182, in interpret_expression
    ret, should_abort = self.interpret_statement(try_expr, local_vars, allow_recursion - 1)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 145, in interpret_statement
    v = self.interpret_expression(expr, local_vars, allow_recursion)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 256, in interpret_expression
    self.interpret_expression(sub_expr, local_vars, allow_recursion)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 449, in interpret_expression
    return eval_method()
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 442, in eval_method
    return obj[member](argvals)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 536, in resf
    ret, should_abort = self.interpret_statement(stmt, var_stack)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 145, in interpret_statement
    v = self.interpret_expression(expr, local_vars, allow_recursion)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 449, in interpret_expression
    return eval_method()
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 371, in eval_method
    argvals = [
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 372, in <listcomp>
    self.interpret_expression(v, local_vars, allow_recursion)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 446, in interpret_expression
    self._named_object(local_vars, eval_method()) + remaining,
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/jsinterp.py", line 405, in eval_method
    for i in range(index, min(index + howMany, len(obj))):
TypeError: 'float' object cannot be interpreted as an integer
 (caused by TypeError("'float' object cannot be interpreted as an integer")); please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, codec:vp9.2, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
ERROR: [youtube] WSSnD8vnJ7Q: Requested format is not available
Traceback (most recent call last):
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1311, in wrapper
    return func(self, *args, **kwargs)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1349, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1399, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "/home/mm2pl/.local/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 2414, in process_video_result
    raise ExtractorError('Requested format is not available', expected=True,
yt_dlp.utils.ExtractorError: [youtube] WSSnD8vnJ7Q: Requested format is not available

Reproduced on my side, up to date using Python 3.9.5.

Thanks for reporting. It should be reproducible with same player URL. I'll take a look.
Although, the jsinterp failing is actually a non-fatal error. The actual error shown in the verbose is due to you providing a wrong format code

Oh yeah this doesn't crash when given a correct format. Thanks for pointing out the typo!


Hardcoded column counts and column drop

Hi. I think some changes with degiro's csv output is breaking the hardcoded column counts and might be a problem for dropping unused columns too. As a workaround I just hardcoded 12 columns as per the csv format I downloaded.
[ERROR] Input data column count incorect! expected: 19

Hi,
thanks for the heads up.
Can you share the header of your "Transactions.cvs", so I can see what needs to be done.
My header still has 19 columns:
Datum,Uhrzeit,Produkt,ISIN,Referenzbörse,Ausführungsort,Anzahl,Kurs,,Wert in Lokalwährung,,Wert,,Wechselkurs,Transaktionskosten,,Gesamt,,Order-ID

Perhaps it varies by country. I'm using the UK site.  By the way, there is also an option to hide Money Market Fund transactions ("Hide cash movements" toggle) in the Degiro UI. Anyway,  I get the following:
12 columns, 10 headers
Date		(eg. 31-12-2018 )
Time		(eg. 08:42 )
Value date	(eg. 31-12-2018 )
Product		(eg. TESLA MOTORS INC. - C)
ISIN			(eg.  US88160R1014)
Description	(eg. Buy 10 TESLA MOTORS INC. - C@305 USD (US88160R1014))
FX			(eg. 1.2621)
Change		(2 columns, eg. USD  | 300.00 )
Balance		(2 columns, eg. USD  | 300.00 )
Order Id		(eg. 270b3e5e-b771-5d34-1n0t-r3a16fc60967a)
I will try to look at your code if I have time. I don't know Pandas very well but I imagine there's a neat solution because the Tradervue format is so simple and they allow a different column order if headers are present.

Okay, yes.
We just would have to define different parsing functions for the different input formats and choose these accordingly.
Would be nice to know if there are even more different formats and how to distinguish between them.
For now, it would just be parseInputDE(inputData) and pasreInputUK(inputData) or something along the lines.

Or ask the user which columns to select with args? So few are used and they have the file. I emailed DEGIRO asking about exports. I’ll let you know.

I would prefer to automate it instead of having the user to think through it and fiddle with it.


Adding Haystack Conference Europe

Added Haystack Conference Europe taking place on the 28th of October

thanks, and sorry I wasn't able to merge this PR on time...


macos-64 build failed

https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=1085501&view=logs&jobId=e0208569-136d-54da-4ec5-14d0e8771cbc&j=e0208569-136d-54da-4ec5-14d0e8771cbc&t=25992e30-3e2a-5788-2b3e-e373b0534bf7
@conda-forge/help-python would you be able to restart this CI so that the macos-64 build finishes and uploads? You can see it's missing here
https://anaconda.org/conda-forge/uv/files
And let me know if there is some other person/group I should ping next time or different procedure to follow to get the build to run again! (For example: could just bump the build number and merge, which would be a bit clunky but would work.)

Kicked. Thanks for the report.
On Wed, Nov 20, 2024, 08:19 Eric Larson @.***> wrote:

https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=1085501&view=logs&jobId=e0208569-136d-54da-4ec5-14d0e8771cbc&j=e0208569-136d-54da-4ec5-14d0e8771cbc&t=25992e30-3e2a-5788-2b3e-e373b0534bf7
@conda-forge/help-python
https://github.com/orgs/conda-forge/teams/help-python would you be able
to restart this CI so that the macos-64 build finishes and uploads? You can
see it's missing here
https://anaconda.org/conda-forge/uv/files
And let me know if there is some other person/group I should ping next
time or different procedure to follow to get the build to run again! (For
example: could just bump the build number and merge, which would be a bit
clunky but would work.)
—
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/uv-feedstock/issues/114, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALCRE6LIZFTBZ5AMHIBRD2BSK7BAVCNFSM6AAAAABSEWEWSGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGY3TMMBRGYZDMNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

All better, thanks!


Core:NavigateToPageAction in XAML with Behavior don't work in Template10

If i try to Navigate to a Page with NavigateToPageAction in XAML in Template 10 App it throws an Exception.

    <Button x:Name="button" Content="Klick">

        <Interactivity:Interaction.Behaviors>
            <Core:EventTriggerBehavior EventName="Click" SourceObject="{Binding ElementName=button}">
                <Core:NavigateToPageAction TargetPage="WindowsApp1.Views.BlankPage1" />
            </Core:EventTriggerBehavior>
        </Interactivity:Interaction.Behaviors>
    </Button>

If i do the same with normal Blank App thats Navigate

Template 10 navigates using the NavigationService. Your best bet is to create a method that navigates for you. Using an action that doesn't understand Template 10 just won't work. By the way, there's a Nav action in Template 10, but it is incomplete.


Release 0.9.22

If #169 is merged, this one should have no effect, but then shows as merged.

This now includes a NRE fix that is not already merged in master, so this PR merge would have an important result.

Now includes the fix for llEmail-triggered exception that halted the script engine on I'z Straits 8.


Don't filter EV alignment

This removes the IIR filter on the attitude offset between vision and EKF frame.
I don't see why this rotation would need to be low pass filtered. Especially if one of the two drifts, this leads to toilet-bowling.
Not sure if I did this right, so any review is appreciated.

Yeah, e.g. the log that triggered this change in the first place:
https://review.px4.io/plot_app?log=21099c0f-75f8-48b0-af7e-0071e0d50089

I think you need to rebase and force-push.


a weird finding about static variable

HI there, here is my code:
use std::sync::atomic::{AtomicU8, Ordering};

fn main() {
    println!("{}", get_num());
    println!("{}", get_num());
    println!("{}", get_num());
}

fn get_num() -> u8 {
    static NUM: AtomicU8 = AtomicU8::new(1);
    NUM.fetch_add(1, Ordering::Relaxed)
}

#[test]
fn test_1() {
    println!("test 1 got {}", get_num());
}

#[test]
fn test_2() {
    println!("test 2 got {}", get_num());
}

#[test]
fn test_3() {
    println!("test 3 got {}", get_num());
}

cargo test -- --nocapture will get 3 different number in each test.

output
running 3 tests
test 2 got 1
test 1 got 2
test 3 got 3
test test_2 ... ok
test test_1 ... ok
test test_3 ... ok

cargo nextest run --no-capture will only get number 1 in each test.

output
running 1 test
test 1 got 1
test test_1 ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 2 filtered out; finished in 0.00s

        PASS [   0.002s] rustplay::bin/rustplay test_1
       START             rustplay::bin/rustplay test_2

running 1 test
test 2 got 1
test test_2 ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 2 filtered out; finished in 0.00s

        PASS [   0.002s] rustplay::bin/rustplay test_2
       START             rustplay::bin/rustplay test_3

running 1 test
test 3 got 1
test test_3 ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 2 filtered out; finished in 0.00s

        PASS [   0.002s] rustplay::bin/rustplay test_3
------------
     Summary [   0.006s] 3 tests run: 3 passed, 0 skipped

should I add any flag for it? in my understanding cargo test equals to cargo nextest run

Thanks for the question! Nextest runs each test in its own process, which means that static memory isn't shared between tests the way it is in cargo test. Please see https://nexte.st/book/how-it-works.html for more information.

oh! Thank you! Is it possible to group them in the same process?

I'm afraid not, see #27 for more.

I will follow the issue! Thank you!


Add literal.rs multiply

Codecov Report

Merging #411 (a284d1d) into main (186b689) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #411      +/-   ##
==========================================
+ Coverage   90.39%   90.45%   +0.05%
==========================================
  Files         144      144
  Lines        8539     8549      +10
==========================================
+ Hits         7719     7733      +14
+ Misses        820      816       -4

Impacted Files
Coverage Δ

src/data/literal.rs
85.77% <100.00%> (+2.36%)
:arrow_up:

Continue to review full report at Codecov.

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


How to cancel "all"?

I have the following construct:
export default function *(): Generator<*, *, *> {
  while (yield take(SET_START)) {
    const allTasks = yield all([
      fork(createRequestRoutine(GET_MOVIES, getMoviesRemote, setMovies, setError)),
      fork(watchNewSearchCriteria)
    ]);

    yield take(SET_STOP);

    yield cancel(allTasks);
  }
}

Attempting to yield cancel(allTasks) produces an error:

uncaught at rootSaga
at effects
Error: cancel(task): argument [object Object],[object Object] is not a valid Task object
(HINT: if you are getting this errors in tests, consider using createMockTask from redux-saga/utils)

It is not clear from the documentation how to cancel tasks forked using all.

Should work with yield cancel(...allTasks) or yield all(allTasks.map(task => cancel(task))). We'll try to add some docs info later.

Thank you


IMGPROXY_FALLBACK_IMAGE is not returned on error 500

If remote url return html code, imgproxy return error 500 with text Internal error, but not fallback image
ERROR   [2024-10-11T06:52:33Z] Completed in 828.215882ms /url/cdSg_FJdwMgBHr-ibNoxnPjL5X7NS4lDgJlj1yw4hE8/rs:fit:1200:900/g:sm/aHR0cDovL3N2ZXRl/cnJhLnJ1L3VwbG9h/ZC9pYmxvY2svODRm/Lzg0ZjU0MjcwYzM1/ZDlkZWY4MTQyYjE3/NjE0NzY0ZWI5Lmpw/Zw  request_id=uA-Za_kRw09ic92B9ddI3 method=GET status=500 client_ip=<IP_ADDRESS> error="XML parse error: unexpected NULL character on line 1 and column 5\n    1: ����··JFIF·····`·`··��·<CREATOR: gd-jpeg v1.0 (using IJG ...\n           ^"
/app/processing_handler.go:194 main.sendErrAndPanic
/app/processing_handler.go:193 main.sendErrAndPanic
/app/processing_handler.go:198 main.checkErr
/app/server.go:114 main.buildRouter.withCORS.func1
/app/server.go:165 main.buildRouter.withPanicHandler.func2
/app/server.go:103 main.buildRouter.withMetrics.func3
/app/router/router.go:102 github.com/imgproxy/imgproxy/v3/router.(*Router).ServeHTTP
/usr/local/go/src/net/http/server.go:2939 net/http.serverHandler.ServeHTTP
/usr/local/go/src/net/http/server.go:2010 net/http.(*conn).serve
/usr/local/go/src/runtime/asm_amd64.s:1651 runtime.goexit
WARNING [2024-10-11T06:52:35Z] Could not load image https://www.***.**//upload/iblock/f94/f94d545ba4954c6f253d7848a5df0d0d.jpg. Using fallback image. Can't download source image: Status: 404; <!DOCTYPE html>
<html dir="ltr" lang="ru">

<head>
    <meta charset="UTF-8" />
     <link rel="icon" href="/favicon.svg" type="image/svg+xml">
    <link rel="shortcut icon" href="/favicon.ico" type="image/x-icon">
    <link rel="apple-touch-icon" sizes="180x180" href="/favicon.png" />
...

Hey @DarthSim !
I use base64 encoded image
  fallbackImage:
    data: "PHN2ZyBoZWlnaHQ9IjcwIiB2aWV3Qm94PSIwIDAgNTEyIDUxMiIgd2lkdGg9IjcwIiB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciPjxnIGZpbGw9IiM4ZThlOGUiPjxwYXRoIGQ9Im0yNzUuMjc3MzQ0IDI5Ni40ODgyODEtNTkuNzY1NjI1LTU5Ljc2NTYyNWMtMTguMjQ2MDk0IDM4LjEyODkwNiAyMS41NjI1IDc4LjA0Njg3NSA1OS43NjU2MjUgNTkuNzY1NjI1em0wIDAiLz48cGF0aCBkPSJtMTY3LjY0NDUzMSAxNDYuNDMzNTk0IDE3LjMyNDIxOS01NS40MzM1OTRoMTQyLjA2MjVsMTguNzUgNjBoNzUuMjE4NzV2MjQwaC04Ljc4OTA2MmwzNS4xODc1IDM0LjE4NzVjNDEuNjA5Mzc0LTQ2LjcxNDg0NCA2NC42MDE1NjItMTA2LjExMzI4MSA2NC42MDE1NjItMTY5LjE4NzUgMC0xNDAuOTE0MDYyLTExNS4wNDY4NzUtMjU2LTI1Ni0yNTYtNjMuMDc0MjE5IDAtMTIyLjQ3MjY1NiAyMy45OTIxODgtMTY5LjE4NzUgNjUuNjAxNTYyem0wIDAiLz48cGF0aCBkPSJtMTU5Ljc4OTA2MiAxODFoLTM4Ljc4OTA2MnYxODBoMjE4Ljc4OTA2MmwtNDIuNDk2MDkzLTQyLjQ5NjA5NGMtMTEuODU1NDY5IDcuODYzMjgyLTI2LjAzMTI1IDEyLjQ5NjA5NC00MS4yOTI5NjkgMTIuNDk2MDk0LTQxLjM1MTU2MiAwLTc1LTMzLjY0ODQzOC03NS03NSAwLTE1LjI2MTcxOSA0LjYzMjgxMi0yOS40Mzc1IDEyLjQ5NjA5NC00MS4yOTI5Njl6bTAgMCIvPjxwYXRoIGQ9Im0zNjkuNzg5MDYyIDM5MWgtMjc4Ljc4OTA2MnYtMjQwaDM4Ljc4OTA2MmwtNjQuMTg3NS02NC4xODc1Yy00MS42MDkzNzQgNDYuNzE0ODQ0LTY1LjYwMTU2MiAxMDYuMTEzMjgxLTY1LjYwMTU2MiAxNjkuMTg3NSAwIDE0MC45MTQwNjIgMTE1LjA0Njg3NSAyNTYgMjU2IDI1NiA2My4wNzQyMTkgMCAxMjIuNDcyNjU2LTIzLjk5MjE4OCAxNjkuMTg3NS02NS42MDE1NjJ6bTAgMCIvPjxwYXRoIGQ9Im0zOTEgMzYxdi0xODBoLTY3LjI4MTI1bC0xOC43NS02MGgtOTcuOTM3NWwtMTUuNDMzNTk0IDQ5LjM4NjcxOSAyMy4xMDkzNzUgMjMuMTA5Mzc1YzExLjg1NTQ2OS03Ljg2MzI4MiAyNi4wMzEyNS0xMi40OTYwOTQgNDEuMjkyOTY5LTEyLjQ5NjA5NCA0MS4zNTE1NjIgMCA3NSAzMy42NDg0MzggNzUgNzUgMCAxNS4yNjE3MTktNC42MzI4MTIgMjkuNDM3NS0xMi40OTYwOTQgNDEuMjkyOTY5bDYzLjcwNzAzMiA2My43MDcwMzF6bTAgMCIvPjxwYXRoIGQ9Im0yMzYuNzIyNjU2IDIxNS41MTE3MTkgNTkuNzY1NjI1IDU5Ljc2NTYyNWMxOC4yNDYwOTQtMzguMTI4OTA2LTIxLjU2MjUtNzguMDQ2ODc1LTU5Ljc2NTYyNS01OS43NjU2MjV6bTAgMCIvPjwvZz48L3N2Zz4="
    ttl: "3600"

If remove url returns 404, its works well.

Hmm, it seems that everything is ok with it. Which version of imgproxy do you use?

v3.20.0
Log messages
WARNING [2024-10-11T18:58:04Z] Could not load image http://sveterra.ru/upload/iblock/8b0/8b03a2d12c9e188ad1eac217afcb0869.jpg. Using fallback image. Can't download source image: Status: 404; <!DOCTYPE html>
ERROR   [2024-10-11T18:58:04Z] Completed in 890.721621ms /url/OEu6ZzrENUa-F1Q0n4GWH6DIu7Tqk_aP4d41HD20yRs/rs:fit:1200:900/g:sm/aHR0cDovL3N2ZXRl/cnJhLnJ1L3VwbG9h/ZC9pYmxvY2svOGIw/LzhiMDNhMmQxMmM5/ZTE4OGFkMWVhYzIx/N2FmY2IwODY5Lmpw/Zw  request_id=f3_r6sjRf-TGE1v6RKW4i method=GET status=500 client_ip=<IP_ADDRESS> error="XML parse error: unexpected NULL character on line 1 and column 5\n    1: ����··Exif··II*·············��··Ducky·······2··��··Adobe·...\n           ^"
WARNING [2024-10-11T18:58:05Z] Could not load image https://cabsystems.ru/upload/iblock/af6/af642eb62e9e5520cb3a42dd520d8477.jpg. Using fallback image. Can't download source image: Status: 404; <!DOCTYPE html>
ERROR   [2024-10-11T18:58:05Z] Completed in 727.716093ms /url/H98TrpWK3Hn1or_YyAr9LXPeK6Kk4agmLbceGUH-X64/rs:fit:1200:900/g:sm/aHR0cHM6Ly9jYWJz/eXN0ZW1zLnJ1L3Vw/bG9hZC9pYmxvY2sv/YWY2L2FmNjQyZWI2/MmU5ZTU1MjBjYjNh/NDJkZDUyMGQ4NDc3/LmpwZw  request_id=Vgv65zbT37D7RI3xC_uR- method=GET status=500 client_ip=<IP_ADDRESS> error="XML parse error: unexpected NULL character on line 1 and column 5\n    1: ����··Exif··II*·············��··Ducky·······2··��··Adobe·...\n           ^"
WARNING [2024-10-11T18:58:08Z] Could not load image http://sveterra.ru/upload/iblock/033/03325f6e812c3fa290173b1fb0c28240.jpg. Using fallback image. Can't download source image: Status: 404; <!DOCTYPE html>
ERROR   [2024-10-11T18:58:08Z] Completed in 1.895895868s /url/BeqAHT6u86P9l8nYggGZnxY8sN5gRHfQIDMB6MSPZq4/rs:fit:1200:900/g:sm/aHR0cDovL3N2ZXRl/cnJhLnJ1L3VwbG9h/ZC9pYmxvY2svMDMz/LzAzMzI1ZjZlODEy/YzNmYTI5MDE3M2Ix/ZmIwYzI4MjQwLmpw/Zw  request_id=LIoqM-11ocaTKKwbvAcYl method=GET status=500 client_ip=<IP_ADDRESS> error="XML parse error: unexpected NULL character on line 1 and column 5\n    1: ����··Exif··II*·············��··Ducky·······2··��··Adobe·...\n           ^"

This is weird as hell. The XML parse error tells about some EXIF stuff, but the fallback image you set to fallbackImage.data is SVG, so it doesn't have EXIF. And if it wasn't SVG, then why would any XML parsing be involved? The saddest part is that I can't reproduce this with your provided data. Do you by any chance use a fork of imgproxy?

I use darthsim/imgproxy:v3.20.0 docker image
Can you try this image sourece?
https://www.meratest.ru/catalog_photos/item_142735/originals/142735_a2c8849dcef8a4f8e76d65a24458fa06.jpg

The same: the fallback image is used without an issue.
Did you try using a fallback image URL instead of base64 data? Maybe it's Helm or K8s messing up with long values, IDK.

I used the URL before, and then changed it to BASE64, and only now I understand that the broken pictures are possibly a cache in CloudFlar, now the pictures are starting to be updated on fallback.
There are errors in the logs, but the fallback picture is displayed normally.
I'll observe some more and write about the results.


Feedback on Tutorial: Create a multitenant container database in HANAExpress

For the CREATE DATABASE statement I receive error message
general error: Database could not be started;start databaseServer for database=3 on : hxehost:30040 failed
select * from "PUBLIC"."M_DATABASES"
returns ACTIVE_STATUS=NO for the new database.
Would be great if there was some help with trouble shooting in this tutorial.

Database could not be started is a general error might due to many reasons.
You need to provide trace file and detail configuration/setup.
Give you an example here: If you only have 8GB memory, you tried to create the 3rd database (2nd tenantDB), then this might be expected behavior and you might see oom trace file in the trace folder.


Image function of BigBrainVolume expects bounding box to be in voxel space

If passing a bounding box to the clip argument to the Image function of BigBrainVolume, it expects the bounding box to be specified in voxel space of the specified resolution, which is highly inconvenient for users.
The bounding box should be defined in millimeter space and the class should do the conversion itself (which it can do using it's affine matrix).

Fixed in v0.1a4 which introduced to have all non-protected method arguments in mm space, and proper volume-of-interest specifications.


CAMEL-18649 + minor fix for CAMEL-14148

Failure seems like a flaky on core. I'll merge and keep an eye on it.


ZSH completions when using glab plugin

Platform or tool
GitLab
Desired or expected behavior
Hi team, really enjoying the GitLab 1password plugin. https://developer.1password.com/docs/cli/shell-plugins/gitlab
One thing I noticed once I installed the glab shell plugin is that my completions stopped working, since op aliases glab
I had to add setopt completealiases to my ~/.zshrc file so zsh does not expand aliases before performing completion.
I was thinking if it would be worth mentioning here: https://developer.1password.com/docs/cli/shell-plugins/gitlab ?
Current behavior
Completions don't work by default with zsh + op
Relevant log output
No response
op CLI version
2.8.0-beta.10

Have not tested, but I would guess this is the same for all shells (except maybe Fish, Fish shell is pretty good about doing that kind of stuff by default) and all Shell Plugins.
Maybe plugins.sh should set the correct equivalent of completealiases in all shells, or otherwise mention it somewhere in the docs that applies to all shell plugins.

Some findings from looking into this a bit:

For zsh, you just need to setopt completealiases
For fish, no changes are needed, Fish shell follows alias completions by default
For bash, there is not any equivalent of setopt completealiases, so you would need to do one of a few things (see below)

For these reasons, I think the best approach is to document it, rather than trying to set it automatically, because the plugins.sh file has no shebang line (e.g. #!/usr/bin/env bash) and so it runs in the user's own shell environment. Trying to set it automatically would mean adding some shell detection, in a syntax that is compatible with all of zsh, bash, and fish, which might be complex; fish syntax is mostly compatible with bash/zsh but not fully.
Options for bash completions:
Disclaimer: I've not validated these, we'll need to audit them before we can officially recommend them. For now, use at your own risk.*

https://github.com/cykerway/complete-alias
Something like this: https://unix.stackexchange.com/questions/4219/how-do-i-get-bash-completion-for-command-aliases/310089#310089

For ZSH we can do the following:
function _glab_command () {
  op plugin run -- glab $@
}

alias glab="_glab_command"
compdef _glab _glab_command

I no tested it yet, but I did it for "doctl" and it works well

I'd prefer not to setopt completealiases in zsh as that would mess up completions of other aliases that include command line args.
I tried this for gh and while @jrpedrianes solution worked, it asked for authentication on every completion, unless I switched the security settings from "ask approval for each new application and terminal session" to "ask approval for each new application". I think this is because a new subshell is spawned for each function call?
My interim solution has been to update the op completion functions to call _gh directly –
see this gist for the changes. This should work for _glab as well.

@aaronkollasch's solution above works for me for op plugin run glab:
% op plugin run glab <TAB>
alias         -- Create, list and delete aliases
api           -- Make an authenticated request to GitLab API
...

However op's aliases default to including the -- separator, e.g. op plugin run -- glab, which doesn't work and autocompletes files/dirs instead:
% op plugin run -- glab <TAB>
bin/               Desktop/           Downloads/
...

zsh 5.9. op autogenerates the aliases, I'd prefer not to mess with those manually. Any idea what could be the issue here? Thank you!

@Forst I finally found the problem – there was an extra -- hanging around before the command in the zsh variable $words.
Adding the following lines before _dispatch "$curcontext" "$line[1]" fixes the issue:
  if [[ "$words[1]" == "--" ]]; then
    words[1]=()
  fi

I've also updated the gist. This should get things to work properly with op's autogenerated aliases, i.e. op plugin run -- glab

Related: #433

The thing that works reliably for me is also monkey patching op completion function with:
function _op_plugin_run_fix() {
    for (( i=2; i < CURRENT; i++ )); do
        if [[ ${words[i]} == -- ]]; then
            shift $i words
            (( CURRENT -= i ))
            _normal
            return
        fi
    done

    # run original op completion
    _op
}

function __load_op_completion() {
  local op_completion_fn="$(op completion zsh)"
  local fix_fn="$(which _op_plugin_run_fix)"

  # replace all references to _op with _op_plugin_run_fix (except for the function definition)
  local patched_op=$(sed -E -e 's/\b_op\b([^()]|$)/_op_plugin_run_fix\1/g' <<< $op_completion_fn)

  echo -e "$patched_op\n\n$fix_fn"
}

eval "$(__load_op_completion)"

The only problem with it is that I need to use eval instead of keeping it on fpath.
With fpath the guard in _op on $funcstack means that when loading completion for the first time I have to press <TAB> twice.


Submitter: check gas balance when setting gas price

Description
Adds an extra checkGasBalance() call to make sure the submitter wallet has enough gas to pay for the given transaction.
Metadata

Addresses #2410

Summary by CodeRabbit

New Features

Enhanced transaction fee calculation to dynamically adjust fees within a set maximum limit.
Transactions can now have their gas price bumped based on their current nonce, improving transaction processing efficiency.
Support for EIP-1559 in setting gas prices and fee caps, ensuring compatibility with the latest Ethereum standards.

Improvements

Increased the default gas bump percentage to better align with current network conditions.
Added a verification step to check for sufficient gas balance before submitting a transaction.

Tests

Updated unit tests to reflect changes in gas price setting logic.

Will merge after 2432


longitudinal planner: disable change cost when stopped or not engaged

This might help remove a small amount of lag on starting from a standstill.

LGTM


Updated Google Driver

Proposed changes
Google is going to shutdown Google+ APIs in March 2019.
Types of changes

Changed user profile endpoint
Updated user building with new data, example of response:

{
  sub: '<id>',
  name: '<given_name> <family_name>',
  given_name: '<given_name>',
  family_name: '<family_name>',
  profile: '<link>',
  picture: '<link>',
  email: '<email>',
  email_verified: true,
  gender: '<gender>',
  locale: '<locale>'
}

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

Checklist

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation (if appropriate)

Coverage remained the same at 67.033% when pulling 4f2b23f5d7f80cb7828b4d98358e35eaccc9526b on keeross:develop into 4e3902f4591bfc7642163bc91cc507a6a68fd1e2 on adonisjs:develop.

Removed commits

Thanks.
Also do you have any link for the documentation of the new API?
I cannot find this endpoint https://www.googleapis.com/oauth2/v3/userinfo.

Hey, that is pretty strange, but the only place we can find some info about this endpoint is Google API Browser, but it references v2 instead of v3.
Then I decided to check what did people with Passport, here is the PR, the same did guys with Omniauth, PR.
Looks legit.

Thanks! 💜
Will release it soon.


onResolved → onFulfilled

R: @domenic @beaufortfrancois etc.
Update as per https://github.com/GoogleChrome/samples/pull/286#discussion_r51310447

Lgtm


I send from ledger to btcmarket.Net and I never received my found. Wallet address and note were set correctly. Please help me with that

I send from ledger to btcmarket.Net and I never received my found. Wallet address and note were set correctly. Please help me with that
Thanks
Transaction: 1e3731a22896a15850a5c8128ad4b3a85089b0e009dffc690e6aa029397b5ae8

At the moment, SDF-maintained Horizon service is down. You can monitor current network status here: https://status.stellar.org/
Once the problem is fixed, exchanges and other services will resume deposits.

Hi Team
Looks like  https://status.stellar.org/ is up & running but I am  yet to
receive my stellar
Transaction
1e3731a22896a15850a5c8128ad4b3a85089b0e009dffc690e6aa029397b5ae8 on Stellar
public network | StellarExpert
https://stellar.expert/explorer/public/tx/1e3731a22896a15850a5c8128ad4b3a85089b0e009dffc690e6aa029397b5ae8
Best Regards
Vijay
On Tue, Apr 6, 2021 at 10:08 PM orbitlens @.***> wrote:

At the moment, SDF-maintained Horizon service is down. You can monitor
current network status here: https://status.stellar.org/
Once the problem is fixed, exchanges and other services will resume
deposits.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/orbitlens/stellar-expert-explorer/issues/402#issuecomment-814067924,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APH5KQIYVVHBNWBQSKKFM6TTHL2TLANCNFSM42OSTURA
.


[WIP] Add support for configuring Google OAuth logins

Known issues:

Upon registration/login you are redirected to an ugly error message.
If you change the URL you can access the application and things are
working, though

User cannot access the user list!

No documentation for the new environment variables

Signed-off-by: Alex Corcoles<EMAIL_ADDRESS>

[ ] I have updated the CHANGELOG file according to the conventions in keepachangelog.com
[ ] This PR contains changes that do not require a mention in the CHANGELOG file

From [private] slack:

IMO it makes sense to open a PR on sourced-ui pointing to your PR :thumbsup::skin-tone-4: where we could eventually add some refinements

Actually I meant issue 😅 but I wrote PR. I think that we could use this as a base and take ownership. But there's no urgency for this to be included now.

Actually I meant issue sweat_smile but I wrote PR. I think that we could use this as a base and take ownership. But there's no urgency for this to be included now.

Don't worry, it took minutes to do it. It still lacks documentation but at least you have something to start with, I guess

another option, I can just close it, copy-paste the code into my PR and take all the praise for the implementation of the feature 😝

:D
I really don't mind. I've had success with using git format-patch / am to get out of these situations. But I'm not proud of that- I feel like I'm doing a dumb thing instead of the "proper" solution.

hahaha
I'd vote for this BTW:

another option, I can just close it, copy-paste the code into my PR and take all the praise for the implementation of the feature 😝

😂
I prefer to avoid having non-properly ready things.


There is no registered "infer" function for node "33" with op = "Floor". Please implement this function in the extensions.

So I'm trying to convert an ONNX model to OpenVINO and I first had this error:
Original exception message: One/both of widths_scale = None and height_scale = None is not defined for Upsample node 44
But I just manually set both variables to 1.0 and that seemed to fix this (hope I didn't destroy the network accuracy, have no idea what the actual values would be).
But now that this problem is semi-fixed, I get this error:
 There is no registered "infer" function for node "33" with op = "Floor". Please implement this function in the extensions.
Is the only way to fix this to manually add the Floor function myself?

Dearest @Aeroxander
Hi again ! I believe I have already answered your IDZ Forum Post
You need to create a custom layer for "Floor", and that should solve your problem.
Thanks for using OpenVino !
Shubha

Ah yes I'll close this one now, thanks!


reading an invalid f64 now returns 0.0

Related to this issue
To handle the cases "NaN.0", "0.NaN", and any other floats that are valid in dxf files and invalid in rust, I changed it so that any invalid f64 would become 0.0 when reading.
it's not the best approach, I know. but it's better than the program crashing, which is what I need

It looks like this is just in the binary reader, is that correct?  I experimented a bit with AutoCAD and when trying to read an entity with NaN it defaults all values of the entity to 0.0.  E.g., a LINE from (1, NaN, 0) to (NaN, 1, 0) comes in as (0, 0, 0) and (0, 0, 0).  While that behavior differs a little with these proposed changes (only setting the one NaN value to 0.0) I'm OK with the discrepancy.
Now to my real question: I'm only familiar with the surface level of IEEE 754 encoding of floats.  Is there a binary pattern that's not a valid float and doesn't evaluate to NaN?  Or are all invalid floats treated as NaN?  If there are non-NaN values that are invalid, I'd prefer those to still throw the error, but if there are no invalid non-NaN values then I'm good with the fallback to 0.0.
Semi-related: AutoCAD doesn't allow any NaN values in an ASCII encoded DXF file, so I'd like to immediately reject a file with this:
...
 10
NaN.0
...

etc.

It looks like this is just in the binary reader, is that correct? I experimented a bit with AutoCAD and when trying to read an entity with NaN it defaults all values of the entity to 0.0. E.g., a LINE from (1, NaN, 0) to (NaN, 1, 0) comes in as (0, 0, 0) and (0, 0, 0). While that behavior differs a little with these proposed changes (only setting the one NaN value to 0.0) I'm OK with the discrepancy.
Now to my real question: I'm only familiar with the surface level of IEEE 754 encoding of floats. Is there a binary pattern that's not a valid float and doesn't evaluate to NaN? Or are all invalid floats treated as NaN? If there are non-NaN values that are invalid, I'd prefer those to still throw the error, but if there are no invalid non-NaN values then I'm good with the fallback to 0.0.
Semi-related: AutoCAD doesn't allow any NaN values in an ASCII encoded DXF file, so I'd like to immediately reject a file with this:
...
 10
NaN.0
...

etc.

To be honest, I'm probably even less knowledgeable about IEEE 754. I only learned what's AutoCAD and dwg/dxf files last week and I found this error while testing the format.
However, I still think returning to 0.0 would be better than raising an error

I am closing this pull request and opening a new request with the code sent by @mrghosti3


Redirects are not handled in fetch()

In other implementations (node, bun, etc.), when fetching the specified URI, a redirect is processed and the final response (status code 200) is returned, but in llrt, the redirect is not processed and status code 301 is returned.
Reproduction procedure:
// fetch.js
const main = async () => {
  try {
    const response = await fetch('https://google.com');
    console.log(response.status);
    console.log(response.headers.get('location'));
  } catch(e) {
    console.log(e);
  }
}

main();

Execution results in llrt:
shinya@MBA2022M2 llrt-test % ./llrt fetch.js
301
https://www.google.com/

Execution results in bun:
shinya@MBA2022M2 llrt-test % bun fetch.js
200
null

Thanks for the well documented report. PR is welcome, want to take shot at it?
This could be implemented as looping over the response until no redirect header is present if redirect option is set to follow on the Request or fetch options.
https://github.com/awslabs/llrt/blob/d05da26103acc1fdf45afe15965ef05013e5f488/src/http/fetch.rs#L81

Hi @richarddavison ,
I would love to work on this assignment if the opportunity arises :)


Identical styled generating different classnames

Version
2.2.1
Hi!
I'm writing a HOC for wrapping and enhancing my styled component. I know this kind of pattern has been frowned upon in previous issues (#316), and without going into the details of what my HOC is doing, just trust me when I say I have a valid use case.
To enhance my wrapped component I'm parsing it's props and generate some styles which I then pass to the styled method of styled-components, passing in my WrappedComponent to be used as a base.
My only problem with this approach is that even though I generate the exact same styles a unique class name is created for every component. I'm concerned that this might turn into a performance hog in the long run and would like to know if there's a better way to "inject" styles into an existing component.

My only problem with this approach is that even though I generate the exact same styles a unique class name is created for every component.

That sounds either like you're confusing the static and dynamic class we create, or it sounds like you're recreating the styled component on every render.

I'm writing a HOC for wrapping and enhancing my styled component. I know this kind of pattern has been frowned upon in previous issues (#316), and without going into the details of what my HOC is doing, just trust me when I say I have a valid use case.

It's not encouraged for a reason, and I don't think there is any valid use case to be honest. Sorry, if that comes across as harsh, but HOCs for styling do not scale, and we do not want to recommend them, or be associated with the pattern :smile:

Thank you for the fast reply @philpl !

Thank you for the fast reply @philpl !
I do understand the concept of dynamic and static styles, but looking at the docs I can't find any other way to compose and reuse functionality the way I would like to.
To give you some context the goal of my HOC is to provide a centralised way of dealing with different breakpoints. The enhanced components receives props in the following format
<Column columns={{ x: 2, l: 3, xl: 4 }} />

Where the name/key columns corresponds to the name of a function/mixin and the value of x, l and xl becomes it's input. I like this approach because it's compassable and provides a common API for dealing with responsive styles in a conditional way.
Could you recommend any other way of achieving the same result with a different pattern?

That sounds either like you're confusing the static and dynamic class we create, or it sounds like you're recreating the styled component on every render.
Is it an expected behaviour for the "dynamic classes" to always get a unique class-name, regardless if i.e. two two created components receive identical props?
In that case I might have come up with a functioning solution to my problem.

That sounds either like you're confusing the static and dynamic class we create, or it sounds like you're recreating the styled component on every render.

Turns out you were spot on with this comment :D I was both confusing the dynamic and static classes AND recreating the styled component on every render, haha.
Moving the styled component out of render solved that issue. Thank you.
I just have one more questing. Is it an expected behaviour for the dynamic classes to always get a unique class-name, regardless if i.e. two created components receive identical props?

@johnnyBira the dynamic class is generated from the generated CSS string. So if you pas the same props, into the same interpolations, generating the same CSS string, you can expect the dunamicclass not to be duplicated.
I’d suggest you to take a look at the css helper, which you can use to apply a mixin pattern. There’s an example here: https://www.styled-components.com/docs/advanced#media-templates and some API descriptions here: https://www.styled-components.com/docs/api#helpers
Essentially you can use it statically by having some interpolations in it or produce one in a factory function. You can then apply it to a styled component by putting it into a interpolation there.
This encourages you to keep styled components as leaf components, I.e. you’d avoid wrapping any is your own components using styled.
This scales really well and you’ll always know where to look for styles. Wrapping your own non-styled components encourages you instead to refer to children, scatter the styling, and to resort to higher specificity to resolve problems. I’ve written about this in some other issues, which I can’t find now unfortunately

@philpl , thank you for taking the time to explain.
I am using the CSS helper and the mixin pattern with interpolation inside my HOC. What mixins/functions the HOC should handle at each breakpoint are passed as arguments to the HOC, tougehter with the wrapped component, making it easy to follow what styles can be applied.
As I see it I'm basically doing what you describe except I've abstracted it inside the HOC. I guess that's why I still struggle to see why this would be a bad pattern.
Let me know if you find the issues you mentioned.
I'm planing to release this as a package and would be very interested in your feedback on the matter.

@johnnyBira I'd still recommend you not to abstract styling into HOCs. It doesn't make much sense either, since the layouting logic will be quite contained, judging from the examples.
Anyway, I hope I answered all your questions for you :smile:

I totally see your arguments agains doing this, personally I think the pros overweight the cons in this particular case.
Another reason for this approach in my option is portabilty. Imagine you've defined media querie with their responsive styles, encapsulated inside each styled component.
Now you want to reuse the component in another project with different breakpoints.
This logic could of course be tailor for every individual component, but I still prefer not having to repeat that logic and also have a common API for responsive props.
Thank you @philpl you've been very helpful.


Make some return types nullable, add StyleProvider

Strange still error
src/treeview.rs:112:20: 112:29 error: no method named `get_depth` found for type `core::option::Option<gtk::auto::tree_path::TreePath>` in the current scope

It need rebase after 279 merge and then regen again to hide TreeModel::filter_new.


Give padding in allergen alert list

Checklist

[x] Bug Report - The issue you are reporting might be there already. Please search on the issues tracker before creating or reporting an issue.
[x] Before reporting, please ensure (if you can) that you have tested it with the latest (dev) version of the software.
[x] Don't forget to tag your issue, it makes it easier to discover.

Summary:
Deleting the last item in allergen alert can be difficult due to the overlapping of the floating action button and the delete button.
Steps to reproduce:
Add a long list of allergens.
Expected behavior:
Floating action button should be present below this list.
Observed behavior:
Floating action button overlaps the Delete button.
Screenshots:

I would like to work on it.


Access to native google.maps.Marker instance

Issue description
Hi, I really like this Angular 2 Google Maps integration -  clean and well documented. Thank you very much!
What's missing for me though, is an easy way to get access to the native google.maps.Marker instances attached to each SebmGoogleMapMarker instance.
Similarly to #209 where I learned how to get access to the native google.maps.Map instance there is probably already some way to grab hold of the native google.maps.Marker instance. However, I could not figure out how to do it on my own.
I would really appreciate any hints on how to get access to the native google.maps.Marker instances attached to each SebmGoogleMapMarker instance.
Steps to reproduce and a minimal demo of the problem
I tried using @ViewChildren but without success. I forked the demo on Plunkr (http://plnkr.co/edit/to 7rdOz3QswDL0pik7SmAS?p=preview) and made some changes to it so that the ViewChildren get logged in the console (an empty array at the moment):
markers after view init []

However, it might as well be the totally wrong way to go so please don't take this demo too serious.
angular2 & angular2-google-maps version

angular2 version RC2 and RC3
angular2-google-maps version 0.12.0

I got it working now thanks to some hints in issue 307

hey @jfmaeck can you share your code? I'm having difficulty with this, even after looking at 307.


Update GridFramework to support flexbox

Related: https://github.com/concrete5/concrete5/issues/4791
Would be nice to see the grid layout editor support flex based framework.
In edit mode it just doesn't work currently.

I use my own framework with themes and started using flexbox a while ago. To get things working with the ‘layout’ option in edit mode I added a bit of code to my header.php.
`isEditMode()) { ?>

`
I had to modify my css to play nicely with C5, so to compromise I renamed some elements. I now have a ‘row’, not ‘ContentHolder’ and ‘span1’, ’span2’, etc instead of ‘boxx10’, ‘boxx20’, etc. Maybe you could try playing with that idea, just try changing the class names to suit foundation.

I decided to go the same route, still working on the editor side css but the framework itself is from the ground up.


[util] ralgen connectivity to topgen

As a part of fixing #8207, we introduced an "alternate" hjson path for modules such as ast.
However, when ralgen is invoked, it does not know to pass this alternate path to topgen, so the resulting register model generated would still be the old one.
We will have to update the flow to make sure the path can be passed through fusesoc generators to topgen.
@sha-ron i think this is the issue you were describing. Let me know if it's filed somewhere else.

Yes we need a way to pass the path through hjson/dvsim.
It is described in 8207 but indeed it is already closed. Thanks for opening this one Tim.

Ralgen is a FuseSoC "generator", which wraps over topgen and ralgen invocations. Unfortunately, my knowledge of FuseSoC is limited, so I am not sure if there is a way to modify a FuseSoC core file (in the open source) that already uses the open source specific settings to create the RAL model externally, from a closed source testbench. Only thing supported is FuseSoC flags, which are boolean values that enable or disable filesets. Luckily, it works for enabling / disabling filesets that are generated.
The solution I have settled on with #9325 will work as follows:

Skip the RAL generation that happens on parsing the hw/top_earlgrey/dv/env/chip_env.core entirely by passing the following FuseSoC flag in your Nuvoton chip_sim_cfg.hjson:

    sv_flist_gen_flags: ["--flag=skip_ral_gen"]

Add your own RAL generation step in your extended chip env core file:

generate:
  ral:
    generator: ralgen
    parameters:
      name: chip
      top_hjson: relpath/to/top_earlgrey.hjson
      hjson_path: path # Must be the parent dir containing ast.hjson

Since this RAL generation step is happening outside of the processing of chip_env.core file, we may run into a issue - we need to somehow ensure that the RAL generated in the step above is placed as a dependency of the open source chip_env.core, so that compilation does not break.

I hope this works, but if it doesn't, #9325 has support for one alternative approach - I will talk about it after you try the above :)

thanks @sriyerg ! I'll have a look at the other PR to better understand the alternative approach (the alternative approach i think is what you said would be more stable right?)


Fixes in unmarshal

There are 2 commits in this pull request to improve Unmarshal:

Fix for unameshal string : Using value of string when unmarshalling string into interface{} and fix some of the corresponding unit tests (the original change, roots in when returning type of reader.string changed to a pointer.

Add null check : Setting nil to v.Set() can lead to error.

Codecov Report

Merging #166 into master will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
+ Coverage   74.90%   74.94%   +0.04%
==========================================
  Files          25       25
  Lines        4857     4865       +8
==========================================
+ Hits         3638     3646       +8
  Misses        724      724
  Partials      495      495

Impacted Files
Coverage Δ

ion/unmarshal.go
68.64% <100.00%> (+0.55%)
:arrow_up:

Continue to review full report at Codecov.

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


Research use case: monitoring local CA bundle for expiring/expired CA certificates

While crafting examples for #314, I ran check_cert --filename /etc/ssl/certs/ca-certificates.crt in a test Ubuntu 20.04 environment and got back a failure for Cybertrust Global Root. The output notes 127 certs in the bundle.
The output is 1127 lines long; listing 127 certificates might be a bit much for plugin output.
Instead, a setting to list only non-OK certificates (expiring soon, already expired) would probably make this feasible.

I'd also need to research what the policy/practice is for expired root certificates. How long are they kept around before the CA bundle maintainers remove expired CA certs?


Unable to pod update

Flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.3, on macOS 13.4 22F66 darwin-arm64, locale en-MY)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.78.2)
[✓] Connected device (2 available)
[✓] Network resources

• No issues found!

Package version
flutter_libphonenumber: ^2.1.3

Expected

able to run pod update without error

Actual

error shows up after running pod update

Specs satisfying the `PhoneNumberKit/PhoneNumberKitCore (= 3.6.0),
PhoneNumberKit/PhoneNumberKitCore (= 3.5.10)` dependency were found,
but they required a higher minimum deployment target.

Make sure you added the entry to your podfile like the readme outlines.

@acoutts-nydig So we need to install melos and bootstrap?

I'll just put a tag for now.
pod "PhoneNumberKit", :git => "https://github.com/marmelroy/PhoneNumberKit", :tag => "3.5.10"

I'll just put a tag for now.
pod "PhoneNumberKit", :git => "https://github.com/marmelroy/PhoneNumberKit", :tag => "3.5.10"

but this isnt working

  [!] CocoaPods could not find compatible versions for pod "PhoneNumberKit/PhoneNumberKitCore":
      In Podfile:
        PhoneNumberKit (from `https://github.com/marmelroy/PhoneNumberKit`, tag `3.5.10`) was resolved to 3.5.10, which depends on
          PhoneNumberKit/PhoneNumberKitCore (= 3.5.10)

        flutter_libphonenumber_ios (from `.symlinks/plugins/flutter_libphonenumber_ios/ios`) was resolved to 1.1.0, which depends on
          PhoneNumberKit/PhoneNumberKitCore (= 3.6.0)

    Specs satisfying the `PhoneNumberKit/PhoneNumberKitCore (= 3.6.0), PhoneNumberKit/PhoneNumberKitCore (= 3.5.10)` dependency were found, but they required a higher minimum deployment target.

Im getting the same error even after putting the tag

can you please try the latest version just published?

Im on   flutter_libphonenumber: ^2.1.4

Did you try pod update?

issue happens when i run pod update.
Im using codemagic for building.

is there a way i can downgrade @acoutts ?

this is odd, i was able to upgrade and use the latest version with the example app.. is it an environment issue in codemagic?

Im sure that codemagic pulls all the deps and runs pod install everytime i run the build

Can it be an issue with Podfile.lock?

ill prolly get my macbook back tommorow, until then if there is way to downgrade and use it would be awesome

I'm not sure on your codemagic setup, but doing a pod update should update the lock file and work after that. Are you able to reproduce the issue with the example app? Then you can narrow down is it an issue with this package, your project, or codemagic.

let me fork and check

check the git tags to see, it will be there

@acoutts  what's the solution for this issue? I'm having this issue too

@BenevidesLecontes @icodelifee For whoever that is still facing this issue, it's probably because of the dependency's version. Since the error says which depends on PhoneNumberKit/PhoneNumberKitCore (= 3.6.0).
In your Podfile, your need to change the version tag to be 3.6.0 as such
pod "PhoneNumberKit", :git => "https://github.com/marmelroy/PhoneNumberKit", :tag => "3.6.0" # here

hi @MingSern thank you, I already did that and is working,  @acoutts i think this not the best solution this issue needs to remain open until a better solution. In our company we are developing plugins that depend on this plugin, but everyone who uses our plugins has to configure this in the podfile.


[FLINK-29644] [docs] Adding flink k8s operator documentation link

What is the purpose of the change
Flink docs don't have any documentation link related to Flink Kubernetes operator. This change lets us see the Flink Kubernetes operator and how to use it.
Brief change log

Added documentation links for Flink Kubernetes operator.

Verifying this change
This change is a trivial rework/code cleanup without any test coverage. I added new documentation links.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

Can you please always squash/rebase commits instead of merging? Makes review and later merging easier

Squashed multiple commits into one.

@gyfora - Do you think creating a separate PR with only this commit would be a good way instead squashing all commits into one? Please let me know.

looks fine +1, I am going to merge this. Please open the same PR against release-1.16 and release-1.15 branches so we can have this in all version of the docs

Thanks!
On Wed, Oct 19, 2022, 20:00 Gyula Fora @.***> wrote:

Merged #21089 https://github.com/apache/flink/pull/21089 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/flink/pull/21089#event-7622228114, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABV5SR64IMWKDOT267OPLBDWEAAYFANCNFSM6AAAAAARHTGYPE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


ERR Cannot read property 'nextElementSibling' of undefined

Testing #119367
Sorry, no steps nor further info but while testing output rendering I have seen these - in a block of 35 repetitions
 ERR Cannot read property 'nextElementSibling' of undefined: TypeError: Cannot read property 'nextElementSibling' of undefined
    at s.updateOutputRendering (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:2179:46116)
    at file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:2179:45667
    at c.fire (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:60:1836)
    at R.replaceData (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:982:29116)
    at l._replaceNotebookCellOutputItems (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:984:27669)
    at l.applyEdits (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:984:20709)
    at m.$applyEdits (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:2176:4697)
    at o._doInvokeHandler (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1589:11903)
    at o._invokeHandler (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1589:11587)
    at o._receiveRequest (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1589:10257)
    at o._receiveOneMessage (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1589:9043)
    at file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1589:7144
    at c.fire (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:60:1836)
    at s.fire (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:76:16186)
    at i._receiveMessage (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:76:21444)
    at file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:76:18330
    at c.fire (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:60:1836)
    at u.acceptChunk (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:76:13551)
    at file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:76:12899
    at Socket.k (file:///Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:2792:52556)
    at Socket.emit (events.js:315:20)
    at addChunk (_stream_readable.js:295:12)
    at readableAddChunk (_stream_readable.js:271:9)
    at Socket.Readable.push (_stream_readable.js:212:10)
    at Pipe.onStreamRead (internal/stream_base_commons.js:186:23)

It seems that output.model.onDidChangeData (which calls updateOutputRendering) may fired before render is called (which sets domNode)


Allow saved queries

Allow users to save their queries from Search box and name them.
How about even sharing them among users? E.g. the user would be the root folder for such queries, and all would be shared? Or should it be private? Probably it is part of customization.

Probably duplicates #500, merge all valuable info into one.


Fix cgroups per workload

This series allows each workload of a experiment to override the default CGroup defined by a target configuration. This can be obtained by specifying the cgroup in which a workload has to be executed under. For example, the following workload configuration fragment:
        "rta" : {
            "type" : "rt-app",
            "loadref" : "big",
            "cgroup" : "/tg1",
            "conf" : {
                "class"  : "profile",
                "params"  : {
                    "p20" : {
                        "kind"   : "periodic",
                        "params" : {
                            "duty_cycle_pct" : 20,
                         },
                    },
                },
            },
        }

will configure the "rta" workload to run under the "/tg1" cgroups.
NOTE: the specified cgroup has to be configured in each target configuration the workload will run, otherwise a run-time error will be generated.
NOTE: to use control groups the "cgroups" module has to be listed among the "modules" to load in the target/test configuration.

Tested on Juno.


onnxruntime-web vs tfjs

Issue Description
tfjs models are slower then onnxruntime-web
Steps to Reproduce
https://github.com/microsoft/onnxruntime-web-benchmark
Expected Behavior
Use the fastest framework for a specific task
Environment
web

this is a veeeery generic statement without any provided information...

which exact models?
using which backend?
what is the performance difference - give specifics

Hi Vladimir,
Thanks for you comment.  I really like your solution.
I am sorry for not being specific.
In my test, I was using a simple image processing with models from https://github.com/PINTO0309/PINTO_model_zoo
There are different formats of the models, so it is easy to compare.
When using onnx models I have seen smaller memory footprint and faster execution.
There is a chance that it is a specifics of the models conversion, in my specific case.
I just want to suggest to try different approach. The onnx implementation is relatively easy to implement and there are plenty models to use.
In my case, I just used simplest single html implementation with:
<script src="https://cdn.jsdelivr.net/npm/onnxruntime-web/dist/ort.min.js"></script>

it really depends on the model and the backend used
different models use different kernel ops and those ops are implemented differently by onnx and tjfs
even more, they are very different comparing tensorflow backend (in nodejs) or wasm or webgl or webgpu.
as far as i know:

onnx really prefers wasm backend and support for webgl is quite limited
tfjs doesn't spend that much time optimizing wasm backend, its the least one preferred and much more development is focused towards webgl (current) and webgpu (future)

that's why i'm asking - which exact model and which exact backend?


Fix Command for OS Feed Type is RetailEval in the instructions

G'day Team,
I think you need to update this last intruction on the page.  OSFeed for retail evaluation should be "RetailEval".  If I try this command with Retail option for evalaution it doesn't work.  When  i changed it to "RetailEval" , was able to update the OS to 20.01
If the OS Feed Type is RetailEval, change it to Retail:
azsphere device-group update --productname  --devicegroupname "Development" --osfeed "Retail"
Best,
Kumar

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

ID: f06ff328-8ce0-ed53-ec16-77557571701c
Version Independent ID: 6f534143-977d-a964-f170-ee07f0a3e083
Content: Deploy your first application quickstart - Azure Sphere
Content Source: AzureSphere/articles/install/qs-first-deployment.md
Product: azure-sphere
GitHub Login: @pennyo
Microsoft Alias: azurespheredocs

@kkomand Thank you for your feedback.
As described in Important News about Retail Evaluation, we recommend that customers create  a device group specifically for Retail Evaluation and maintain their development devices on the Retail OS feed. We believe that this approach will lead to the best customer experience.

G'day Team,
(1) When I tried the command with the Flag "Retail" as in the document for evaluation. It did not update the device to 20.01. Only when I tried the command with "RetailEval" flag it updated the device to 20.01.  Please review the command
azsphere device-group update --productname --devicegroupname "Development" --osfeed "Retail"
(2) The instructions are slightly convoluted and would request a sequenced instructions for the scenarios below:
(a) Procedure to update a device to Retail Evaluation version  for those who are using 19.11 OS
version  &  corresponding known issues/fixes.
(b) Procedure to update a device to Retail Evaluation version for those who are running Out-of-the box OS such as 18.x.  (Ex: New kits). The newer commands (such as azsphere enable cloud-test) doesn't work with 18.x OS versions. So if you can provide instructions on how to update a device that's on older version to  20.01 , that would ease the process .
Best,
Kumar

@kkomand Were you unable to deploy an app to the device with 19.11 OS/SDK?


Initial work for the onboarding/community documentation

This is the initial work for the OpenEXR Bindings project onboarding and community documentation.
It isn't complete at the moment, but is meant to be the starting point of discussions.
Closes #31

Looks like a great start, thanks Scott! My only thought is that we could use “openexr-bind” rather than “the OpenEXR Rust Bindings Project”, both for brevity and because on a diagonal read I kept thinking I was looking at documentation from OpenEXR itself so might be good to separate this more clearly?

Looks like a great start, thanks Scott! My only thought is that we could use “openexr-bind” rather than “the OpenEXR Rust Bindings Project”, both for brevity and because on a diagonal read I kept thinking I was looking at documentation from OpenEXR itself so might be good to separate this more clearly?

Yeah, I was wondering about that. I did borrow some of the text from OpenEXR, so some of the structure is the same.

Only other thing to do would be to add a section to the README linking to these

Notes from a discussion with one of the contributors for OCIO:

Since our goal is to hand off ownership of the bindings, we need to figure out some of the legal stuff (CLA, signoff, etc.) Even though we probably don't need to worry about it now, OpenEXR, and other projects do need to worry.
How to integrate our documentation with the wrapped libraries. For example, https://opencolorio.readthedocs.io/en/latest/api/grading_transforms.html has the Python and C++ bindings all in one page. They'd likely want the Rust bindings on the same page as well.
Which license should we use (I think that it is BSD, but John Mertic is probably the best person to talk to).
Make sure we go through the process of "I want to contribute", creating issues and pull requests, and having the PR approved. Also, a good thing to think about with diversity/inclusivity is to think about what are our expectations for someone contributing. For example, what's the minimal amount of Rust should someone know to contribute.
Defining some goals/non goals of the OpenEXR crate. So, stuff like how close to the wrapped library do we want the crate, what functionality are we explicitly leaving out, etc.

Looks good to me. Ready to merge?