[tool_crash] FileSystemException: Cannot open file, OS Error: Operation did not complete successfully because the file contains a virus or potentially unwanted software., errno = 225

Command
flutter create --platforms windows .

Steps to Reproduce

...
...
...

Logs
FileSystemException: Cannot open file, OS Error: Operation did not complete successfully because the file contains a virus or potentially unwanted software.
, errno = 225
#0      _File.throwIfError (dart:io/file_impl.dart:675:7)
#1      _File.openSync (dart:io/file_impl.dart:490:5)
#2      ForwardingFile.openSync (package:file/src/forwarding/forwarding_file.dart:74:16)
#3      ErrorHandlingFile.copySync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:312:22)
#4      _runSync (package:flutter_tools/src/base/error_handling_io.dart:590:14)
#5      ErrorHandlingFile.copySync (package:flutter_tools/src/base/error_handling_io.dart:311:5)
#6      Template.render.<anonymous closure> (package:flutter_tools/src/template.dart:379:18)
#7      _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:633:13)
#8      Template.render (package:flutter_tools/src/template.dart:281:24)
#9      CreateBase.renderMerged (package:flutter_tools/src/commands/create_base.dart:483:21)
<asynchronous suspension>
#10     CreateBase.generateApp (package:flutter_tools/src/commands/create_base.dart:506:23)
<asynchronous suspension>
#11     CreateCommand.runCommand (package:flutter_tools/src/commands/create.dart:351:31)
<asynchronous suspension>
#12     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1350:27)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#14     CommandRunner.runCommand (package:args/command_runner.dart:212:13)
<asynchronous suspension>
#15     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:348:9)
<asynchronous suspension>
#16     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.16.0, on Microsoft Windows [Version 10.0.19045.3570], locale en-US)
    • Flutter version 3.16.0 on channel stable at C:\Users\pc\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (3 months ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\pc\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop Windows apps (Visual Studio Professional 2019 16.9.0)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional
    • Visual Studio Professional 2019 version 16.9.31025.194
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2023.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)

[✓] VS Code (version 1.86.2)
    • VS Code at C:\Users\pc\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.82.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.3570]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 121.0.6167.185
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 120.0.2210.91

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

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: db7ef5bf9f59442b0e200a90587e8fa5e0c6336a
Plugins
cloud_firestore-4.13.1
cloud_firestore_web-3.8.5
file_picker-6.1.1
file_selector_linux-0.9.2+1
file_selector_macos-0.9.3+3
file_selector_windows-0.9.3+1
firebase_auth-4.15.2
firebase_auth_web-5.8.11
firebase_core-2.24.2
firebase_core_web-2.10.0
firebase_storage-11.5.5
firebase_storage_web-3.6.17
flutter_barcode_scanner-603ea693b43575dafa4c6672e6232f5a99e261ef
flutter_local_notifications-15.1.3
flutter_plugin_android_lifecycle-2.0.17
google_sign_in-6.1.5
google_sign_in_android-6.1.20
google_sign_in_ios-5.6.4
google_sign_in_web-0.12.1
image_picker-1.0.4
image_picker_android-0.8.8
image_picker_for_web-3.0.1
image_picker_ios-0.8.8+2
image_picker_linux-0.2.1+1
image_picker_macos-0.2.1+1
image_picker_windows-0.2.1+1
onesignal_flutter-5.1.0
path_provider-2.0.14
path_provider_android-2.0.25
path_provider_foundation-2.2.2
path_provider_linux-2.1.11
path_provider_windows-2.1.7
shared_preferences-2.2.2
shared_preferences_android-2.2.1
shared_preferences_foundation-2.3.4
shared_preferences_linux-2.3.2
shared_preferences_web-2.2.1
shared_preferences_windows-2.3.2
sign_in_with_apple-4.3.0
sign_in_with_apple_web-1.0.1
sqflite-2.2.6
url_launcher-6.1.10
url_launcher_android-6.0.27
url_launcher_ios-6.1.4
url_launcher_linux-3.0.6
url_launcher_macos-3.0.7
url_launcher_web-2.1.0
url_launcher_windows-3.0.8
video_player-2.8.1
video_player_android-2.4.10
video_player_avfoundation-2.5.1
video_player_web-2.1.2

Closing this as a duplicate of https://github.com/flutter/flutter/issues/61997


Add "pause" menu

Barely solves #138
Closes #586
Closes #585

Fixened

Setting this as draft again since it's waaaay more work than I assumed 🙃


feat(phisherman): add url validation

in the past 24 hours phisherman has had some invalid url requests from us
2022-02-13 04:19:14    "request_url":"https://api.phisherman.gg/v2/domains/check/16.5k"
2022-02-13 04:19:07    "request_url":"https://api.phisherman.gg/v2/domains/check/3.5k"
2022-02-13 04:19:07    "request_url":"https://api.phisherman.gg/v2/domains/check/2.5m"
2022-02-13 04:19:07    "request_url":"https://api.phisherman.gg/v2/domains/check/4.5m"
2022-02-13 04:18:50    "request_url":"https://api.phisherman.gg/v2/domains/check/18.5M"
2022-02-13 04:18:40    "request_url":"https://api.phisherman.gg/v2/domains/check/1.5m"
2022-02-13 04:18:27    "request_url":"https://api.phisherman.gg/v2/domains/check/1.2m"
2022-02-13 04:18:19    "request_url":"https://api.phisherman.gg/v2/domains/check/53.8m"

so i thought it would be better to validate urls before we fetch them

Honestly, I'm very against this, you should validate URLs yourself using Result.from(() => new URL(url)).ok, I believe it's the developer's job to properly validate the URLs.

Honestly, I'm very against this, you should validate URLs yourself using Result.from(() => new URL(url)).ok, I believe it's the developer's job to properly validate the URLs.

But still it's a hefty stock of invalid requests from the developer and just how we validate the api key, validating the domain would be a bit safer for the requests


Correct the capitalization of Xcode in README

This pull request corrects the capitalization of Xcode :sweat_smile:
https://developer.apple.com/xcode/
Created with xcode-readme.

:laughing: :+1:

Thank you.

:smile:


Cart transform - multiple merge operations with same parent variant

Describe the bug
We have a problem when multiple numbers of the same bundle exist in the cart.
After navigating to checkout, the price update modal is displayed and child components are added back to the cart.
Example function output:
{ "operations": [ { "merge": { "cartLines": [ { "cartLineId": "gid://shopify/CartLine/9dff30c8-b084-4e53-ab6c-38b1d0ee5d0a", "quantity": 1 }, { "cartLineId": "gid://shopify/CartLine/96dbc3d2-73b6-4544-bf27-3c434f74d43f", "quantity": 1 } ], "parentVariantId": "gid://shopify/ProductVariant/47599165669668", "price": { "percentageDecrease": { "value": 25 } }, "image": null, "title": "Bundle 34" } }, { "merge": { "cartLines": [ { "cartLineId": "gid://shopify/CartLine/9dff30c8-b084-4e53-ab6c-38b1d0ee5d0a", "quantity": 1 }, { "cartLineId": "gid://shopify/CartLine/96dbc3d2-73b6-4544-bf27-3c434f74d43f", "quantity": 1 } ], "parentVariantId": "gid://shopify/ProductVariant/47599165669668", "price": { "percentageDecrease": { "value": 25 } }, "image": null, "title": "Bundle 34" } } ] }
The problem also exists when we assign different titles to operations.
Example function output:
{ "operations": [ { "merge": { "cartLines": [ { "cartLineId": "gid://shopify/CartLine/075d4d14-4bf2-4389-b4e4-f7886cd7bb2e", "quantity": 1 }, { "cartLineId": "gid://shopify/CartLine/2a56590c-acd7-4595-adfe-a80b11994646", "quantity": 1 } ], "parentVariantId": "gid://shopify/ProductVariant/47599165669668", "price": { "percentageDecrease": { "value": 25 } }, "image": null, "title": "1" } }, { "merge": { "cartLines": [ { "cartLineId": "gid://shopify/CartLine/075d4d14-4bf2-4389-b4e4-f7886cd7bb2e", "quantity": 1 }, { "cartLineId": "gid://shopify/CartLine/2a56590c-acd7-4595-adfe-a80b11994646", "quantity": 1 } ], "parentVariantId": "gid://shopify/ProductVariant/47599165669668", "price": { "percentageDecrease": { "value": 25 } }, "image": null, "title": "2" } }, { "merge": { "cartLines": [ { "cartLineId": "gid://shopify/CartLine/075d4d14-4bf2-4389-b4e4-f7886cd7bb2e", "quantity": 1 }, { "cartLineId": "gid://shopify/CartLine/2a56590c-acd7-4595-adfe-a80b11994646", "quantity": 1 } ], "parentVariantId": "gid://shopify/ProductVariant/47599165669668", "price": { "percentageDecrease": { "value": 25 } }, "image": null, "title": "3" } }, { "merge": { "cartLines": [ { "cartLineId": "gid://shopify/CartLine/075d4d14-4bf2-4389-b4e4-f7886cd7bb2e", "quantity": 1 }, { "cartLineId": "gid://shopify/CartLine/2a56590c-acd7-4595-adfe-a80b11994646", "quantity": 1 } ], "parentVariantId": "gid://shopify/ProductVariant/47599165669668", "price": { "percentageDecrease": { "value": 25 } }, "image": null, "title": "4" } } ] }
Expected behavior
We needed to add multiple bundles to the cart that would work properly at checkout.
Additional context
We are only using merge functionality and have not implemented expand operations.
Everything works fine if there is only one bundle on the cart.

hey facing the same issues, were u able to find some resolution?

As it's clearly mentioned here in Docs, only one merge operation will be applied and the reset with the same cartline ids will be discarded.

As it's clearly mentioned here in Docs, only one merge operation will be applied and the reset with the same cartline ids will be discarded.

As it's clearly mentioned here in Docs, only one merge operation will be applied and the reset with the same cartline ids will be discarded.
So I think this issue is closed!

For example, the following code from the documentation
{   "operations": [     "expand": {       "cartLineId": "gid://shopify/CartLine/1",       "title": "Awesome TV with Warranty",       "image": null,       "expandedCartItems": [         {           "merchandiseId": "gid://shopify/ProductVariant/1",           "quantity": 1,           "price": {             "adjustment": {               "fixedPricePerUnit": {                 "amount": "1000.00"               }             }           }         },         {           "merchandiseId": "gid://shopify/ProductVariant/2",           "quantity": 1,           "price": {             "adjustment": {               "fixedPricePerUnit": {                 "amount": "150.00"               }             }           }         }       }     }   ] }
In the above example, if a customer adds three items to the cart, "gid://shopify/ProductVariant/1" and "gid://shopify/ProductVariant/2", only one "Awesome TV with Warranty" is added to the cart. and "gid://shopify/ProductVariant/1" and "gid://shopify/ProductVariant/2" are purchased separately, two at a time.
In such a case, the correct behavior would seem to be to purchase three "Awesome TV with Warranty", but this is not possible with the current "Cart transform" API specification.
Is there any way to do this here?


