Cancel Schedule When App is Terminated

I am trying to make a missed call notification.  When a call type notification is displayed, it launched a 15 second scheduled notification.  If someone presses the accept or cancel button, I call the function to cancel the notification scheduled by group key.  This works with the App in the background, but if the App is terminated, the scheduled notification is never canceled.  Is there a way to do this, is it not possible, or is there a specific way to do it?  Thanks

I am also facing a similar issue, I am trying to cancel all the sheduled notifcations but after restarting the app if i call the function to cancel All Notifications or cancel all schedules, the scheduled notifications are never cancelled. If i call AwesomeNotifications().listAllSchedules function it does not show all the schedules that were added before restarting the app

It seems a problem with Shared Preferences for Android not saving the informations.
Can you confirm to me what plugins your app are using and what Gradle version is your app?

It seems a problem with Shared Preferences for Android not saving the informations. Can you confirm to me what plugins your app are using and what Gradle version is your app? Did you change anything in your Android project folder?

gradle-6.7
plugins list
provider: ^6.0.2
get: ^4.6.1
sizer: ^2.0.15
flutter_launcher_icons: ^0.9.2
http: ^0.13.4
flutter_spinkit: ^5.1.0
modal_progress_hud_nsn: ^0.1.0-nullsafety-1
shared_preferences: ^2.0.12
awesome_notifications: ^0.6.21
location: ^4.3.0
fluttertoast: ^8.0.2

and i have not made any changes to android folder

My Project have this plugins:
environment:
sdk: ">=2.12.0 <3.0.0"
dependencies:
animated_splash_screen: ^1.1.0
auto_size_text: ^3.0.0
avatar_glow: ^2.0.2
awesome_notifications: ^0.6.21
badges: ^2.0.1
cloud_firestore: ^3.1.8
cloud_functions: ^3.0.3
cupertino_icons: ^1.0.2
device_info_plus: ^3.2.1
email_validator: ^2.0.1
empty_widget: ^0.0.5
firebase_auth: ^3.0.1
firebase_core: ^1.4.0
firebase_messaging: ^11.2.6
flutter:
sdk: flutter
flutter_easyloading: ^3.0.0
flutter_local_notifications: ^9.3.2
flutter_localizations:
sdk: flutter
flutter_mobx: ^2.0.2
flutter_svg: ^1.0.3
font_awesome_flutter: ^9.1.0
google_nav_bar: ^5.0.5
google_sign_in: ^5.0.5
http: ^0.13.3
jitsi_meet: ^4.0.0
line_icons: ^2.0.1
lint: ^1.7.2
mobx: ^2.0.4
modal_bottom_sheet: ^2.0.0
move_to_background: ^1.0.2
package_info: ^2.0.2
provider: ^6.0.0
rflutter_alert: ^2.0.4
share_plus: ^3.0.4
shared_preferences: ^2.0.12
shimmer: ^2.0.0
dev_dependencies:
build_runner: ^2.1.2
flutter_launcher_icons: ^0.9.1
flutter_test:
sdk: flutter
mobx_codegen: ^2.0.3
Gradle: 6.7

Hi! Sorry for taking so long to respond to your issue on GitHub. I was focused on releasing the new 0.7.0 version of awesome_notifications and awesome_notifications_fcm and i didn't had time enough to do both. But now I can answer all your questions.
So now i'm asking you to recreate this topic using the new issue template. There's a lot of missing informations that i need to understand your problem, and all the instructions are already in the new issue template. Also, remember to check beforehand if your issue was posted by another user.
So I will automatically close all previous issues so far.
Sorry for the inconvenience and i will be waiting for your new issue request.
Thank you so much for your support!


libsimdpp 2.1 [fix] :snowflake:

libsimdpp 2.1 [fix] :snowflake:
Destination channel: Snowflake
Links

PKG-4203
dev_url: https://github.com/p12tic/libsimdpp/tree/v2.1

Explanation of changes:

fix copy *.hpp files as well, add test for them

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_4b1la_bn6t/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:21: patch_must_be_in_build: patch must be in build when source/patches is set.
===== Final Report: =====
1 Error and 0 Warnings were found


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_43540c13-9948-445c-8ef5-4bb891708bec/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_43540c13-9948-445c-8ef5-4bb891708bec.git

Docs Build status updates of commit cb34371:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-source-1' (branch: Release_Archive_main_2022-02-21-07-46-54) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Header Links not Showing Header Name

