"kubectl exec" fails but "Execute Shell" via UI works with a custom project role.

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a custom project role with the below rules

{
   "administrative":false,
   "annotations":{
      "cleanup.cattle.io/rtUpgradeCluster":"true",
      "lifecycle.cattle.io/create.mgmt-auth-roletemplate-lifecycle":"true"
   },
   "clusterCreatorDefault":false,
   "context":"project",
   "created":"2020-10-23T05:11:54Z",
   "creatorId":"user-bsjnb",
   "external":false,
   "hidden":false,
   "id":"rt-w82xf",
   "labels":{
      "cattle.io/creator":"norman"
   },
   "locked":false,
   "name":"pod_exec_test",
   "ownerReferences":[

   ],
   "projectCreatorDefault":false,
   "roleTemplateIds":[

   ],
   "rules":[
      {
         "apiGroups":[
            "*"
         ],
         "resources":[
            "daemonsets"
         ],
         "type":"/v3/schemas/policyRule",
         "verbs":[
            "get",
            "list",
            "patch",
            "update",
            "watch"
         ]
      },
      {
         "apiGroups":[
            "*"
         ],
         "resources":[
            "pods/exec"
         ],
         "type":"/v3/schemas/policyRule",
         "verbs":[
            "get",
            "list",
            "patch",
            "update",
            "watch"
         ]
      },
      {
         "apiGroups":[
            "*"
         ],
         "resources":[
            "pods/status"
         ],
         "type":"/v3/schemas/policyRule",
         "verbs":[
            "get",
            "list",
            "watch"
         ]
      },
      {
         "apiGroups":[
            "*"
         ],
         "resources":[
            "pods"
         ],
         "type":"/v3/schemas/policyRule",
         "verbs":[
            "get",
            "list",
            "patch",
            "update",
            "watch"
         ]
      },
      {
         "apiGroups":[
            "*"
         ],
         "resources":[
            "pods/log"
         ],
         "type":"/v3/schemas/policyRule",
         "verbs":[
            "get",
            "list",
            "watch"
         ]
      }
   ],
   "uuid":"67ac3781-e026-413b-bc16-a460c86deb14"
}

Create new user with a "User base" global role.
Assign project role to the new user.
Open UI shell and execute kubectl exec to the pod.

Result:
> kubectl exec -it myapp-789ffb4694-jqpbm -- bash
Error from server (Forbidden): pods "myapp-789ffb4694-jqpbm" is forbidden: User "u-pprzw" cannot create resource "pods/exec" in API group "" in the namespace "default"
>

Other details that may be helpful:
Opening shell via the UI option "Execute shell" from container works fine

As per kubectl verbose output, the call is POST and this might need create permission, but in that case, UI shell also shouldn't work.
I1023 06:28:13.987358     123 round_trippers.go:423] curl -k -v -XPOST  -H "User-Agent: kubectl/v1.17.6 (linux/amd64) kubernetes/d32e40e" -H "Authorization: Bearer kubectl-shell-u-pprzw:92gk9449mf829l89r7tfbqq59jjmjn2wtgrnk9m8fkbkhlh2kqwdjv" -H "X-Stream-Protocol-Version: v4.channel.k8s.io" -H "X-Stream-Protocol-Version: v3.channel.k8s.io" -H "X-Stream-Protocol-Version: v2.channel.k8s.io" -H "X-Stream-Protocol-Version: channel.k8s.io" 'https://rancher244.xxx/k8s/clusters/c-2w9xn/api/v1/namespaces/default/pods/myapp-789ffb4694-jqpbm/exec?command=bash&container=myapp&stdin=true&stdout=true&tty=true'

I1023 06:28:14.017735     123 round_trippers.go:443] POST https://rancher244.xxx/k8s/clusters/c-2w9xn/api/v1/namespaces/default/pods/myapp-789ffb4694-jqpbm/exec?command=bash&container=myapp&stdin=true&stdout=true&tty=true 403 Forbidden in 30 milliseconds

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.4.8
Installation option (single install/HA): HA

gz#12963

Thank it works for me!!!
Same Racnher 2.4.8 version


Not an issue

I fixed and updated the code so it would work @AstraaDev
You have to install a specific module for it to work with the edited code which is discord.py-self which u can install by the following cmd
pip install discord.py-self
selfbot fixed py file.zip

not work?


Nonce is forced

Hi, it seems that the nonce parameter of the request url is always there.
Google doesn't accept requests with that parameter.
A quick fix would be to add the nonce only if it is provided like in the original oauth-ng plugin:
function init(){
...
       if (scope.nonce != undefined){
              scope.nonce = scope.nonce || generateState();
       }
...

service.init = function(params) {
...
       if (params.nonce){
              service.url += '&' + 'nonce=' + encodeURIComponent(params.nonce)
       }
...

I've updated this but to maintain backwards compatibility I've done it in a slightly different way - you can disable the auto generation of the nonce by setting the auto-generate-nonce attribute to false:
<oauth2 .... auto-generate-nonce="false" ...>
It's not as clean as the above but if I use the lack of a nonce to trigger it then it breaks existing clients where the server is expecting a nonce.
I may revisit on v2.0.0.
The update is in v1.0.3


appbio_quantstudio: 'allele color' should be returned as reporter dye setting

For QuantStudio Genotyping experiments as in appbio_quantstudio_example03.txt the entries for "Allele1 Color" and "Allele2 Color' should returned as the reporter dye setting in the respective measurement documents for each target DNA description

Addressed in PR #102


Add Iliad Italy (Mobile Telephony Provider) Sensor

Description:
Added Iliad Italy Sensor
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): https://github.com/home-assistant/home-assistant.io/pull/7981
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: iliad_italy
    username: USERNAME
    password: PASSWORD

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Added the documentation of this sensor here: https://github.com/home-assistant/home-assistant.io/pull/7981

@dgomes Something like 30 minutes should be fine, together with 1 hour of scan interval.
Adding it now.

Removed 😎
On Fri, Jan 4, 2019, 11:44 Diogo Gomes<EMAIL_ADDRESS>wrote:

@dgomes commented on this pull request.
In homeassistant/components/sensor/iliad_italy.py
https://github.com/home-assistant/home-assistant/pull/19645#discussion_r245265010
:

+from homeassistant.components.sensor import PLATFORM_SCHEMA
+from homeassistant.const import CONF_PASSWORD, CONF_USERNAME
+from homeassistant.helpers.aiohttp_client import async_get_clientsession
+import homeassistant.helpers.config_validation as cv
+from homeassistant.helpers.entity import Entity
+from homeassistant.util import Throttle
+import homeassistant.util.dt as dt_util
+
+REQUIREMENTS = ['aioiliad==0.1.1']
+
+_LOGGER = logging.getLogger(name)
+
+ICON = 'mdi:phone'
+
+SCAN_INTERVAL = timedelta(hours=1)
This is not used
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/19645#pullrequestreview-189324949,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH2OZ4NnJL5tPpO_LPKrHAzZsF8e9IwFks5u_zCUgaJpZM4Zkr7Q
.

It might be OK if it's just you :) but if more Iliad users start using your sensor, it might become an issue for Iliad...

Tested it with multiple requests for second and it didn't have problems so far :)


Removed the inputFormatters from the text input fields used by the Date Pickers

Description
The current Date Picker implementation uses inputFormatters on the text input fields to restrict the input text to only digits and a date separator. However, there are some locales that don't have a single character separator, so this restriction is causing more problems than it is helping (see #63373). This PR removes the inputFormatters and just lets the user enter what ever characters they want and lets the validation take care of the error.
Related Issues
Fixes: #63373
Tests
Updated the invalid format text test to check that non-digit characters are allowed, but will generate an error if submitted.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them?

[x] No, no existing tests failed, so this is not a breaking change.

If the solution is that we choose to remove usage of MaterialLocalizations.dateSeparator as a text input formatter, should we just completely remove dateSeparator from our arb files as well? I did a quick sweep of the flutter repo and it seems like the only two places where it's used is already removed in this PR.

Yeah, we can remove this field as it will no longer be needed after this PR. I didn't include it here as I was trying to keep the PR  somewhat contained so that we would have a better change of getting it cherry picked.


libMicrosoft.CognitiveServices.Speech.core.a(speechapi_c_audio_config.cpp.o)' does not contain bitcode.

Does the ios system not turn on bitcode?    The error is as follows:

More Errors will appear if the local project has bitcode turned off.

xcode：13.4.1
SDK： native c++ 1.21.0 or 1.22.0;         name: Xamarin.iOS  -> libMicrosoft.CognitiveServices.Speech.core
How should this be handled?
Thanks！

@Eddiegooo Thanks for the report. We provide bitcode enabled version of Xamarin.iOS also https://www.nuget.org/packages/Microsoft.CognitiveServices.Speech.Xamarin.iOS
Could you please try with that ?

@jhakulin I tried, still no effect.   Still reporting the original error.
libMicrosoft.CognitiveServices.Speech.core.a(speechapi_c_audio_config.cpp.o)' does not contain bitcode. You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target. for architecture arm64 clang: error: linker command failed with exit code 1 (use -v to see invocation)

@Eddiegooo Thanks for the report. We provide bitcode enabled version of Xamarin.iOS also https://www.nuget.org/packages/Microsoft.CognitiveServices.Speech.Xamarin.iOS Could you please try with that ?

i had tried last week,  using 1.22.0 version, it does not work!

libMicrosoft.CognitiveServices.Speech.core.a.zip
Check if this is the latest SDK. 1.22.0version.    i used this. @jhakulin
I tried, still no effect. Still reporting that error.
libMicrosoft.CognitiveServices.Speech.core.a(speechapi_c_audio_config.cpp.o)' does not contain bitcode. You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target. for architecture arm64 clang: error: linker command failed with exit code 1 (use -v to see invocation)

@Eddiegooo Sorry about that, we will investigate and let you know.

@Eddiegooo Sorry about that, we will investigate and let you know.

OK, Thank u, waiting for your good news!

OK, Thank u, waiting for your news!     or send me a compressed package that you test normally on iOS system, and I will try to test it again @jhakulin

i download SDK from https://www.nuget.org/packages/Microsoft.CognitiveServices.Speech.Xamarin.iOS.
Integrating it into the iOS(Objc language) project will report an error: does not contain bitcode.

@Eddiegooo Could you please provide sample project, e.g. as zip as attachment here ?

TestSpeech.zip
This is my Demo. The compilation error is as follows:

@Eddiegooo Thanks for the sample! I can see that libMicrosoft.CognitiveServices.Speech.core.a binary in your project does not contain bitcode. You can check if the binary contains bitcode by following command in terminal: otool -l yourlib.a | grep bitcode.
if I use the libMicrosoft.CognitiveServices.Speech.core.a binary from the package linked below, the bitcode issue is resolved.
https://www.nuget.org/packages/Microsoft.CognitiveServices.Speech.Xamarin.iOS
In addition to build your project successfully, you need to add AudioToolbox.framework to "Link Binary With Libraries" configuration in your project as that is required dependency.

I add AudioToolbox.framework to "Link Binary With Libraries" and switch Enable Bitcode to YES, it still reporting that error of *** does not contain bitcode.
if switch Enable Bitcode to NO.   then compilation succeeded~

@Eddiegooo I would like to add that, I was able to reproduce the same bitcode issue directly with your sample. The problem with your sample is that libMicrosoft.CognitiveServices.Speech.core.a binary bundled inside the sample, did not indeed have bitcode. This can be possible if you have taken that binary from regular https://www.nuget.org/packages/Microsoft.CognitiveServices.Speech NuGet package, however if you take the iOS binary from https://www.nuget.org/packages/Microsoft.CognitiveServices.Speech.Xamarin.iOS, it will contain bitcode and project can be built successfully when bitcode is enabled.


Duplicate node error- singularity 20.2.0

What version of fMRIPrep are you using?
20.2.0
What kind of installation are you using? Containers (Singularity, Docker), or "bare-metal"?

Yes
Did fMRIPrep generate the visual report for this particular subject? If yes, could you share it?
No

Can you find some traces of the error reported in the visual report (at the bottom) or in crashfiles?
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/workflows.py", line 766, in _check_nodes
    raise IOError('Duplicate node name "%s" found.' % node.name)
OSError: Duplicate node name "func_preproc_task_srDoors_run_01_wf" found.

I tried switching scratch folders and killing all user processes. also looked for hidden files
Are you reusing previously computed results (e.g., FreeSurfer, Anatomical derivatives, work directory of previous run)?
No
fMRIPrep log
If you have access to the output logged by fMRIPrep, please make sure to attach it as a text file to this issue.

Can you list the directory contents for that subject.

Thank you for the quick responses this week! I killed and restarted my processes and that looks to have fixed it.


Add lodging found on TripIt

This PR adds support for fetching lodging activities in TripIt.
It saves them as a purchase of type "hotel".
It also adds versioning to the tripit integration to make sure we fetch historical lodging.
Finally, I had to also make sure the hotel model runs only on the right activities.

I'll review this today, just need to setup TripIt and import some lodgings :)

I'd be happy to give one of you lot access to my TripIt account for a bit? I've got months worth of hotels, trains, ferries, you name it.

@madsnedergaard I don't think you need to test it yourself (I've done it)
However, would be great if you could look at the code to check for edge cases etc..