Added storm summary functions

Produces a dataframe of typical stats for a collection of storms.

Coverage decreased (-0.01%) to 88.25% when pulling 797b2e6408e66876949445e662104eaf3322c357 on phobson:storm-summary-fxn into d7b916a9976e56f5f043c975037399c95e073d69 on Geosyntec:master.

Coverage increased (+0.02%) to 88.28% when pulling 797b2e6408e66876949445e662104eaf3322c357 on phobson:storm-summary-fxn into d7b916a9976e56f5f043c975037399c95e073d69 on Geosyntec:master.

Coverage decreased (-0.01%) to 88.25% when pulling 5a966c00c727151cb6dcef41e37dcc4e542f640c on phobson:storm-summary-fxn into d7b916a9976e56f5f043c975037399c95e073d69 on Geosyntec:master.


Send Linux distro name and version when resolving ArtifactTool to download

Part of fix for #217
Basically we need to send the distro name up to blobstore service, so that it can send down the alpine-x64 variant of ArtifactTool rather than the linux-x64 variant.
Tested on Alpine, makes request:
vsts.vss_client : GET https://vsblob.dev.azure.com/mseng/_apis/clienttools/ArtifactTool/release?distroVersion=3.8.1&distroName=alpine&arch=x86_64&osName=Linux

Thanks, I'll target that branch as well. For my future knowledge, is dev effectively dead or just temporarily forked? Also, I didn't internally see vsts-cli listed as a governed repository, so please let me know if you'll need anything to move the OSS approval forward.

Dev is effectively dead - mostly yes. We are not planning on releasing another version unless really necessary.
OSS approval will follow up on email with you.


OpenModsLib crashing Forge Server

Hello! i'm having a crash when a start a server with the OpenModsLib.
Crash: http://pastebin.com/n46YBgVG
Versions:
OpenModsLib: 0.6-snapshot-295.jar and 0.6-snapshot-300.jar
Forge Version: <IP_ADDRESS>3
On client is working, the crash is only on server.
Thanks for the atention :)

I'm unable to reproduce and I have almost no idea what triggers this bug. There was some room to improvement, but I can't confirm fix, Please reopen if persist with recent snapshot.

Hello! Tks for the fast repply.
Updated to snapshot-301 and still crashing, but another error: http://pastebin.com/ESzxv4kZ
Tks again :)

That's basically same as #27 and #28 - you are keeping something weird in mods folder or even modifying minecraft.jar.

Yes yes, i tried withou the other mods and the serer starts ok. I will add one by one to see if it's a incopatibilty and i'll post here again. Tks again :)


config: update 'resinOS 2.x' to 'balenaOS 2.x'

Change-type: patch
Signed-off-by: Matthew McGinn<EMAIL_ADDRESS>

BTW doing a quick search shows there's also a couple of other pages which still say "resinOS" which should probably be properly parameterized so that they say "balenaOS" instead?
https://github.com/balena-io/docs/blob/master/shared/getting-started/getDeviceOnDash/bananapi-m1-plus.md
https://github.com/balena-io/docs/blob/master/pages/reference/OS/advanced.md

BTW doing a quick search shows there's also a couple of other pages which still say "resinOS" which should probably be properly parameterized so that they say "balenaOS" instead?
https://github.com/balena-io/docs/blob/master/shared/getting-started/getDeviceOnDash/bananapi-m1-plus.md
https://github.com/balena-io/docs/blob/master/pages/reference/OS/advanced.md

Good idea to update those as well! I have updated this PR, thanks @lurch!