When linking to headers the header name isn't shown.
Arm Muscles.md
- [[Arm Muscles#Deltoid]]
- [[Arm Muscles#Biceps Brachii|Biceps Brachii]]

## Deltoid
...

## Biceps Brachii
...

How it looks on GitHub Pages:

Both link to the correct header, but the top link doesn't show the header name.

There are 2 ways of fixing this bug. Either display "Note Name" >"Header Name" or just "Header Name". Obsidian does the first method:

but I strongly recommend against this and instead it should simply show the headers name.

This is intentional. Wikipedia does this as well afaik. You can configure this in Quartz 3/4 but this is not a bug.

Can you elaborate, I was also going to reference Wikipedia as that is a good standard of doing things. How so do they have that system in place?
Also, this isn't really an issue because one almost never wants the name of the header as is because the header is in caps and one mostly wants to reference it without caps


Function search

This tool supports more and more features, which is a good thing. I think it's time to add a search feature

It's possible, check the settings

It's possible, check the settings

Oh, there it is. Maybe we can make it a little more obvious


Styling headings

Hi,
First of all, nice wrapper!
Is it possible to style the headings with font-weight so the labels don't float together with the columns?
Now it's a bit hard to see the actual labels and the column width is also a bit tiny.

Hello @nahojj , Thank you.
About this lib: it is an adaptation for "Vue 3" of an existing version for "Vue 2" https://www.npmjs.com/package/vue-xlsx
About your doubt; this lib uses in the background a lib called "xlsx", its documentation is here; https://www.npmjs.com/package/xlsx.
When you want to generate an XLSX from a JSON, the "json_to_sheet" method of the "xlsx" lib is called in the background, so you can consume the official "xlsx" documentation to understand the possibilities of the "vue3-xlsx" lib .

@joaomede Can you please provide an example of what you are meaning? I understand I can use and hijack into the official package but not how I should write this in Vue.

@nahojj
When you declare <xlsx-sheet>, you pass a prop, called "collection", this property needs to meet 2 types of structure, because the library under the hood, if called "xlsx", and in the documentation it talks about 2 methods
"aoa_to_sheet"
"json_to_sheet"
So the JSON structure is the second method; in this case, "json_to_sheet", if you look at this link "https://www.npmjs.com/package/xlsx" about this method (json_to_sheet) or (aoa_to_sheet), you will see several possibilities of use, including styling.
To make it clear, under the hood "things" happen, and these things belong to the "xlsx" library, this project here makes a wrapper on top of this "xlsx" library and it respects the entries, so many things under the hood can be done, feel free to study the "xlsx" package and test various possibilities.
obs: be sure to read the components of this project to understand how properties are received, this can help you

@joaomede Thanks! I'll try the collection prop and hijack me from there down to xlsx.


Fix broken 2.4.x and 2.5.0 builds

Fixes an issue in DS v2.4.x and 2.5.0 where the cjs export from pcln-icons is incorrectly ingested by the esm rollup of core in core/Stepper/Step

Codecov Report

Merging #602 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #602   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          63     63
  Lines         664    664
=====================================
  Hits          664    664

Impacted Files
Coverage Δ

packages/core/src/Step.js
100% <ø> (ø)
: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 c5d07e0...832f160. Read the comment docs.


return error when findDevice failed

This is already returning the error using a named result parameter: https://golang.org/doc/effective_go.html#named-results

Thanks for the patch, as explained above, this is not necessary, so I'm going to close it out. (There's an argument that we should omit the named return parameter entirely, for explicitness, but in that case it'll be a more invasive change, so please file a new PR).

Thank you, you are right. That's my bad.  Yes, in device.go, there is no consistency to use named result parameters.


Fix link to latest WordPress version

As it is, without this fix, WordPress setup is broken as the link is not valid without the wordpress- bit. Updated to the latest patch because might as well. Thanks!

I signed it!

Great catch, thank you @bionicbrian!

Can we revert it back to https://wordpress.org/latest.tar.gz?
@ryanmats WDYT?

@tmatsuo i'll run through everything again soon to see if the issue with 'latest' has been resolved!


Metrics for Service Binding States

Is your feature request related to a problem? Please describe.
Hi folks!
We were delighted to see that the interoperator already comes with metrics regarding cluster usage & service instance states (and a sample dashboard for it).
Describe the solution you'd like
However, for service bindings, there is currently no metrics. Having metrics service_binding_state similar to service_instance_state would be very nice (especially now that interoperator supports async bindings).
Describe alternatives you've considered
None
Additional context
Would you accept a PR for this? Happy to contribute!

Hi @MatthiasWinzeler
Thank you for reaching out and proposing to also raise PR for the same. We would in general be happy to take the PRs.
But, since we are in planning phase for Q2, we can may be add the above mentioned feature request in our scope and plan to work on it. Would that be fine with you?

Hi @swati1102
Sounds great! Q2 would be perfect for us. Thanks for your effort and let us know when we can support.

@MatthiasWinzeler we have added service_binding_state metrics to show the binding_id and the id of the corresponding service_instance this binding belongs to. Here is a sample promql output:
interoperator_service_bindings_state{app_kubernetes_io_managed_by="Helm", binding_id="0abc2107-de86-408d-8617-800935b84028", control_plane="interoperator-multiclusterdeployer-controller-manager", instance="xx.xx.xx", instance_id="80ca2362b-6561-4673-ad24-111d7ac86cfd", job="kubernetes-service-endpoints", kubernetes_name="interoperator-multiclusterdeployer-metrics-service", kubernetes_namespace="interoperator", kubernetes_node="shoot--interop--pooja-dev-worker-j5mor-z1-6b645-jb924"} | 1
interoperator_service_bindings_state{app_kubernetes_io_managed_by="Helm", binding_id="0abc2107-de86-408d-8617-800935b84038", control_plane="interoperator-multiclusterdeployer-controller-manager", instance="xx.xx.xx", instance_id="80ca2362b-6561-4673-ad24-111d7ac86cfd", job="kubernetes-service-endpoints", kubernetes_name="interoperator-multiclusterdeployer-metrics-service", kubernetes_namespace="interoperator", kubernetes_node="shoot--interop--pooja-dev-worker-j5mor-z1-6b645-jb924"} | 1
interoperator_service_bindings_state{app_kubernetes_io_managed_by="Helm", binding_id="0ceb2107-de86-408d-8617-800935b84108", control_plane="interoperator-multiclusterdeployer-controller-manager", instance="xx.xx.xx", instance_id="65ca2362b-6561-4673-ad24-111d7ac86cfd", job="kubernetes-service-endpoints", kubernetes_name="interoperator-multiclusterdeployer-metrics-service", kubernetes_namespace="interoperator", kubernetes_node="shoot--interop--pooja-dev-worker-j5mor-z1-6b645-jb924"} | 0
interoperator_service_bindings_state{app_kubernetes_io_managed_by="Helm", binding_id="1abc2107-de86-408d-8617-800935b84038", control_plane="interoperator-multiclusterdeployer-controller-manager", instance="xx.xx.xx", instance_id="81ca2362b-6561-4673-ad24-111d7ac86cfd", job="kubernetes-service-endpoints", kubernetes_name="interoperator-multiclusterdeployer-metrics-service", kubernetes_namespace="interoperator", kubernetes_node="shoot--interop--pooja-dev-worker-j5mor-z1-6b645-jb924"} | 0

Also, attached are few sample dashboards that can be generated using the binding metrics:
service_binding_state : represents the state of all the bindings
service_binding_count: represents the total count of the bindings present in the cluster
service_bindings_grouped_by_instances: represents the bindings grouped by the service instances
service_bindings_in_failed_state: represents the failed bindings in the cluster

Please let us know if this helps.

@Pooja-08 sounds really cool! do you already have a plan when it should land in a release?

@MatthiasWinzeler tentatively by July end.

@Pooja-08 cool, sounds great!

Closing this feature request as it has been released in v0.17.0


Should the compiler warn about a type that is both globally-isolated and Sendable?

Motivation
Something I'm noticing people do is both mark a class MainActor and also make it Sendable. I don't see why this should be allowed - it seems like a logical mistake. The developer is simultaneous expressing that this type is both only usable on one actor and also safe to pass to any actor.
I don't know if an error is appropriate but maybe a warning?
@MainActor
class MyClass: Sendable {}

Proposed solution
I think this should produce a warning.
Alternatives considered
No response
Additional information
No response

Something I'm noticing people do is both mark a class MainActor and also make it Sendable. I don't see why this should be allowed - it seems like a logical mistake.

Hmm, can you please elaborate on why you think this is the case? Making a class isolated to the @MainActor implies a conformance to Sendable. You can pass an instance of the value across isolation boundaries and have concurrent references to it specifically because its state is protected by actor isolation -- to touch any state, you need to get back on the main actor. You can still call any nonisolated methods on the class because those cannot touch main actor isolated state. The Sendable conformance is just being explicit, but a Sendable conformance is inferred if you don't write it!

I think I can chock this up to just misunderstanding on my part. Sorry for the noise!


delete-federation.sh script doesn't take care of FederatedItems

If this script is used without deleting the federated objects beforehand the ControlPlane won't be deleted.
I think this script should wether show a disclaimer or ask the user for confirmation in order to delete federated objects.
To my understanding, it should be enough if FederatedClusterRole and Namespaces associated with a FederatedNamespacePlacemenare deleted from the Host Cluster.
What do you think?

@mvazquezc +1 to your proposal, can you please create a PR to update the document at https://github.com/kubernetes-sigs/federation-v2/blob/master/docs/userguide.md#cleanup to claim this first?
Actually, we already cleaned up the resources at https://github.com/kubernetes-sigs/federation-v2/blob/master/docs/userguide.md#example-cleanup , but it would be great if we can highlight this again before delete federation.

@mvazquezc can you close this issue as the PR was merged?

Sure.


Implement smart threading improvements

[ ] Start pulling gsuite resources as soon as inventory starts, do not wait for CAI export to complete.
[ ] Remove blocking API calls from the visit() function and move them all into child resources
[ ] Aggressively fork API requests into a separate thread queue from CAI sourced data

Start pulling gsuite resources as soon as inventory starts, do not wait for CAI export to complete.
Remove blocking API calls from the visit() function and move them all into child resources
Aggressively fork API requests into a separate thread queue from CAI sourced data

Analysis shows little time gain kicking off GCP API gathering before CAI. This work would only gain us a few mins improvement.
Closing as Will not Fix.


Add crosspost resolver

Changes done:

Added and using Crosspost URL resolver
Cleaned up the cleanup hook, removed unused functionality, dependencies follow in next version
Added more domains to generic downloader
Fixed issue with environment system
Switched stats dictionary to Counter


removing CAPEX

CAPEX is an acronym that people might not know. It would be better not to use lingo. Broke it into "money or resources."

--
Daniel Rothamel
434-962-7305
~sdg~
On Thu, Jan 13, 2022 at 8:34 AM Catherine Paganini @.***>
wrote:

@.**** approved this pull request.
Good point, thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/cncf/glossary/pull/306#pullrequestreview-851717039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUSQ65AI4ZUXFA6VFSEDG33UV3IH5ANCNFSM5LZVBOJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Great job, thank you!

Thank you, Jason!
On Fri, Jan 14, 2022 at 9:08 AM Jason Morgan @.***>
wrote:

Merged #306 https://github.com/cncf/glossary/pull/306 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/cncf/glossary/pull/306#event-5895027457, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AUSQ65A2MK3URB5JN74PBULUWAU4DANCNFSM5LZVBOJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Daniel Rothamel
434-962-7305
~sdg~


add snippets to method/function matches

I had to rethink my approach because SublimeText (and perhaps other editors) have some serious limitations regarding autocompletion.
The purpose of this MR is to add snippet to the function match, so that editors can paste it on completion. That snippet has placeholders for function arguments, so it is easier to fill them. I really like this feature and would like to have it for Rust.
I used ??? string to separate snippet from the rest of the match string, because both snippets and context may contain commas, which would mess splitting. I'm open to suggestions if you have better idea how to handle this.
The snippet format seems to be supported by popular editors, i.e. SublimeText, Atom, Emacs and Vim. Other editors should be able to adjust it to their needs.

After selecting completion you should be able to switch between arguments with TAB.

Hi @krzat,
Nice! This seems like a good approach. I need to get a bit more familiar with snippets.
I'm afraid I can't merge this as-is because the ??? corrupts the other editor plugins output a bit. Could you maybe work this in as a separate command for the short term (e.g. complete-with-method-info or something)
Thanks!
Phil

Done. I used this format: MATCH {};{};{};{};{};{:?};{}, where snippet is at second position.
Maybe I should just use JSON instead? I don't know how much the performance would suffer, but it would allow later for more customizability.

Hi @krzat, This is awesome! thanks for adding this. When I get some time I'll have a play with snippets on emacs.
Regarding json, I agree it would be a more flexible format. The advantage of the current csv-style format is that it's easy for each of the editors to parse without creating a dependency on a json library. I'm not sure if this is a good tradeoff long term; I'd want to know what was involved in parsing json in at least emacs and vim before switching though.
Thanks again,
Phil


NO-JIRA: remove CPO/CPPKIO from the hypershift-operator image build

hypershift-operator image builds are taking a long time in prow ~20m.
Don't build more than we need.

/area ci-tooling

/lgtm

/override ci/prow/e2e-aws
it already passed once


Install Failure: Mojo::Template

When installing with panda I received the following error:
===SORRY!===
Could not find Template::Mojo in any of: /home/lancew/.panda-work/1428163693_5/blib/lib, /home/lancew/.panda-work/1428163693_5/lib, /home/lancew/.perl6/2015.03, /usr/languages/perl6/lib, /usr/languages/perl6, /usr/languages/perl6/site
build stage failed for Bailador: Failed building lib/Bailador/Template/Mojo.pm
in method install at lib/Panda.pm:122
in method resolve at lib/Panda.pm:198
in sub MAIN at /home/lancew/.perl6/2015.03/45:20
in sub MAIN at /home/lancew/.perl6/2015.03/45:18
in block  at /home/lancew/.perl6/2015.03/45:82
Attempted to manually install Temaple::Mojo. This did not resolve the issue.
Attempted: panda install Bailador::Template::Mojo This also did not solve issue

FWIW https://github.com/tadzik/Template-Mojo/pull/17 fixes that.

Fixed by PR 17, thanks for reporting :)


Replaced Image Path with Body

Added Body field instead of just Image File field so we can add any widget we want in splash screen. For example, custom animated text.

Hi @VrajGohil , the context of splash is to show a logo right, if I merge this pr, someone else will complain me - stating that I got overflow error.

Okay! Animated Text was what i was going for. Thanks for letting me know.


[FLINK-18484][core] - Add length difference information to arity exception

What is the purpose of the change

Include length difference details when RowSerializer throws RuntimeException due to this difference

Brief change log

Exception message updated

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
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/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

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

Thanks! I merged this manually. 👍


UnicodeDecodeError

I am trying to run the master branch on Arch Linux.  When I run python2.7 -m ice I get the following output:
=========== Starting Ice ===========
Detected Emulator: Dolphin
Detected Console: Nintendo Wii => Dolphin
=========== User: 37281274 ===========
[ERROR] An exception occurred while running Ice
Traceback (most recent call last):
File "ice/runners/ice_engine.py", line 153, in run
self.main(dry_run=dry_run)
File "ice/runners/ice_engine.py", line 133, in main
self.run_for_user(user_context, dry_run=dry_run)
File "ice/runners/ice_engine.py", line 145, in run_for_user
self._create_backup(user, dry_run=dry_run)
File "ice/runners/ice_engine.py", line 167, in _create_backup
shortcuts.write_shortcuts(backup_path, shortcuts.get_shortcuts(user))
File "/usr/lib/python2.7/site-packages/pysteam-1.0.0b2-py2.7.egg/pysteam/shortcuts.py", line 30, in write_shortcuts
vdf_contents = ShortcutGenerator().to_string(shortcuts)
File "/usr/lib/python2.7/site-packages/pysteam-1.0.0b2-py2.7.egg/pysteam/_shortcut_generator.py", line 20, in to_string
string = x00 + 'shortcuts' + x00 + self.generate_array_string(shortcuts) + x08 + x08 + x0a
File "/usr/lib/python2.7/site-packages/pysteam-1.0.0b2-py2.7.egg/pysteam/_shortcut_generator.py", line 29, in generate_array_string
string += x00 + str(i) + x00 + self.generate_shortcut_string(shortcut)
File "/usr/lib/python2.7/site-packages/pysteam-1.0.0b2-py2.7.egg/pysteam/_shortcut_generator.py", line 37, in generate_shortcut_string
string += self.generate_keyvalue_pair("icon",shortcut.icon)
File "/usr/lib/python2.7/site-packages/pysteam-1.0.0b2-py2.7.egg/pysteam/_shortcut_generator.py", line 52, in generate_keyvalue_pair
return x01 + key + x00 + value + (x00 if more else x08)
UnicodeDecodeError: 'ascii' codec can't decode byte 0x84 in position 109: ordinal not in range(128)
Close the window, or hit enter to exit...
These are all my rom file names, nothing non-ascii in them:
/s/games/wii/roms/Skyward Sword.iso
/s/games/wii/roms/DKCR.iso
/s/games/wii/roms/Mario Galaxy.iso
/s/games/wii/roms/Metroid Prime 3 Corruption.iso
/s/games/wii/roms/Goldeneye 007.iso
/s/games/wii/roms/Mario Galaxy 2.iso
/s/games/wii/roms/Twilight Princess.iso

I have the same problem on windows 7 and my snes roms. I run the master branch, too.
2016-01-09 13:14:05,956 [DEBUG][steam_shortcut_synchronizer.py][sync_roms_for_user:99]: Saving shortcuts
2016-01-09 13:14:05,957 [ERROR][ice_engine.py][run:155]: An exception occurred while running Ice
Traceback (most recent call last):
File "ice\runners\ice_engine.py", line 153, in run
self.main(dry_run=dry_run)
File "ice\runners\ice_engine.py", line 133, in main
self.run_for_user(user_context, dry_run=dry_run)
File "ice\runners\ice_engine.py", line 148, in run_for_user
self.shortcut_synchronizer.sync_roms_for_user(user, roms, self.config, dry_run=dry_run)
File "ice\steam_shortcut_synchronizer.py", line 100, in sync_roms_for_user
shortcuts.set_shortcuts(user, updated_shortcuts)
File "C:\Python27\lib\site-packages\pysteam-1.0.0b2-py2.7.egg\pysteam\shortcuts.py", line 41, in set_shortcuts
write_shortcuts(paths.shortcuts_path(user_context), shortcuts)
File "C:\Python27\lib\site-packages\pysteam-1.0.0b2-py2.7.egg\pysteam\shortcuts.py", line 30, in write_shortcuts
vdf_contents = ShortcutGenerator().to_string(shortcuts)
File "C:\Python27\lib\site-packages\pysteam-1.0.0b2-py2.7.egg\pysteam_shortcut_generator.py", line 20, in to_string
string = x00 + 'shortcuts' + x00 + self.generate_array_string(shortcuts) + x08 + x08 + x0a
File "C:\Python27\lib\site-packages\pysteam-1.0.0b2-py2.7.egg\pysteam_shortcut_generator.py", line 29, in generate_array_string
string += x00 + str(i) + x00 + self.generate_shortcut_string(shortcut)
File "C:\Python27\lib\site-packages\pysteam-1.0.0b2-py2.7.egg\pysteam_shortcut_generator.py", line 35, in generate_shortcut_string
string += self.generate_keyvalue_pair("Exe",shortcut.exe)
File "C:\Python27\lib\site-packages\pysteam-1.0.0b2-py2.7.egg\pysteam_shortcut_generator.py", line 52, in generate_keyvalue_pair
return x01 + key + x00 + value + (x00 if more else x08)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xfc in position 92: ordinal not in range(128)

Same issue here on Debian/SteamOS. Glad to know it's not the Debian package I made specifically.

This is an issue with pysteam. I encountered this problem as well and made an issue about it. One thing to mention right away is that, since Ice has to re-create the shortcuts.vdf file, this problem could occur due to a non-Steam app shortcut that you have created; your ROMs might not be causing the problem. The problem for me was due to a custom non-Steam app shortcut for the game Star Wars: Dark Forces. In my case, a Registered Sign symbol (U+00AE) is used in the game name (STAR WARS®: Dark Forces).
https://github.com/scottrice/pysteam/pull/7
Unicode Character 'REGISTERED SIGN' (U+00AE)
http://www.fileformat.info/info/unicode/char/00ae/index.htm

Thank you very much @Ryochan7. This explains why I didn't get this before as it was likely a clean install still needs fixed, but I'll do a clean install soon to confirm. I'll susbcribe to the issue you linked. I had thought my pysteam packaging was wrong, as it worked before. I actually debianized all the pypi packages per the required versuons, using py2dsc.

I'll test this PR you made with my fork.


fix(core/services/onedrive): remove @odata.count for onedrive list op

This @odata.count does not seem to exist before in their official docs...
Recently I found they don't return this field anymore so all list op would raise errors.
I'm going to remove it.
In this repo, this field does not exist for all the time: https://github.com/sreeise/graph-rs-sdk/blob/afda1dcfbf5d491e56c6753152a1ce7e1c97fb44/test_files/drive_ep/drive_root_child.json#L4

for the record, I verified this change locally on my branch

end to end
I don't think a local run of the behavior test is needed because this PR is deleting an unused field

Thanks!

thanks for the quick review! 🫡


Connecting to sysrepo failed (libyang error).

Hi,
Environment Version: libyang1
The following problems occasionally occur after a long run.
root@OpenWrt:~# cat /tmp/log/netopeer2-server.log
[INF]: LY: Plugin "/usr/lib/libyang1/extensions/yangdata.so" successfully loaded.
[INF]: LY: Plugin "/usr/lib/libyang1/extensions/nacm.so" successfully loaded.
[INF]: LY: Plugin "/usr/lib/libyang1/extensions/metadata.so" successfully loaded.
[INF]: LY: Plugin "/usr/lib/libyang1/user_types/user_yang_types.so" successfully loaded.
[INF]: LY: Plugin "/usr/lib/libyang1/user_types/user_inet_types.so" successfully loaded.
[INF]: LY: Reading module "ietf-yang-metadata".
[INF]: LY: Module "ietf-yang-metadata@2016-08-05" successfully parsed as implemented.
[INF]: LY: Reading module "yang".
[INF]: LY: Resolving "yang" unresolved schema nodes and their constraints...
[INF]: LY: All "yang" schema nodes and constraints resolved.
[INF]: LY: Module "yang@2017-02-20" successfully parsed as implemented.
[INF]: LY: Reading module "ietf-inet-types".
[INF]: LY: Resolving derived type "union" failed, it will be attempted later.
[INF]: LY: Resolving derived type "union" failed, it will be attempted later.
[INF]: LY: Resolving derived type "union" failed, it will be attempted later.
[INF]: LY: Resolving derived type "union" failed, it will be attempted later.
[INF]: LY: Resolving "ietf-inet-types" unresolved schema nodes and their constraints...
[INF]: LY: All "ietf-inet-types" schema nodes and constraints resolved.
[INF]: LY: Module "ietf-inet-types@2013-07-15" successfully parsed as implemented.
[INF]: LY: Reading module "ietf-yang-types".
[INF]: LY: Module "ietf-yang-types@2013-07-15" successfully parsed as implemented.
[INF]: LY: Searching for "ietf-yang-types" in /mnt/sysrepo/yang.
[INF]: LY: Resolving "sysrepo" unresolved schema nodes and their constraints...
[INF]: LY: All "sysrepo" schema nodes and constraints resolved.
[INF]: LY: Module "sysrepo@2021-01-18" successfully parsed as implemented.
[ERR]: LY: Failed to find matching hash for a child of "installed-module". (/sysrepo:sysrepo-modules/installed-module)
[ERR]: SR: Failed to find matching hash for a child of "installed-module".
[ERR]: NP: Connecting to sysrepo failed (libyang error).
[ERR]: NP: Server init failed.
[INF]: NP: Server terminated.

root@OpenWrt:~# sysrepoctl -l
[ERR]: Failed to find matching hash for a child of "installed-module". (path: /sysrepo:sysrepo-modules/installed-module)
sysrepoctl error: Failed to connect (libyang error)

After deleting SYSREPO, restart the deployment to return to normal.
Thanks.

Depending on the exact libyang version, it may be fixed in more recent version (of v1). But if not, it seems like a serious LYB format error (alternatively caused by corrupted LYB file), which are really difficult to debug. So, unless there was a simple way to reproduce, I will not be spending time on this obsolete version.


Drop the minRank parameter to getPwPairs

Client code can implement its own filtering here.

@nondebug also

LGTM


Unable to pull Docker image from private repo

I'm trying to pull an image from Dockerhub as part of a PythonScriptStep, but the job fails with the error Error response from daemon: pull access denied for base_training, repository does not exist or may require 'docker login'. Username and pw are provided, but this specific image is actually public. Pulling from my local machine works with docker pull docker.io/kandoenv/base_training:prod works.
To reproduce:
from azureml.core import Workspace, Experiment, ScriptRunConfig, Environment
from azureml.core.runconfig import RunConfiguration
from azureml.pipeline.core import Pipeline
from azureml.pipeline.steps import PythonScriptStep

ws = Workspace.from_config()
experiment = Experiment(ws, "first_experiment")

compute_target = ws.compute_targets["cpu-cluster"]
aml_run_config = RunConfiguration()
aml_run_config.target = compute_target

aml_env = Environment("env_name")
aml_env.python.user_managed_dependencies = True
aml_env.docker.base_image = "base_training:prod"
aml_env.docker.base_image_registry.address = "docker.io"
aml_env.docker.base_image_registry.username = "kandoenv"
aml_env.docker.base_image_registry.password = "*********"
aml_env.environment_variables = {"foo": "bar"}
aml_env.register(ws)
aml_run_config.environment = aml_env

aml_env.get_image_details(workspace=ws)
> {'imageExistsInRegistry': True, 'ingredients': None, 'pythonEnvironment': {'interpreterPath': 'python', 'condaEnvironmentName': None, 'condaEnvironmentPath': None}, 'dockerImage': {'name': 'base_training:prod', 'registry': {'address': 'docker.io', 'username': 'kan***', 'password': '***'}}, 'warnings': []}

train_src = ScriptRunConfig(
    source_directory=".",
    script="train.py",
    compute_target=compute_target,
    environment=aml_env,
)

train_step = PythonScriptStep(
    name="train_step",
    script_name="train.py",
    compute_target=compute_target,
    runconfig=train_src.run_config
)

pipeline = Pipeline(workspace=ws, steps=[train_step])
pipeline_run = experiment.submit(pipeline)

Closing legacy issue.
Please consider upgrading to AzureML v2 CLI/SDK. https://learn.microsoft.com/en-us/azure/machine-learning/concept-v2


Is it possible to disable autoplay?

yes how to disable auto play I too want to have this feature

Hello @zakharchanka @muthu1001
To disable autoplay, you can use :
player.playWhenReady = false in Kotlin
player.setPlayWhenReady(false) in Java


Entity Report Error When using CRFEntityExtractor

Describe the bug
I have changed the default NLU pipeline and added CRFEntityExtractor rasa component
pipeline:
  - name: WhitespaceTokenizer
  - name: CountVectorsFeaturizer
  - name: EmbeddingIntentClassifier
  - BILOU_flag: true
    name: CRFEntityExtractor
    features:
      - [low, title, upper]
      - [low, bias, prefix5, prefix2, suffix5, suffix3, suffix2, upper, title, digit, pattern]
      - [low, title, upper]
  - name: rasa_addons.nlu.components.gazette.Gazette
  - name: EntitySynonymMapper

Getting the below Error
 Warning: Failed prop type: The prop `predictions` is marked as required in `EntityReport`, but its value is `undefined`.
    in EntityReport (created by Tab)
    in Tab (created by Evaluation)
    in form (created by Form)
    in Form (created by Evaluation)
    in Loading (created by Evaluation)
    in div (created by Segment)
    in Segment (created by TabPane)
    in TabPane (created by Evaluation)
    in Evaluation (created by ForwardRef)
    in ForwardRef
    in ForwardRef (created by NLUModel)
    in div (created by Container)
    in Container (created by NLUModel)
    in div (created by NLUModel)
    in NLUModel (created by Context.Consumer)
    in Unknown (created by ForwardRef)
    in ForwardRef
    in ForwardRef (created by Connect(Component))
    in Connect(Component) (created by RouterContext)
    in div (created by Project)
    in Unknown (created by Project)
    in div
    in Pane
    in div
    in SplitPane (created by Project)
    in div (created by Project)
    in div (created by Project)
    in Project (created by ForwardRef)
    in ForwardRef
    in ForwardRef (created by Connect(Component))
    in Connect(Component)
    in ErrorBoundary
    in Unknown (created by RouterContext)
    in RouterContext (created by Router)
    in Router (created by Routes)
    in Provider (created by Routes)
    in ApolloProvider (created by Routes)
    in ApolloProvider (created by Routes)
    in DocumentTitle (created by SideEffect(DocumentTitle))
    in SideEffect(DocumentTitle) (created by Routes)
    in Routes

To Reproduce
Steps to reproduce the behavior:

Go to 'Evaluation'
Click on 'Start Evaluation'

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Botfront Version:   0.26.0-rc.1
Rasa for Botfront Version: v1.10.10-bf.3

Hi @engahmed1190, thanks for the report. Could you scroll up to "the above error" mentioned? It doesn't look related to the one the prop warning you pasted for predictions in EntityReport.

Thanks for getting back @pheel

I get this from development mode

The problem mainly is:  report, accuracy , precision ,  f1_score ,  predictions and conditionalRowFormatter  are undefined

Oh, I just realized this is because we only show evaluations for entity extractions from DIETClassifier now. I think we should consider a tab for each extractor in the entity evaluation, but at the very least we need to handle this error and prevent this crash. What do you think, @znat ?


odin: dev-2024-07 -> dev-2024-08

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for odin is: Fast, concise, readable, pragmatic and open sourced programming language
meta.homepage for odin is: https://odin-lang.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found dev-2024-08 with grep in /nix/store/sj3dxp6siiqvgagxk1x3mvppxrsc2v7l-odin-dev-2024-08
found dev-2024-08 in filename of file in /nix/store/sj3dxp6siiqvgagxk1x3mvppxrsc2v7l-odin-dev-2024-08

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

odin
ols

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sj3dxp6siiqvgagxk1x3mvppxrsc2v7l-odin-dev-2024-08 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/d3d9af4522d8e94c32511945f2e86ba8f10e77e6#odin

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sj3dxp6siiqvgagxk1x3mvppxrsc2v7l-odin-dev-2024-08
ls -la /nix/store/sj3dxp6siiqvgagxk1x3mvppxrsc2v7l-odin-dev-2024-08/bin

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

1 package failed to build:

ols

1 package built:

odin

Maintainer pings
cc @astavie @luc65r @znaniye for testing.

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

1 package failed to build:

ols

1 package built:

odin


schema validator: add ability to fail on deprecated or WiP

Also add basic tests to the validator.
Finally, some cleanups in the core traversal code to prep for plumbing
through generic PGV validation.
Risk Level: Low
Testing: Added tests for the tool
Docs Changes: Added
Release Notes: Added
Platform Specific Features: N/A

Good enhancement to ensure all deprecated things clean up!

This new code is actually covered by the new schema validator tool tests, but for some reason it's not counting towards coverage. Will try to add some additional tests.

/retest

@htuch updated per comments

@htuch thanks this should be ready for another pass and includes all the changes bundled together.

@htuch this should be ready to go. Coverage is here: https://storage.googleapis.com/envoy-pr/2075ad3/coverage/source/common/protobuf/index.html.
I honestly don't know how to fix tidy. It seems like an issue in the tclap code we should probably be ignoring. cc @lizan

/lgtm deps

I'm going to merge this so I can use this in a deployment. I will follow up and try to figure out the clang-tidy issue.


using overlayfs?

Hey! I just started looking at pfs and noticed it is using btrfs. I was curious what features it uses and if it could possibly use overlayfs instead. Happy to discuss via email too.

Hi!! Just read a bit more about OverlayFS and it seems like it should be doable.
Here's how we use btrfs:

read only subvolume snapshots for commits, obvious analog in ofs
writeable subvolume snapshots for commits, The docs I read didn't explicitly says this but I assume ofs doesn't mind if 2 file systems share a lower fs given that they use it in a read only manner.
what changed - btrfs has a what changed command which tells you which files changed between snapshots. If I understand correctly, in ofs we could accomplish the same thing by just looking at the upper filesystem by itself.
send / receive are used for data transfer, this should work the same way as what changed
parent info, btrfs records where volumes came from which we use to record our commit history. This seems like the only thing that ofs doesn't have a direct analog for but it wouldn't be too hard to record it on the side.

Do you happen to know much about the performance characteristics of ofs? One thing I would worry about is that under normally usage we expect users to create commits with something like a per minute granularity. Depending on how ofs is implemented lookups on files that haven't changed in a long time could get very costly because it will need to traverse several layers to reach them.

I agree with your overview. The send/receive thing is something that is left to userspace so you will have to figure out some way to address the contents of lower filesystems (hash or something) to ensure you are talking about the same thing across RPC boundaries.
overlayfs should work fine with a commit workload of minute granularity although I haven't measured it myself. Essentially you would want to set some reasonable policy after measuring of when to flatten the commits again.
Of course the other option would be to just continue using btrfs even on systems that don't have a btrfs root filesystem by using a loopback block device.

We are now using FUSE in lieu of btrfs, which suits our needs for now


Upgrade packages and workspace settings

I managed to get the build working without having to use Tailwind in PostCSS 7 config mode. This allows us to upgrade Tailwind and use the new JIT mode.
We also had some custom stuff in the next.config.js that had dependencies on versions of PostCSS before version 8, so I had to adjust a few things to remove those, but now everything seems to work smoothly this way, without negatively impacting UX in any way.
With JIT mode, at least on my machine, the project runs and updates significantly faster in dev mode (because it just generates the classes you need, rather than locally generating all the classes, and then deleting the ones you don't need at build time). This also allows us to use the new [] syntax to break out of the Tailwind design system / theme when necessary (which didn't seem to be working with the PostCSS7 compatability build):
<div className="p-[-10.5px] bg-[#ff00ff] max-h-[650px]">

instead of extending the Tailwind theme or making a class, or resorting to inline-styles.
This PR also upgrades Next to version 11, which adds ESLint, so I also resolved a bunch of ESLint errors and added that to the project.
I also removed a couple old component files from old Explorer that aren't used in Beta, as well as added ./vscode for a custom VSCode theme and recommended extensions for the project.

merged in as #544


DHCPv4 Renew source MAC is wrong

Describe the bug
When we are testing DHCPv4 for IPoE sessions, it appears that the source MAC address in the ethernet layer 2 headers are wrong, while the client MAC address presented in the DHCP request packet is correct. It appears to be some kind of encoding issue since the last byte of the MAC address does appear in the layer 2 ethernet header, maybe just an oversight in the source code?
To Reproduce
Currently testing on dev build, however I suspect this also happens on stable since we've been testing for the past 3 weeks and run into the same problem where our sessions flap after the expire timer (because the source mac is wrong)
Version (bngblaster -v):
Version: DEV
Compiler: GNU (11.3.0)
IO Modes: packet_mmap_raw (default), packet_mmap, raw

JSON configuration:
{
        "interfaces": {
                "access": {
                        "interface": "ens21",
                        "type": "ipoe",
                        "outer-vlan-min": 1000,
                        "outer-vlan-max": 1002,
                        "inner-vlan-min": 1,
                        "inner-vlan-max": 4094,
                        "ipv4": true,
                        "ipv6": true,
                        "dhcp": true,
                        "dhcpv6": true
                }
        },
        "ipoe": {
        },
        "dhcp": {
                "enable": true,
                "timeout": 5,
                "retry": 30,
                "broadcast": true
        },
        "dhcpv6": {
                "enable": true,
                "timeout": 5,
                "retry": 30,
                "access-line": true,
                "rapid-commit": true,
                "ia-na": false,
                "ia-pd": true
        },
        "access-line": {
                "agent-remote-id": "bng-demo-{session-global}",
                "agent-circuit-id": "<IP_ADDRESS>/<IP_ADDRESS> eth0:{session-global}"
        }
}

Steps to reproduce the behavior:

Create an IPoE session
Packet Capture either via the bngblaster CLI or on the incoming interface of the router connected to BNGBlaster
Wait for the lease time to expire (DHCP ACK is not sent back by our BNG because the DHCP session is identified by the client MAC address internally)
Notice source MAC address of the request is different to the client-id as per screenshot:

Expected behavior
Layer 2 Ethernet Header Source MAC Address should be the same as the original source generated for the subscriber/session in the DHCP Discover process (Client MAC Address which is encoded correctly)
For example, if the client MAC address in the DHCP discover/renew is:
02:00:00:00:00:44
Then I expect the Layer 2 Ethernet MAC Source Address to also be 02:00:00:00:00:44
Screenshots
DHCP Renew

Additional context
Packet Capture will be sent to the email with the issue # in the subject , thanks

In the picture the last 4 bytes of destination + first two bytes of source are the actual source address, so maybe just encoding in Wireshark is wrong for ethernet over MPLS...
DST: 48:0C:02:00:00:00
SRC: 00:44:81:00:03:e8
The source address is even followed by 0x8100 (ether type for VLAN).

Could you please start BNG Blaster with -P test.pcap and compare packet send out by Blaster with your capture including MPLS.

You are correct, it appears that I had just decoded the single mpls header and not 2, hence why its shifting everything in the packet capture
That is fixed now, this leads me onto a different issue, would you like me to close this issue and open a new issue? We're seeing some very weird issues with the DHCP Renew messages when the Source IP in the ipv4 packet is set to <IP_ADDRESS>, all of our CPEs we are testing with set their source IP in the header itself (3 different vendors we are testing against our BNG) and this works on the CPEs but not on BNG blaster.
I'm not entirely sure if this is defined in the RFC but is there a reason why the source IP is set to <IP_ADDRESS> for the DHCP renew messages from BNG Blaster?

This is a bug, will fix it.

I'm seeing our BNG now process the DHCP requests and its renewing now, thanks so much for fixing this! Is there any chance I could request a feature we can enable in the config to allow this to be a unicast requests to the original server identifier sent during the DORA process? I'll be happy to open up a separate feature request if this is something you would consider

My fault, should be actually unicast:

The Blaster will already send unicast in renew if you set broadcast mode to false:

The Blaster will already send unicast if you set broadcast mode to false:

It appears to break for me when I set broadcast false for the DHCP discover to reach the BNG, I will be able to check later tonight if maybe it was a configuration issue on our end but when I set to false, the DHCP discovers break, I will look into the packet level and report back information on a new issue, thanks again for all your help

Okay, will wait for your feedback. Changes in this part of the code are not that complex, so can be done quickly.

I have done a few more minor changes in regards to be compliant with RFC2131. Now all messages will be send to broadcast IP except of renew and release (https://www.rfc-editor.org/rfc/rfc2131#section-4.3.6).

Fixed with 0.8.12. Please verify and close!


Enable optional visualization of complex query atoms in a more compact form

This PR adds a new useComplexQueryAtomSymbols boolean option to MolDrawOptions to enable a more compact visualization of certain complex atom queries supported by MDL molblocks and CXSMILES, e.g.:
from rdkit import Chem
from rdkit.Chem.Draw import IPythonConsole

mol = Chem.MolFromMolBlock("""
  MJ201100

  6  6  0  0  0  0  0  0  0  0999 V2000
   -1.7633    0.8919    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -2.4778    0.4794    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -2.4778   -0.3456    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.7633   -0.7580    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.0488   -0.3456    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.0488    0.4794    0.0000 A   0  0  0  0  0  0  0  0  0  0  0  0
  1  2  1  0  0  0  0
  2  3  1  0  0  0  0
  3  4  1  0  0  0  0
  4  5  1  0  0  0  0
  1  6  1  0  0  0  0
  5  6  1  0  0  0  0
M  END
""")

# current depiction (that I maintained as default)
mol

# with useComplexQueryAtomSymbols set to True
IPythonConsole.drawOptions.useComplexQueryAtomSymbols = True
mol

or
mol = Chem.MolFromMolBlock("""
  MJ201100

  7  7  0  0  0  0  0  0  0  0999 V2000
   -2.1205    2.4545    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -2.8349    2.0420    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -2.8349    1.2169    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -2.1205    0.8044    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.4060    1.2169    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -1.4060    2.0420    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
   -0.6916    0.8044    0.0000 X   0  0  0  0  0  0  0  0  0  0  0  0
  1  2  2  0  0  0  0
  2  3  1  0  0  0  0
  3  4  2  0  0  0  0
  6  1  1  0  0  0  0
  4  5  1  0  0  0  0
  5  6  2  0  0  0  0
  5  7  1  0  0  0  0
M  END
""")

# current depiction (that I maintained as default)
mol

# with useComplexQueryAtomSymbols set to True
IPythonConsole.drawOptions.useComplexQueryAtomSymbols = True
mol

@greglandrum Good point - done (and tested).

@DavidACosgrove do you think it makes sense to enable this by default?

Yes, makes sense.  I wonder how often Astatine is a hit?


[BUG]Virtual Machine termination doesn't remove the attached volume

Describe the bug
When the VM is deleted using the api, the VM is deleted but the attached volume is not deleted. When you manually try to delete that volume on UI, it fails with error -
admission webhook "validator.harvesterhci.io" denied the request: can not delete the volume 62a6b86fbc35454796a6dc6d808fb4b5-disk-0-0ihni which is currently owned by these VMs: 62a6b86fbc35454796a6dc6d808fb4b5
But In this case, the VM is already deleted
To Reproduce
Steps to reproduce the behavior:

Try to delete a VM using api.
Try to delete Volume attached to VM manually on UI

Expected behavior
When the VM is deleted using API the attached Volume should delete along with it
Manually on UI, you should be able to delete Volume if VM is already deleted
Support bundle

Environment:

Harvester ISO version: master
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):Harvester iPXE Boot Using Vagrant Libvirt

Additional context
Add any other context about the problem here.

@guangbochen can you take a look at this?

When you try to delete this via the UI it hangs and won't finish terminating.

The root cause of this issue is the same as the one in https://github.com/harvester/harvester/issues/970.
It should be fixed together.

duplicate of https://github.com/harvester/harvester/issues/970


Change disk default to 10GB

Thanks to @kherner for pointing this out.
We were using an outdated value for our default.  This was the document, which has since been updated:
https://cdcvs.fnal.gov/redmine/projects/fife/wiki/Information_about_job_submission_to_OSG_sites

Passes unit tests and a simple submission check of the .cmd file.

Closes #277

So I'm not sure if we shouldn't have default=None, to let the schedd/glideins pick the default(?).


ODB Connection error

Hi Riazraza,
Very useful project. Thank you.
I am not able to connect to my Jira instance when opening the template. I get an OLE DB or ODBCerror: [DataSource.Error] Web.Contents failed to get contents from 'https://fscedge.atlassian.net/rest/api/3/search?jql=project IN (JPPS)&maxResults=100&startAt=0&fields=*all' (400): Bad Request.
I've generated the api token.  Any thoughts?
Thanks!
Patrick

Have you configure in PowerBI how you connect to the Data or just generated the token and did nothing with it?
To setup the connection parameters, go to Home->Transform Data->Data source settings
Select the first only entry and Edit Permissions
Use as Password the API Token


Docker-from-Docker (Windows) Docker Compose EXTRA string=

The latest version of docker-from-docker breaks ${LOCAL_WORKSPACE_FOLDER} bind volumes.
devcontainer.json configuration:
"features": {
        "ghcr.io/devcontainers/features/docker-from-docker:1": {}
    },

docker-compose volume configuration:
volumes:
      - type: bind
        source: ${LOCAL_WORKSPACE_FOLDER}/<rest-of-path>
        target: /root/<rest-of-path>

docker-compose up results in
Resulting error:
Error response from daemon: /workspaces/<path-to-workspace>/c:\Users<Path-to-folder>%!(EXTRA string=is not a valid Windows path)
Without changing the docker-compose file and simply changing the docker-from-docker to:
"features": {
        "ghcr.io/devcontainers/features/docker-from-docker:1.0.8": {
            "moby": false,
            "version": "5:20.10.22~3-0~ubuntu-focal"
},

results in successful builds.
This may be a bug with the moby image. Please advise.

Hi 👋
I couldn't reproduce this error for a docker compose scenario on a Windows machine. 😓

The latest version of docker-from-docker breaks ${LOCAL_WORKSPACE_FOLDER} bind volumes.

The only change from 1.0.8 to 1.0.9 (ie 1 / latest) for docker-from-docker was a Feature rename to docker-outside-of-docker - https://github.com/devcontainers/features/pull/373
There wasn't any source code change so it's strange that it works with 1.0.8 but not with 1.
Not sure if there's a bug in dual publishing the artifacts, the SHA for docker-from-docker & docker-outside-of-docker is the same. So ideally it shouldn't. However, can you give this a try? 👇
"features": {
        "ghcr.io/devcontainers/features/docker-outside-of-docker:1": {},
}

This may be a bug with the moby image. Please advise.

Interesting. Does 👇  work?
"features": {
        "ghcr.io/devcontainers/features/docker-outside-of-docker:1": {
            "moby": false
        }
},

Also, can you share docker compose logs?

Closing as stale, feel free to reopen with details.


[DOC] Add doc for Authorization in REST Layer

What do you want to do?

[ ] Request a change to existing documentation
[x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
Description of the Feature
The Security plugin has recently introduced a new feature. This addition enhances the plugin's capabilities by enabling user authorization for requested REST routes. Let's delve into the details of how this feature functions.
NOTE: This feature sits on top of existing authorization check at Transport Layer. No changes were made at Transport Layer.
During the registration of a new route, the plugin requires a unique name to be assigned to each route. This ensures that no duplicate names are used during the registration process. Additionally, the plugin provides the option to register a set of "legacy" cluster permissions for each route. These permissions, such as cluster:admin/transport/, serve as the criteria against which the route is evaluated when a user attempts to access it.
Functionality of the Feature
The authorization process for this feature operates as follows: when a user initiates a REST request, their roles are examined, and each permission associated with the user is evaluated. The evaluation is conducted to determine if either of the following conditions is met: a match with the unique name assigned to the route or a match with any of the legacy actions defined during the route's registration.
For instance, let's consider a scenario where a user is assigned to role A. Here is an example configuration in YAML format:
abcplugin_read_access:
  reserved: true
  cluster_permissions:
    - 'cluster:admin/opensearch/abcplugin/route/get'
    - 'cluster:admin/opensearch/abcplugin/route/put'

Suppose the plugin has two registered routes under the abcplugin namespace as instance of NamedRoute:

/_plugins/_abcplugin/route/get registered with the unique name routeGet and associated with the legacy transport action cluster:admin/opensearch/abcplugin/route/get.
/_plugins/_abcplugin/route/delete registered with the unique name routeDelete and associated with the legacy transport action cluster:admin/opensearch/abcplugin/route/delete.

In this scenario, when the user makes a REST API call to /_plugins/_abcplugin/route/get, they will be granted authorization. The request flow will proceed as normal.
However, if the same user attempts a REST call to /_plugins/_abcplugin/route/delete, they will be denied authorization and receive an error message.
Goal of the documentation

It should help plugin developers point to the correct resources, for registering named routes
It should help end user define correct permission actions in the roles.

Related issue: https://github.com/opensearch-project/security/issues/2590

Getting this started now.

As per Dave, this has been moved to 2.10.

@cwillum This is not going to make it for 2.10, moving to 2.11 - sorry for the shuffle.
CC: @davidlago @hdhalter


Skip invalid BODYSTRUCTURE in server responses

Skip invalid BODYSTRUCTURE in server responses
Motivation:
A common parser failure is inside BODYSTRUCTURE server responses.
A server has to parse a MIME message to build up the RFC 3501 body data, and since there are a lot of badly formatted messages in the wild, servers can sometimes end up generating “bad” body data. The most common source is from junk messages, that are more-or-less intentionally ill-formated or even malicious. Another source are messages generated by scripts.
Currently, when the parser could fail on such messages, and a (client) library using the parser has no reasonable way to recover from such a failure — other than potentially skipping the entire line returned by the server (even though that is also non-trivial since there may be literals containing line breaks).
If the parser can still “succeed” overall, but note that just the BODYSTRUCTURE part failed, the client code using this parser would be able to implement some strategy to recover if it chooses to.
Modifications:
This adds a new wrapper type
extension MessageAttribute {
    public enum BodyStructure: Hashable {
        case valid(NIOIMAPCore.BodyStructure)
        case invalid
    }
}

which gets used instead of NIOIMAPCore.BodyStructure inside MessageAttribute.body.
Result:
The parser can now “skip” invalid / unparsable BODYSTRUCTURE data.
It is worth noting, that the MessageAttribute.BodyStructure.invalid case can be encoded into an IMAP stream, but it will (obviously?) not be valid BODYSTRUCTURE data, and it will not preserve the original parsed data. That is intentional: the code might otherwise re-encode malicious content.

Late to the party but LGTM


Width is calculated from the container element but height not

Hi @kirjs
Thank you for a great component :)
As the title implies i am having a problem with the chart height.
In highchart documentation it says that if width and height are not set in the config, then the chart takes the parent container size.
In the example here It works well on pure highchart
http://jsfiddle.net/o1pnxfnv/
But when i use react-highchart, the height seems fixed and does not stretch to the parent div height.
Example (can be edited thorugh gist if clicking on the button on the top-right)
http://output.jsbin.com/gobukah
Any idea?

@omerKenshoo

@mosheKenshoo I noticed this as well and was going to file a bug report when I got around to it. Another example of height not adjusting - https://jsfiddle.net/ktppqv0g/1/.
In the meantime, I used the following for a work around.
const parentElementHeight = <https://github.com/STRML/react-resizable>;
const config = {
  chart: {
    ...
    height: parentElementHeight
  }
  ...
};
<ReactHighstock config={config} />

@Jakehp thanks!

This has to do with the offsetHeight of a block element inside a block:

The offsetHeight of .highcharts-root is 0, so according to the documentation, chart.height gets automatically set to 400px.
Note that automatic resize on container resize is outside the scope of highcharts.

Feel free to reopen (also if anyone wants to send a docs PR, would be great, maybe just mention height and link here?

does somebody can give me a solution for this problem? can't solve it...

I added an event listener on page resize that updates a height local slice of state which is used by my ReactHighStock component to determine the height.
`    componentDidMount() {
window.addEventListener("resize", this.updateDimensions);
this.updateDimensions();
}
updateDimensions() {
    let height = this.getDims();
    this.setState({
        height: height
    });
}

getDims() {
    let navBarHeight = document.querySelector('.navbar-container').clientHeight;
    let height = window.innerHeight - navBarHeight;
    return(height);
}`

@STRML How to get the onResize event on GridItem as mentioned in  STRML/ResizableBox
 onResize = (event, {element, size}) => {
    this.setState({width: size.width, height: size.height});
  };

I am expecting this event in my code so, that I can re flow the highcharts in child component.
<ResponsiveReactGridLayout
                    breakpoints={{lg: 1200, md: 996, sm: 768, xs: 480, xxs: 0}}
                    cols={{lg: 18, md: 10, sm: 6, xs: 4, xxs: 2}}
                    rowHeight={20}
                    autoSize={false}
                    draggableCancel="input,textarea"
                    useCSSTransforms={true}
                    // onLayoutChange={this.onLayoutChange}
                    // onBreakpointChange={this.onBreakpointChange}
                    containerPadding={[5, 5]}
                    margin={[10, 10]}
                    {...this.props}
                >
                {_.map((this.dashboardStore.currentDashboard.layout), el => this.createElement(el))}
                </ResponsiveReactGridLayout>

In child component of  I use,

 this.tileContainerEl = tileContainerEl} >
                            {/*
{JSON.stringify(this.props.grid, null, 1)}
 */}
                            <ReactHighcharts
                                config={config}
                                callback={this.getChart}
                            />

componentDidMount() {
        var height = this.tileContainerEl.clientHeight;
        var width = this.tileContainerEl.clientWidth;
        this.chart.setSize(width, height);
        this.chart.reflow();
    }

Thank you.

Hey @abelkbil , How did you prevent your "ResponsiveReactGridLayout" to trigger "onLayoutChange" when user click on grid?

@nywooz Click will not trigger, I confuse your click might have made some changes to click either by css or any.

Anyone can give me a solution for this problem? can't solve it！

I finally find a solution with the help of react-sizeme library.

npm install react-sizeme
See: https://github.com/ctrlplusb/react-sizeme
Make a wrapper component ReactHighstockWrapper.

import cloneDeep from 'lodash/cloneDeep';
import React, { PureComponent } from 'react';
import { withSize } from 'react-sizeme';
import ReactHighstock from 'react-highcharts/ReactHighstock.src';

class ReactHighstockWrapper extends PureComponent {
  render() {
    const {
      size: { height },
      config,
      ...otherProps
    } = this.props;
    let newConfig = cloneDeep(config);
    newConfig.chart = {
      height,
    };

    return (

        <ReactHighstock config={newConfig} {...otherProps} />

    );
  }
}

ReactHighstockWrapper component will automatically extend to 100% height of its container's div.

@huangzhuolin very thanks

solution is
Config.chart = {       width:'200',       height:'200',,     };
changed hight


Failed To fetch

I log into rosetta (Fluency builder), enable the extension, Do an exercice, when i add the minutes it shows an error message with the "failed to fetch" , drari t3amlo qbel mayte7o ela had faille Deadline qerreb.

khouya wax l 9iti l 7el?

Is there anything more in the error ?


Time Travel Debugging missing or not working

Version
6.1.4
Browser and OS info
Chrome 100, Ubuntu 20.04
Steps to reproduce
You can skip steps 1-4 by just cloning this example project, running yarn install, then running yarn serve: https://github.com/amcsi/demonstrate-time-travel-not-working
In a Vue 2 app with Vuex:

go to the Vue DevTools
go to Timeline
go to Vuex Mutations
trigger Vuex mutations in your app if you haven't done so already
try to time-travel debug using the mutations list in DevTools

What is expected?
Either: when hovering over a row, it should offer you a button to time travel to that mutation; or on double-click it should time-travel to that point in time with mutations.
Here is a video of an older version of Devtools and how hovering over each row offers you a bunch of buttons related to time-travel debugging (Commit, Revert, Time-Travel): https://user-images.githubusercontent.com/49567934/136670141-9cf404ff-50eb-4e3a-a87f-e8ef30d88724.gif
What is actually happening?
It doesn't offer a time travel button, also, double-clicking on a mutation results in that row getting set as "selected", but that doesn't seem to do anything, and time travel is not happening.

🎥 Here is a link to a video demonstrating how Time Travel debugging isn't working using the example project mentioned above:
https://www.dropbox.com/s/b93a52n889qqt56/2022-04-13 21-05-15.mp4?dl=0

Duplicate of https://github.com/vuejs/devtools/discussions/1692


fix docs.rs build

see: https://docs.rs/crate/memory-rs/0.2.5/builds/547841

Oh this is really cool! didn't know you could do that, thanks!

Hey @CzBiX, thanks to your contributions we now have docs in docs.rs! thank you a lot.


(General) Save your last export settings

When you use ExportSheetData every day on various document, you end up applying always the same settings several time in a day. If the sidebar could memorize your last settings, it would be much easier to keep the same export formatting every time.
In fact, it works as long as you stay on the same document. But as soon as you open a new spreadsheet, your sidebar come back to the default settings.
Something store in your google drive settings ?

This is the next step for saving/loading export settings after the initial roll out of the feature in V39. Currently, the settings are saved and loaded per document, but there is no way to transfer those settings between sheets. Based on the way settings are stored, I'd prefer to always default to basic settings in new documents, while allowing users to import templates that they have set up. It would also be nice to allow a base template to be set up in place of the default settings.
I think I may be able to do this, so long as I can get the interface for templates to be displayed in an intuitive way. Both this and the automation workflow are likely going to have their own menus, distinct from the sidebar.

Took a look at initial implementation of systems like this while developing Nested Elements. Unfortunately, it seems like properties (add-on version of preferences) are locked to individual documents. Not an impossible problem, but I will have to think more on how to approach this. Best likely solution currently is a JSON file with setting preferences in it somewhere in your Drive folder, but that is far from ideal.

Any update on this issue? I hate having to configure the same settings over and over for each export. Always make a mistake and lose a 5 minutes of time on it.

Sorry for the extremely delayed response. I haven’t been able to come up with a great solution for this yet. I don’t particularly like having a text file that we read from at a hard set path, and as I noted before, add-on preferences are per document. I could probably allow users to export a blob of JSON settings that can be pasted into another document for setup, but that process would be a little clunky. I apologize that there hasn’t been much progress on this issue due to the lack of a clear path forward.

Importing/Exporting settings now has its own UI and should be implemented once v65 releases.

Importing/Exporting settings now has its own UI and should be implemented once v65 releases.


Update README.md Link is broken upload new one

README link to paper is broken
fix #555

I'm not sure this is the correct link either. If you scroll down to the bottom of the blogpost you linked in this PR it says "Read the paper ->" and that link goes to https://arxiv.org/abs/2304.02643
So I guess it's the arxiv preprint that is the link that should go here.

hello @GenevieveBuckley I made new pr is it good now


Add network-policies helm release

What this PR does / why we need it
Checklist

[ ] Update changelog in CHANGELOG.md.

Trigger e2e tests

/run cluster-test-suites

/run cluster-test-suites

Do we need these on all providers? (I guess yes.) In that case I think we can add them to the cluster chart right away.

@nprokopic I think this has already been done on CAPV and CAPVCD and besides CAPA here only CAPZ doesn't have this changed, but I'm not sure what the state there is and if it's ready for it as there were some cilium changes that needed to be done first.

@nprokopic I think this has already been done on CAPV and CAPVCD and besides CAPA here only CAPZ doesn't have this changed, but I'm not sure what the state there is and if it's ready for it as there were some cilium changes that needed to be done first.

Thanks for the info. This sounds then that we really need it for all providers, meaning that we should have this in the cluster chart. I mean, you can add it here now, but then we'll just probably move it to the cluster chart (probably in the following weeks) together with cilium and all other provider-independent HelmReleases that are currently in cluster-aws.
Provider-independent .global.* values for features like this (like global.connectivity.network.allowAllEgress here) should be something that makes sense and works for all providers (this here looks like that). And in general, if there are any differences across different providers, then the resources would be just more configurable via Helm values (this is probably not the case here for network policies, as this here looks like it would be identical for all providers).
P.S. I just wanted to comment here, so we avail double work if possible in any way.

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites

@nprokopic I agree this should end up in the cluster chart, but I would rather do it separately and not block this PR on it. I think we first need to spend the time to bring this change to CAPZ and then spend the effort to extract it into the cluster chart when all providers are ready.


Blazor 0.5.0 is heavy

Since the new update of blazor, the standard blazor project size changed from 2.1mb to 4.48 wich is the double size. the community wont use a ui framework that eats 4mb from the user bandwidth.

I agree that minimising the payload size is extremely important. We hope to put in a lot of optimisations to bring down the size.
The good news however is that it hasn't increased to 4MB :) You're looking at the uncompressed size there. The real amount of data transferred over the network is the figure on the bottom to the right, i.e., 1.83MB, which is almost exactly the same as it was before the 0.5.0 update.

Thank you @SteveSandersonMS


Mv to native Chef platform helpers, remove cpe_utils deps

This proposed change will do a few things:

Replace instances of node['platform'] == 'macos' || node['platform'] == 'mac_os_x' with macos? => I introduced the change to this longer form of detection for macOS platforms awhile ago (See: https://github.com/pinterest/it-cpe-cookbooks/pull/29) when there was a bug with an early release of Chef v16 (IIRC) that wasn't returning the expected platform value. This bug has now been fixed and the community can resume using macos? instead of this long-form incantation.
Replace all instances of node.windows?, node.macos? with windows?, macos? (respectively) => I recently noticed that Chef has some native helper functions available (See: below code block) for discerning platform family (i.e. macos?, windows?, debian?, linux?, et al) that make Facebook's helper node functions moot.
Removes all explicit dependencies on cpe_utils => Since the platform family helpers from Facebook are being removed and since those were the sole node functions pulled from cpe_utils, we can remove all cookbooks' dependence on cpe_utils. This is a joyous change as it makes all your cookbooks ever-so-slightly more "portable" (insofar that there is less reliance upon Facebook's open-source repository). This is especially joyous as cpe_utils is no longer offered in Facebook's repo so it makes it difficult (for newcomers to CPE teams using Chef) to grok how to use your (and our) open-source cookbooks.

$ sudo chef-shell
loading configuration: none (standalone session)
true
Session type: standalone
Loading.................done.

Welcome to the chef-shell 17.7.29
For usage see https://docs.chef.io/chef_shell/

run `help' for help, `exit' or ^D to quit.

chef (17.7.29)> require 'pry'
 => true
chef (17.7.29)> binding.pry

Frame number: 0/14
[1] pry(main)> show-method macos?

From: /opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-utils-17.7.29/lib/chef-utils/dsl/platform_family.rb:72:
Owner: ChefUtils::DSL::PlatformFamily
Visibility: public
Signature: macos?(node=?)
Number of lines: 3

def macos?(node = __getnode)
  node ? node["platform_family"] == "mac_os_x" : macos_ruby?
end
[2] pry(main)> show-method windows?

From: /opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-utils-17.7.29/lib/chef-utils/dsl/platform_family.rb:249:
Owner: ChefUtils::DSL::PlatformFamily
Visibility: public
Signature: windows?(node=?)
Number of lines: 13

def windows?(node = __getnode(true))
  # This is all somewhat complicated.  We prefer to get the node object so that chefspec can
  # stub the node object.  But we also have to deal with class-parsing time where there is
  # no node object, so we have to fall back to RUBY_PLATFORM based detection.  We cannot pull
  # the node object out of the Chef.run_context.node global object here (which is what the
  # false flag to __getnode is about) because some run-time code also cannot run under chefspec
  # on non-windows where the node is stubbed to windows.
  #
  # As a result of this the `windows?` helper and the `ChefUtils.windows?` helper do not behave
  # the same way in that the latter is not stubbable by chefspec.
  #
  node ? node["platform_family"] == "windows" : windows_ruby?
end
[3] pry(main)> exit
 => nil
chef (17.7.29)> windows?
 => false
chef (17.7.29)> macos?
 => true
chef (17.7.29)> debian?
 => false
chef (17.7.29)> linux?
 => false
chef (17.7.29)> exit

Thanks @ChefAustin!


Support multiple audio tracks / languages for southpark.de

Checklist

[x] I'm reporting a site feature request
[x] I've verified that I'm running youtube-dl version 2019.05.20
[x] I've searched the bugtracker for similar site feature requests including closed ones

Description

The Problem is: southpark.de is offering its video content with two audio languages. First off english, as it is the original language of the series. Then in german as this is the country at which southpark.de is addressed to. When using the website to play a video, the language can be switched using a combination of javascript buttons which causes the wesite to reload and switch to the selected audio track.
When using youtube-dl to download a video from southpark.de, there is no option (at least none that i could find) to select which audio tracks are downloaded - downloading with youtube-dl yields a video with just the local language audio track (in this case german).
This could be fixed by finding a way to download the english audio track.
My proposed solution would look this way: When downloading a video from southpark.de that is being offered in multiple languages, youtube-dl will output a video file with all available languages as seperate audio tracks embedded into it.
Ideally, this should be done in a way so that it applies to all southpark sites for all countries, but this will sadly be impossible to test properly without proxies or similar since any southpark site always redirects to it's local site due to georestriction.

Post concrete example URLs.

Sorry. I concluded that this wasn't nessesary since the topic concerns the whole site along with every video as far as I know.
Example URL:
http://www.southpark.de/alle-episoden/s22e08-buddha-box#source=e3748950-6c2a-4201-8e45-89e255c06df1:896b2aa3-ac17-11e8-b956-70df2f866ace&position=8&sort=!airdate

Sorry, i don't follow.
As far as i can tell, your commit causes the extractor for SouthparkDE to check what language is set in the cookies, after which the according language is downloaded. But how do i change this when i simply run youtube-dl from command line? Is there some switch that i missed or do i have to use a cookie file?
And: Does youtube-dl currently have any infrastructure to support multiple audio tracks in one video?

Is your code only able to get the cookie if downloading an url under http://www.southpark.de/alle-episoden/? Would be a good time to mention that URLs like the following should also be covered:
http://www.southpark.de/clips/152307/garrisons-hohle
http://www.southpark.de/clips/sz9szy/its-a-miracle
http://www.southpark.de/clips/sz9szy
http://www.southpark.de/collections/9996/all-about-stan
http://www.southpark.de/collections/9996/all-about-stan/1
http://www.southpark.de/collections/1256/season-7-creator-commentary/1
http://www.southpark.de/collections/7988/super-heldenschurken/1
http://www.southpark.de/collections/7988
And also the commentaries which are only offered with english audio:
http://www.southpark.de/collections/1256/season-7-creator-commentary/1
Also this change seems pretty simple, as all necessary extractor work is already implemented and this is only a question of specifying the correct language. Would it be safe to implement this for all  southpark extractors of all country divisions since they're all using the same backend?

N.B. on the requirement of example URLs: Only now i start to grasp the workflow of a youtube-dl developer and the role of example URLs and why they always should be provided.
 While it is perfectly comprehensible to expect that contribution guidelines will just be followed - you're the ones doing the work here - it might be good to give some understanding to the issue requester. In my case a clause like "Remember we're all very short on time here. Give your example URLs so that even if a contributor will exclusively use the provided URLs to try out a new implementation they represent every possible URL sheme that can be found on the site." would have made clear what to post & why.
 While it should be clear to just follow the contribution guidelines i was kind of left alone on deciding which URLs would qualify as a useful example. My reasoning was: since it's an issue concerning the enire website, all URLs do qualify so why put up examples and especially which? I'm sorry that that's been all that came to my mind that moment. Now i definitely know better.
This reminds me of #18303 where the user Siddhant suggested to change an error message for users who don't get it.
 While reading this i also thought at first "What's not clear in the error message?". But after reading Siddanth's comment i also thought it might be better to just change the message and make a clear point even for users who don't get it.
Maybe i'm the only one who didn't get it with the example URLs but if this happens often i hope changing the relevant paragraph will be considered.

In my case a clause like "Remember we're all very short on time here. Give your example URLs so that even if a contributor will exclusively use the provided URLs to try out a new implementation they represent every possible URL sheme that can be found on the site." would have made clear what to post & why.

In my opinion, this is not correct. Every properly implemented extractor has a host of test cases representing known "good" cases. These tests can and should be run, in order to avoid regressions. Unless you're trying to improve a known restricted extractor, you shouldn't be forced to provide previously good cases.
The problem is: I have stumbled across this myself, when trying to improve extractors. (Improving them myself meaning: Fix or amend the python code, test with my particular use case, and try to create a PR (pull request).) My impression was (don't take this as full fact): Some testcases are outdated, some actually don't work anymore, some require a login (but AFAICT the test infrastructure doesn't allow for providing login credentials, while the CLI does), some of the checked details have changed but have never been updated... So before fixing something, I need to go through the existing tests first. Oh, and I haven't found a way to say "run all tests", they apparently need to be specified with "_1", "_2", "_3". This makes it very tedious, and the benefit of the test cases becomes slightly questionable.
Sorry for this rant. youtube-dl is an excellent tool, and I'm impressed by its quality and how much progress it makes with limited developer time. These issues of mine belong in a separate ticket. I'm just trying to say: You probably needn't have worried about providing existing URLs.


chore(deps): update dependency stylelint to v13.2.0

This PR contains the following updates:

Package
Type
Update
Change

stylelint (source)
devDependencies
minor
13.1.0 -> 13.2.0

Release Notes

stylelint/stylelint
v13.2.0
Compare Source

Security: updated to postcss-selector-parser@6 due to a vulnerability in one of postcss-selector-parser@3 dependencies (#4595). Due to this update:

selector-descendant-combinator-no-non-space will ignore selectors containing comments
selector-pseudo-class-parentheses-space-inside can't autofix pseudo-classes that contain comments

Added: --stdin CLI flag that accepts stdin input even if it is empty (#4594).
Fixed: autofix will ignore sources containing disable comments or nested tagged template literals - this is workaround to make autofix safer to use until we can resolve the underlying issues (#4573).

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 3.0.44 :tada:
The release is available on:

v3.0.44
GitHub release

Your semantic-release bot :package::rocket:


ImportError when trying to use 'Linear8bitLt' from 'lit_llama.quantization'

I'm encountering an error while trying to use bitsandbytes in lit-llama. Here's the error message:
Loading model ...
bin /home/mostafa/anaconda3/envs/ll/lib/python3.11/site-packages/bitsandbytes/libbitsandbytes_cuda117.so
Traceback (most recent call last):
File "/home/mostafa/programming/lit-llama/generate.py", line 170, in <module>
CLI(main)
File "/home/mostafa/anaconda3/envs/ll/lib/python3.11/site-packages/jsonargparse/_cli.py", line 85, in CLI
return _run_component(component, cfg_init)
File "/home/mostafa/anaconda3/envs/ll/lib/python3.11/site-packages/jsonargparse/_cli.py", line 147, in _run_component
return component(**cfg)
File "/home/mostafa/programming/lit-llama/generate.py", line 129, in main
with fabric.init_module(empty_init=True), quantization(mode=quantize):
File "/home/mostafa/anaconda3/envs/ll/lib/python3.11/contextlib.py", line 137, in enter
return next(self.gen)
File "/home/mostafa/programming/lit-llama/lit_llama/utils.py", line 142, in quantization
from .quantization import Linear8bitLt
ImportError: cannot import name 'Linear8bitLt' from 'lit_llama.quantization' (/home/mostafa/programming/lit-llama/lit_llama/quantization.py)

I receive this error when I run the following command:
python generate.py --quantize llm.int8 --prompt "Hello, my name is"

I'm using Python 3.11, and the error seems to be with importing 'Linear8bitLt' from 'lit_llama.quantization'. Could you please help me with this issue?
Thank you.

It might be caused by https://github.com/TimDettmers/bitsandbytes/issues/544: pip install scipy should fix it

It works now. Thank you so much for your great help.

It might be caused by TimDettmers/bitsandbytes#544: pip install scipy should fix it in that case

It works now. Thank you so much for your great help.


Remove @ts-ignore comments

Updates types
Removes @ts-ignore comments

Thank you this is great!

Np!
By the way, adding an eslint configuration file to this repo might make sense. I tried running eslint just now, but it threw an error saying it needed a configuration file

Oh nice, looks like #3 will end up taking care of it

Yeah realized it was getting ignored in .gitignore. #3 adds it to git and adds eslint to the Github actions as well


Init container failing due to Jsonnet import failure

Describe the bug
The init container bombs out during the "Start Kubernetes Job" section due to being unable to import /tmp/job-env-<id>
To Reproduce
steps:
  - key: k8s-build
    label: "Kubernetes Build"
    command:
    - "echo \"Hello World\""
    plugins:
    - greymatter-io/k8s:
        image: alpine:latest
        secret-name: <secret-name>
        agent-token-secret-key: <buildkite-agent-token-key>
        mount-secret: <some mounting secrets>
        git-ssh-secret-name: <secret-name>
        git-ssh-secret-key: <secret-key>

Screenshots
Starting Kubernetes Job | 0s
-- | --
  | RUNTIME ERROR: couldn't open import "/tmp/job-env-01865403-5c84-4531-b02f-061cd146d4362409616407": no match locally or in the Jsonnet library paths.
  | tla:stepEnvFile:1:1-73	thunk <stepEnvFile>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:57:26-37	thunk <str>
  | std.jsonnet:102:17-20
  | std.jsonnet:102:8-21	thunk
  | std.jsonnet:102:8-33	function <anonymous>
  | std.jsonnet:102:8-33	function <anonymous>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:57:16-44	thunk
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:(51:5)-(59:6)	function <anonymous>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:(51:5)-(59:6)	thunk <stepEnv>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:62:5-12	thunk <podEnv>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:314:18-24	object <anonymous>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:(310:11)-(343:26)	thunk <array_element>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:(309:21)-(344:10)	object <anonymous>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:(302:13)-(350:8)	object <anonymous>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:(295:15)-(351:6)	object <anonymous>
  | /buildkite/plugins/github-com-greymatter-io-k8s-buildkite-plugin/lib/job.jsonnet:(290:9)-(352:4)	object <anonymous>
  | During manifestation

Additional context

This is using out fork which is even with this repository.
This is our first time using init images.

@clbx This should be resolved with #63 I believe!

Latest main is verified working; so this should be fixed now.

This is fixed


Multiple Script Tags

Describe the bug
Every time the owner of the shop update the widget customization, a new pair of script tags are added to the shop.
To Reproduce
Steps to reproduce the behavior:

Go to 'Digital Climate Strike Shopify App' in your shop dashboard
Update which ever field you want
Submit form.
Go to the shop
Open browser inspect
Look into head tag. You will see multiple script tags.

Expected behavior
The shop should only have two script tags: one that contains the shop customized params and the actual digital climate strike script.
Screenshots

Desktop (please complete the following information):

macOS High Sierra
Chrome
Version 76.0.3809.100 (Official Build) (64-bit)

We can update the script vars tag. For that we need to save the script tag id return by Shopify. This is because ActiveStorage will create a diff blob id every time we update the script file in amazon.


Added 2 tutorials on Pandas. One on Series and one on DataFrames

Summary
I have added two articles/tutorials, one on the premises and functions involving Pandas Series and the other on the premises and functions involving Pandas DataFrames. I discuss the various ways you can create them, access their data, manipulate them and combine them. They go into good detail and have plenty of explanation.
Basic example
Overview of Series in Pandas
A Pandas Series is a one-dimensional array-like object that can hold any data type. It is very similar to a column in a spreadsheet or a SQL table. Each element in a Series is assigned a label whcih is called an index. The index can be numeric or string-based.
In this tutorial, you will learn how to create, manipulate, and analyze Series using Pandas
Creating Series
In order to create a Series all you need is the data to pass in and the index is assigned automatically or you can pass in an index with the data.
import pandas as pd

# Create a new Series
my_Series = pd.Series([1, 3, 5, 6])

# Creating  Series with a string index to a normal array
my_Series = pd.Series([8, 6, 1, 4], index=['a', 'b', 'c', 'd'])

# Creating  Series with an integer index to a normal array
my_Series = pd.Series([-3, 2, 9, 1], index=[3,6,4,9])

Motivation
Why are we doing this? What use cases does it support? What is the expected outcome?
The end goal is to provide a good document to allow people to learn about the Pandas module. When I set out to learn Pandas I found I had to use lots of documents and recourses to do so.  By combining all the key aspects into two documents I have made it a lot more easy to follow and learn.
Checks

[x] Read Create a Pull Request
[x] Supports all screen sizes (if relevant)
[x] Supports both light and dark mode (if relevant)
[x] Passes npm run test

Any chance of this being reviewed?


Save rendered templates in memory rather than serving from disk (or both?).

Right now, html files are served from disk every time. Keep rendered templates in memory to reduce disk load and increase serving speed.

Reopening issue because this should (hypothetically) be feasible


[tricore] float-point in DVADJ operand

@mumbel, if you dont mind, i will continue describe bugs in tricore sleigh here.
Describe the bug
According to p-Code operation manual, abs() function, that used in dvadj constructor, dealing with float-point operands, but any values, used in dvinit / dvstep / dvadj should be integer.
To Reproduce
Look at decompiled function, that uses dvadj
Expected behavior
I didnt fully understand, how dvXXX works, but according to tricore architecture manual, it should be redone as follows:
:dvadj Ree2831/Reo2831,Ree2427/Reo2427,Rd1215 is PCPMode=0 & Rd1215 & op0007=0x6b & op0811=0x0 ; Ree2427 & Reo2427 & Ree2831 & Reo2831 & op1623=0xd0
{
	#TODO  divide sequence
	local quotient:4 = Ree2427;		# E[d]  0..31
	local remainder:4 = Reo2427;	# E[d] 32..63
	local divisor:4 = Rd1215;		# D[b]

	local x_sign = remainder[31,1];
 	local q_sign = quotient[31,1];

	if (q_sign) goto <no_inc_quot>;
		quotient = quotient + 1;
	<no_inc_quot>
	if (((remainder == divisor) || (remainder == - divisor)) && x_sign) goto <nonzero_remainder>;
		remainder = 0;
	<nonzero_remainder>
	Reo2831 = remainder;
	Ree2831 = quotient;
}

It looks like the issue, across my whole implementation, is the usage of abs().
Hopefully this isn't in the wrong direction, but I have put a PR in https://github.com/NationalSecurityAgency/ghidra/pull/1286

Thanks again @esaulenka   looks merged in master


elements links have no effect

I try to override link elements and it doesn't work:
elements: {
    link: '{{link}}',
}

But using messageLink work:
elements: {
    messagelink: '{{message}} {{link}}'
}

I'm wrong, I read again the doc and understand the concept of layouts...


support enable_shielded_nodes

Hi,
I'd like to be able to enable Shielded Nodes feature
https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/container_cluster#enable_shielded_nodes
and have its default set to true as it is a best practice in the sec-hardening guide. I'll submit a PR later on.

Closing due to repo sunset


refactor money column

Why
https://github.com/Shopify/shopify/issues/121143
Money column was missing some key features namely

proper reload fixes #91
legacy support for saving 1.23 strings and 1.23 floats in addition to money objects
proper handling of invalid currencies from the DB
changing the currency invalidates the money column cache
make the currency column explicit (remove default name)

What
Tests and moved the code in functions for better readability

This also looks to change the currency_column from defaulting to currency if not specified

@sunblaze that's correct, reducing the amount of magic and making things explicit

@sunblaze please let me know if there's anything else, thx


Fix to 'repo-svn push' command

Trying to push an existing podspec with an increased version number
fails to update the repository.
Issue fails with:
svn: warning: W150002:
'/Users/user/.cocoapods/repos/PrivateRepo/MyProject’ is already under
version control
The solution is to add the —force command to the ‘svn add’ operation,
and the commit happens successfully.

Hey @briansbledsoe ! It's Eric, the former intern. I guess I must have introduced that bug when I implemented the repo push command, heh.

Hey @amorde ! We're definitely appreciative and using everything you wrote! Just ironing out the kinks :) I hope things are going well.

Hey guys! I'm driving back from the beach ATM, but I'll get this merged in and released tonight. :smile:

Thanks much for the fix @briansbledsoe! :cake: :ice_cream: :+1:

Sure! Thank you for the quick merge! :)


[release-0.23] Update the manifests and dependencies

This PR updates the dependencies and the manifests for the release-0.23.

/lgtm

/test pull-knative-operator-eventing-upgrade-tests