Sorry @madsnedergaard , I might have a merged a bit too quickly (I saw the approved message for the other PR and mistakenly thought it was for this one)

Haha no worries, only had the .0 that I think could be left out in future, but it's fine for now :)


[BUG] Roblox Failed To Launch, Brings Up Crash Page, Then Launches Fine [REOPEN REQUEST]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.8.1
What problem did you encounter?
I wanna say that I just faced the same error once again, cause I had discord open.
Since I don't have the permission to reopen the issue I made this to bring attention to this.

Not everyone has a high-end gaming pc, I was just trying to help so you guys can make it low-end pc friendly.
I'm reopening the issue now to show you this. Roblox opened normally but showed me this same error popup.
I told you a process doesn't immediately become a valid process for everyone, let alone start logging.
This is proof that a process doesn't immediately become valid for every computer. Especially work computers.
Where my computer was originally a work computer, even with upgrades it's still not powerful enough to have discord open and run bloxstrap.
And as we all know, a lot of roblox players don't have the luxury of using a high end gaming pc.
If you think otherwise, I would like you to join a random game and if you don't see a random bacon hair having a seizure from lag then I stand corrected.
Bloxstrap Log
N/A

I get the exact same error, although I have nothing open when I opened Bloxstrap.

Just so I can catch it again, how long does Roblox typically take for you to launch? When using the bootstrapper, I mean from the time you click the green Play button to when the game client window shows.

Just so I can catch it again, how long does Roblox typically take for you to launch? When using the stock bootstrapper, I mean from the time you click the green Play button to when the game client window shows.
Also, excuse my technicality, you initially described it as "take upto 2mins for robloxplayerbeta.exe to become a detectable process" - I thought you were saying that it takes up to 2 minutes for the process to be registered, which doesn't make any sense at all. What I now realize you were trying to say is that the process may be suspended for a brief period of time when it starts which, yes, is true.
Writing an entire paragraph seriously accusing me of not knowing that poor performance PCs exist is pretty funny though won't lie