Add Java security CVE dependency bump CVE info [#180651488] [#180637081]

Signed-off-by: Jeremy Alvis<EMAIL_ADDRESS>

I merged and pushed to prod but I couldn't close out the Tracker story because I don't have write access to that Tracker.
https://docs.pivotal.io/healthwatch/2-1/release-notes.html#2-1-8


[advice] add check of PREFIX for termux-widget

Now in android, when $SHELL exists, it will use linux config, such as ~/.config/XXX, else it will use android config.
However, for termux-widget, it will not set $SHELL.
Such as create a /data/data/com.termux/files/home/.shortcuts/test
echo $SHELL

return nothing.
Why not add a check not only for $SHELL but also for $PREFIX? the latter is all programs running in termux will set. Thanks.
https://github.com/termux/termux-widget

PR welcome.


dd can not print object string

Laravel Version: 5.8.27
PHP Version: 7.1
Database Driver & Version: mysql 5.7

Description:
can not print object message using dd in command
Steps To Reproduce:
vendor/symfony/var-dumper/Dumper/CliDumper.php:488
vendor/symfony/var-dumper/Dumper/CliDumper.php:494
comment this lines perform ok

It's not laravel issue dd is actually using VarDumper so you need to create issue in VarDumper repo

Please create an issue on the symfony repo.


Dynamic nuxt module supporting in static mode.

Is your feature request related to a problem? Please describe.
I want to develop a portal app with dynamic menus to integrate other modules. Currently, all sub modules are added in nuxt.config and built to activated, which causes the whole to build and deploy when any module changes.
It will be great for nuxt to support dynamic loading modules.
Describe the solution you'd like
A register API like this.$nuxt.registerModule(module). Partial functions like router, pages, layouts, stores, and middlewares are loaded and registered. I can load this module through async components or third-party CDN network.
Describe alternatives you've considered
I tried requirejs. It load components I want, but lack the context of nuxt. I also dont know how to dynamic register middlewares and layouts.
Additional context
Our applications built on yarn workspace. But package independently.

Modules in Nuxt 3 are build-time only, so I don't think we can support this within Nuxt itself. But as for registering features, I think this should be very possible for any individual module to enable. So you would enable the module as normally but pass a configuration option to it, and it would inject a minimal function that you could call to lazy load further features.


GitPython and Cloning Down 7k+ repositories

Environment

Nautobot version (Docker tag too if applicable): 1.6.4
Python version: python 3.8
Database platform, version: postgres
Middleware(s):

With this many repositories, and Nautobot not needing the historical information on the repository, it would make sense for Nautobot to shallow clone the repository.  Alternatively perhaps there is a caching option so that this process is mitigated.
Known issues in GitPython to workaround:

https://github.com/gitpython-developers/GitPython/issues/718
I’m thinking getting Nautobot Core current thoughts on the shallow clone mitigation, and also if any ongoing caching fix is appropriate/preferred/alternative means on addressing.  Perhaps also potentially deleting the local copy after pushing upstream.

I understand this example references Golden Config, but the problem is inherently in Nautobot Core per conversations with Glenn.

Steps to Reproduce

Have 7k+ sites with 7k+ git repositories
Perform a GC compliance job

Expected Behavior
Updates to git repos

Observed Behavior
Perform a clone down of all repositories for an app such as Golden Config to perform updates
See 50GB in memory required
Host OOMs and kills the process

For Golden Config, we should approach it this way. https://github.com/nautobot/nautobot-app-golden-config/issues/683
For Nautobot Core in general, I am not sure what the best path forward is.

As far as I understand the original issue, the current biggest problem is https://github.com/gitpython-developers/GitPython/issues/718, which may be mitigated by ensuring that we explicitly delete any git.Repo objects when done with them, most likely either by explicitly calling Repo.__del__() at the end of our ensure_git_repository() utility function, or by calling Repo.__del__() when a nautobot.core.utils.git.GitRepo instance is deleted.
Shallow clone might be a nice feature to mitigate disk space requirements but to the best of my knowledge the memory leak is the current problem to solve.

Thus far in some manual testing I'm unable to trigger the GitPython memory leak. Even after re-cloning a repository 1000 times, or cloning 100 separate repos, there are no extraneous open file handles and no apparent increase in memory consumption. This leads me to believe that my previous understanding was perhaps incorrect, and perhaps the "OOM" referenced in the issue was due to disk space exhaustion rather than memory exhaustion -- @mattmiller87 can you provide any additional clarity?


typo correction

Issue no #32
Corrected typo error in "how it works"  component.
@rishipurwar1

Issue no #32
Corrected typo error in "how it works" component.
@rishipurwar1

can you please update your repo code and then make pr again?
I have made some changes.

Issue no #32
Corrected typo error in "how it works" component.
@rishipurwar1

can you please update your repo code and rerun npm i and then make pr again?
I have made some changes in the main branch.
this will resolve conflict issues.
step-1 git fetch origin
step-2 git rebase origin/main
and then check your changes working properly.

@rishipurwar1
I did it and it say nothing to commit in git status .

Issue no #32
Corrected typo error in "how it works" component.
@rishipurwar1

can you please update your repo code and rerun npm i and then make pr again?
I have made some changes in the main branch.
this will resolve conflict issues.
step-1 git fetch origin
step-2 git rebase origin/main
and then check your changes working properly.

@rishipurwar1
I did it and it say nothing to commit in git status .

but here it's showing two files are conflicting.
Check these files and see where they are conflicting?


Export makes a new query

Hi,
After I've got the SQL result, I want to export it with "Export Resultset..."
But instead of exporting the already received data, it initiates a new SQL query run.
This cause a time waste because I need to wait for the same query to run second time.
Can you please make it export simply the already retrieved data?
Thanks.

This is already implemented. Enable option "Selected rows only" in export wizard:

Thanks a lot! super cool!


applied glass effect on navbar

Related Issue
none
Fixes:  # 838
Description
I've applied glass effect on the navbar as it was creating visibility issue of the names of the tabs written on navbar.
issue# 838 is fixed in this PR.
Type of PR
UI is enhanced.

[X] Bug fix
[X] Feature enhancement

Screenshots / videos (if applicable)
Take a look on the navbar
Before:

After:

Checklist:

[X] I have made this change from my own.
[X] I have performed a self-review of my own code.
[X] My changes generate no new warnings.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

@PriyaGhosal Kindly review my PR and let me know if there are any changes to be made.

Kindly also add labels of hacktoberfest, hacktoberfest-accepted and gssoc-ext.

@Kiran-F Please see the conflicts . Resolve the conflicts . Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Remove usage of ES ConfigMap and PublicCertSecret

These resources are no longer used now that Linseed has taken over this functionality
This has a companion commit in Calico Enterprise here
Description

For PR author

[x] Tests for change.
- [ ] If changing pkg/apis/, run make gen-files
- [ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

@Josh-Tigera a few conflicts to resolve, though


Support "kubernetes.io" metrics prefix

Previously, GKE metrics had the prefix "container.googleapis.com," but with recent changes, the new metrics are now prefixed with "kubernetes.io."
So, it seems necessary to support the "kubernetes.io" prefix.
This change will allow the exporter to correctly handle and process the new metrics from GKE clusters.

Oh, wait.. After examining the Stackdriver code, I initially thought that the metrics with the "kubernetes.io" prefix were not being collected.

However, upon testing, I have confirmed that the "kubernetes.io" metrics are indeed being collected.

I will go ahead and close the issue.


Epinio: support of k8s 1.26

This issue is to implement the chart changes & perform the testing required to upgrade to k8s 1.26. Please perform below steps.

[ ] Check whether the chart has any references to the below deprecated features of k8s 1.26. You can find information about deprecated APIs here on Kubernetes Deprecation guide.

flowcontrol.apiserver.k8s.io/v1beta1 API version of FlowSchema

flowcontrol.apiserver.k8s.io/v1beta1 API version of PriorityLevelConfiguration

autoscaling/v2beta2 API version of HorizontalPodAutoscaler

[ ] Update kube-version annotation in Chart.yaml. catalog.cattle.io/kube-version: '>= 1.23.0-0 < 1.27.0-0'
Note:  If this is an upstream chart and if the upstream chart doesn't support k8s 1.26 then do not update the kube-version annotation

[ ] Perform below tests

Fresh install/uninstall/upgrade/downgrade of the charts on K8s 1.26 provisioned on Rancher. Test the chart functionality

Provision rancher on k8s 1.25 or lesser

Install chart on this cluster

Upgrade this cluster in rancher to k8s 1.26

Make sure the existing chart and the app is functional after cluster upgrade

Please note that the k8s 1.26 upgrade is targeted for the 2023-Q2-v2.7x. Please complete this task by 05/19/2023.

The PR was made and validated, see report in the PR

@andreas-kupries Are missing something to close this issue?

AFAIK no.

Closing, feel free to reopen if we missed something


Overlay network + custom IP

How can I define the custom IP for containers in compose-file? I use overlay network with network_mode: my-net
So I cannot specify the IP via networks, because I get the following error:
ERROR: 'network_mode' and 'networks' cannot be combined

Can you open this on https://github.com/docker/compose ?
Thanks!


Reloading the video player with a fresh ad after an event trigger is not working

Issue
We are facing an issue with Reloading the video player with a fresh ad after an event trigger. In our case an event trigger can be a set timer for auto-refresh or click of a specific button on the UI.
Overview

We are using videojs ima plugin to display preroll ads. We have a custom play button and on click of this button we display the video player and it runs the ads; on the “ads end” event we are hiding the player. Everything is working perfectly till here in the first load of the video player. Problem starts when we want to Re-trigger and reload the player with a fresh ad on the same page without refreshing the entire page.
Here is the link of test page: https://ascendeum.com/videoAd/test/index.html

Hi,
Can you confirm whether or not the ended event to which your player registered is being fired. This can be the reason why the ad playback doesn't occur when the content playback is completed.
The part below:
player.on('ended', function(){
      player.load();
      player.currentTime(0);
      player.ima.initializeAdDisplayContainer();
      player.ima.setContentWithAdTag(null, adTagUrl1, true);
      // player.ima.changeAdTag(adTagUrl1);
      // player.adsManager.start();
      player.ima.requestAds();
    });

Hi,
I’m closing this issue because it has been inactive for more than a month. Please reopen if you still encounter this issue with the latest stable version
Thank you!


Support long-term and short-term memory in Conversation chain

The current Memory objects allow a user to store the entire conversation history, a subset of history, and a summary of history.
Users might want to store a summary of the conversation in addition to a fixed-size history.
You could imagine the following algorithm:
1. Check if current conversation history has reached maximum size
2. If so, generate a conversation summary.
3. Optionally, merge the prior summary with the new summary.
4. Trim the conversation history
5. Append new exchange to conversation history

If the user wants to maintain a fixed-size conversation history as well as a summary, a simpler approach might be to create a more general MemoryBank container. There would need to be validation that the memory key is unique.
class MemoryBank(Memory, BaseModel):
    resources: list[Memory]

    def load_memory_variables(self, inputs: Dict[str, Any]) -> Dict[str, str]:
        """Return history buffer."""
        return {r.memory_key: r.buffer for r in self.resources}

     ...

i love both the specific memory algorithm you desribed as well as the general MemoryBank abstraction! i think both are worth doing. probably two separate PRs/issues. any idea which one you want to do first?


App crash when trying to create expense

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs Reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @mountiny
Slack conversation (hyperlinked to channel name): expensify_bugs
Action Performed:

Launch the app
Tap the Green + and select Create expense

Expected Result:
Create expense screen displayed with options to scan or manual or distance and user able to select manual and continue
Actual Result:
App crashes
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[x] iOS: Standalone
[x] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/b432d2b0-d29b-4dd8-8d7f-1858792e35bb
https://github.com/user-attachments/assets/e93305a4-078c-4992-9658-b45f39142193
logs-2024-12-15 15_25_14.073.txt
App_Info_9.0.76-6 2.json
logs-2024-12-15 15_24_16.466.txt
App_Info_9.0.76-6.json
View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Aaah i believe this i because of using the profiling debugging tool - i believe there is an issue for this already- i havent realized before

@mountiny do you know where the other issue is?

Here, I even created it 🙈 https://github.com/Expensify/App/issues/53061

We cn close this, the app was only crashing when profiling so its that issue


Hot Exit in Workspaces

Azure Data Studio Version: 1.41.1
OS Version: W10

Steps to Reproduce:

Open a workspace
Create e temp file
Close azure data studio

Does this issue occur when all extensions are disabled?: Yes

When i'm in a workspace i'm not enable anymore to use temp files , every file need to be saved or i'll lost it.
In settings of workspace there isn't the possibility to to set the files.hotExit.

@mad4hit I'm missing something in the repo.  When I open a workspace, create a temp file, close ADS, then reopen ADS the temp file is restored and I'm not asked to save it during this process.  Am I missing some detail in the repo?

Hi @kburtram, the problem occurs if you open a second workspace then i create a temp file and i try to close the second one, or if you have an open workspace and try to open a second one in substitution.
I attach a video, i hope it self-explain.
https://user-images.githubusercontent.com/5448105/226494795-e0025363-621f-4d1e-aa3a-6cb3756cd50f.mp4


[NODE] global singleton object, set_body_typed IRPrinter default

Three misc improvements:

Enable global_key field for serialization, this is used for things like nnvm::Op where we don't really want to serialize the object content, but simply a global_key(op->name) and then recover by calling Op::Get
set_body_typed in TVM_REGISTER_API to make it easy to use TypedPackedFunc signature.
IRPrinter now have a good default if print is not registered for the specific node type

@jroesch please review


[BEAM-1542] Make AbstractSpannerFn public

Make AbstractSpannerFn public to support custom user defined functions that use Cloud Spanner.
R: @jkff

Coverage decreased (-0.003%) to 69.971% when pulling 65a3b669eb1168fcc7a526ed6ac9d98958bc52d9 on mairbek:spannerfnvisibility into aadbe36ff66b69e7eb55fb4977cb59b93b3269e2 on apache:master.

Can you elaborate what kind of functions you expect users to write based on this? Is an abstract DoFn base class the best API to enable those use cases?

So, I'm building a pipeline that exports all the database records. For this I've build a function that reads system tables and builds the schema.
I don't want to rewrite the spanner client lifecycle management code.

It should be possible to do this via composition rather than inheritance: you can create a helper class SpannerAccessor with a constructor from SpannerConfig and getters getSpanner(), getDatabaseClient() and a close() method. Then your new DoFn will have 1-line Setup and Teardown methods which I think is acceptable. In return, you'll get:

The ability to unit-test SpannerAccessor without a pipeline or a DoFnTester
The ability to have multiple SpannerAccessor's in a single DoFn (say, you want to do something like DynamicDestinations but for Spanner)
The ability to use it in the context of something that is not a DoFn
The ability to use a different base class for a DoFn that wants to use Spanner

I've updated the code, please take a look.

Coverage decreased (-0.02%) to 69.955% when pulling fa194b82f9d371fc15446a2996c229e53827dcfa on mairbek:spannerfnvisibility into aadbe36ff66b69e7eb55fb4977cb59b93b3269e2 on apache:master.

retest this please

Coverage decreased (-0.01%) to 69.965% when pulling fa194b82f9d371fc15446a2996c229e53827dcfa on mairbek:spannerfnvisibility into aadbe36ff66b69e7eb55fb4977cb59b93b3269e2 on apache:master.


Improve Update Container

This PR improve the following:

Add setting to keep Terminal Window opened after a successful Container Update
Add setting for the max number of Container Update retries
Add a new containerStatus function to clean the code
Hide Terminal Window when the Container Update is done
Only retry the Container Update if number of retries < max value from settings
Change version to 0.1.6

This PR also fixes https://github.com/LedgerHQ/ledger-vscode-extension/issues/2

@cedelavergne-ledger as this PR does not change any host command executed, we should be fine with not testing on all supported platforms (no need to test on windows or mac, it should still work)


Update backup-azure-vms-automation.md

The value being passed for -KeyEncryptionKeyVaultId must begin with a “/” or IaaS VM creation will fail with ErrorCode: KeyVaultAccessForbidden.

Hi @ejarvi, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Eric Jarvi). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@ejarvi : Thanks for your contribution to the Azure documentation! The author, @markgalioto, has been notified to review your proposed change.

@ejarvi , thank you for finding this mistake, we'll make this update.

#sign-off


npm doc is not up to date

According to npm doc I should pass an url but it does not work as I must pass a videoId.
Sounds stupid but I lost 20 minutes

That link goes the npm page for a package called lapanoid-react-youtube…which seems to be an exact copy of an older version of this module. (?)
This module is published to npm at https://npmjs.com/package/react-youtube, and the docs are up to date there :)

The strangest thing is that the github repo link comes here ...

Not sure what happened there...
It seems like a bit of bad manners to upload someone else's module to NPM :confused:
perhaps this is the same person: @lapanoid ?
If so would you be willing to remove your package from NPM (or at least removing this repository from the repository field in package.json so people do not confuse the packages)

I don't think unpublish it is the right solution as people whom relies on it will experience some troubles.
Plus I think it is not possible to unpublish a package older than 24 hours, there have been a big issue about this a time ago.
See here for npm unpublish policy

There are no dependents (npm packages that depend on his package) so he is free to remove it by contacting NPM support.
As far as I can tell it is a direct copy of our package, if it isn't and there is a reason for it to stay on NPM (say people are using it as a modified version of our package) then he should (at the very least) not link it to this repository.
As it is right now I would dare say it is dangerous for that package to stay up on NPM because people will think they are installing the code from this repository but actually they could be installing anything (which is what caused this whole thing about un-publishing in the first place).


After the script is deployed, the network card cannot get the IPV6 global address

After the script is deployed, the network card cannot get the IPV6 global address. Therefore, the IPv6 address cannot be bound in the openvpn configuration. Does this support IPv6 network? It works normally under IPv4.
Test at armbian7 of debian core

use script remove openvpn, the network card can get the IPV6 global address.

There is a language barrier here and we can't understand what the issue which you are trying to report is.

install script , the network adapter (eth0) cannot obtain the IPv6 global address. After uninstall script, the network adapter can obtain the IPv6 address normally.


provide shebang: /bin/sh for fish compatibility, closes #118

obviously things are not as easy as I originally expected, any chance for help on failing fish travis-ci job?

@typekpb thanks for investigating this. It seems that fish is pretty straightforward about their support of the POSIX standard (or lack thereof).
Personal opinions aside, this looks to be a non-trivial amount of work to convert shpec to use fish-supported constructs; there's also the possibility that we do some things in shpec that would need to be rewritten entirely.
Given that this is the first request for fish compatibility, I'm inclined to believe that there is not a strong need for it, but I am always happy to be corrected by the community at large on that assertion. I'm more than happy to accept a PR that provides fish compatibility, of course.
@typekpb it does look like fish gives you useful feedback in regards to what it doesn't like; between that and the test suite(s) you should have some decent guard rails for getting this working if you so choose. I'm more than happy to answer more pointed questions if you do get stuck on a particular problem, too. Feel free to ask on this PR, or on gittr, or wherever you can find me ;)

@typekpb are there any updates here? Does this PR work for you locally, or do the test failures provide an accurate picture of using shpec with fish?

@rylnd no progress on my side. never worked for me locally. I was originally expecting this one to be a low hanging fruit, which obviously is not :) closing


Proof of concept: serialize config directly to json

This is a sibling PR to: https://github.com/ansible-community/ansible-nomad/pull/62 please see the PR on nomad for further details and discussion.

Failing tests, lack of activity.


CI | Change Deprecating set-output

Explain the changes

Change deprecating set-output (instead we use GITHUB_OUTPUT), more information here.

Issues: Fixed #xxx / Gap #xxx

none

Testing Instructions:

none

[ ] Doc added/updated
[ ] Tests added

I could fully test this PR only after it was merged.
After this PR was merged I manually run the workflow Manual Build Dispatch (here). I inserted the inputs in master as the branch and alpha as a tag (in the GitHub Action UI).
checks:

date
 echo "date=$(date +'%Y%m%d')" >> $GITHUB_OUTPUT
VERSION="${{ steps.date.outputs.date }}"
output: VERSION="20230501"
suffix
echo suffix="-${{ github.event.inputs.tag }}" >> $GITHUB_OUTPUT

BASE_TAGS="${DOCKER_BASE_IMAGE}:${{ github.event.inputs.branch }}-${VERSION}${{ steps.suffix.outputs.suffix }}"
BUILDER_TAGS="${DOCKER_BUILDER_IMAGE}:${{ github.event.inputs.branch }}-${VERSION}${{ steps.suffix.outputs.suffix }}"
CORE_TAGS="${DOCKER_CORE_IMAGE}:${{ github.event.inputs.branch }}-${VERSION}${{ steps.suffix.outputs.suffix }}"

outputs:
 BASE_TAGS="${DOCKER_BASE_IMAGE}:master-${VERSION}-alpha"
 BUILDER_TAGS="${DOCKER_BUILDER_IMAGE}:master-${VERSION}-alpha"
 CORE_TAGS="${DOCKER_CORE_IMAGE}:master-${VERSION}-alpha"

I also checked that we have the newly pushed images in Docker Hub.


vmi and pool controller: Use patchset for patch operations

What this PR does
Use the patch package to replace the hardcoded patch operations. This improves the readability and make the code more cohesive.
Before this PR:
Hardcoded operation in vmi controller and pool controller.
After this PR:
Replaced all the patch operations with the patch.PatchSet in the vmi and pool controller
Fixes #
Partially address: https://github.com/kubevirt/kubevirt/issues/11654
Release note

NONE

/approve
thanks!


Checkout page displays 'Internal Server Error'

I am running the Saleor React Storefront with Saleor Cloud as my backend.  My storefront is slightly modified with theme changes, but the logic has not changed.
The storefront is running here.
When I try to checkout, I get the following error on the console when attempting to access the checkout page:

I can access the checkout customization page without issues.
/.env
SALEOR_API_URL=https://mysneakers.eu.saleor.cloud/graphql/
STOREFRONT_URL=http://<IP_ADDRESS>/
CHECKOUT_APP_URL=http://<IP_ADDRESS>:3001/
CHECKOUT_STOREFRONT_URL=http://<IP_ADDRESS>:3001/checkout-spa/
CLOUD_DEPLOYMENT_URL=https://prod.demo.saleor.cloud

SENTRY_DSN=
SENTRY_ENVIRONMENT=
SENTRY_RELEASE=
VERCEL_URL="http://<IP_ADDRESS>/"

ENABLE_EXPERIMENTAL_COREPACK=1

/apps/storefront/.env
 NEXT_PUBLIC_API_URI=$SALEOR_API_URL
NEXT_PUBLIC_VERCEL_URL=$STOREFRONT_URL
NEXT_PUBLIC_CHECKOUT_URL=$CHECKOUT_STOREFRONT_URL
NEXT_PUBLIC_CHECKOUT_APP_URL=$CHECKOUT_APP_URL

NEXT_PUBLIC_STOREFRONT_NAME="My Sneakers"
NEXT_PUBLIC_DEFAULT_CHANNEL=default-channel
NEXT_PUBLIC_HOMEPAGE_MENU=navbar
NEXT_PUBLIC_DEMO_MODE=false
NEXT_PUBLIC_GEOLOCATION=false
NEXT_PUBLIC_ALLOWED_IMAGE_DOMAINS=img.youtube.com,i.ytimg.com

/apps/saleor-app-checkout/.env
# Default values for tests
# They will work only when replaying requests with Polly.js
# To record new requests, use real credentials in .env.local
TEST_MOLLIE_KEY=mollie_test_key
TEST_MOLLIE_PROFILE_ID=mollie_profile_id

TEST_ADYEN_MERCHANT_ACCOUNT=Saleor
TEST_ADYEN_CLIENT_KEY=adyen_test_client_key
TEST_ADYEN_API_KEY=adyen_test_api_key
TEST_ADYEN_HMAC=adyen_hmac
TEST_ADYEN_WEBHOOK_PASSWORD=adyen_webhook_password
TEST_ADYEN_WEBHOOK_USERNAME=adyen_webhook_username

TEST_STRIPE_SECRET_KEY=sk_test_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
TEST_STRIPE_PUBLISHABLE_KEY=pk_test_yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy
TEST_STRIPE_WEBHOOK_SECRET=whsec_zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz

NEXT_PUBLIC_SENTRY_DSN=${SENTRY_DSN}
NEXT_PUBLIC_SENTRY_ENVIRONMENT=${SENTRY_ENVIRONMENT}
NEXT_PUBLIC_GIT_BRANCH=${VERCEL_GIT_COMMIT_REF}
SENTRY_RELEASE=${VERCEL_GIT_COMMIT_SHA}
NEXT_PUBLIC_SENTRY_RELEASE=${VERCEL_GIT_COMMIT_SHA}

APL=file
SETTINGS_ENCRYPTION_SECRET=beabravedude

Looks like your 3001 port is refusing connection for some reason, maybe try restarting, and one small look at your logs showing me an extra / this could be problem also in some cases when running in production.
Thanks

Looks like the issue was that the .env was not updating on the live server.  A simple build and run solved the issue.

I am getting same issue. Can you please help ?
saleor-app-checkout:start:
storefront:start: Failed to proxy http://localhost:3001/checkout-spa/?checkout=Q2hlY2tvdXQ6MzUyN2ZlM2ItMGUwOS00YWEzLWFmZGYtMzk2ZmU4MDk5YzQw&locale=en-US&channel=default-channel&saleorApiUrl=http%3A%2F%2Flocalhost%3A8000%2Fgraphql%2F&domain=localhost Error: connect ECONNREFUSED ::1:3001
storefront:start:     at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1494:16) {
storefront:start:   errno: -4078,
storefront:start:   code: 'ECONNREFUSED',
storefront:start:   syscall: 'connect',
storefront:start:   address: '::1',
storefront:start:   port: 3001
storefront:start: }

Can you please help in finding cause of this? Will be really helpful

Please open an issue with a minimal reproduction if you believe there's a problem with the project or a discussion if you need help.
You're also welcome to join our Discord: discord.com/invite/H52JTZAtSH


ValueTriple has "measurements" should be "measurement" to match corim draft

The reference-triple-record has 1 environment-map and 1 measurement-map. The ValueTriple's "measurements" collection does not follow that document structure, so I don't know how this produces a valid comid.
It would certainly seem to me that 1 environment could have multiple measurements of different names and value kinds the way this code allows for, but the draft doesn't allow for that. The mkey I thought was a way for multiplicity to be allowed, but also no.

Looks like the code matches rev00 of the draft, and the triples structure was never brought into alignment with the later versions. We need to fix this.

Noting that this change would impact how PSA and CCA reference values are currently encoded.

Noting that this change would impact how PSA and CCA reference values are currently encoded.

Yup, this will have ripple effect on a lot of existing code. Fixing this is not going to be trivial...

Yes, this was a known issue however, we had not scoped it fully so had put on hold for a while.
Has impact on PSA, CCA and also Parsec-CCA Scheme under Veraison

@deeglaze this should be resolved by https://github.com/veraison/corim/pull/124


VM Probers send to wrong address

https://github.com/istio/istio/blob/889db80ef8b3889a2e5c6ee0d4f4214ddd92f822/pkg/istio-agent/health/health_check.go#L74
This should be to the pod IP, matching our rewrite prober

mimic https://github.com/istio/istio/pull/31121


Can multiple types of reducers be declared in the same zkApp?

Or another way to ask is it possible to define multiple different types of sequence events in the same zkApp? (This should require a zkApp to have multiple sequence states?)
It should look like this:
reducer = Experimental.Reducer({ actionType: Note });
reducer2 = Experimental.Reducer({ actionType: Field});