Didn't mean to offend you when saying that, if roblox doesn't start logging within 2-5 mins that's cause there's been an error. Yes, processes can sometimes be suspended for 5 mins on laggier devices. But 2 mins should be enough here.
And to answer your question, from clicking play button to the window being shown, for me personally usually takes 10 seconds to a minute, usually the time increases when I have programs open.
Before when my pc had about 1-2 gigs of ram (I don't actually remember) it would take wayy longer. Ig around 2 mins.
I hope that answers your question.

@pizzaboxer Hello good sir, when will the issue be fixed? I got it multiple times the past week, forget poor performance pcs it does this on even decent devices.

Unclear, I've got a lot of work to do. Sorry to keep you waiting.

Unclear, I've got a lot of work to do. Sorry to keep you waiting.

Unclear? What do you mean what info do you need?

Client issues are no longer accepted.

Client issues are no longer accepted.

 You just closed my issue after we all have been waiting so long for this issue to be fixed. I'm getting the error at least 5 times a day now. Yk what bloxstrap is just unusable to me at this point I'm gonna stop using it.

Client issues are no longer accepted.

Okay, so you seriously just closed the issue for a simple fix like this?

Sorry, I didn't mean to close this issue.

Good stuff. Hoping for the issue to be fixed soon.


TestPurgeFile fail in semaphore CI

--- FAIL: TestPurgeFile (0.16s)
	purge_test.go:56: filenames = [5.test 6.test 7.test 8.test 9.test], want [7.test 8.test 9.test]

Go 1.5.1
latest master
I think it needs more time, but i am not sure how much it needs.

@yichengq is this still an issue?

yes. I met it several times.


feat: add storage for creator reward recipients

Current dependencies on/for this PR:

main

PR #267

PR #268   👈

This comment was auto-generated by Graphite.


chore(master): release 0.18.0-pre.1

:robot: I have created a release beep boop
0.18.0-pre.1 (2022-12-21)
Features

update bee-dashboard to latest (#309) (392ad56)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ethersphere/swarm-desktop/releases/tag/untagged-2afd2661ce4df2544691 :sunflower:


Improve djadmin filtering and search

Some useful filters and searches to the djadmin to help with generic troubleshooting.

Nice!


integration tests publish all methods

Fixes #1190  .
Changes proposed in this PR:

publish pool/ fixed priced/ dispenser assets
add validate method aqua
validate ddo for each published asset

added metadata proofs for publishing  (see https://github.com/oceanprotocol/multi-repo-issue/issues/128)


Fix metadata load performance

Codecov Report

Merging #13235 (1da5075) into master (46d5e18) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master   #13235      +/-   ##
============================================
- Coverage     43.95%   43.89%   -0.06%
+ Complexity     8951     8942       -9
============================================
  Files          1359     1359
  Lines         78112    78234     +122
  Branches       9496     9504       +8
============================================
+ Hits          34332    34339       +7
- Misses        40875    40983     +108
- Partials       2905     2912       +7

Impacted Files
Coverage Δ
Complexity Δ

...main/java/alluxio/master/file/InodeSyncStream.java
71.33% <100.00%> (-0.42%)
79.00 <0.00> (ø)

...mmon/src/main/java/alluxio/underfs/UfsManager.java
68.18% <0.00%> (-13.64%)
0.00% <0.00%> (ø%)

...on/src/main/java/alluxio/collections/LockPool.java
74.03% <0.00%> (-6.74%)
15.00% <0.00%> (-1.00%)

...thentication/AuthenticatedChannelServerDriver.java
87.27% <0.00%> (-3.64%)
11.00% <0.00%> (-1.00%)

...lluxio/client/file/cache/store/LocalPageStore.java
76.82% <0.00%> (-2.44%)
19.00% <0.00%> (-1.00%)

...c/main/java/alluxio/proxy/AlluxioProxyProcess.java
62.79% <0.00%> (-2.33%)
8.00% <0.00%> (-1.00%)

.../alluxio/worker/block/io/LocalFileBlockReader.java
63.33% <0.00%> (-2.30%)
8.00% <0.00%> (-1.00%)

.../main/java/alluxio/underfs/AbstractUfsManager.java
61.36% <0.00%> (-2.28%)
12.00% <0.00%> (-1.00%)

...ava/alluxio/client/block/stream/BlockInStream.java
56.25% <0.00%> (-1.95%)
36.00% <0.00%> (+1.00%)
:arrow_down:

...main/java/alluxio/concurrent/jsr/ForkJoinPool.java
29.80% <0.00%> (-1.85%)
58.00% <0.00%> (-3.00%)

... and 27 more

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 46d5e18...1da5075. Read the comment docs.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@46d5e18). Click here to learn what that means.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master   #13235   +/-   ##
=========================================
  Coverage          ?   17.36%
  Complexity        ?     2630
=========================================
  Files             ?     1359
  Lines             ?    78251
  Branches          ?     9504
=========================================
  Hits              ?    13588
  Misses            ?    63461
  Partials          ?     1202

Impacted Files
Coverage Δ
Complexity Δ

...main/java/alluxio/master/file/InodeSyncStream.java
8.65% <0.00%> (ø)
3.00 <0.00> (?)

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 46d5e18...4214ef5. Read the comment docs.

helps with #13259

@gpang @ZacBlanco PTAL, addressed some comments.

I think this works, but could you add more information to the PR description, as to what was the undesired behavior, and how this PR fixes it?

One request to change the title of the PR to be a little more descriptive. I propose:

Prevent loading child inode metadata when not required

alluxio-bot, cherry-pick this to branch-2.5 please

alluxio-bot, cherry-pick this to branch-2.5 please


[Enhancement] Extraction of constants [example module grpc http directory]

Search before asking

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

Enhancement Request

Describe the solution you'd like
Located at:eventmesh-examples/src/main/java/org/apache/eventmesh/grpc and http.demo
Extract these constants from the image and put them in the ExampleConstants class in the common directory, and change all the classes that use these constants.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

i will try my best to do it，please assign to me

Hi, can you assign this issue to me? I would like to start work on it as it would be my first-time contribute.

I am interested to solve this issue, so can you please assign me ?

I would like to contribute. Can you please assign me?

I am Interested, Can you please assign to me?

I would like to work on this and I'm a beginner, please assign to me


KeyError: 'RANK'

when I run tools/test_recognizer_heavy.py file, I get the following error:
Traceback (most recent call last):
  File "tools/test_recognizer_heavy.py", line 211, in <module>
    main()
  File "tools/test_recognizer_heavy.py", line 170, in main
    init_dist(args.launcher, **cfg.dist_params)
  File "/media/liuxy/dataset/mmaction/mmaction/apis/env.py", line 16, in init_dist
    _init_dist_pytorch(backend, **kwargs)
  File "/media/liuxy/dataset/mmaction/mmaction/apis/env.py", line 27, in _init_dist_pytorch
    rank = int(os.environ['RANK'])
  File "/usr/local/src/python37/lib/python3.7/os.py", line 678, in __getitem__
    raise KeyError(key) from None
KeyError: 'RANK'

Can anyone give me some idea to solve this? Thanks!

i have the same question ,how to solve it?


Add name to childApp instance and getName method

With this, now childApp instances will have their name attached to instance itself. A appName can be retrieved by using the new getName method. If an app is a childApp then the appName will be returned. However if the app is not a childApp or is a parentApp then undefined will be returned. This commit includes src code, tests and docs. This resolves  #103

Why deleting the name on remove?

Well currently a 'parentApp' isn't being named; with this new functionality only childApps will be named. So the only relevance of naming is between a 'parent' and a childApp. Once a childApp has been removed from the parent, now that app is an autonomous app, like the original parent app.
What is your use-case for keeping the name around?

No use case, just wondering. I dont even have use case of removing a child app, only destroy.

@JSteunou Ok, cool...yea it just make since that if we are removing a childApp from it's parent to remove the name as well. This just keeps things consistent.

:+1:


Remove Not Collected and Not provided filter for Tumor type in filter box

Description
Currently, values for Tumor type filter contain Not Collected and Not provided values.

We agreed this change is more appropriate to be done in the ETL instead of the UI.
The proposed solution is to modify the search_facet transformation to exclude Not Collected/Not Provided from the values in Tumor type.
Acceptance criteria

[x] Filter does not include Not Collected/Not Provided.

this needs to be done at the level of ETL - needs discussion

@zperova done


internal processing hardware D435

Hello,

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win (10)

Platform
PC

SDK Version
legacy / 2.12.0

Language
opencv/python

Issue Description
Is the internal processing hardware of the cameraD435 (D4 ASIC) exclusively devoted to provide the best images and depth precision? Is there any ways to do some processing in the camera as an embedded system or all the high-level processing must be performed inevitably on the host computer ?
Thank you in advance !

The Vision Processor D4 is designed to perform processes such as rectification to reduce the amount of calculations that the computing host-processor has to do.  The results of those calculations are then output over USB.  Some form of computing board should be required though to do something useful with that data output.
You would not necessarily have to connect the camera to a full PC.  System integrators could incorporate small 'single board' computers such as the Up Board and connect the camera to that.  Or if computing power closer to a desktop PC is needed in a small form-factor, an Intel NUC mini PC could be used.
If you would like to explore this possibility, bundle deals for the 400 Series cameras and Up Board or NUC are both available in the RealSense section of Intel's online click store.
https://click.intel.com/realsense.html
Here's a specification sheet for the D4.
https://www.mouser.com/pdfdocs/Intel_Vision_Processor_D4_ProductBrief.pdf

This totally answer my question, Thank you!!!


Add simplifier support for bisimilarity reasoning

Currently, when reasoning with bisimilarities, we need to apply core bisimilarities manually. This leads to long and hard-to-create proofs. Especially the need for manual application of compatibility (preservation) laws and associativity of parallel composition constitutes a hardship.
Our goal is to employ Isabelle’s simplifier to automate the creation of such boring proof steps. A challenge is that the simplifier seems to work only for equational reasoning by default. We want to make it work for bisimilarity reasoning as well.

[x] Research how to best use the simplifier for reasoning with arbitrary congruences
[ ] Implement the desired simplifier support

Our new, more ambitious, goal is to implement simplifier support for reasoning not only with bisimilarities but with arbitrary equivalence relations.


fix css compile, bring default ::selection effect back

What did
Make the esbuild-plugin-postcss3 support replace a CSS file with another when building, works the same as webpack NormalModuleReplacementPlugin.
Related PR: https://github.com/baurine/esbuild-plugin-postcss3/pull/6
Preview
Before:

After:

How it works

You are my hero!

I'm not sure.. But seems that it is not working in my environment (dev):


router: support prefix_rewrite & strip_query for redirects

title: support prefix_rewrite & strip_query for redirects
Description:
adding support to create a redirect message by rewriting
the matched prefix or path with a new path string.
adding support to remove the query portion of the
request for redirects.
the features are enabled by configuring prefix_rewrite & strip_query
fields under redirect section.
Risk Level: Medium
Testing:
new unit tests added for both.
Docs Changes:
envoyproxy/data-plane-api#456
Release Notes: Included in the PR
Fixes #2343
API Changes envoyproxy/data-plane-api#456
Signed-off-by: Madhavan Balasubramanian<EMAIL_ADDRESS>

@zuercher Updated the PR based on your comments PTAL. Thanks

You'll need to merge master to resolve the release notes conflict.

Thanks @zuercher . I will rebase the PR with master now. Do you prefer squashing the commits or leave it as is ? Thanks.

You should just merge master.

@bmadhavan I was too slow, looks like a merge conflict crept in.


Refactor sidebar HTML

When including the 'actions' sidebar using content_for :sidebar HTML boilerplate needs to be included. Refactor into partial.

i think this is obsolete as of https://github.com/hpi-swt2/wimi-portal/pull/580


Explore options for automatically creating a table of events to the readme that have been added

It's hard to tell exactly what events have been added  to this package -- we can explore using GitHub actions or something to automatically generate a markdown table that clearly shows what's been added and when.

Briefly looked into how we can do this today, seems like the dart:mirrors package could be useful for extracting a list of the named constructors for the Event class.. from there we could use this list to publish some kind of document into the github repo via github actions.

I've added this as an app script to this sheet, which runs each morning: and grabs the fields in the enums and event files on main. This way there's an easy way to view all the fields that have been added as a table.
https://docs.google.com/spreadsheets/d/12fUrmwmShlWYeBGl0_TEOt_FVTB1KTxru0mPOT1qFA8/edit#gid=76461167


Update metaconfig-typesafe-config to 0.9.16

Updates com.geirsson:metaconfig-typesafe-config from 0.9.15 to 0.9.16.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.geirsson", artifactId = "metaconfig-typesafe-config" } ]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Not sure we actually use that dependency
Look at Fmt.ScalaFmtVersionConfig

Bumped in https://github.com/VirtusLab/scala-cli/pull/756.


Add APIs and API Products

Proposed changes:

Restructure root
Add Open AI deployments
Add APIM API definitions for models
Add APIM API product
Add APIM API logging

Successfully deployed e2e:


Tag 0.4.0 release of specification

We plan to release the SDK - https://github.com/servicemeshinterface/smi-sdk-go/issues/81

dep - https://github.com/servicemeshinterface/smi-sdk-go/issues/81

We're currently at the working draft for v0.4.0 of the spec which includes

TrafficSplit v1alpha3
TrafficTarget v1alpha1
TrafficSpecs v1alpha2
TrafficMetrics v1alpha1

We have not yet defined a release process for the spec. I think we should open a PR that bumps the version in SPEC.md to v0.4.0 and points to the API versions listed above. Then, cut a release branch tagged v0.4.0. We should also update the readme to point to the SPEC.md file in that branch.

If we go this route, we also probably need to create a spec-v0.4.0.md file or spec-latest-stable.md or something along those lines so we can still have a working-draft file for the spec.

Release plan:

Create a PR with following changes:

Create a file called spec-v0.4.0.md
Rename SPEC.md to spec-working-draft.md and update version in the file to v0.5.0-WD
Update README.md to point to spec-v.0.4.0.md and add a table in README like the one in the CloudEvents spec which points to the latest release of each doc as well the current working draft.

Merge PR.
Create a tag v0.4.0
Cut a GitHub release v0.4.0


logger.info() messages should not be sent to stderr

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?

No
Description

logger.info() statements from ngcc are sent to stderr causing the informational messages to be displayed (in red) as ERRORs under a maven build. Please see the rejected PR at https://github.com/angular/angular/pull/35310 which documents the incorrect behavior. The ngcc team believes this should be fixed in the cli.
🔬 Minimal Reproduction

In maven's pom.xml:

  ...
  <execution>
    <id>npm angular build</id>
      <goals>
        <goal>npm</goal>
      </goals>
      <configuration>
        <arguments>run build-dev</arguments>
      </configuration>
  </execution>
  ...

In package.json:

    "scripts": {
        ...
        "build-dev": "ng build",
        ...
    },

🔥 Exception or Error
Output from maven build:

[INFO] > ng build
[INFO]
[ERROR]
[ERROR] Compiling @angular/core : module as esm5
[ERROR]
[ERROR] Compiling @angular/common : module as esm5
[ERROR]
[ERROR] Compiling @angular/platform-browser : module as esm5
[ERROR]
[ERROR] Compiling @angular/platform-browser-dynamic : module as esm5
[ERROR]
...

🌍 Your Environment

Angular CLI: 9.0.1
Node: 12.13.1
OS: darwin x64

Angular: 9.0.0
... animations, cdk, common, compiler, compiler-cli, core, forms
... language-service, material, platform-browser
... platform-browser-dynamic, router, upgrade
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.900.1
@angular-devkit/build-angular     0.900.1
@angular-devkit/build-optimizer   0.900.1
@angular-devkit/build-webpack     0.900.1
@angular-devkit/core              9.0.1
@angular-devkit/schematics        9.0.1
@angular/cli                      9.0.1
@angular/flex-layout              9.0.0-beta.29
@ngtools/webpack                  9.0.1
@schematics/angular               9.0.1
@schematics/update                0.900.1
rxjs                              6.5.4
typescript                        3.7.5
webpack                           4.41.2

Anything else relevant?

Progress, diagnostic and status messages should be written to stderr. stdout should only be used for logs intended to be piped to another command.
//cc @clydin as he definitely has more stronger arguments why it should be stderr.

Most likely we should offer a way to silence these, maybe hide this logs when using the no-progress or only show them in verbose mode.
Related to https://github.com/angular/angular-cli/issues/16899

Is frontend-maven-plugin being used in that maven pom?  If so, than version 1.8.0+ (changelog & related issue) appears to contain a fix so that it does not assume that anything written to stderr is an error.

@clydin - Thank you! Upgrading frontend-maven-plugin from 1.7.6 to 1.8.0 resolves the issue:
[INFO] --- frontend-maven-plugin:1.8.0:npm (npm angular build) @ newton-frontend ---
[INFO] Running 'npm run build-dev' in /Users/kolson/dev/Newton/webapps/newton-frontend
[INFO]
[INFO] ><EMAIL_ADDRESS>build-dev /Users/kolson/dev/Newton/webapps/newton-frontend
[INFO] > ng build
[INFO]
[INFO]
[INFO] Compiling @angular/core : module as esm5
[INFO]
[INFO] Compiling @angular/common : module as esm5
[INFO]
[INFO] Compiling @angular/platform-browser : module as esm5
[INFO]
[INFO] Compiling @angular/platform-browser-dynamic : module as esm5
[INFO]
[INFO] Compiling @angular/upgrade/static : module as esm5
...

Good to hear.  Since the issue appears to be resolved, I'm going to close.


Could not find com.facebook.react:react-android

What went wrong:
Could not determine the dependencies of task ':maplibre_maplibre-react-native:compileDebugAidl'.

Could not resolve all task dependencies for configuration ':maplibre_maplibre-react-native:debugCompileClasspath'.
Could not find com.facebook.react:react-android:.
Required by:
project :maplibre_maplibre-react-native

The version that i installed
   "@maplibre/maplibre-react-native": "9.1.0",
React native version 0.69.9
I went into this problem, anyone knows how to solve it? If so i would be appreciate.

i was able to install this library when i updated my react native project to 0.73.3, i guess react native projects below 0.70.0 are not suitable to install this lib. I'll close this issue. Thank you


Allow Generated Mocks to Accept Other Mock Interfaces as Input to Avoid Cyclic Imports

Requested feature
Consider two interfaces, InterfaceA and InterfaceB. InterfaceA has a function that takes an object of InterfaceB as input. I want to generate MockInterfaceA, with the function accepting MockInterfaceB as input instead of InterfaceB.
Why the feature is needed
If the generated mock files are placed in a separate folder, and the mocked interface is used in a test file located in the main package, it will result in a cyclic import: main package -> mock package -> main package.

Here is a simple example why this does not work:
example.zip
TL;DR:
// compilation error
// Type HumanMock does not implement Human
// need the method: Call(dog Dog)
// have the method: Call(_ DogMock)
var human Human = mock.HumanMock{}

Possible solutions to the problem:

Generate mocks in the same directory as interfaces.
Use the _test postfix in tests. Then the dependency graph will look like this:

flowchart LR

some_package
mock --> some_package
some_package_test --> some_package
some_package_test --> mock


rotating schematic retains original orientation for rails

When you rotate a schematic, the original orientation for certain blocks (most notably rails) is retained. The blocks reorient as they're placed, but are then highlighted as incorrectly placed.

This is for version <IP_ADDRESS> (1.11).

The mapping is most likely outdated, have to check if it still happens in 1.12.x. Won't be fixing it in 1.11.x or earlier though.

Yes this happens in 1.12.2.


Support Templating

Personally, I am not a fan of how Dictu handles formatting strings. I think the addition of a templating system similar to Python's f-Strings or JSX's expression embedding system would be beneficial to the language:
var name = "Lambo";
var job = "Developer";

print("My name is {name} and I'm a {job}."); // My name is Lambo and I'm a Developer.

The ability to perform logic inside these templates (like JavaScript's template literals) would be a bonus:
print("I am {10 + 7} years old."); // I am 17 years old.

I can see the benefit of more seamless string interpolation, however from a parsing perspective allowing expressions inside strings adds a lot of complexity for very little benefit. That being said, simple symbol resolution wouldn't be too costly.


Audio stops playing after stopping and restarting editor a number of times

Using the latest commit 693233d and UE 4.14 Preview 2, if I play in editor and stop multiple times, eventually the audio will stop playing when the video starts. When this happens, the condition is permanent until the editor is restarted.
I have been able to reproduce this bug consistently. I had to play and stop anywhere from 2 times to 10 times to reproduce.
I also verified that another application was able to play the audio from the same NDI stream while the UE editor was in the failure condition, so the problem is isolated to the plugin.
Reloading the editor unloads and reloads the plugin, which alleviates the problem.

A little update. I noticed the UE4Editor.exe stays running indefinitely in the Background Processes section of Task Manager in Windows 10 after this issue occurs, even after you close it and it disappears from the desktop. My theory is the thread locking is not allowing the program to terminate fully.

Are you able to catch this in the Visual Studio debugger? Would be interesting to know which thread is getting stuck and where. I'll try to repro as well.

I ran it with Visual Studio debugger attached. I am not 100% positive, but it looks like this is the culprit:
FNdiMediaAudioSampler::~FNdiMediaAudioSampler()
{
Thread->Kill(true); // <----- this is stuck
delete Thread;
Thread = nullptr;
}

Can you try the latest plug-in code? I wasn't able to reproduce, but Newtek fixed an issue with their thread synchronization.

By the way, how are you able to play NDI audio? 4.14 does not have support for quadrophonic audio. That's something I added for 4.15. Are you using UE4 Github Master?

Oh nevermind, I guess you're using your own test content. I was using the NDI Test Patterns app, which only outputs quadrophonic.

Yep, you're right about that. I was streaming my own content which has stereo audio. I am working on testing the new drop. So far, I've been unable to update an existing project as I got an error after recompiling and trying to load the editor.
"PLugin 'NdiMedia' failed to load because module 'NdiMediaEditor' could not be loaded. There may be an operating system error or the module may not be properly set up."
Then I get a crash and stack trace.
--- BEGIN COPY & PASTE ---
Assertion failed: GIsHotReload [File:D:\Build++UE4+Release-4.14+Compile\Sync\Engine\Source\Runtime\CoreUObject\Private\UObject\UObjectBase.cpp] [Line: 579]
Trying to recreate class 'UNdiMediaSettings' outside of hot reload!
UE4Editor_CoreUObject!UObject::ConditionalBeginDestroy() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\coreuobject\private\uobject\obj.cpp:750]
UE4Editor_CoreUObject!StaticExit() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\coreuobject\private\uobject\obj.cpp:3887]
UE4Editor_CoreUObject!TBaseStaticDelegateInstance<void __cdecl(void)>::ExecuteIfSafe() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\core\public\delegates\delegateinstancesimpl.h:1018]
UE4Editor!TBaseMulticastDelegate::Broadcast() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\core\public\delegates\delegatesignatureimpl.inl:922]
UE4Editor!FEngineLoop::AppPreExit() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\launch\private\launchengineloop.cpp:3442]
UE4Editor!FEngineLoop::Exit() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\launch\private\launchengineloop.cpp:2488]
UE4Editor!GuardedMain() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\launch\private\launch.cpp:163]
UE4Editor!GuardedMainWrapper() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\launch\private\windows\launchwindows.cpp:126]
UE4Editor!WinMain() [d:\build++ue4+release-4.14+compile\sync\engine\source\runtime\launch\private\windows\launchwindows.cpp:202]
UE4Editor!__scrt_common_main_seh() [f:\dd\vctools\crt\vcstartup\src\startup\exe_common.inl:264]
kernel32
ntdll
--- END COPY & PASTE ---
I did a cursory search and it looks like this can happen when API interfaces change that UE4 blueprints rely on. However, I'm still trying to see if I can get this to work. My next step if I can't get it to upgrade in my existing project is to try a clean project.