I know that we can use a composite CircuitValue to combine these types as a sequence event, but since snarkyjs does not support plain if statements, what I understand is that this can only handle relatively simple single data or relatively simple logic. For a sequence event with different types of data, you also need to use dummy data (Not all processing can use dummy data, and calculations that do not conform to the Circuit.if judgment also need to be side-effect-free). For example, it would be difficult for a zkApp that holds two or more different merkle roots to use the same sequence event to perform the update calculation of these roots.

no, not possible. to implement multiple types of actions, you can create an actionType that represents one of many actions at the same time. for example:
class MyCompositeAction extends CircuitValue {
  @prop note: Note;
  @prop fields: Field;
  @prop isNote: Bool
}

class MyContract extends SmartContract {
  reducer = Experimental.Reducer({ actionType: MyCompositeAction });
}

you then could have one reduce operation which ignores actions that have isNote = true and one which ignores ones with isNote = false

Ok, I get it, it's just that sometimes it's really tricky to implement ignore 🤦🏻

maybe this helps:
(state, action) => {
  let { note, field, isNote } = action;
  let stateForNewNote = reduceNote(state, note);
  let stateForNewField = reduceField(state, fields);
  return Circuit.if(isNote, stateForNewNote, stateForNewField);
}

Thanks. In fact, the most troublesome thing is the handling of dummy data.


ncc internal typescript build bug workaround

This provides a workaround for https://github.com/zeit/ncc/issues/435 by using the previous TypeScript build.
Hopefully we can get a proper bug fix in soon as well to avoid this workaround.

Seems like the tests didn't like it

Codecov Report

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

@@           Coverage Diff           @@
##           master     #445   +/-   ##
=======================================
  Coverage   74.68%   74.68%
=======================================
  Files          13       13
  Lines         395      395
=======================================
  Hits          295      295
  Misses        100      100

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 f640557...88be8d0. Read the comment docs.

Looks good 👍


5 ➡ 6

➡ Forward port
Port ign-gazebo5 to main
Branch comparison: https://github.com/ignitionrobotics/ign-gazebo/compare/main...ign-gazebo5
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)
🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

Closing in favor of #940


[OSD-10257] update operator-sdk and dependencies version

update the dependency version
operator-sdk 0.18.2
controller-runtime 0.10.0
kube-* 0.22.1

We haven't normally included the vendor dir for this project, was its inclusion as part of this PR an accident?

/hold
Adding a hold till the query on vendoring deps is addressed

Tested with some self created SubjectPermission, did not see errors in the operator log, and the corresponding rolebindings created in the target namespace.

Tested it locally as well and all seems fine. Adding lgtm for now to let it pass through e2e tests over weekend to capture issues if any.
/lgtm
/approve

/unhold


libs/utils/android/screen: Add screenrecord utility

Implementation isn't the prettiest thing ever. I hit the same problem as @qperret in that I have to send signals to the process that is spawned on the device, and not the ADB command that resides on the host.
I put the code in the Screen class, but with how that class is used we don't have any context, only static classes, so the way to use screenrecord is:
data = record_start(target, ...)
<dostuff>
record_stop(target, data)

I find that absolutely barbaric, but I also didn't want to put it in the Workload class as it wouldn't be available outside of android workloads. If you guys think it is fine to move it in the Workload class, that'll get us rid of that ugly data passing.

@qperret I didn't want to create another class as, after all, this is a screen-related utility. But I think you're right, a non-static class in screen.py is probably the cleanest solution.

Maybe a Recorder class in Screen ? You could the use it like:
rec = Screen.Recorder(target)
rec.start()
# do stuff
rec.stop()

I agree putting members to the Screen class is very bad but using it to encapsulate a class like that is OK I think.

@bjackman || @derkling, as LISA senseis what do you think ?

Moved the code to Screen.Recorder to get rid of that ugly context passing

Added wait() method. As @qperret suggested, stop() always stops the recording whereas wait() waits for max_time to be elapsed.
If no max_time is defined, a warning is fired and we'll wait for the end of times, or for the device's disk to get full, whichever comes first. Not the greatest thing, but like I said, I don't know what to do of that wait() in this case.

We can probably do 2 things about that:

use a default max_time with a large value. Not sure what large means though (1 hour ?)
if we really don't have a max_time because it was explicitly set to None, stop() then raise an exception with a message such as: "max_time must be set to use wait(). Recorder has stopped()"

Thinking about if, just the second option is probably the better.

Quick Q - any reason why we can't put this in devlib as in Instrument? Then we get it for WA3 too...

Hmmm very good point. I think it makes sense. And actually it could even make sense to move all the content of screen.py into devlib. What do you think @valschneider ?

Yeah about that I had a chat with @derkling - thing is, there's quite a lot of code (like 90%) we have in system.py and screen.py that could make its way over to Devlib, but I don't really know how much of it all we can move over to WA nor how much we should move over.

Just from skimming the code this bit doesn't seem to have any functional dependencies so we could always start by just doing this in devlib? Should also get @setrofim's view on whether it belongs in devlib/what form it should take.

@bjackman should we close this ? I think it does make more sense to have it in devlib.

Yeah I agree


SHAP Explainer throws an error while using transformers model for abstractive summarization

Issue Description
Hello,
I am trying to generate an explanation of abstractive text summarization output for a long piece of text input.  I have been using various transformers models e.g. Bigbird-Pegasus, LongT5, Lonformer Encoder Decoder etc. But none of it worked and throws different errors. Following is the piece of code I used.
I get the TypeError. 'int object is not callable'.
Minimal Reproducible Example
LONG_ARTICLE = """"anxiety affects quality of life in those living
with parkinson 's disease ( pd ) more so than
overall cognitive status , motor deficits , apathy
, and depression [ 13 ] . although anxiety and
depression are often related and coexist in pd
patients , recent research suggests that anxiety
rather than depression is the most prominent and
prevalent mood disorder in pd [ 5 , 6 ] . yet ,
our current understanding of anxiety and its
impact on cognition in pd , as well as its neural
basis and best treatment practices , remains
meager and lags far behind that of depression .
overall , neuropsychiatric symptoms in pd have
been shown to be negatively associated with
cognitive performance . for example , higher
depression scores have been correlated with lower
scores on the mini - mental state exam ( mmse ) [
8 , 9 ] as well as tests of memory and executive
functions ( e.g. , attention ) [ 1014 ] . likewise
, apathy and anhedonia in pd patients have been
associated with executive dysfunction [ 10 , 1523
] . however , few studies have specifically
investigated the relationship between anxiety and
cognition in pd . one study showed a strong
negative relationship between anxiety ( both state
and trait ) and overall cognitive performance (
measured by the total of the repeatable battery
for the assessment of neuropsychological status
index ) within a sample of 27 pd patients .
furthermore , trait anxiety was negatively
associated with each of the cognitive domains
assessed by the rbans ( i.e. , immediate memory ,
visuospatial construction , language , attention ,
and delayed memory ) . two further studies have
examined whether anxiety differentially affects
cognition in patients with left - sided dominant
pd ( lpd ) versus right - sided dominant pd ( rpd
) ; however , their findings were inconsistent .
the first study found that working memory
performance was worse in lpd patients with anxiety
compared to rpd patients with anxiety , whereas
the second study reported that , in lpd , apathy
but not anxiety was associated with performance on
nonverbally mediated executive functions and
visuospatial tasks ( e.g. , tmt - b , wms - iii
spatial span ) , while in rpd , anxiety but not
apathy significantly correlated with performance
on verbally mediated tasks ( e.g. , clock reading
test and boston naming test ) . furthermore ,
anxiety was significantly correlated with
neuropsychological measures of attention and
executive and visuospatial functions . taken
together , it is evident that there are limited
and inconsistent findings describing the
relationship between anxiety and cognition in pd
and more specifically how anxiety might influence
particular domains of cognition such as attention
and memory and executive functioning . it is also
striking that , to date , no study has examined
the influence of anxiety on cognition in pd by
directly comparing groups of pd patients with and
without anxiety while excluding depression . given
that research on healthy young adults suggests
that anxiety reduces processing capacity and
impairs processing efficiency , especially in the
central executive and attentional systems of
working memory [ 26 , 27 ] , we hypothesized that
pd patients with anxiety would show impairments in
attentional set - shifting and working memory
compared to pd patients without anxiety .
furthermore , since previous work , albeit limited
, has focused on the influence of symptom
laterality on anxiety and cognition , we also
explored this relationship . seventeen pd patients
with anxiety and thirty - three pd patients
without anxiety were included in this study ( see
table 1 ) . the cross - sectional data from these
participants was taken from a patient database
that has been compiled over the past 8 years (
since 2008 ) at the parkinson 's disease research
clinic at the brain and mind centre , university
of sydney . inclusion criteria involved a
diagnosis of idiopathic pd according to the united
kingdom parkinson 's disease society brain bank
criteria   and were confirmed by a neurologist (
sjgl ) . patients also had to have an adequate
proficiency in english and have completed a full
neuropsychological assessment . ten patients in
this study ( 5 pd with anxiety ; 5 pd without
anxiety ) were taking psychotropic drugs ( i.e. ,
benzodiazepine or selective serotonin reuptake
inhibitor ) . patients were also excluded if they
had other neurological disorders , psychiatric
disorders other than affective disorders ( such as
anxiety ) , or if they reported a score greater
than six on the depression subscale of the
hospital anxiety and depression scale ( hads ) .
thus , all participants who scored within a
depressed  ( hads - d > 6 ) range were excluded
from this study , in attempt to examine a refined
sample of pd patients with and without anxiety in
order to determine the independent effect of
anxiety on cognition . this research was approved
by the human research ethics committee of the
university of sydney , and written informed
consent was obtained from all participants . self
- reported hads was used to assess anxiety in pd
and has been previously shown to be a useful
measure of clinical anxiety in pd . a cut - off
score of > 8 on the anxiety subscale of the hads (
hads - a ) was used to identify pd cases with
anxiety ( pda+ ) , while a cut - off score of < 6
on the hads - a was used to identify pd cases
without anxiety ( pda ) . this criterion was more
stringent than usual ( > 7 cut - off score ) , in
effort to create distinct patient groups . the
neurological evaluation rated participants
according to hoehn and yahr ( h&y ) stages   and
assessed their motor symptoms using part iii of
the revised mds task force unified parkinson 's
disease rating scale ( updrs ) . in a similar way
this was determined by calculating a total left
and right score from rigidity items 3035 ,
voluntary movement items 3643 , and tremor items
5057 from the mds - updrs part iii ( see table 1 )
. processing speed was assessed using the trail
making test , part a ( tmt - a , z - score ) .
attentional set - shifting was measured using the
trail making test , part b ( tmt - b , z - score )
. working memory was assessed using the digit span
forward and backward subtest of the wechsler
memory scale - iii ( raw scores ) . language was
assessed with semantic and phonemic verbal fluency
via the controlled oral word associated test (
cowat animals and letters , z - score ) . the
ability to retain learned verbal memory was
assessed using the logical memory subtest from the
wechsler memory scale - iii ( lm - i z - score ,
lm - ii z - score , % lm retention z - score ) .
the mini - mental state examination ( mmse )
demographic , clinical , and neuropsychological
variables were compared between the two groups
with the independent t - test or mann  whitney u
test , depending on whether the variable met
parametric assumptions . chi - square tests were
used to examine gender and symptom laterality
differences between groups . all analyses employed
an alpha level of p < 0.05 and were two - tailed .
spearman correlations were performed separately in
each group to examine associations between anxiety
and/or depression ratings and cognitive functions
. as expected , the pda+ group reported
significant greater levels of anxiety on the hads
- a ( u = 0 , p < 0.001 ) and higher total score
on the hads ( u = 1 , p < 0.001 ) compared to the
pda group ( table 1 ) . groups were matched in age
( t(48 ) = 1.31 , p = 0.20 ) , disease duration (
u = 259 , p = 0.66 ) , updrs - iii score ( u =
250.5 , p = 0.65 ) , h&y ( u = 245 , p = 0.43 ) ,
ledd ( u = 159.5 , p = 0.80 ) , and depression (
hads - d ) ( u = 190.5 , p = 0.06 ) . additionally
, all groups were matched in the distribution of
gender (  = 0.098 , p = 0.75 ) and side - affected
(  = 0.765 , p = 0.38 ) . there were no group
differences for tmt - a performance ( u = 256 , p
= 0.62 ) ( table 2 ) ; however , the pda+ group
had worse performance on the trail making test
part b ( t(46 ) = 2.03 , p = 0.048 ) compared to
the pda group ( figure 1 ) . the pda+ group also
demonstrated significantly worse performance on
the digit span forward subtest ( t(48 ) = 2.22 , p
= 0.031 ) and backward subtest ( u = 190.5 , p =
0.016 ) compared to the pda group ( figures 2(a )
and 2(b ) ) . neither semantic verbal fluency (
t(47 ) = 0.70 , p = 0.49 ) nor phonemic verbal
fluency ( t(47 ) = 0.39 , p = 0.70 ) differed
between groups . logical memory i immediate recall
test ( u = 176 , p = 0.059 ) showed a trend that
the pda+ group had worse new verbal learning and
immediate recall abilities than the pda group .
however , logical memory ii test performance ( u =
219 , p = 0.204 ) and logical memory % retention (
u = 242.5 , p = 0.434 ) did not differ between
groups . there were also no differences between
groups in global cognition ( mmse ) ( u = 222.5 ,
p = 0.23 ) . participants were split into lpd and
rpd , and then further group differences were
examined between pda+ and pda. importantly , the
groups remained matched in age , disease duration
, updrs - iii , dde , h&y stage , and depression
but remained significantly different on self -
reported anxiety . lpda+ demonstrated worse
performance on the digit span forward test ( t(19
) = 2.29 , p = 0.033 ) compared to lpda , whereas
rpda+ demonstrated worse performance on the digit
span backward test ( u = 36.5 , p = 0.006 ) , lm -
i immediate recall ( u = 37.5 , p = 0.008 ) , and
lm - ii ( u = 45.0 , p = 0.021 ) but not lm %
retention ( u = 75.5 , p = 0.39 ) compared to
rpda. this study is the first to directly compare
cognition between pd patients with and without
anxiety . the findings confirmed our hypothesis
that anxiety negatively influences attentional set
- shifting and working memory in pd . more
specifically , we found that pd patients with
anxiety were more impaired on the trail making
test part b which assessed attentional set -
shifting , on both digit span tests which assessed
working memory and attention , and to a lesser
extent on the logical memory test which assessed
memory and new verbal learning compared to pd
patients without anxiety . taken together , these
findings suggest that anxiety in pd may reduce
processing capacity and impair processing
efficiency , especially in the central executive
and attentional systems of working memory in a
similar way as seen in young healthy adults [ 26 ,
27 ] . although the neurobiology of anxiety in pd
remains unknown , many researchers have postulated
that anxiety disorders are related to
neurochemical changes that occur during the early
, premotor stages of pd - related degeneration [
37 , 38 ] such as nigrostriatal dopamine depletion
, as well as cell loss within serotonergic and
noradrenergic brainstem nuclei ( i.e. , raphe
nuclei and locus coeruleus , resp . , which
provide massive inputs to corticolimbic regions )
. over time , chronic dysregulation of
adrenocortical and catecholamine functions can
lead to hippocampal damage as well as
dysfunctional prefrontal neural circuitries [ 39 ,
40 ] , which play a key role in memory and
attention . recent functional neuroimaging work
has suggested that enhanced hippocampal activation
during executive functioning and working memory
tasks may represent compensatory processes for
impaired frontostriatal functions in pd patients
compared to controls . therefore , chronic stress
from anxiety , for example , may disrupt
compensatory processes in pd patients and explain
the cognitive impairments specifically in working
memory and attention seen in pd patients with
anxiety . it has also been suggested that
hyperactivation within the putamen may reflect a
compensatory striatal mechanism to maintain normal
working memory performance in pd patients ;
however , losing this compensatory activation has
been shown to contribute to poor working memory
performance . anxiety in mild pd has been linked
to reduced putamen dopamine uptake which becomes
more extensive as the disease progresses . this
further supports the notion that anxiety may
disrupt compensatory striatal mechanisms as well ,
providing another possible explanation for the
cognitive impairments observed in pd patients with
anxiety in this study . noradrenergic and
serotonergic systems should also be considered
when trying to explain the mechanisms by which
anxiety may influence cognition in pd . although
these neurotransmitter systems are relatively
understudied in pd cognition , treating the
noradrenergic and serotonergic systems has shown
beneficial effects on cognition in pd . selective
serotonin reuptake inhibitor , citalopram , was
shown to improve response inhibition deficits in
pd , while noradrenaline reuptake blocker ,
atomoxetine , has been recently reported to have
promising effects on cognition in pd [ 45 , 46 ] .
overall , very few neuroimaging studies have been
conducted in pd in order to understand the neural
correlates of pd anxiety and its underlying neural
pathology . future research should focus on
relating anatomical changes and neurochemical
changes to neural activation in order to gain a
clearer understanding on how these pathologies
affect anxiety in pd . to further understand how
anxiety and cognitive dysfunction are related ,
future research should focus on using advanced
structural and function imaging techniques to
explain both cognitive and neural breakdowns that
are associated with anxiety in pd patients .
research has indicated that those with amnestic
mild cognitive impairment who have more
neuropsychiatric symptoms have a greater risk of
developing dementia compared to those with fewer
neuropsychiatric symptoms . future studies should
also examine whether treating neuropsychiatric
symptoms might impact the progression of cognitive
decline and improve cognitive impairments in pd
patients . previous studies have used pd symptom
laterality as a window to infer asymmetrical
dysfunction of neural circuits . for example , lpd
patients have greater inferred right hemisphere
pathology , whereas rpd patients have greater
inferred left hemisphere pathology . thus ,
cognitive domains predominantly subserved by the
left hemisphere ( e.g. , verbally mediated tasks
of executive function and verbal memory ) might be
hypothesized to be more affected in rpd than lpd ;
however , this remains controversial . it has also
been suggested that since anxiety is a common
feature of left hemisphere involvement [ 48 , 49 ]
, cognitive domains subserved by the left
hemisphere may also be more strongly related to
anxiety . results from this study showed selective
verbal memory deficits in rpd patients with
anxiety compared to rpd without anxiety , whereas
lpd patients with anxiety had greater attentional
/ working memory deficits compared to lpd without
anxiety . although these results align with
previous research , interpretations of these
findings should be made with caution due to the
small sample size in the lpd comparison
specifically . recent work has suggested that the
hads questionnaire may underestimate the burden of
anxiety related symptomology and therefore be a
less sensitive measure of anxiety in pd [ 30 , 50
] . in addition , our small sample size also
limited the statistical power for detecting
significant findings . based on these limitations
, our findings are likely conservative and
underrepresent the true impact anxiety has on
cognition in pd . additionally , the current study
employed a very brief neuropsychological
assessment including one or two tests for each
cognitive domain . future studies are encouraged
to collect a more complex and comprehensive
battery from a larger sample of pd participants in
order to better understand the role anxiety plays
on cognition in pd . another limitation of this
study was the absence of diagnostic interviews to
characterize participants ' psychiatric symptoms
and specify the type of anxiety disorders included
in this study . future studies should perform
diagnostic interviews with participants ( e.g. ,
using dsm - v criteria ) rather than relying on
self - reported measures to group participants ,
in order to better understand whether the type of
anxiety disorder ( e.g. , social anxiety , phobias
, panic disorders , and generalized anxiety )
influences cognitive performance differently in pd
. one advantage the hads questionnaire provided
over other anxiety scales was that it assessed
both anxiety and depression simultaneously and
allowed us to control for coexisting depression .
although there was a trend that the pda+ group
self - reported higher levels of depression than
the pda group , all participants included in the
study scored < 6 on the depression subscale of the
hads . controlling for depression while assessing
anxiety has been identified as a key shortcoming
in the majority of recent work . considering many
previous studies have investigated the influence
of depression on cognition in pd without
accounting for the presence of anxiety and the
inconsistent findings reported to date , we
recommend that future research should try to
disentangle the influence of anxiety versus
depression on cognitive impairments in pd .
considering the growing number of clinical trials
for treating depression , there are few if any for
the treatment of anxiety in pd . anxiety is a key
contributor to decreased quality of life in pd and
greatly requires better treatment options .
moreover , anxiety has been suggested to play a
key role in freezing of gait ( fog ) , which is
also related to attentional set - shifting [ 52 ,
53 ] . future research should examine the link
between anxiety , set - shifting , and fog , in
order to determine whether treating anxiety might
be a potential therapy for improving fog ."""

import torch
from transformers import AutoTokenizer, LongT5ForConditionalGeneration

tokenizer = AutoTokenizer.from_pretrained("Stancld/longt5-tglobal-large-16384-pubmed-3k_steps")
model = LongT5ForConditionalGeneration.from_pretrained("Stancld/longt5-tglobal-large-16384-pubmed-3k_steps", return_dict_in_generate=True).to("cuda")
import shap