I just upgraded my test project and tried it out. I got the editor to hang on the 2nd or 3rd play/stop cycle in PIE mode. I'll see if this happens in a clean project.

Can you explain how to repro this? Do you open the NDI Media Source in your level blueprint's BeginPlay event, and then you simply play/stop PIE several times in a row?

I do it just the way you describe sometimes, or I setup hotkeys using input mapping and Open/Close the NdiMediaPlayer multiple times.

I reproduced the editor hang using commit #1805260 in a fresh UE4 4.14 Preview 3 project.
It's happening pretty reliably after 3 or so play/stop cycles as I described above.
I am opening a source on Begin Play and not closing it on End Play, although I can try closing it as well (I've tried both with previous builds--it did not seem to make a difference).

I still haven't been able to repro, but I suspect that it's because NDIlib_recv_destroy is called while NDIlib_recv_capture is in a waiting state. Newtek is investigating now.

Ok, so I am running the latest commit, #1805260 in UE4 4.14 Preview 3. I switched to DebugGame mode and ran the editor with Visual Studio's debugger attached. I'm opening a source on Begin Play and closing it on End Play.
It took me like 7 tries, but I got the editor to freeze. Then I broke into the debugger, and here's what immediately came up in WindowsPlatformProcess.cpp:
1053: bool FEventWin::Wait(uint32 WaitTime, const bool bIgnoreThreadIdleStats /= false/)
1054: {
1055: 	WaitForStats();
1056:
1057:	SCOPE_CYCLE_COUNTER( STAT_EventWait );
1058: 	check( Event );
1059:
1060:	FThreadIdleStats::FScopeIdle Scope( bIgnoreThreadIdleStats );
1061: 	return (WaitForSingleObject( Event, WaitTime ) == WAIT_OBJECT_0);  // <------ Worker thread "PoolThread 0" is waiting here indefinitely, causing the editor to stop responding.
1062: }
The crash reporting thread in UE4Editor-Core.dll's next statement to execute is in linee 390 of WindowsPlatformCrashContext.cpp:
385: /** Main loop that waits for a crash to trigger the report generation */
386: FORCENOINLINE uint32 Run()
387: {
388:	while (StopTaskCounter.GetValue() == 0)
389:	{
390:		if (WaitForSingleObject(CrashEvent, 500) == WAIT_OBJECT_0)
391:		{
392:			ResetEvent(CrashHandledEvent);
393:			HandleCrashInternal();
394:			ResetEvent(CrashEvent);
395:			// Let the thread that crashed know we're done.
396:			SetEvent(CrashHandledEvent);
397:			break;
398:		}
399:	}
400:	return 0;
401: }
MainThrd's next statement to execute is line 36 of ScopeLock.h:
26: /**
27: * Constructor that performs a lock on the synchronization object
28: *
29: * @param InSynchObject The synchronization object to manage
30: /
31: FScopeLock( FCriticalSection InSynchObject )
32:		: SynchObject(InSynchObject)
33: {
34: 	check(SynchObject);
35:	SynchObject->Lock();
36: }

Repeated the same test as above but without closing the Media Player on End Play. Same results.
I'm going to try and step through the code that runs after End Play sequentially to see where exactly this chain of events results in a deadlock.
I may need symbols for either XAudio or the NDI SDK to see what's going on on a deeper level. I know there is an XAudio debug version, but do you know if NDI's SDK comes with symbols? I uninstalled it when you included the libraries as part of the plugin.

There are no debug symbols in the SDK, unfortunately.
Can you copy/paste the callstack of both the Main thread and the Processing.NDI.Lib.x64 thread?

Here is MainThrd's call stack:
[External Code]

UE4Editor-NdiMedia.dll!FNdiMediaAudioSampler::SetReceiverInstance(void * InReceiverInstance) Line 32	C++
UE4Editor-NdiMedia.dll!FNdiMediaPlayer::Close() Line 212	C++
UE4Editor-MediaPlayerEditor.dll!FMediaPlayerEditorModule::HandleEditorEndPIE(bool bIsSimulating) Line 218	C++
UE4Editor-MediaPlayerEditor.dll!TBaseRawMethodDelegateInstance<0,FMediaPlayerEditorModule,void cdecl(bool)>::ExecuteIfSafe(bool <Params_0>) Line 639	C++
UE4Editor-UnrealEd.dll!TBaseMulticastDelegate<void,bool const >::Broadcast(const bool <Params_0>) Line 922	C++
UE4Editor-UnrealEd.dll!UEditorEngine::EndPlayMap() Line 218	C++
UE4Editor-UnrealEd.dll!UEditorEngine::Tick(float DeltaSeconds, bool bIdleMode) Line 1670	C++
UE4Editor-UnrealEd.dll!UUnrealEdEngine::Tick(float DeltaSeconds, bool bIdleMode) Line 371	C++
UE4Editor.exe!FEngineLoop::Tick() Line 2859	C++
UE4Editor.exe!GuardedMain(const wchar_t * CmdLine, HINSTANCE * hInInstance, HINSTANCE__ * hPrevInstance, int nCmdShow) Line 152	C++
UE4Editor.exe!WinMain(HINSTANCE__ * hInInstance, HINSTANCE__ * hPrevInstance, char * __formal, int nCmdShow) Line 191	C++
[External Code]

There are three Processing.NDI.Lib.x64.dll threads.
http://i.imgur.com/PttJKP2.png
Call stacks are as follows:
15528:
[External Code]

Processing.NDI.Lib.x64.dll!00007ffa6f3299f9()	Unknown
Processing.NDI.Lib.x64.dll!00007ffa6f328919()	Unknown
[External Code]

6364:
[External Code]

Processing.NDI.Lib.x64.dll!00007ffa6f32a7b0()	Unknown
Processing.NDI.Lib.x64.dll!00007ffa6f328939()	Unknown
[External Code]

1088:
[External Code]

Processing.NDI.Lib.x64.dll!00007ffa6f32a173()	Unknown
Processing.NDI.Lib.x64.dll!00007ffa6f328929()	Unknown
[External Code]

Do you have the callstack of the "FNdiMediaAudioSampler" thread?

Please try the following change:

void FNdiMediaPlayer::Close()
{
{
FScopeLock Lock(&CriticalSection);

	if (ReceiverInstance != nullptr)
	{
		NDIlib_recv_destroy(ReceiverInstance);
		ReceiverInstance = nullptr;
	}

	CurrentState = EMediaState::Closed;
	CurrentUrl.Empty();

	LastAudioChannels = 0;
	LastAudioSampleRate = 0;
	LastVideoDimensions = FIntPoint::ZeroValue;
	LastVideoFrameRate = 0.0f;

	SelectedAudioTrack = INDEX_NONE;
	SelectedVideoTrack = INDEX_NONE;
}

UpdateAudioSampler();

MediaEvent.Broadcast(EMediaEvent::TracksChanged);
MediaEvent.Broadcast(EMediaEvent::MediaClosed);

}

Here is the callstack of the FNdiMediaAudioSampler thread:
[External Code]

UE4Editor-NdiMedia.dll!FNdiMediaPlayer::HandleAudioSamplerSample(const NDIlib_audio_frame_t & AudioFrame) Line 787	C++
UE4Editor-NdiMedia.dll!TBaseRawMethodDelegateInstance<0,FNdiMediaPlayer,void __cdecl(NDIlib_audio_frame_t const & __ptr64)>::ExecuteIfSafe(const NDIlib_audio_frame_t & <Params_0>) Line 639	C++
UE4Editor-NdiMedia.dll!FNdiMediaAudioSampler::Run() Line 49	C++
UE4Editor-Core.dll!FRunnableThreadWin::Run() Line 74	C++
UE4Editor-Core.dll!FRunnableThreadWin::GuardedRun() Line 23	C++
[External Code]

I will try your changes to FNdiMediaPlayer::Close() right now and report back.

I haven't been able to reproduce the issue after I made the change you suggested above. It's looking good so far!

Ok, the old code deadlocked when shutting down while audio samples were being returned.


Logo Design

Hello!
I'm a graphic designer and I like to support open source projects. I would like to design a logo for your project if you interested, I will be happy to work with you! :)
Best Regards
Baran Pirinçal
Graphic Designer

Thank you for the proposal. Replaykit is an affiliate project of Appetizer and it is not currently expecting an independent new logo.


Make it possible to set timeouts on slack URL comms

On very busy slack networks the default timeout of 20000 can be a
problem. This timeout can mean that the extension is unable to download
all the necesarry data before weechat times out the connection.
This patch allows for the timeout to be adjusted so that for larger
networks the end user can specify it.

This change is

👍 looks good once the globals are out.


v0.2.0 of Rum Goggles, Crashes after signing in to the Chat bot

Every time i Add my username and password and stream url clicked save and it Crashed.

Are you including "https://" at the beginning of your stream url? I got it to crash if I copied the url without "https://". I can work on fixing this on my stream tomorrow. Let me know if you're still experiencing the crash with "https://" included.

Are you including "https://" at the beginning of your stream url? I got it to crash if I copied the url without "https://". I can work on fixing this on my stream tomorrow. Let me know if you're still experiencing the crash with "https://" included.

I added https:// manually like its in my Muscle memory, but it still crashed after Pressing save.


[CI] Benchmark activity

Description
Please include a summary of the change and which is fixed.

Add BM cache key for whole 'meta' directory
Fix BM scores after review and adding new repo

How has this been tested?
Please describe the tests that you ran to verify your changes.

[x] Benchmark

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.53%. Comparing base (21a54c9) to head (f96cb2e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #520   +/-   ##
=======================================
  Coverage   90.53%   90.53%
=======================================
  Files         127      127
  Lines        4394     4394
  Branches      707      707
=======================================
  Hits         3978     3978
  Misses        274      274
  Partials      142      142

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Creates a sha512.cpp file for sha512's non-template functions.

I shall make a new pull request with the headers in the .cpp file.


Can not loadTinyFaceDetectorModel after build from react

Hello,
Thanks for the cool project !
It does quite work well with creat-react-app "dev mode" but doesn't work after build the sources.
Error log is such as below;
tf-core.esm.js:17 Uncaught (in promise) Error: Based on the provided shape, [1,1,32,64], the tensor should have 2048 values but has 807

The test code is here
https://github.com/UihyunKim/face-api-build-test
Am I miss something?

It seems the issue from the backend related thing.. The main reason is that tiny_face_detector_model-shard1 is not loaded from tf-core because I ran temporary server from serve -s build. It doesnt serve the xxxshard1 file appropriately.
When I deploy this app on netlify the xxxshard1 file is loaded and it's working.
I'll close this issue.


JSON is not copatible with v2.2 spec - "packages" and "files" lists should be outside "documentDescribes"

I see a major gap between the the v2.2 spec and the current python-tool implementation of the JSON format.
In https://github.com/Yash-Varshney/tools-python/issues/9 discussion with @Yash-Varshney, @goneall have processed to add spec issue around it.

Python-tool (presently)
Java-tool (v2.2)
proposed
Spec

[MAJOR CHANGE] documentDescribes contains Package and Files
documentDescribes is only a list and packages and files another separate entity
Add spec issue
documentDescribes should only be the top-level package/file that the  document describes, not the entire list - may need more research

Package is a dict
Package is a list in which there is dict ex. [{}]
Add spec issue
This is related to the documentDescribes difference

The v2.2 spec points to https://github.com/spdx/spdx-spec/blob/master/schemas/spdx-schema.json schema. In addition, all the other spdx tools implementations are aligned with the spec (java-tools, golang, etc).
I think that we should align the python-tool as well, to complete the v2.2 support (formally declared as WIP now).
Do you guys agree?
Do you have any plans/roadmap of closing the v2.2 support gap?
Thanks,
Yan

I am also having this issue. The json generated by tools-python does not validate against spdx-schema.json.


using pre-commit for pre commit hooks

Tested trying to commit something that breaks a test and worked as expected. this commit resolves #41

Today I learned that you put  # to relate a pull request to an issue


Syntax coloring indicates error on object in lower case even without double-quote [SNOWFLAKE]

Description
In Snowflake DB, name of DB objects (schema, tables, fields etc.) are by default interpreted as uppercased from the SQL Client, unless (and only unless) the name is explicitely enclosed by double quote.
For instance, the standard schema INFORMATION_SCHEMA and its standard view TABLES can be either reached out to via calling them INFORMATION_SCHEMA.TABLES, information_schema.tables, inForMatioN_SChema.TablEs etc. The SF engine will always look for INFORMATION_SCHEMA.TABLES.
Only if you use double-quotes, then only the case is taken into consideration : INFORMATION_SCHEMA <> "information_schema".
However, the newly added auto-coloring feat in DBeaver, indicating potential errors in syntax, takes the case in consideration even without double-quotes...

vs

DBeaver Version
CE Version <IP_ADDRESS>401072024
Operating System
Win11 23H2 (Build 22631.2715)
Database and driver
Snowflake via JDBC 3.13.29
Steps to reproduce
(see screenshots provided)
Additional context
No response

Thank you for bug report.

@ufuk-ergin-carbon is it Snowflake too?

@ufuk-ergin-carbon is it Snowflake too?

Hi @E1izabeth , yes, I am also using Snowflake.

We will investigate and fix it. As a temporary workaround if you don't want to see red highlighting, you can turn it off here.

We will investigate and fix it. As a temporary workaround if you don't want to see red highlighting, you can turn it off here.

What I did is this for the time being as a workaround: I went to the connection by right clicking, Edit Connection, then changed the Insert case under Code Completion to Default (previously set to lowercase).

[like]  Leillet, Thomas /FR/EXT reacted to your message:

From: Elizabeth @.>
Sent: Monday, January 8, 2024 12:36:08 PM
To: dbeaver/dbeaver @.>
Cc: Leillet, Thomas /FR/EXT @.>; Author @.>
Subject: Re: [dbeaver/dbeaver] Syntax coloring indicates error on object in lower case even without double-quote [SNOWFLAKE] (Issue #22368)
We will investigate and fix it. As a temporary workaround if you don't want to see red highlighting, you can turn it off here.
image.png (view on web)https://github.com/dbeaver/dbeaver/assets/28875055/bf80edcb-fc10-4e92-aac9-6bac815fc1b5
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/22368#issuecomment-1880925934, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7ECPLNFXAVVAYMLNVT2GS3YNPR3RAVCNFSM6AAAAABBRBOD26VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQHEZDKOJTGQ.
You are receiving this because you authored the thread.Message ID: @.***>


[Bug]: Using ChatPromptTemplate with context_str no longer works in 0.9 because of faulty _get_available_chunk_size logic

Bug Description
The behavior of ChatPromptTemplate in llama_index 0.9 has changed so that it no longer works with context_str. It fails in the function for checking available chunk size. In version 0.9 a commit was added (95e77ab9e1e61719e7e759523638cbb1b980417a) which does custom logic for this specific class, which I believe is faulty.
The specific code snippet that errors is in llama_index/indices/prompt_helper.py:
# get string variables (if any)
template_vars = [
    var
    for _, var, _, _ in Formatter().parse(str(message))
    if var is not None
]

# figure out which variables are partially formatted
used_vars = {}
for var_name, val in prompt.kwargs.items():
    if var_name in template_vars:
        used_vars[var_name] = val

# format partial message
if len(used_vars) > 0 and partial_message.content is not None:
    partial_message.content = partial_message.content.format(
        **used_vars
    )

I kept trying to understand what this part of code is trying to do, and according to the code and the comment I could only guess that it is trying to replace only parts of the prompt string. But this is not how the Python format method works.
For example:
>>> "{arg1} {arg2}".format(arg1=5)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
KeyError: 'arg2'

If this is truly what the code is supposed to do then I suggest adding a default value to used_vars like so:
# get string variables (if any)
template_vars = [
    var
    for _, var, _, _ in Formatter().parse(str(message))
    if var is not None
]

# figure out which variables are partially formatted
used_vars = {template_var: f"{{{template_var}}}" for template_var in template_vars}
for var_name, val in prompt.kwargs.items():
    if var_name in template_vars:
        used_vars[var_name] = val

# format partial message
if len(used_vars) > 0 and partial_message.content is not None:
    partial_message.content = partial_message.content.format(
        **used_vars
    )

This change would replace the fields that do not have values back to their unformatted selves.
But since this function is trying to estimate the length, then they could maybe be replaced with any dummy value that would be an estimate.
In any case, some default value has to be given to the missing fields so it doesn't error out. If you have a suggestion what this could be I can submit a PR.
Version
0.9
Steps to Reproduce
I tried to create a mock code that doesn't depend on anything else:
import os

from llama_index import ChatPromptTemplate, PromptTemplate, ServiceContext
from llama_index.llms.base import ChatMessage, MessageRole
from llama_index.llms.mock import MockLLM
from llama_index.response_synthesizers import get_response_synthesizer
from llama_index.schema import NodeWithScore, TextNode

os.environ["OPENAI_API_KEY"] = "..."

PROMPT_TEMPLATE_STR = """
SYSTEM
{context_str}
Query: {query_str}
Answer:
"""

CHAT_PROMPT_TEMPLATE = """
SYSTEM
__&&__
{context_str}
Query: {query_str}
Answer:
"""

prompt_template = PromptTemplate(PROMPT_TEMPLATE_STR)

system_user_delimiter = "__&&__"
system_prompt, user_prompt = CHAT_PROMPT_TEMPLATE.split(system_user_delimiter)
message_templates = [
    ChatMessage(content=system_prompt, role=MessageRole.SYSTEM),
    ChatMessage(content=user_prompt, role=MessageRole.USER),
]
chat_prompt_template = ChatPromptTemplate(message_templates=message_templates)

service_context = ServiceContext.from_defaults(llm=MockLLM(10))
mock_nodes = [NodeWithScore(node=TextNode(text="abc"))]

chat_prompt_synthesizer = get_response_synthesizer(
    service_context=service_context,
    text_qa_template=prompt_template,
)
prompt_response = chat_prompt_synthesizer.synthesize(query="...", nodes=mock_nodes)
print(prompt_response)
print("=" * 10)

chat_prompt_synthesizer = get_response_synthesizer(
    service_context=service_context, text_qa_template=chat_prompt_template
)
chat_prompt_response = chat_prompt_synthesizer.synthesize(query="...", nodes=mock_nodes)
print(chat_prompt_response)

In llama_index 0.8 it outputs:
text text text text text text text text text text
==========
text text text text text text text text text text

While in llama_index 0.9 it errors with (paths are redacted):
text text text text text text text text text text
==========
Traceback (most recent call last):
  File ".../scripts/bug_report_example.py", line 53, in <module>
    chat_prompt_response = chat_prompt_synthesizer.synthesize(query="...", nodes=mock_nodes)
  File ".../.venv/lib/python3.10/site-packages/llama_index/response_synthesizers/base.py", line 146, in synthesize
    response_str = self.get_response(
  File ".../.venv/lib/python3.10/site-packages/llama_index/response_synthesizers/compact_and_refine.py", line 38, in get_response
    return super().get_response(
  File ".../.venv/lib/python3.10/site-packages/llama_index/response_synthesizers/refine.py", line 127, in get_response
    response = self._give_response_single(
  File ".../.venv/lib/python3.10/site-packages/llama_index/response_synthesizers/refine.py", line 169, in _give_response_single
    text_chunks = self._service_context.prompt_helper.repack(
  File ".../.venv/lib/python3.10/site-packages/llama_index/indices/prompt_helper.py", line 270, in repack
    text_splitter = self.get_text_splitter_given_prompt(
  File ".../.venv/lib/python3.10/site-packages/llama_index/indices/prompt_helper.py", line 228, in get_text_splitter_given_prompt
    chunk_size = self._get_available_chunk_size(
  File ".../.venv/lib/python3.10/site-packages/llama_index/indices/prompt_helper.py", line 198, in _get_available_chunk_size
    partial_message.content = partial_message.content.format(
KeyError: 'context_str'

Relevant Logs/Tracbacks
No response

@martinkozle The original code is basically attempting to only call format() on the chat message that actually a) has the proper string variables and b) has the var available to format, in order to estimate length yea.
Your suggested fix seems good enough for now -- I will use that with some minor tweaks (thanks a ton btw for diagnosing)


Static Library Projects

Description
Allow users to define a new kind of ABS project, a static library.. Other projects may declare a dependency on the library (by way of a path or git URL). As a result, the following steps will happen when building a root project (e.g., an exe, dll, or static library):

All remote (just git, for now) dependencies will be copied into the abs directory

This is potentially recursive, since a downloaded project may have its own dependencies.
When to re-download previously downloaded remote dependencies? Should it happen automatically? Should there be an explicit command?

Copies of header files will be maintained in the following directory structure, somewhere inside the abs directory: include/{depender_project_name}/{dependee_project_name}

Then, source files in each depender project will have abs/include/{depender_project_name} added to their include path. Then they can write #include <dependee_project_name/header_name.h>
If multiple dependers depend on the same project, that projects headers will be duplicated for each depender. This prevents projects from being able to "see" the headers that other projects depend on.

They could still write #include <../other_depender_project_name/i_dont_explicitly_depend_on_this/header.h>

But I don't think it's that important to worry about this. If I change my mind, I can generate random strings instead of using the depender_project_name.

An acyclic graph of the dependencies between projects will be constructed

If it is not acylic, emit an error

A suitable compilation order will be determined from the graph in the previous step
Compilation for each project will proceed as normal.

Notes

C++ options must match exactly in all projects in the dependency graph, except for the standard version, which must be greater than or equal to that of all dependencies (recursively).

All code will be compiled with the standard version specified in the root project.
I should consider whether it’s possible to relax some of these options in the future, so that different projects in the graph might be compiled using different settings.

Project names must be unique across the entire dependency graph.
The list of dynamic link libraries will be merged across all projects in the dependency graph.
The list of supported targets for a project must be a subset of that of all of its dependencies. For example, if A depends on B and B supports only win32, it would be an error for A to declare support for both win32 and macOS.
Static libraries mustn't have assets directories.

Future directions

It's worth considering allowing dynamic library projects to be depended on by other projects.
Third-party libraries on GitHub (for example) won't be using ABS, as a general rule. So, some basic support for integrating with non-ABS projects would be useful.

There are also other potential sources for remote dependencies, like nuget.

Update: no, my original idea of transparent libraries is kind of silly. I'm just going to allow static libraries (which I thought I already did), then support depending on a static library project.

Update: all that is left to do now is copy headers from dependee projects to the root project's abs directory.


Bug fix for SPServer objects that use FQDNs

We initially saw this bug about 10 months ago but couldn't reproduce so left it alone. It's finally reared its head again and I've been able to squash it. The scenario is that in some SharePoint installations you will see SPServer objects referred to using a FQDN (like "server.domain.name") instead of just the server name. This breaks xSPServiceInstance, xSPUserProfileSync, xSPSearchTopology and xSPDistributedCacheService as they all refer to these server objects in some form, so this PR covers a fix for each that sees us try to load a service instance from the computer name alone first, and if that returns null we re-run the check using the FQDN.
Code review time @caadam @camiloborges @ykuijs @anlynes

Added the same question on several places in the code. The rest looks fine.


Issue during Creation - cannot find Chrome binary

I think its a relly stupid issue but for the moment I am unable to solve it.
So maybe someone can give me a hint. :)
selenium.common.exceptions.WebDriverException: Message: unknown error: cannot fi
nd Chrome binary

Google Chrome browser has to be installed for this to work, do you have it? That's usually what that error means

Yep Chrome is installed (Version 54.0.2839.0)

What operating system? Is it installed in the default location? On OSX it is the applications folder, on Windows it should be the program files folder.
Are you running a dev version of chrome? The stable version should be v52, the dev branch should be the v54 if I remember correctly from a recent issue

Windows 7
Yep default location ("C:\Users*USER*\AppData\Local\Google\Chrome SxS\Application\chrome.exe")
Yep should be dev version. Might this be the culprit?

The dev version throws a different error from what you pasted (see #39). I'm not familiar with Windows, but could you double check with https://github.com/SeleniumHQ/selenium/wiki/ChromeDriver to see if that's the correct location?

The "Chrome SxS" is diffrent from the given in your link. Maybe this is related to canary version of chrome

I just noticed the link may also be outdated. Maybe try installing the stable branch of Chome and see where that installs to?

Working thx :)

Curious, did you have to install the stable binary or did it work with the dev branch?

Stable version -> created teh necessary folder


Make time_spent in Annotation optional

Can you change the commit msg to use semantic commit prefix? Probably fix: in this case :wink:


Secure boot should be turned off in Local Cellery Setup - Ubuntu 18.04

Description:
When installing cellery runtime on local setup on Ubuntu 18.04. Secure boot should turned off which is not included in doc.
Suggested Labels:
Doc improvements
Suggested Assignees:
@MadhukaHarith92
Affected Product Version:
Cellery 0.2.0
Steps to reproduce:
Related Issues:

This should be an issue with my machine. So closing this issue.


Expression template containing double-quoted string fails

Summary
tl;dr
Argo failed to evaluate an expression template containing a double-quoted string.
value: '{{=jsonpath(workflow.parameters.config, "$.a")}}'

But it successfully evaluated an expression template containing a single-quoted string.
value: "{{=jsonpath(workflow.parameters.config, '$.a')}}"

According to the expr language definition, it supports either.
Full test cases
Fails
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: expression-flatten-json-
spec:
  arguments:
    parameters:
      - name: config
        value: '{"a": "1"}'
  entrypoint: main
  templates:
    - name: main
      inputs:
        parameters:
          - name: a
            value: '{{=jsonpath(workflow.parameters.config, "$.a")}}'
      script:
        env:
          - name: A
            value: "{{inputs.parameters.a}}"
        image: debian:9.4
        command: [bash]
        source: |
          echo "$A"

Output is:
{{=jsonpath(workflow.parameters.config, "$.a")}}

Succeeds
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: expression-flatten-json-
spec:
  arguments:
    parameters:
      - name: config
        value: '{"a-hyphenated": "1"}'
  entrypoint: main
  templates:
    - name: main
      inputs:
        parameters:
          - name: a
            value: "{{=jsonpath(workflow.parameters.config, '$.a')}}"
      script:
        env:
          - name: A
            value: "{{inputs.parameters.a}}"
        image: debian:9.4
        command: [bash]
        source: |
          echo "$A"

Output is:
1

Why it matters
There's no easy/intuitive way to use a string that contains single-quotes. I would expect to be able to use {{=jsonpath(workflow.parameters.config, "$['a-hyphenated']")}}, but that's not possible with double-quoted strings off the table.
I have also tried escaping the inner quotes, but that does not work either. I also don't see any documentation in expr for escaping characters. (Escaped once for YAML, another time for expr.)
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: expression-flatten-json-
spec:
  arguments:
    parameters:
      - name: config
        value: '{"a-hyphenated": "1"}'
  entrypoint: main
  templates:
    - name: main
      inputs:
        parameters:
          - name: a
            value: "{{=jsonpath(workflow.parameters.config, '$[\\\'a-hyphenated\\\']')}}"
      script:
        env:
          - name: A
            value: "{{inputs.parameters.a}}"
        image: debian:9.4
        command: [bash]
        source: |
          echo "$A"

Diagnostics
Kubernetes provider: Docker Desktop
Argo version: 3.1.0
Executor: The default from install.yaml... I assume it's Docker.
It's not a regression, since expression templates is a new feature.
 Give it a 👍. We prioritise the issues with the most 👍.

Here is an even simpler test:
value: '{{="test"}}'  # evaluates to `{{="test"}}`
value: "{{='test'}}"  # evaluates to `test`

would you like to submit PR this?

@sarabala1979 maybe. I ran the controller locally and got some obtuse error back from expr (something about the backslash character being invalid, even when it wasn't present???). Will post that when I get the chance. If the problem turns out to be simple, I'll be happy to contribute a fix.
If this proves more involved, I might just have to submit a PR documenting the bug.

FastTemplate is escaping double-quotes in the tags before they're fed to expr. I believe adding this line before evaluating the expression would solve the issue:
s = strings.Replace(s, "\\\"", "\"", -1)

I'll try to put up a PR, but might not find the time. At least now there's a clearer idea what the issue is. :-)

the template is marshalled to JSON in the SubstituteParams function (i.e. the interface becomes JSON as a JSON-encoded string [i.e. double quotes are escaped])
this string is passed to the Replace function
which is then passed to NewTemplate which calls fasttemplate's NewTemplate
which calls Reset
which dutifully retrieves as a tag everything between {{ and }}, which includes the JSON-escaped double quotes

It would be safe and sufficient to do a string replace before evaluating the expression. I'll put up a PR, but it might be a few days. Gotta finally set up a decent dev environment. :-P


Hot fixes and cypress test changes

Description
This branch was used in a demo environment for reinvent, therefore it contains couple of different hot fixes. It also includes changes:

Skipped some cypress tests that are testing against unreleased visualizations, and those that needs to be rewritten and improved
Resolved rest of failing cypress tests of event analytics

This PR is for updating 2.x with latest fixes, event analytics refactoring, and updating PR #1304 to be backported to main.
Issues Resolved
#1272
#1274
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1327 (5cf66a5) into 2.x (17b8771) will increase coverage by 28.27%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##                2.x    #1327       +/-   ##
=============================================
+ Coverage     41.82%   70.10%   +28.27%
  Complexity      315      315
=============================================
  Files           302       46      -256
  Lines         17840     2482    -15358
  Branches       4285      253     -4032
=============================================
- Hits           7462     1740     -5722
+ Misses        10205      600     -9605
+ Partials        173      142       -31

Flag
Coverage Δ

dashboards-observability
?

opensearch-observability
70.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ty/common/query_manager/antlr/ppl_syntax_parser.ts

...mon/query_manager/ast/builder/stats_ast_builder.ts

...shboards-observability/common/utils/query_utils.ts

.../public/components/custom_panels/helpers/utils.tsx

...ublic/components/visualizations/charts/bar/bar.tsx

...ponents/visualizations/charts/helpers/viz_types.ts

...ic/components/visualizations/charts/lines/line.tsx

...-observability/common/constants/trace_analytics.ts

...lic/components/visualizations/assets/chart_bar.tsx

.../query_manager/antlr/output/OpenSearchPPLParser.ts

... and 246 more

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


Remove uppercase enforcement of environment variables

Currently environment vars must be uppercase, which prevents us from populating terraform input variables directly with this action.  For example, we cannot do TF_VAR_secret_1 directly with this action and then need an intermediate action to take this output and set the correct terraform environment variable.
I think this should be allowed given the enormous popularity of terraform.  Can't see any documented reasoning why you force the name to uppercase here https://github.com/aws-actions/aws-secretsmanager-get-secrets/blob/main/src/utils.ts#L189C54-L189C65

Thank you for the request, we will look into this.

We've added option name-transformation. Your use-case is covered by name-transformation: none. Please see the documentaiton or https://github.com/aws-actions/aws-secretsmanager-get-secrets/pull/139 for more info.


Build for windows

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@monaldoj I'm hoping that this w'd make hazm be available for windows too. Was there an issue in particular that prevented it when you created the recipe?

@conda-forge-admin, please rerender

originally skipped the windows build because the requirement libwapiti is not available for windows in conda-forge


Correct URL to Example Source Code Repo

It was pointing to the manuscript repo

Good catch, thanks @carltongibson!

No worries!
Aside: is the Multi Process Mac Apps book still alive?

Yes, it's alive. I'm going to add a bunch of meat to it this summer when I have refactored an existing app into this pattern.

OK. I look forward to it. Ta!


docs: rename dvc ls to dvc list to link

Alias does not seem to get automatically linked if they there are other flags.

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.
Please choose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏

I have submitted a solution to allow Highlight alias command with arguments.

Thanks.


add Tests

Codecov Report

Merging #4 (6a7121f) into master (19df043) will increase coverage by 75.86%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           master        #4       +/-   ##
============================================
+ Coverage   24.13%   100.00%   +75.86%
============================================
  Files           5         2        -3
  Lines          58        16       -42
  Branches        8         1        -7
============================================
+ Hits           14        16        +2
+ Misses         37         0       -37
+ Partials        7         0        -7

Impacted Files
Coverage Δ

lib/module.js
100.00% <ø> (ø)

lib/plugins/helpers/console.js

lib/store/index.js

lib/store/modules/console.js

lib/plugins/index.js

example/nuxt.config.js
100.00% <0.00%> (ø)

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 19df043...6a7121f. Read the comment docs.


feat: add auth helpers glue code

Not yet doing anything useful

Should I add an end-to-end test in pixi with fake pip server, similar to the one implemented in rip?

That would be awesome!

@vlad-ivanov-name Is this ready for a review?


#161421169 Create a db-powered endpoint for store owners to view a single sale

What does this PR do?
This PR creates the endpoint for store owners, and an attendant who made a sale to view it
Description of Task to be completed
The main tasks to be completed are:
Creation of an endpoint for store owners to view a single sale
Ensure a store attendant who made the sale can view it too
Connect it to the database
How should this be manually tested?

Create  a virual enviroment with the command
$ virtualenv -p python3 env

Activate the venv with the command
$ source env/bin/activate

Install git
sudo apt install git

clone this repo
$ git clone https://github.com/codeMarble254/Store-Manager-v1.git"

cd into the folder Store-Manager-V1

export required enviroment variables
$ export FLASK_APP=run.py
$ export DB_NAME="storemanager"
$ export DB_HOST="fill in your localhost"
$ export DB_USER="fill in the db user"
$ export DB_PASSWORD="fill in your db user password"
$ export SECRET_KEY="<your key>"

create the dev database
createdb storemanager

create the testing database
$ createdb test_db

Switch to the current branch
$ git checkout ft-view-single-sale-v2-161421169

install requirements
$ pip install -r requirements.txt

now we are ready to run.
8.now we are ready to run.
for tests
export APP_SETTINGS="testing"
pytest app/test/v2 -v --cov=app/test/v2

for the application run
export APP_SETTINGS="development"
$ flask run
If you ran the application you can test the  api end points using postman as follows

Endpoint
functionality
contraints(requirements)

get /api/sales/saleId
View a single sale
User authentication

What are the relevant pivotal tracker stories?
#161421169

Pull Request Test Coverage Report for Build 135

253 of 255 (99.22%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.0%) to 99.216%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/test/v2/test_users.py
83
85
97.65%

Totals

Change from base Build 67:
2.0%

Covered Lines:
253

Relevant Lines:
255

💛  - Coveralls


Add config option for pruning repos

Scala Steward can prune the list of repos it will work on to reduce its overall workload (see https://github.com/fthomas/scala-steward/issues/123 and https://github.com/fthomas/scala-steward/issues/319#issuecomment-504208183 for explanations why we're doing this). This pruning can be disabled by commenting this line in StewardAlg and uncommenting the line below. Pruning makes sense when running Scala Steward very frequently and with many repos but not if it is only running once a day / or week with only a few repos. And I guess most private instances of Scala Steward are used this way. The default behaviour can also lead to errors when checking dependencies for updates like https://github.com/fthomas/scala-steward/issues/319. Therefore we should add a new config option to enable pruning repos and disable it by default.

Implemented in #711.


Running tests with --maxWorkers=1 or in GitHub Actions several test suites fail.

🐛 Bug Report
When running tests with --maxWorkers=1, 7 out of 15 test suites fail. (running without this flag they all pass)
When running the tests individually or in smaller sub groups (3-5 test suites at a time), they all pass as well.
When running on GitHub Actions without the --maxWorkers flag it also gave the same errors, so running with the flag seems to be required.
Using this as the config jest.config.js:
module.exports = {
  preset: 'ts-jest',
  testEnvironment: 'node',
};

To Reproduce
Have --maxWorkers=1 and several test suites.
Expected behavior
Tests pass/fail regardless of --maxWorkers flag
Input
jest --silent --coverage --maxWorkers=1
Clearing the cache (jest --clearCache) or running with --no-cache has no difference.
Error Output
Several tests will pass then (in this case 7) then the rest fail (8).
 FAIL  src/frontend/app.test.tsx
  ● Test suite failed to run

    jest: failed to cache transform results in: [USER_PATH]/AppData/Local/Temp/jest/jest-transform-cache-d857afcac7a5b1cfb398c940ea96865b-d0e7c439bf6eb5a4bf3a6371fdd259d1/50/apptest_50062bb11ef8a5300b3bc1eb1d34a237.map
    Failure message: ENOENT: no such file or directory, open '[USER_PATH]/AppData/Local/Temp/jest/jest-transform-cache-d857afcac7a5b1cfb398c940ea96865b-d0e7c439bf6eb5a4bf3a6371fdd259d1/50/apptest_50062bb11ef8a5300b3bc1eb1d34a237.map.2375307809'

      at Function.writeFileSync [as sync] ([NODE_PATH]/node_modules/@jest/transform/node_modules/write-file-atomic/index.js:203:17)
      at writeCacheFile ([NODE_PATH]/node_modules/@jest/transform/build/ScriptTransformer.js:751:32)
      at ScriptTransformer.transformSource ([NODE_PATH]/node_modules/@jest/transform/build/ScriptTransformer.js:492:7)
      at ScriptTransformer._transformAndBuildScript ([NODE_PATH]/node_modules/@jest/transform/build/ScriptTransformer.js:523:40)
      at ScriptTransformer.transform ([NODE_PATH]/node_modules/@jest/transform/build/ScriptTransformer.js:579:25)

Running on GitHub Actions gives a similar error
FAIL src/frontend/app.test.tsx
  ● Test suite failed to run

    jest: failed to cache transform results in: C:/Users/runneradmin/AppData/Local/Temp/jest/jest-transform-cache-9e25f567cea78ff63e1ef5ae25643a38-d0e7c439bf6eb5a4bf3a6371fdd259d1/44/apptest_4436a4bd0e44e8cc88ad357e40d9e4e8.map
    Failure message: ENOENT: no such file or directory, open 'C:/Users/runneradmin/AppData/Local/Temp/jest/jest-transform-cache-9e25f567cea78ff63e1ef5ae25643a38-d0e7c439bf6eb5a4bf3a6371fdd259d1/44/apptest_4436a4bd0e44e8cc88ad357e40d9e4e8.map.3706536467'

      at Function.writeFileSync [as sync] (node_modules/@jest/transform/node_modules/write-file-atomic/index.js:203:17)
      at writeCacheFile (node_modules/@jest/transform/build/ScriptTransformer.js:751:32)
      at ScriptTransformer.transformSource (node_modules/@jest/transform/build/ScriptTransformer.js:492:7)
      at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:523:40)
      at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:579:25)

Note
When running with --maxWorkers=2 more tests pass (only 2 suites fail from the 15), again with the same error.
envinfo
Local: Windows 10 x64
GitHub Actions: ubuntu-latest, windows-latest

Node: 12

jest: ^24.9.0
ts-jest: ^24.2.0

needed to use @jest-runner/electron in the jest.config.js
module.exports = {
  preset: 'ts-jest',
  runner: '@jest-runner/electron',
  testEnvironment: '@jest-runner/electron/environment',
};


manage windows.vadyarascan.VadYaraScan as yarascan.YaraScan

Currently starred rules can be assigned by default only to yarascan.YaraScan.
Add support for windows.vadyarascan.VadYaraScan because at the moment only manual upload is supported.

Added in: https://github.com/LDO-CERT/orochi/commit/33cd18c5a5c79510c24f3f3d0e240d11d1769eb5

Thanks it works


🐛 Bug Report: OpenAI Assistant API instrumentation doesn't use token stats from response

Which component is this bug for?
OpenAI Instrumentation
📜 Description
OpenAI Assistant API instrumentation does not use the token usage that one could get from the run.usage object. In addition, gen_ai.system=openai is not sent
👟 Reproduction steps

Use sample app to call an Assistant.
[Optional] add print(run.usage) after line 34, when run.status == "completed".

👍 Expected behavior

The resulting span must contain gen_ai.system = openai in its attributes.
Tokens must be added as gen_ai.usage attributes even when enrich_tokens flag is not set, because they are returned in the response.

👎 Actual Behavior with Screenshots

The resulting span does not have the system attribute
There are no gen_ai.usage.*_tokens attributes on the resulting span

🤖 Python Version
3.9
📃 Provide any additional context for the Bug.
(I haven't tried with the token enricher/in-place counter, but I think we should not rely on it, now that the API returns token usage)
👀 Have you spent some time to check if this bug has been raised before?

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

Are you willing to submit PR?
Yes I am willing to submit a PR!

Thanks for reporting @dinmukhamedm! Looking forward for the fix 🙏


Remove --print-ast CLI flag as PsiViewer provides the same features

Closes #5417

As a side note, @samuelneff, even we don't want to keep the maintenance of these feel free to copy these code and create your own stand-alone cli for printing the ast tree.
And I think that ktlint has this feature on their cli too so that's other option.


Add current rune state to timeline

I assume some rune prediction was worked on (as there was ideas about rune drift at some point) and it might not be finished yet. But whenever that part is finished, current rune state should be added to timeline of the fight if possible.

Finished in an earlier issue


notifyHeaderChanged Bug

@florent37
But there is a bug in notifyHeaderChanged()
If i didn't used the tabs atleast once, the position variable will be still -1 and even if i update header and called notifyHeaderChanged now. the lastPosition is equal to previous position (both are -1) and it wont update my header.
So when we call setCurrentItem() you need to save it to position variable. to solve this.
May be set Default position as -2 instead of -1 for small fix.

@florent37 will this be fixed in next update ?

@florent37 Thanks, it was fixed in this update(1.1.2) closing issue


Ensure generated .d.ts files compile with no errors under noImplicitAny

We recently just had https://github.com/microsoft/TypeScript/issues/36630 opened up on us. This is just a few weeks after #36216.
I'm pretty sure we already check that our generated .d.ts files are valid when the input was valid, but apparently we're not checking under noImplicitAny. We would've avoided these bugs if we'd checked with noImplicitAny.

I tried changing https://github.com/Microsoft/TypeScript/blob/9889c7493990e7e24393832e53591e684ac7eadb/src/harness/harnessIO.ts#L511 to force the options with
            const adjustedOptions: ts.CompilerOptions = {
                ...origOptions,
                declaration: true,
                skipLibCheck: false,
            };

but it doesn't seem to uncover the existing issues.


Can't sign in on earlier versions

VS Code Version:  1.85.2
OS Version: Windows 10

Unable to sign in using any version other than v0.12.*. The latest version is basically unusable with the highlight to use context change and the lack of syntax highlighting for HCL code. I'm trying to use previous versions until these issues are fixed, but copilot chat just will not sign in, even though my github account is signed in:

If I sign out and in again, it opens a window but nothing loads:

Steps to Reproduce:

Upgrade to v0.12.*
Install any version prior to this
Sign in

First off, why are you downgrading?
Second off, you need to also downgrade the Copilot extension to 1.156.0

That worked. Thank you. HCL syntax highlighting which is broken in the new version (please see https://github.com/microsoft/vscode-copilot-release/issues/881) also works on this version:

This can be closed now.

Thanks for confirming!


Exclude buffertype

I see that on file explorers like chadtree or symbols-outline that are sidebar I get the word underline.
It is possible to exclude filetype in this way the behaviour will be more clean.

@Mte90 could you please check if my pull request works for you?
Here is the fork and the branch is feature/disable-conditionally.
With packer.nvim:
packer.use {
  "Hrle97/nvim-cursorline",
  branch = "feature/disable-conditionally",
  config = function() ...  disable filetypes here as instructed on the fork link ... end
}

I don't know about other plugin managers, so I can't help you there.

I came looking for this too, except for cursorline in terminals.


Incorrect image or text

This image does not align with the text provided above this image.
https://docs.microsoft.com/en-us/azure/devops/repos/git/media/pull-request-templates/additional-pull-request-template.png?view=azure-devops
Text says dev.md is the default template while image shows pull_request_template.md as default template.

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

ID: a297d54f-48b3-1b68-e828-15d1559d9b2d
Version Independent ID: fd9dcb35-72f9-f5a9-e0c0-75c3e59ba8d9
Content: Improve pull request descriptions with pull request templates - Azure Repos
Content Source: docs/repos/git/pull-request-templates.md
Product: devops
Technology: devops-code-git
GitHub Login: @vijayma
Microsoft Alias: vijayma

@pkg01 -- thank you for your close reading.
@vijayma -- please look into this issue.


Ability to specify time limit (-W)

Hello, it would be nice to be able to set the job time limit (-W).
It seems like this would require adding an entry to a rule's resources, but it's not clear to me that there is a standard key for this in snakemake.

Hi @funnell I am in the process of adding the ability to specifc "non-standard" LSF parameters for rules. Follow #7 for updates. I should hopefully have it implemented today or tomorrow. The idea will be that you can specify additional cluster parameters in a special cluster-specific config file lsf.yaml. Say you have a rule foo and you want to indicate this job should only run for an hour and 5 minutes and should only run on a specific project (say bigmem) you could do
lsf.yaml
foo:
  - "-W 1:05"
  - "-P bigmem"


return None instead of '' as payee if it's undefined

Use None rather than empty string if payee is undefined as empty string is ignored by CountVectorizer fixes #1

I am surprised by the use of 'None' in your code, i.e., a string value.
Shouldn't it be None, i.e., the python null value?

I used 'None' to have the CountVectorizer get something, the problem with empty string or None is that it will not find any and fail with ValueError: empty vocabulary; perhaps the documents only contain stop words, the easiest solution I found was to return the string None for empty payees which can then be vectorized (haven't done anything with sklearn before, so just played around to find a solution for the exception)

just tried it out and thought it through. i think that using a string literal such as 'None' wil work just fine for the predict_postings usecase. But i think it will mess up the predict_payees decorator, which would suggest payees named 'None'.
Alternative idea: we could calculate the number of distinct payees in the training data, i.e., the vocabulary size, and only conditionally include the payee step in the machine learning pipeline if the vocabulary size is >= 2.
What do you think?

Makes sense.

ok, would you want to give it a try and implement it that way? that would be great. (I've had a long day in front of the computer already, so if I were to do it, I'd rather postpone the task for a few days)

Done, didn't find a good way to create a test case for this, maybe have to think about a better way to do some test cases (maybe some data driven approach could work nicely)

thank you!