explainer = shap.Explainer(model, tokenizer)
shap_values = explainer(s)

### Traceback

```shell
TypeError                                 Traceback (most recent call last)
Cell In [23], line 1
----> 1 shap_values = explainer(s)

File /usr/local/lib/python3.9/dist-packages/shap/explainers/_partition.py:125, in PartitionExplainer.__call__(self, max_evals, fixed_context, main_effects, error_bounds, batch_size, outputs, silent, *args)
    121 def __call__(self, *args, max_evals=500, fixed_context=None, main_effects=False, error_bounds=False, batch_size="auto",
    122              outputs=None, silent=False):
    123     """ Explain the output of the model on the given arguments.
    124     """
--> 125     return super().__call__(
    126         *args, max_evals=max_evals, fixed_context=fixed_context, main_effects=main_effects, error_bounds=error_bounds, batch_size=batch_size,
    127         outputs=outputs, silent=silent
    128     )

File /usr/local/lib/python3.9/dist-packages/shap/explainers/_explainer.py:267, in Explainer.__call__(self, max_evals, main_effects, error_bounds, batch_size, outputs, silent, *args, **kwargs)
    265     feature_names = [[] for _ in range(len(args))]
    266 for row_args in show_progress(zip(*args), num_rows, self.__class__.__name__+" explainer", silent):
--> 267     row_result = self.explain_row(
    268         *row_args, max_evals=max_evals, main_effects=main_effects, error_bounds=error_bounds,
    269         batch_size=batch_size, outputs=outputs, silent=silent, **kwargs
    270     )
    271     values.append(row_result.get("values", None))
    272     output_indices.append(row_result.get("output_indices", None))

File /usr/local/lib/python3.9/dist-packages/shap/explainers/_partition.py:150, in PartitionExplainer.explain_row(self, max_evals, main_effects, error_bounds, batch_size, outputs, silent, fixed_context, *row_args)
    148 # if not fixed background or no base value assigned then compute base value for a row
    149 if self._curr_base_value is None or not getattr(self.masker, "fixed_background", False):
--> 150     self._curr_base_value = fm(m00.reshape(1, -1), zero_index=0)[0] # the zero index param tells the masked model what the baseline is
    151 f11 = fm(~m00.reshape(1, -1))[0]
    153 if callable(self.masker.clustering):

File /usr/local/lib/python3.9/dist-packages/shap/utils/_masked_model.py:69, in MaskedModel.__call__(self, masks, zero_index, batch_size)
     66         return self._full_masking_call(full_masks, zero_index=zero_index, batch_size=batch_size)
     68 else:
---> 69     return self._full_masking_call(masks, batch_size=batch_size)

File /usr/local/lib/python3.9/dist-packages/shap/utils/_masked_model.py:146, in MaskedModel._full_masking_call(self, masks, zero_index, batch_size)
    143         all_masked_inputs[i].append(v)
    145 joined_masked_inputs = tuple([np.concatenate(v) for v in all_masked_inputs])
--> 146 outputs = self.model(*joined_masked_inputs)
    147 _assert_output_input_match(joined_masked_inputs, outputs)
    148 all_outputs.append(outputs)

File /usr/local/lib/python3.9/dist-packages/shap/models/_model.py:28, in Model.__call__(self, *args)
     27 def __call__(self, *args):
---> 28     out = self.inner_model(*args)
     29     is_tensor = safe_isinstance(out, "torch.Tensor")
     30     out = out.cpu().detach().numpy() if is_tensor else np.array(out)

File /usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py:1130, in Module._call_impl(self, *input, **kwargs)
   1126 # If we don't have any hooks, we want to skip the rest of the logic in
   1127 # this function, and just call forward.
   1128 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1129         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1130     return forward_call(*input, **kwargs)
   1131 # Do not call functions when jit is used
   1132 full_backward_hooks, non_full_backward_hooks = [], []

File /usr/local/lib/python3.9/dist-packages/transformers/models/longt5/modeling_longt5.py:1991, in LongT5ForConditionalGeneration.forward(self, input_ids, attention_mask, decoder_input_ids, decoder_attention_mask, head_mask, decoder_head_mask, cross_attn_head_mask, encoder_outputs, past_key_values, inputs_embeds, decoder_inputs_embeds, labels, use_cache, output_attentions, output_hidden_states, return_dict)
   1988 # Encode if needed (training, first prediction pass)
   1989 if encoder_outputs is None:
   1990     # Convert encoder inputs in embeddings if needed
-> 1991     encoder_outputs = self.encoder(
   1992         input_ids=input_ids,
   1993         attention_mask=attention_mask,
   1994         inputs_embeds=inputs_embeds,
   1995         head_mask=head_mask,
   1996         output_attentions=output_attentions,
   1997         output_hidden_states=output_hidden_states,
   1998         return_dict=return_dict,
   1999     )
   2000 elif return_dict and not isinstance(encoder_outputs, BaseModelOutput):
   2001     encoder_outputs = BaseModelOutput(
   2002         last_hidden_state=encoder_outputs[0],
   2003         hidden_states=encoder_outputs[1] if len(encoder_outputs) > 1 else None,
   2004         attentions=encoder_outputs[2] if len(encoder_outputs) > 2 else None,
   2005     )

File /usr/local/lib/python3.9/dist-packages/torch/nn/modules/module.py:1130, in Module._call_impl(self, *input, **kwargs)
   1126 # If we don't have any hooks, we want to skip the rest of the logic in
   1127 # this function, and just call forward.
   1128 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1129         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1130     return forward_call(*input, **kwargs)
   1131 # Do not call functions when jit is used
   1132 full_backward_hooks, non_full_backward_hooks = [], []

File /usr/local/lib/python3.9/dist-packages/transformers/models/longt5/modeling_longt5.py:1396, in LongT5Stack.forward(self, input_ids, attention_mask, encoder_hidden_states, encoder_attention_mask, inputs_embeds, head_mask, cross_attn_head_mask, past_key_values, use_cache, output_attentions, output_hidden_states, return_dict)
   1392     raise ValueError(
   1393         f"You cannot specify both {err_msg_prefix}input_ids and {err_msg_prefix}inputs_embeds at the same time"
   1394     )
   1395 elif input_ids is not None:
-> 1396     input_shape = input_ids.size()
   1397     input_ids = input_ids.view(-1, input_shape[-1])
   1398 elif inputs_embeds is not None:

TypeError: 'int' object is not callable

```

Expected Behavior
No response
Bug report checklist

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest release of shap.
[X] I have confirmed this bug exists on the master branch of shap.
[X] I'd be interested in making a PR to fix this bug

Installed Versions
Latest version used from github repo.

I can reproduce your example on master but do not think that this is really a shap issue here. The following lines already result in similar errors:
>>> model([LONG_ARTICLE])
AttributeError: 'list' object has no attribute 'size'
>>> model(LONG_ARTICLE)
AttributeError: 'str' object has no attribute 'size'

Can you somehow get this to run so that the model actually returns something?

I have tried but it didn't work.

Okay, this indicates that we do not really have a shap error here since the model is not running at all.

Can you show me how to generate the output with the model you are using? That's what I tried above but that failed.

Following is a piece of code to generate the output using longt5.
!pip install transformers[sentencepiece] datasets sacrebleu rouge_score py7zr -q
LONG_ARTICLE = """"anxiety affects quality of life in those living
with parkinson 's disease ( pd ) more so than
overall cognitive status , motor deficits , apathy
, and depression [ 13 ] . although anxiety and
depression are often related and coexist in pd
patients , recent research suggests that anxiety
rather than depression is the most prominent and
prevalent mood disorder in pd [ 5 , 6 ] . yet ,
our current understanding of anxiety and its
impact on cognition in pd , as well as its neural
basis and best treatment practices , remains
meager and lags far behind that of depression ."""

import torch
from transformers import AutoTokenizer, LongT5ForConditionalGeneration

tokenizer = AutoTokenizer.from_pretrained("Stancld/longt5-tglobal-large-16384-pubmed-3k_steps")
input_ids = tokenizer(LONG_ARTICLE, return_tensors="pt").input_ids

model = LongT5ForConditionalGeneration.from_pretrained("Stancld/longt5-tglobal-large-16384-pubmed-3k_steps", return_dict_in_generate=True)

sequences = model.generate(input_ids).sequences

summary = tokenizer.batch_decode(sequences)

For the second model bigbird pegasus the output can be generated using the code below.

from transformers import BigBirdPegasusForConditionalGeneration, AutoTokenizer

tokenizer = AutoTokenizer.from_pretrained("google/bigbird-pegasus-large-pubmed")
model = BigBirdPegasusForConditionalGeneration.from_pretrained("google/bigbird-pegasus-large-pubmed")

text = "text article"
inputs = tokenizer(text, return_tensors='pt')
prediction = model.generate(**inputs)
prediction = tokenizer.batch_decode(prediction)

Hey I encountered a similar issue. I think the issue is SHAP passed in a numpy array instead of tensor object when calling transformers API.

@Yangliule Did you find any solution.?

Hey @SVC04, I solved the issue by reverting transformers version and pytorch version. Also, make sure you are using AutoModelForSeq2SeqLM.

@Yangliule could you please show the versions that are working?


Adding TEMPLATE_README.md to DscResource.Template

Pull Request (PR) description

Adding TEMPLATE_README.md to DscResource.Template
Cleaning up template files.
Adding information of the Plaster template project.

This Pull Request (PR) fixes the following issues

Fixes #278

Task list

[ ] This PR changes documentation
[x] This PR changes DscResource.Template
[ ] This PR changes Tests.Template

[ ] The template version number has been increased appropriately.
[ ] An descriptive entry has been added to the CHANGELOG.md under the
Tests.Template folder.

This change is

Since I don't want to rename the README.md file in the DscResource.Template I added a new file TEMPLATE_README.md which has information about the template. I just added some base information for now.

@PlagueHO Could you please review this when you have some time? No hurry.


Add typespecs for Timex.Protocol and adjust irregularities

Adding the typespecs to the protocol will enable dialyzer
to verify the typespecs of the implementations as well as
the calls from the main Timex module.
The typespecs in the Timex module was updated to use
Types.valid_datetime instead of the more verbose multi type
version (also Types.calendar_types does not exist).
Error tuple returns was added to those missing it in the
Timex module.
The protocol implementations type specifications is
narrowed down to match those of the protocol/caller module.
The typespecs for to_julian was changed to float, which is
what is returned.
Also fixes a few bugs:

Date.from_iso_day now returns a Date struct instead of a map
Date.beginning_of_quarter now works instead of crashes

Summary of changes
I have som more typespec changes coming up, but think that this is a good start to build upon an merge separately.
This greatly improves dialyzers ability to find errors in the typespecs for all the implementations and calling code. A possible downside is if any client implements their own implementation of the protocol, in which case I think they will get dialyzer errors since their types are not the expected types. Is this a dealbreaker?
Checklist

[x] New functions have typespecs, changed functions were updated
[x] Same for documentation, including moduledocs
[x] Tests were added or updated to cover changes
[x] Commits were squashed into a single coherent commit

Coverage increased (+0.7%) to 69.236% when pulling b4ab964e5dd9c015f7ecc9da6800c6b7ff780be6 on alvinlindstam:typespec-protocol into 90741ce9263402597dc53341995c5c1acc3de44e on bitwalker:master.

Coverage increased (+0.6%) to 69.125% when pulling b4ab964e5dd9c015f7ecc9da6800c6b7ff780be6 on alvinlindstam:typespec-protocol into 90741ce9263402597dc53341995c5c1acc3de44e on bitwalker:master.

The protocol was designed to make implementations for the various types cleaner, I would expect it to be very rare that someone would need to implement the protocol types other than those supported, especially since plain maps are supported.
This is great! Thanks for digging in and helping clean up the code! Timex has evolved a ton since the beginning, so there is unfortunately still cruft hanging around that was missed. One thing I want to introduce are property based tests to help fuzz test the API and surface any latent bugs, it's tricky with date/time because we'll need custom generators and I haven't figured out a good way to do shrinking yet.

Yes, improved tests would be very nice.
Happy to help out.


Any plans to add Tiled Convolutional layer?

Tiled Convolutional layer was introduced in this paper from NIPS 2010: http://papers.nips.cc/paper/4136-tiled-convolutional-neural-networks.pdf. Per the authors, it provides scale and rotational invariance which can be tweaked with the tiling parameter (k, in the paper). This can be a useful alternative to augmenting your data and thereby increasing its size.
Excerpt from the paper:

tiled convolution neural networks ... use a regular “tiled” pattern of tied weights that does not require that adjacent hidden units share identical weights, but instead requires only that hidden units k steps away from each other to have tied weights. By pooling over neighboring units, this architecture is able to learn complex invariances (such as scale and rotational invariance) beyond translational invariance.

I'm not sure if this can be trained like a regular CNN.
The code would require some work but the API won't need much modification. The 'tiling parameter' can be added cleanly to the Covolution1D and Convolution2D classes.
Thoughts and comments are welcome!
Thanks for all the great work on Keras!
-Angad

Stop the bot ;) and let it live/open.
Would be great to add this features!


Enforcing version consistency

… between latest git tag and version embedded in src/libraries.ts
Let's see if the CI catches the current inconsistent state! 🤞

One issue I see with this approach is rolling out a new version without the tag already generated would be impossible. You would have to kind of do both at the same time. Which isn't bad - we could tweak the release steps to not rely on GitHub-Release-generated tag (as we do for all other deno-slack-* repos) but instead create the tag manually from your own terminal at the same time you are bumping the version inside the code.

One issue I see with this approach is rolling out a new version without the tag already generated would be impossible. You would have to kind of do both at the same time. Which isn't bad - we could tweak the release steps to not rely on GitHub-Release-generated tag (as we do for all other deno-slack-* repos) but instead create the tag manually from your own terminal at the same time you are bumping the version inside the code.

🤔 I have a similar concern, namely that you can create the tag without a commit to update the version. We can fix it via process like you mention, but to fix it through CI I think the problem needs to be caught when we (or GH) pushes a tag rather than pushing a commit. I apologize for bringing a problem without any proposal for a solution 😅


Hotkey to start testing from checkpoint last tested from

Some initial testing showed that we can use void TestMapFromCoord(int3 Coord, CGameEditorPluginMap::ECardinalDirections Dir); to start at an integer coordinate. But that uses the basic shootmania arena mode.
Trying to launch into test mode using the TrackMania/TM_RaceTest_Local.Script.txt mode did not seem to work and actually locked up the editor. More investigation is needed here.

There appears to be some *.LaunchedCP.Gbx files in the ProgramData cache that may be related to this. It bears an investigation as to what might be in those files.


RenameVariables should have option to skip normalizing ${_}

There should be a way to define such variable without normalizing it to ${}:
${_}  ${second_return}    Keyword Call  # I dont need first return value so I assigned it to ${_}

Same goes for some special variables (like ${/n}, it looks better than ${/N}). I'm also on fence for ${True} -> ${TRUE}. I could introduce some variables that should generally be ignored (it could be configurable). If you have some examples where you think the RenameVariables shouldn't rename - please let me know.

Fixed - it will be released in 4.4.0 today.


Web5 v0.8.1 updates

Updating knowledge base to reflect changes in Web5 version 0.8.1

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1205327658751426
https://app.asana.com/0/0/1205327658751432

thanks - deploying now


Setup Kanban board for JavaScript capstone

[ ] The link of the project:

https://github.com/mahdinoori2000/javascript-capstone/projects/1

[ ] We are 2 people working on the capstone project:

Mahdi Noori

Hamza Ramalan

STATUS: APPROVED 👍🎯
Hello @mahdinoori2000  and @H-Ramalan 👋
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights ⭐
✔ kanban board is set up properly
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Update RemoveFirstPartyAppsTestRule.java

Issue : This PR https://github.com/AzureAD/microsoft-authentication-library-common-for-android/pull/2056/files introduced an uninstall for MsalTestApp. But the pkgName for this is same as the pkgName for msalAutomationAPp. So, this may break the existing tests.
Fix : Commenting out the MsalTestAPp uninstall statement until the pkg name is changed.

Can the comment also include what you wrote in the description?

Done.


Share parameters

Hi guys,
Thank you for sharing your amazing work!
For training I am using the dataset posted here https://app.activeloop.ai/crossvivit/SunLake, which can be accessed with deeplake.
Furthermore, I am using some of the parameters which you have shared in your paper, however, I got some errors that mat1 could not be multiplied with mat2, because of different matrix size.
In order to solve this little problem, I have changed ctx_channels from 8 to 23, ts_channels from 16 to 8 in cross_vivit_bis.yaml.
However, I am not feeling that it is the correct solution to the problem.
Could you please share all your parameters from cross_vivit_bis.yaml, txcontext_datamodule.yaml and cross_vivit.yaml?

Hello friend, I would like to ask how to download a dataset and run the program? I encountered this error ，deeplake.util.exceptions.DatasetHandlerError: A Deep Lake dataset does not exist at the given path (/network/scratch/g/ghait.boukachab/o/EUMETSAT/tsf_eumetsat_bis-64_64). Check the path provided or in case you want to create a new dataset, use deeplake.empty().
Perhaps this is a very simple question, but it has troubled me for a long time. Looking forward to your answer, thank you very much.

I have added the parameter reset = True -> self.deeplake_ds = deeplake.load(self.data_dir, reset=True). It has solved the problem for me.
I used the command python main.py to start training, but I am not sure, it is correct

Thanks for your answer. Whether i need to modify these two parameters , the data_dir and stats_path?

It is not necessary to download the dataset from deeplake. Just go to configs/paths/default.yaml and change data_dir param. Put there the link to deeplake dataset: "data_dir: hub://crossvivit/SunLake"

Thank you very much！
------------------ 原始邮件 ------------------
发件人:                                                                                "gitbooo/CrossViVit"                                                                                @.>;
发送时间: 2023年9月15日(星期五) 下午4:24
@.>;
<EMAIL_ADDRESS>主题: Re: [gitbooo/CrossViVit] Share parameters (Issue #2)
It is not necessary to download the dataset from deeplake. Just go to configs/paths/default.yaml and change data_dir param. Put there the link to deeplake dataset: "data_dir: hub://crossvivit/SunLake"
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Thank you very much!

Thanky @deep404 for helping @yy0127-ai with the issue. We're going to update the configs accordingly. Please note that the parameters used in the papers will be the ones assigned in configs/experiment.

My please to help other.
Great, I'm looking forward to it

Hey,
I want to add that for any model you run, you have to specify experiment=model_you_want to the command so that it knows which parameters to run, this way you won't have any problems. Can you try that and tell me if the problem persists?

My please to help other. Great, I'm looking forward to it

Hello friend, I would like to ask how to download a dataset and run the program? I encountered this error ，deeplake.util.exceptions.DatasetHandlerError: A Deep Lake dataset does not exist at the given path (/network/scratch/g/ghait.boukachab/o/EUMETSAT/tsf_eumetsat_bis-64_64). Check the path provided or in case you want to create a new dataset, use deeplake.empty(). Perhaps this is a very simple question, but it has troubled me for a long time. Looking forward to your answer, thank you very much.

Hello, do you know how to run crosslive on Linux? I only have one computer and haven't used Slurm before. I don't know how to run it. I would greatly appreciate it if you could provide assistance

It is not necessary to download the dataset from deeplake. Just go to configs/paths/default.yaml and change data_dir param. Put there the link to deeplake dataset: "data_dir: hub://crossvivit/SunLake"

Hi, I am importing sunlake dataset by using deeplake but the process is taking a long time (more than 2 hours) which is causing me to get stuck here without outputting the results. May I ask if you encounter this problem?
Is it possible to download the Sunlake dataset and use it locally? Looking forward to your reply!
import deeplake

ds = deeplake.load('hub://crossvivit/SunLake')

Thank you very much!

"Excuse me, have you successfully imported the SunLake dataset from Deeplake? I'm having a lot of trouble because it's very slow when I use ds = deeplake.load('hub://crossvivit/SunLake'). Can I discuss this issue with you?"


[Bug] Collection Runner does not use client certificates

When using client-certificates like this (bruno.json)
{
  "version": "1",
  "name": "hello",
  "type": "collection",
  "clientCertificates": {
    "enabled": true,
    "certs": [
      {
        "domain": "*.test.tech",
        "certFilePath": "/Users/Martin/.local/share/certs/some-service.client.crt",
        "keyFilePath": "/Users/Martin/.local/share/certs/some-service.client.key",
        "passphrase": ""
      }
    ]
  }
}

then individual requests work and properly use the certificate.
However, when using the collection runner, the certificate does not seem to be used, and I run into http 401 errors on each of my requests:

fyi @phoval

@panda7789 refactor is a priority for v2.
Right now, we copy paste code :)

Just retested to make sure -- it works! 🎉


Sergkanz/new metrics api

Hi @Jangesou88, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@PProvost

Hi @Jangesou88, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@Jangesou88 what is this PR about?