Coral USB changing ID and Vendor

Running Proxmox + LXC Container + Frigate on docker.
I'm setting up the Coral USB as it's being shown by the lsusb command as
Bus 002 Device 002: ID 1a6e:089a Global Unichip Corp.
I create a UDEV rule, then I restart to apply the changes and the ID/Vendor changes as
Bus 002 Device 003: ID 18d1:9302 Google Inc.
It is listed as 18d1:9302 Google Inc. only AFTER classify_image.py has been successfully run. No idea why. Does anyone experiencing a similar issue or knows what's happening?
Another reference is here

Hi @jcastro it is an expected behavior. When Coral USB connected it would list as Global Unichip Corp and after first inference it would change to 'Google Inc'.

Hi @jcastro it is an expected behavior. When Coral USB connected it would list as Global Unichip Corp and after first inference it would change to 'Google Inc'.

How is it possible to map it then? This behaviour makes it really difficult to use it on Linux

Can you please refer to the below link and try to passthrough the port than passing the device ID.
https://www.reddit.com/r/Proxmox/comments/nmsknx/proxmox_vm_ubuntu_2004_connect_google_coral_usb/

Can you please refer to the below link and try to passthrough the port than passing the device ID. https://www.reddit.com/r/Proxmox/comments/nmsknx/proxmox_vm_ubuntu_2004_connect_google_coral_usb/

Thanks, I will try this soon and see if it helps. Need to figure out how to do it as I'm not using a VM but a LXC (Container) on Proxmox

Can you please refer to the below link and try to passthrough the port than passing the device ID. https://www.reddit.com/r/Proxmox/comments/nmsknx/proxmox_vm_ubuntu_2004_connect_google_coral_usb/

This solution is for a VM not a LXC container-

For reference, this solution using udev works like a charm:
foo@bar:~$ lsusb -d 1a6e:089a
Bus 002 Device 004: ID 1a6e:089a Global Unichip Corp.
foo@bar:~$ echo 'SUBSYSTEMS=="usb", ATTRS{idVendor}=="1a6e", ATTRS{idProduct}=="089a", MODE="0664", TAG+="uaccess"' | sudo tee -a /etc/udev/rules.d/71-edgetpu.rules > /dev/null
foo@bar:~$ echo 'SUBSYSTEMS=="usb", ATTRS{idVendor}=="18d1", ATTRS{idProduct}=="9302", MODE="0664", TAG+="uaccess"' | sudo tee -a /etc/udev/rules.d/71-edgetpu.rules > /dev/null
foo@bar:~$ sudo udevadm control --reload-rules && sudo udevadm trigger

@emepetres What exactly is that supposed to do?

@melyux they are rules so linux can detect the edge-tpu and assing proper permissions even after the id change.

Hi @jcastro it is an expected behavior. When Coral USB connected it would list as Global Unichip Corp and after first inference it would change to 'Google Inc'.

@hjonnala  But why is this necessary? It should just have a stable ID all the time (like basically every other USB device)... is there a technical reason for this or is it just a very very poor design choice?

For reference, this solution using udev works like a charm:
foo@bar:~$ lsusb -d 1a6e:089a
Bus 002 Device 004: ID 1a6e:089a Global Unichip Corp.
foo@bar:~$ echo 'SUBSYSTEMS=="usb", ATTRS{idVendor}=="1a6e", ATTRS{idProduct}=="089a", MODE="0664", TAG+="uaccess"' | sudo tee -a /etc/udev/rules.d/71-edgetpu.rules > /dev/null
foo@bar:~$ echo 'SUBSYSTEMS=="usb", ATTRS{idVendor}=="18d1", ATTRS{idProduct}=="9302", MODE="0664", TAG+="uaccess"' | sudo tee -a /etc/udev/rules.d/71-edgetpu.rules > /dev/null
foo@bar:~$ sudo udevadm control --reload-rules && sudo udevadm trigger

I seem to be running into this issue too, and found this. I have LXC container, and am interested in implementing this solution. However I am not that proficient in Linux. Where do I input this information? Thanks for the patience/help.

@augelr this isn't a support forum, but it looks like you just run the commands as shown. You could put them into a script if you want, then run the script.
If anyone wants a docker-ized answer to this problem, I've documented my solution here.
I am still curious to know if there is a technical reason to require this, or if it's just a dumb design decision... do you happen to know @hjonnala? Looks like there's definitely interest in the answer that question...

For reference, this solution using udev works like a charm:
foo@bar:~$ lsusb -d 1a6e:089a
Bus 002 Device 004: ID 1a6e:089a Global Unichip Corp.
foo@bar:~$ echo 'SUBSYSTEMS=="usb", ATTRS{idVendor}=="1a6e", ATTRS{idProduct}=="089a", MODE="0664", TAG+="uaccess"' | sudo tee -a /etc/udev/rules.d/71-edgetpu.rules > /dev/null
foo@bar:~$ echo 'SUBSYSTEMS=="usb", ATTRS{idVendor}=="18d1", ATTRS{idProduct}=="9302", MODE="0664", TAG+="uaccess"' | sudo tee -a /etc/udev/rules.d/71-edgetpu.rules > /dev/null
foo@bar:~$ sudo udevadm control --reload-rules && sudo udevadm trigger

An additional rule to ease the access would be:
... TAG+="uaccess", SYMLINK+="usbedgetpu"

You can then access the device by its new "dev path" (which is a symlink to the real path) /dev/usbedgetpu. You can see the symlink via ls -la /dev/*tpu:
root@pve:/etc/udev/rules.d# ls -la /dev/*tpu
lrwxrwxrwx 1 root root 15 Apr 28 20:36 /dev/usbedgetpu -> bus/usb/004/005

Just in case anybody else has this issue in the future, I made a Proxmox hookscript that can be used to correct the TPU name whenever the VM boots.

Create snippets in /var/lib/vz/
Create coral-bin in snippets
Download the coral firmware inside of coral-bin with wget https://github.com/google-coral/libedgetpu/raw/master/driver/usb/apex_latest_single_ep.bin
Save script to /var/lib/vz/snippets/coral-init.sh and make executable:

#!/bin/bash
if [ "$2" = "pre-start" ]; then
    echo "Running pre-start hookscript for VM $1" > /var/log/coral-init.log
    lsusb | grep 18d1:9302 > /dev/null
    if [ $? -eq 0 ]; then
        echo "Google Coral TPU already intialized!" | tee -a /var/log/coral-init.log
        exit 0
    fi
    lsusb | grep 1a6e:089a > /dev/null
    if [ $? -ne 0 ]; then
        echo -e "Didn't detect uninitialized Google Coral TPU:\n$(lsusb)" | tee -a /var/log/coral-init.log
        exit 1
    fi
    echo "Initializing coral..." | tee -a /var/log/vm-pre-start.log
    dfu-util -D /var/lib/vz/snippets/coral-bin/apex_latest_single_ep.bin -d "1a6e:089a" -R
    if [ $? -ne 251 ]; then
        echo "Error occured!" | tee -a /var/log/coral-init.log
        exit 2
    fi
    echo -e "Done! USBs:\n$(lsusb)" | tee -a /var/log/coral-init.log
fi

Then run: qm set <VM_ID> --hookscript local:snippets/coral-init.sh
Now every time the VM boots, it will check that the Coral Name/ID has been initialized.

Awesome, thank you! Don't forget to install dfu-util (apt install dfu-util) before using ab3lsons script.

As always, I'm disappointed in Google's inability to ever justify any of its design decisions.
I cannot think of a single other USB device that pulls these shenanigans. If there is a technical reason for it, I think we'd all accept the explanation and sleep more comfortably knowing that there is truly no better way. But when Google acts like this, it gives me the impression that they are so ashamed of their own decisions that they would rather remain silent than open the floor to its paying customers to suggest possible improvements.
Hacky workarounds like the ones in this thread (my own included) are especially frustrating when we are never even given the courtesy of understanding the root cause. Is anyone willing to explain to us why this is the case?
/rant

@hjonnala please respond

any solution for LXC docker??  my problem in ID 1a6e:089a Global Unichip Corp.

any solution for LXC docker??  my problem in ID 1a6e:089a Global Unichip Corp.

I did this : https://www.saninnsalas.com/passing-google-coral-usb-edge-tpu-to-an-unprivileged-container-in-proxmox/


rust-analyzer: 2020-08-24 -> 2020-09-14 and some clean-up

Motivation for this change
Bump version and do some clean-up.

unstable-2020-09-07 requires rustc 1.46.0 to compile but we still have 1.45.2 in master. Patches are introduced for a temporary fix.
Feature jemalloc is removed now, and a new features mimalloc is introduced.
Remove fake rustup in check phase, since we currently test only sub-crate rust-analyzer, which doesn't need it anymore.
Install check works now. Enable it by default.

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

r? @danieldk

This update touches node package, since I literally know nothing about node, I'll leave up the reviewing to someone else.

@oxalica This still needs to go through committer review, but you might want to squash the commits in the meantime.

The global node-packages (pkgs/development/node-packages/node-packages.nix) is updated too frequently.
I'm thinking about using a standalone generated node-packages.nix for the package.
What's the best practice about these npm packages as only build-dependency?
@jonringer

I refactored the node packages part to use generated standalone node-packages.nix to avoid touching global nodePackages.
@midchildan Have a look? What do you think about it?

Using node2nix outside of nodePackages is generally frowned upon [1,2] even if it's only used as a build dependency [3,4]. The guideline for node.js packaging [5] states that "the package set should only provide end user software package," and this has been pointed out in another issue [6], but I think it's best to stick with the currently accepted practice for this specific PR.

@midchildan I'm just tired of generating and generating again over ALL NPM packages used by nixpkgs every time having merge conflict (it's updated quite often). We have package-lock.json but node2nix ignores them and fetches all packages every time, which takes about an hour for me and (accidentally) cause ~100 package rebuild (which may be not needed or even lead to build failure)
(Also node2nix is poor designed and the fetching is synchronous. It takes years in a proxy environment with high latency but large bandwidth.)
Note that just when I typing this comment, there's another update on pkgs/development/node-packages/node-packages.nix.
MERGE CONFLICT AGAIN :disappointed:

The current situation isn't ideal, but I believe this discussion would be best done in a separate issue because this affects how node packaging should be done in general, and isn't limited to this package. #82662 seems like a good place to start.
For now, I'd recommend ignoring merge conflicts until you receive committer approval. You can perform a rebase afterwards.


Feature: allow vehicle dragger tool to attach cars to tracks

Leftovers:

[ ] - TODOs, HACKs.

@Basssiiie I need stuff from the first branch so maybe review here? lol

I can also just close this and reopen at your repo after we merge the first PR.

wrong branch demmit


Frame group perf

:sparkles: Description by Cal
PR Description
This PR introduces performance improvements and refactoring related to frame and group handling in the application. It optimizes the rendering process, reduces redundant calculations, and enhances the logic for determining element containment within frames.

Diagrams of code changes
sequenceDiagram
    participant User as User
    participant GitHub as GitHub Actions
    participant CallstackAI as Callstack.ai Action
    participant App as App Component
    participant Frame as Frame Module
    participant Groups as Groups Module
    participant Renderer as Renderer Module

    User->>GitHub: Trigger Callstack.ai PR Review
    GitHub->>CallstackAI: Dispatch workflow with inputs (config, head, base)
    CallstackAI->>App: Review PR changes
    App->>Frame: Check if elements are in frame
    Frame->>App: Return elements in frame
    App->>Groups: Process group IDs
    Groups->>App: Return processed group IDs
    App->>Renderer: Render elements with new logic
    Renderer->>App: Return rendered elements
    CallstackAI->>GitHub: Complete PR review

Key Issues
None
Files Changed

File: /.github/workflows/callstack-reviewer.yml
				Added a new GitHub Actions workflow for Callstack.ai PR Review with inputs for configuration, head commit SHA, and base commit SHA.

File: /src/components/App.tsx
				Introduced a `processedGroupIds` map to optimize the selection of elements within frames.

File: /src/frame.ts
				Refactored functions to improve performance by optimizing frame and element intersection checks. Removed redundant functions and added tolerance to bounds checking.

File: /src/groups.ts
				Added a `processedGroupIds` set to prevent redundant processing of group IDs during element selection.

File: /src/renderer/renderScene.ts
				Refactored the rendering logic to optimize the handling of visible elements and frame clipping. Introduced a `processedGroupIds` map to avoid redundant group processing.

@callstackai review


PM2 doesn't start after server reboot

Hi,
I have a weird problem, if I restart the remote server and do not log in after wards to it the pm2 doesn't start automatically, however if after restart I log in to the server the pm2 starts as expected.
What can be the issue? I'm using Windows Server 2012 R2.

+1

Yes same here. I am using it on a Win10 SBC. PM2 only seem to start when I log in with a user.

Everyone who is looking this issue, I have found this article: https://blog.cloudboost.io/nodejs-pm2-startup-on-windows-db0906328d75
I have implemented Solution 2 (installing pm2-windows-service), and it works. Now I have a system service and Node Red is being started on Windows startup.

I used an easier solution - created a windows scheduled task which runs "pm2 resurrect" on start up as SYSTEM user. Works without anyone having to log in.

tl;dr

npm install pm2 -g
npm install pm2-windows-startup -g
pm2-startup install
pm2 start myApp.js --name mySuperApp
pm2 save
reboot
pm2 ls

I used an easier solution - created a windows scheduled task which runs "pm2 resurrect" on start up as SYSTEM user. Works without anyone having to log in.

did you create a .bat file that only contains "pm2 resurrect" in it and set it to run on startup from Win Task Scheduler? Is that what you mean?


[Formatting] Stop replace tabs into spaces

Hello
I have a problem/question about Format Document option - is it possible to stop replace tabs into spaces? I prefer tabs (not spaces)

If I may add, it should also respect the .editorconfig, if existing

Any chance of this getting fixed? Unfortunately preventing me from being able to use this extension.

We'd really like this too!

This has finally been resolved with the most recent update to the extensions (0.9.0). You can now choose the indentStyle in the config.
Thank you so much for your patience!


feat: Add proper error messages when something went wrong

:tada: This issue has been resolved in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


An issue on Retrosynthesis Tasks

synthon_optimizer = torch.optim.Adam(synthon_task.parameters(), lr=1e-3)
synthon_solver = core.Engine(synthon_task, synthon_train, synthon_valid,
synthon_test, synthon_optimizer,
gpus=[0], batch_size=128)
synthon_solver.train(num_epoch=10)
synthon_solver.evaluate("valid")
synthon_solver.save("g2gs_synthon_model.pth")

Thanks for interest in TorchDrug. This is duplicate of #42 and has been fixed. Please checkout the latest repo. Note conda/pip packages may have some lag, since we can't afford releasing a package for every bug fix.

So the code on github is right now?

Thanks a lot for solving my problem


Extended ImpliedValueUtil

This PR adds 2 overrides to ImpliedValueUtil.getMdibMdibVersion() that allow applying it to MdibAccess and MdibVersion.
These are neccessary to handle situations in which no Mdib is availlable, but the Architectural Rule still needs to be followed.
// Describe the purpose of the pull request here and remove this line. Keep the checklist.
Checklist
The following aspects have been respected by the author of this pull request, confirmed by both pull request assignee and reviewer:

Adherence to coding conventions

[x] Pull Request Assignee
[ ] Reviewer

Adherence to javadoc conventions

[x] Pull Request Assignee
[ ] Reviewer

Changelog update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

README update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

config update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

SDCcc executable ran against a test device (if necessary)

[x] Pull Request Assignee
[ ] Reviewer

You are right. This PR is hence unneccessary. Closing it.


🧱 include MQT Core via FetchContent instead of as a submodule

Description
https://github.com/cda-tum/mqt-core was included as a submodule so far. This PR removes this submodule and uses the existing fetch content commands to include this dependency.
Furthermore, it adds a dependabot-like update workflow for the FetchContent-managed mqt-core dependency.
Checklist:

[x] The pull request only contains commits that are related to it.
[x] I have added appropriate tests and documentation.
[x] I have made sure that all CI jobs on GitHub pass.
[x] The pull request introduces no new warnings and follows the project's style guidelines.

The failure here will be resolved by #405


Update URL for ORCID to begin with https://

In the 'created by' field, the URL for the creator's ORCID should begin with https:// instead of http://

[ ] please add https://github.com/obophenotype/human-phenotype-ontology/blob/master/src/sparql/orcid-contributor-violation.sparql to src/sparql/qc/general so that this issue does not happen again
[ ]  in https://github.com/monarch-initiative/mondo/blob/master/src/sparql/qc/general/qc-permitted-properties.sparql#L18 remove oboInOwl:createdBy and change dce:creator to dc:creator
[ ] Change all createdBy in mondo to dc:creator

This way we are all aligned.

do Nico's instructions first, and then Moni's errors will show up in QC
fix them with search and replace

In protege (after the above steps are done), cc @sabrinatoro

go to New entities metadata
in creator property, search for creator
click dcterms:contributor
edit manually - dcterms:creator

@matentzn when I copy and replace created_by with dc:creator, then I save as in Protege, the annotations look weird/wrong:

yes, the syntax is a bit different.
Match:
created_by: https?://orcid.org/([A-Z0-9-]+)

Replace:
property_value: http://purl.org/dc/elements/1.1/creator https://orcid.org/$1

After that, do a search for the remaining created_by annotations and fix them manually, using the pattern above:
property_value: http://purl.org/dc/elements/1.1/creator https://orcid.org/..

thank you @matentzn! I'm working on this now.


better template inheritance

Create a method for a search engine to use, for instance, the blob template for each template, but a custom one, say, my_search for the search_form. (Right now, you'd have to create a custom search_form/my_search.template, query_builder/my_search.js, detail_view/my_search.template, and list_view/my_search.template even if you only wanted to customize one.)

we should also allow setting template names by page type
'jeb-bush-emails': {
       list_view: 'email',
       detail_view: 'email',
       search_view: 'jeb-email',
       query_builder: 'jeb-email'
     }

if you wanted only one or two templates to be special and use existing ones for others.
now that this is set thru google spreadsheets, solving it will be a little different, maybe a third Spreadsheet sheet that overrides the data_type column in the first sheet (and that joins on index_name like the sample searches one does).


[Bug] RayService with GCS FT HA issue

Search before asking

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

KubeRay Component
Others
What happened + What you expected to happen
I set up a KubeRay RayService using an external Redis for fault tolerance. The RayService hosted a web service, and I used "mobilenet" as a sample. However, when I removed the head pod, the following issues arose: even though the worker pod remains active, the service named "svc/rayservice-mobilenet-serve-svc" can't process requests after the head's deletion.
I expect the svc to keep serving the request properly after the head is down.
...
{"prediction": ["n02099601", "golden_retriever", 0.786209762096405]}
{"prediction": ["n02099601", "golden_retriever", 0.786209762096405]}
Traceback (most recent call last):
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen
    httplib_response = self._make_request(
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/connectionpool.py", line 449, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/connectionpool.py", line 444, in _make_request
    httplib_response = conn.getresponse()
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/http/client.py", line 1377, in getresponse
    response.begin()
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/http/client.py", line 320, in begin
    version, status, reason = self._read_status()
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/http/client.py", line 289, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/requests/adapters.py", line 489, in send
    resp = conn.urlopen(
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/connectionpool.py", line 787, in urlopen
    retries = retries.increment(
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/util/retry.py", line 550, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/packages/six.py", line 769, in reraise
    raise value.with_traceback(tb)
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen
    httplib_response = self._make_request(
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/connectionpool.py", line 449, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/urllib3/connectionpool.py", line 444, in _make_request
    httplib_response = conn.getresponse()
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/http/client.py", line 1377, in getresponse
    response.begin()
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/http/client.py", line 320, in begin
    version, status, reason = self._read_status()
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/http/client.py", line 289, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/yqw/Desktop/ray-playground/mobilenet_req.py", line 12, in <module>
    response = requests.post(url, files=files)
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/requests/api.py", line 115, in post
    return request("post", url, data=data, json=json, **kwargs)
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/requests/sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/requests/sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)
  File "/Users/yqw/.pyenv/versions/3.9.15/lib/python3.9/site-packages/requests/adapters.py", line 547, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))

Reproduction script
All the code can be found here: https://github.com/YQ-Wang/ray-playground
Step 1: Install KubeRay Operator
helm install kuberay-operator kuberay/kuberay-operator --version <0.6.0 or 1.0.0-rc.0>
Step 2: Install Ray Service
The yaml file has the Ray head integrated with an external Redis
kubectl --context kind-kind apply -f ray-service.mobilenet-ft.yaml
Step 3:
kubectl port-forward svc/rayservice-mobilenet-serve-svc 8000
Step 4: Run the client code
python mobilenet_req.py
Step 5: Delete the head pod
kubectl --context kind-kind delete pod <head pod>
Step 6: Monitor the client output
You will find the client side raise exception
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

I can reproduce this issue. This may be a Ray Serve bug or misconfiguration. cc @edoakes @akshay-anyscale would you mind taking a look? Thanks!

kubectl describe svc rayservice-mobilenet-serve-svc: There are 1 head node and 1 worker node in the RayCluster. However, the service only has 1 endpoint from Ray head.

The Ray head has a HTTPProxyActor.

However, there is no HTTPProxyActor in the worker node.

The value of ProxyLocation is "EveryNode". Hence, every node should have HTTPProxyActor.

@kevin85421 nodes/pods without a replica do not have httpproxy. Looks like that is what is going on here

cc @shrekris-anyscale

nodes/pods without a replica do not have httpproxy.

I'm not sure whether this is correct. I've conducted benchmarks where I created two Ray nodes. Only the worker node had 16 replicas, while the head node didn't have any. If I remember correctly, the head node has an HTTPProxyActor. @shrekris-anyscale will check the details. Thanks!

The head node always has a proxy actor. @GeneDer can you confirm the behavior?

Yes, just remembered there are some behavior change. TLDR is like Akshay mentioned in the terminal state there will only be proxy on head node + active worker nodes (defined as nodes that has replicas).
Previously we will deploy proxy to all nodes (given the proxy location is everywhere). If the node has no replicas, then we will mark that proxy to be in the DRAINING state and failing the health check. This will help upscaling and downscaling to work correctly without affecting ongoing requests and allow enough time for a node to process requests. Proxy will be remained on the inactive cluster, but unreachable.
The current behavior is that we won't even start a proxy on an inactive worker node. During each update loop, we will first check which nodes are active, then we put the proxy on the active node + head node. And during downscaling the same logic applies when a worker node turns inactive, it's proxy will be marked as DRAINING state to allow enough time to process ongoing requests, and then gets shutdown. So in the terminal states only head node + the active worker nodes will have proxies running

Thanks, @akshay-anyscale and @GeneDer!

in the terminal state there will only be proxy on head node + active worker nodes (defined as nodes that has replicas).

Could you point me to the function I should read? Thanks!

The code that decides which proxy to start and stop is https://github.com/ray-project/ray/blob/master/python/ray/serve/_private/proxy_state.py#L672-L673

You might also be interested in https://github.com/ray-project/ray/blob/master/python/ray/serve/_private/cluster_node_info_cache.py#L17 as it determines the active nodes


Function new not implemented for IDataObject_Vtbl

Suggestion
It looks like the new function doesn't exist on the IDataObject_Vtbl struct, which the implement macro is trying to call.
Can this be added please? Or am I doing something wrong maybe?
Error message:
error[E0599]: no function or associated item named `new` found for struct `IDataObject_Vtbl` in the current scope
  --> src\main.rs:22:1
   |
22 | #[implement(IDataObject)]
   | ^^^^^^^^^^^^^^^^^^^^^^^^^ function or associated item not found in `IDataObject_Vtbl`
   |
   = note: this error originates in the attribute macro `implement` (in Nightly builds, run with -Z macro-backtrace for more info)

Here is my Cargo.toml:
[package]
name = "dragsample"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
windows = { version = "0.56", features = [
    "implement",
    "Win32_Graphics_Gdi",
    "Win32_System_LibraryLoader",
    "Win32_UI_WindowsAndMessaging",
    "Win32_System_Com",
    "Win32_System_Ole",
    "Win32_UI_Shell"
] }

And my code:
use windows::{
    core::*, Win32::{
        Foundation::*,
        Graphics::Gdi::ValidateRect,
        System::{
            LibraryLoader::GetModuleHandleA,
            Com::{
                IDataObject,
                IDataObject_Vtbl
            }
        },
        UI::{Shell::SHDoDragDrop, WindowsAndMessaging::*}},
};

#[implement(IDataObject)]
struct MyDataObject(usize);

fn main() -> Result<()> {
    unsafe {
        let instance = GetModuleHandleA(None)?;
        debug_assert!(instance.0 != 0);

        let window_class = s!("window");

        let wc = WNDCLASSA {
            hCursor: LoadCursorW(None, IDC_ARROW)?,
            hInstance: instance.into(),
            lpszClassName: window_class,

            style: CS_HREDRAW | CS_VREDRAW,
            lpfnWndProc: Some(wndproc),
            ..Default::default()
        };

        let atom = RegisterClassA(&wc);
        debug_assert!(atom != 0);

        CreateWindowExA(
            WINDOW_EX_STYLE::default(),
            window_class,
            s!("This is a sample window"),
            WS_OVERLAPPEDWINDOW | WS_VISIBLE,
            CW_USEDEFAULT,
            CW_USEDEFAULT,
            CW_USEDEFAULT,
            CW_USEDEFAULT,
            None,
            None,
            instance,
            None,
        );

        let mut message = MSG::default();

        while GetMessageA(&mut message, None, 0, 0).into() {
            DispatchMessageA(&message);
        }

        Ok(())
    }
}

extern "system" fn wndproc(window: HWND, message: u32, wparam: WPARAM, lparam: LPARAM) -> LRESULT {
    unsafe {
        match message {
            WM_PAINT => {
                println!("WM_PAINT");
                _ = ValidateRect(window, None);
                LRESULT(0)
            },
            WM_DESTROY => {
                println!("WM_DESTROY");
                PostQuitMessage(0);
                LRESULT(0)
            },

            WM_LBUTTONDOWN => {
                println!("buttondown");
                // SHDoDragDrop(window, pdata, pdsrc, dweffect);
                LRESULT(0)
            },

            _ => {
                // println!("unhandled msg: {}", message);
                DefWindowProcA(window, message, wparam, lparam)
            },
        }
    }
}

Additionally, it seems IDataObject_Impl does not exist. I am trying to implement IDataObject so I believe I would need this?

IDataObject relies on types that are unavailable in your crate.
In your Cargo.toml, add windows features:
Win32_Graphics_Gdi,
Win32_System_Com_StructuredStorage,

Then implement IDataObject_Impl as you were going to.
Maybe @kennykerr can speak to how we can potentially emit a better warning/error here.

Yep, I have a few things in the works to improve the compiler diagnostics around COM authoring. Stay tuned.

Closing in favor of #3036.


Documentation: Document global forwards

❗ please do not add sensitive information in issues, you can provide extra information via email using issue number as reference ❗
Describe the issue
PR https://github.com/ibissource/iaf/pull/3240 does not properly describe global forwards yet. This has been postponed to a later PR.
Reporter
Martijn Dirkse

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots
If applicable, add screenshots to help explain your problem.
Configuration
Please provide the configuration of the Pipe or Receiver with the problem.

Input
Please provide an example of the input message.
Alternatively, provide a Ladybug report.

Environment
Please go to the console of your Frank, click on 'Information' and copy-paste the results here.

Additional Environment

DBMS: [e.g. Oracle, MSSQL, MariaDB, MySql, PostgreSQL]
Browser: [e.g. Chrome, Safari, Edge, Firefox]

Additional Context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)

Some documentation was added in #2733. Closing this for now, let's see what that page will look like once we're using our new design.


[SolidJS] Browser Main Fields override Export Conditions

Describe the bug
Expected Behavior
When an export condition matches it doesn't use the "browser" main field to remap it just because "browser" was set.
Actual Behavior
In our case it matches on "worker" and then gets upgraded to the "browser" version unexpectedly.
Additional Information
https://github.com/solidjs/solid-start/issues/263 Lays out the issue.
This issue isn't only unexpected but it makes things very difficult. We have 3 things trying to balance:

Server vs Browser runtimes are different for SolidJS
Cloudflare Worker environments are Server environments for us but most tools call them browser, understandably because of Service/Web Workers. That's ok we can control this with export conditions.
Browser mainfield support is necessary for legacy tooling support. But its existence causes the work we do to map workers to be undone.

All these seem unmoveable. But perhaps main field overriding export conditions is not. The linked issues shows our attempts at trying to fix this. And backlinks relevant Github issues.
Reproduction
https://github.com/patdx/solid-start-graphql-cloudflare/tree/main/solid-with-standard-adapter
System Info
System:
    OS: macOS 12.6
    CPU: (8) arm64 Apple M1
    Memory: 97.66 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.16.0 - ~/.nvm/versions/node/v16.16.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v16.16.0/bin/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.16.0/bin/npm
  Browsers:
    Chrome: 105.0.5195.125
    Edge: 105.0.1343.53
    Firefox: 102.0.1
    Safari: 16.0

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Is there any workaround for this?

I have the same problem, except I'm not using SolidJS. I'm using React.
I've got my issue detailed in the discord discussions tab.
I'll copy some of the more important details here, as I don't expect everyone to have access to Discord.
Vite module resolution seemingly doesn't respect the exports package.json field
So BWIP-JS released V4, with an improved ES Module interface.
This paragraph is directly from the Discord thread

The only trouble is, Vite's module resolution isn't respecting the exports field in bwip-js's package.json. I was expecting Vite to resolve to line 8 of https://github.com/metafloor/bwip-js/blob/master/package.json (exports.browser.import), but it appears to be resolving to line 9 instead (exports.browser.require). Either that, or it's resolving to line 33 (the legacy browser field).

I then linked this image of mine
 which I generated with vite --debug 2>&1 | tee vite-log.txt.
Note that ESBuild's module resolution is spec-compliant. It correctly prioritises the exports map over the legacy browser field.
I was able to confirm that Vite was ignoring the exports map, by using yarn patch bwip-js to point the legacy browser field to the ESM build instead of the CJS/legacy build. With that patch, I was able to see that Vite was resolving the ESM file, which confirmed the fact that the legacy browser field was being resolved instead of the exports map.
I was originally using Vite v3, but I tried moving to Vite v4 and got the exact same problems, and was able to reproduce the above yarn patch hack.
I've made a reproduction repo, just in case it's helpful.

Any traction on this? It's a blocker for me to start using the Bundler module resolution in my tsconfig

I've discussed with @sapphi-red and we think this is working as expected. The browser field currently overrides the exports condition in many bundlers today. This shouldn't be the case if browser field is a plain string, e.g. "browser": "./my-lib.browser.js" since it's a lower-priority entrypoint specifier.
The object form of browser field isn't specifying an entrypoint, but instead the files/modules it should replace if you're in a browser or browser-ish environment (e.g. workers). So on contrary it would have higher priority.
Closing this as expected behavior.


fix(persisted-scope): Prevent memory leaks, fixes #274

It's not actually a memory leak the plugin just breaks the .persisted-scope file, bloating it up (to 100mb after a few uses/restarts if you actually use the scope) to a point where it gets too large to work correctly.
This PR adds more pattern than strictly necessary to fix existing files seamlessly - well, hopefully at least.
Fixes #274

We may have to look into deleting files that are too large before reading them in the future because some users reported OOM crashes. In the PR tests Windows didn't report any memory usage raise when reading a 50MB, but no idea if Rust just handles this extremely well or if Windows just doesn't want to report it in time (even though it has like 5 secs lol)
Edit: Tested it with the current dev branch and Windows does report the insane memory usage correctly so maybe Rust is indeed doing a great job here 👀
Edit2: Yes, both of these edits were added before i initially posted this comment 😂


distro/rhel8: update RHUI client RPM for Azure SAP 8.10 images (COMPOSER-2254)

Since 8.10 is the last minor release of RHEL8, and it's not E4S one / has unversioned repos, its RHUI client rpm name is different.
See COMPOSER-2254

While the snapshot job is still running, the repository we need for this is already done, so let's use it here and we'll update the rest of the repo snapshots later.

The bug caught by the linter is fixed in #716

Discovered a fun little edge case race condition in the rules for rebuilding images involving the ostree containers.  The full rebuild in #716 (triggered by changing the osbuild commit ID) changes the commit ID inside an ostree container, but the manifest ID is the same.  The ref of the ostree container is based on the manifest ID.  So in this PR, even though the image definitions didn't change for edge-based images (disks and installers), the CI found a different commit ID so is rebuilding those.  This basically only happens if a pipeline rebuilds an ostree container and another pipeline uses it before the first pipeline manages to build the disk images and ISOs that use it.


The screenshot says "Create Outbound sync rule" which is not correct and misleading..

in "Negative filtering: "do not sync these"
The screenshot says "Create Outbound sync rule" which is not correct and misleading..
the article asks to create an Inbound rule..

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

ID: faed3c33-d05e-7a05-1327-27a79325f677
Version Independent ID: 151a61e3-ea34-7716-bbc8-66177f768b7b
Content: Azure AD Connect sync: Configure filtering
Content Source: articles/active-directory/hybrid/how-to-connect-sync-configure-filtering.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@AminPasha7 Thank you for pointing this out . We will have this reviewed and update the thread.

@AminPasha7 Thanks for the feedback. We will engage the content development team for further investigation, they will verify and perform necessary changes in the document as needed.
@billmath Can you have a look at this ?

@AminPasha7
Thank for submitting this. I have created a work item for it and will reviewing this in the coming weeks. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


Parallelizes protocol tests

Description
Breaks protocol tests into seperate CI jobs to speed up the workflow and mitigate effects of flakiness
Other changes
Adds support for running protocol tests by subfolder (ex. yarn test identity/ runs all tests under identity subfolder)
Tested
Testing infrastructure change so passing the existing tests is the best test we have.
Related issues

NA

Backwards compatibility
Not applicable

Just looking at the GitHub UI, doesn't look like these checks are required to pass for a PR to merge, which I believe they should?

Great catch. I'll look into how that's configured


Consider Including URDF file in addition to XACRO

Hello all,
I maintain another repository that uses the URDF/XACRO files in this one: https://www.mathworks.com/matlabcentral/fileexchange/65316-designing-robot-manipulator-algorithms
Right now, I have a script that takes the XACRO file and converts it to URDF with a few hard-coded changes, e.g., replacing expressions like "${PI}" with "3.1416"
Would it be possible to include an automatically generated URDF version of the XACRO file as well? You can do this with the "xacro" ROS package:
http://wiki.ros.org/xacro
Thanks, and happy to help with this as needed.

This issue will be closed since there were no actions for a while. You can reopen this issue to show this issue to the users whenever. Thanks.


Please add support for Wify Scene Switch

Scene switch Wify has 2 buttons with 2 programable scenes for each button "single click" and "long press". Tuya app does not provide "double click".
Device details are
{
"result": {
"category": "wxkg",
"functions": [],
"status": [
{
"code": "switch_mode1",
"lang_config": {
"click": "Single Click",
"double_click": "",
"press": "Long Press"
},
"name": "Switch 1",
"type": "Enum",
"values": "{"range":["click","press"]}"
},
{
"code": "switch_mode2",
"lang_config": {
"click": "Single Click",
"double_click": "Double Click",
"press": "Long Press"
},
"name": "Switch 2",
"type": "Enum",
"values": "{"range":["click","press"]}"
},
{
"code": "battery_percentage",
"lang_config": {
"unit": "%"
},
"name": "Battery",
"type": "Integer",
"values": "{"unit":"%","min":0,"max":100,"scale":0,"step":1}"
}
]
},
"success": true,

wxkg is already supported in #136 , have you tried the latest beta version?

the product name is a bit confusing, wxkg actually is wireless switch, represents a physical stateless button. It can't be controlled in the homekit app, but can be used as a trigger to do some automation (requires home hub).

Thank you. I updated to the latest version and the stateless switch is now exposed to homekit.
Possible bug - while activating a scene all scenes seem to get activated, it seems to be looping through and triggering all buttons with the click of any buttons.
Automations configured to trigger with single click button 1 get triggered when button 2 is pressed, along with automations configured for button 2
The switch is exposed as a single switch and only one automation is possible, instead of 4

Do you have any logs during these problems happened? Maybe related to https://github.com/0x5e/homebridge-tuya-platform/issues/125#issuecomment-1342043783 if you can see some warnings about "Message received with wrong order".
About the last issue I have no idea now... switch_mode1 and switch_mode2 should be created with two buttons in one accessory, can you take a picture of your switch's setting page?

Thank you. Below is the log file, please let me know if this helps
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] Message received with wrong order.
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] LastMessage: dataId = xxx, t = xxx
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] CurrentMessage: dataId = xxx, t = xxx
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] Fallback to use API fetching the latest device status.
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenAPI] Request:
method = get
endpoint = https://openapi.tuyaus.com
path = /v1.0/iot-03/devices/xxx/status
query = undefined
headers = {
"t": "xxx",
"client_id": "xxx",
"nonce": "xxx",
"Signature-Headers": "client_id",
"sign": "xxx",
"sign_method": "HMAC-SHA256",
"access_token": "xxx",
"lang": "en",
"dev_lang": "javascript",
"dev_channel": "homebridge",
"devVersion": "1.7.0-beta.5"
}
body = null
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] onMessage:
topic = cloud/token/in/xxx
protocol = 4
message = {
"dataId": "xxx",
"devId": "xxx",
"productKey": "xxx",
"status": [
{
"1": true,
"code": "switch_led_1",
"t": "xxx",
"value": true
}
]
}
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] onMessage:
topic = cloud/token/in/xxx
protocol = 4
message = {
"dataId": "xxx",
"devId": "xxx",
"productKey": "xxx",
"status": [
{
"1": true,
"code": "switch",
"t": "xxx",
"value": true
}
]
}
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] Message received with wrong order.
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] LastMessage: dataId = xxx, t = xxx
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] CurrentMessage: dataId = xxx, t = xxx
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] Fallback to use API fetching the latest device status.
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenAPI] Request:
method = get
endpoint = https://openapi.tuyaus.com
path = /v1.0/iot-03/devices/xxx/status
query = undefined
headers = {
"t": "xxx",
"client_id": "xxx",
"nonce": "xxx",
"Signature-Headers": "client_id",
"sign": "xxx",
"sign_method": "HMAC-SHA256",
"access_token": "xxx",
"lang": "en",
"dev_lang": "javascript",
"dev_channel": "homebridge",
"devVersion": "1.7.0-beta.5"
}
body = null
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] onMessage:
topic = cloud/token/in/xxx
protocol = 4
message = {
"dataId": "xxx",
"devId": "xxx",
"productKey": "xxx",
"status": [
{
"1": true,
"code": "switch",
"t": "xxx",
"value": true
},
{
"2": 0,
"code": "countdown_1",
"t": "xxx",
"value": 0
}
]
}
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] Message received with wrong order.
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] LastMessage: dataId = xxx, t = xxx
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] CurrentMessage: dataId = xxx, t = xxx
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] Fallback to use API fetching the latest device status.
[2022-12-12, 10:16:37 p.m.] [@0x5e/homebridge-tuya-platform] [TuyaOpenAPI] Request:
method = get
endpoint = https://openapi.tuyaus.com
path = /v1.0/iot-03/devices/xxx/status
query = undefined
headers = {
"t": "xxx",
"client_id": "xxx",
"nonce": "xxx",
"Signature-Headers": "client_id",
"sign": "xxx",
"sign_method": "HMAC-SHA256",
"access_token": "xxx",
"lang": "en",
"dev_lang": "javascript",
"dev_channel": "homebridge",

Are they the same device? From the log, the code switch and switch_led_1 didn't contained in your wxkg's schema list [switch_mode1, switch_mode2, battery_percentage]

I have a few tuya devices and this one might have got included in the log while i was copying and pasting Sorry about that

Ok, does the problem still exist? You can see "readme - troubleshooting" to get all device info list json, which maybe much complete than find it through debug logs.

I updated to the latest beta and the problem still exists. I am currently using the switch with only one click and it works well.
I had already posted the device information before so am not sure what more to provide. I am happy that at-least one click works which is very useful, Thank you.
After updating to the latest beta, 2 of my dimmers do not report the correct brightness in homekit. I can turn the switch on and off but am unable to set the brightness level (which was working previously) So something changed in the beta version.
This switch is made by Topgreener below is the device information
{
"result": {
"active_time":<PHONE_NUMBER>,
"category": "dj",
"category_name": "Light Source",
"create_time": XX,
"gateway_id": "",
"icon": "smart/icon/XX",
"id": "XX",
"ip": "XX",
"lat": "XX",
"local_key": "XX",
"lon": "XX",
"model": "TWF500D",
"name": "Second floor",
"online": true,
"owner_id": "XX",
"product_id": "XX",
"product_name": "DimmerSwitch",
"sub": false,
"time_zone": "-05:00",
"update_time":<PHONE_NUMBER>,
"uuid": "XX"
},
"success": true,
"t": XX,
"tid": "XX"

I need mqtt log to see why the second button didn't work, please see the updated readme: https://github.com/0x5e/homebridge-tuya-platform#3-collecting-logs
During collecting the log, please don't operate other device, and operate scene switch (button 1 and button 2, short press and long press) physically, and there should have some logs. This's important for me to diagnosis.
The dimmer's code didn't changed during 1.7.0 beta versions. Is it related to #32 ? If not, please create a new issue. And your information is missing status and schema property, I suggest get device info from local json file: https://github.com/0x5e/homebridge-tuya-platform#1-get-device-information
And also, I need the mqtt log during you setting the brightness level physically.
Thanks for the feedback!


Support for Multicast messages in service to service communication

This is a feature request to be able to send a multicast request to all instances of a service using Envoy with Service Discovery.
This would be useful for cases like a fully replicated cache, where cache update one node needs to be multicasted to all other nodes.
Desired behavior:

Multicast request indicated to envoy probably through a header.
Usable with gRPC (HTTP2).
Request can be best effort without any retries by default or with configurable retries. Any failure to one of the multicast nodes would return request failure.

How does service discovery figure into this?

I wasn't clear about this earlier. The intention is that multicast request would be sent with service name and Envoy would query service discovery for all instances of this service to use that as destination pool of multicast request.

Sounds like an interesting feature. I think a design doc or more detailed inline proposal would be the next step here.

@srujann agree with @htuch that a design doc would be useful here. This isn't hard to implement but it's not trivial either. I would probably implement along the lines of the shadow writer in the router and create the ability to fire and forget the request to all of the healthy backends.


Add a changelog entry for the performance improvement of std.algorithmm.iteration.joiner

Follow-up to https://github.com/dlang/phobos/pull/6492
Preview:

@wilzbach feel free to force merge as Jenkins is still marked as required.

Can we make that gist live under the dlang organisation, instead of your personal account ?

I'm not sure there's an easy way to do so :/
We could create a special benchmark repository, but I'm not sure it's worth the effort.
If you are worried about the link being dead in the future, we could link to the respective PR instead (though of course you never know what will happen to GitHub in the MS-age).

(though of course you never know what will happen to GitHub in the MS-age)

Well... Doesn't that reasoning applies to gist as well ? :)
Do you see any way to add it here ?


DynamicForm does not load or save correctly when field name starts with special character

Category
[ ] Enhancement
[*] Bug
[ ] Question
Version
Please specify what version of the library you are using: [3.5.0]
Expected Behavior
DynamicForm should display and save all field values, regardless of internal field name in SharePoint
Observed Behavior
When a field is created in SharePoint with a special character at the start of the field name, SharePoint replaces this character with a unicode representation like \_x####\_ - where #### is the unicode character reference. For example: if you name a field "% Complete" - SharePoint will call this field "_x0025_Complete" internally. To complicate matters further, the property returned from the API will be named "OData__x0025_Complete". At present, because of this difference, fields named in this way will not work in DynamicForm.
This does not affect fields where a special character is used in the middle or at the end of the field name. Only those used at the beginning.
Steps to Reproduce

Add a SharePoint field to a list, beginning with a special character such as %
Create or edit a list item to ensure that this field has a value
Amend ControlsTest.tsx#L886 to uncomment and display the DynamicForm control, point this control at the list and list item, then view the test in Workbench
Observe that the list item value does not load correctly, and values cannot be saved back to the list

Proposed Solution
The necessary data to fix this issue is already being retrieved. Instead of using field.InternalName, use field.EntityPropertyName. In most cases this matches field.InternalName - but it also matches the field property name as the SharePoint API expects it.
So on DynamicForm.tsx#L326 use:
defaultValue = item[field.EntityPropertyName];

and on DynamicForm.tsx#L458 use:
columnInternalName: field.EntityPropertyName,

Any idea why the "Person Group" type fields on the list, renders an arbitrary (user) value when left blank?
Any workaround to mitigate the behavior


Change order of subarrays to cover all cases

Since in all tests each subarray has a larger max than the previous subarray, if the user does not reset max for each subarray, the tests still pass.

Verified that this works. Thank you for your awesome contribution to our open source project, @biellls!


adds maximum tensor op

@sunny-g, do you want to keep this PR open? @nathanielsimard has added min and max ops, so you want to look into them. This has been open for some time and it's out of date. Let us know. Thanks.

Sorry about that, let's close it!


fix space in OriginalName issue

If OriginalName contains spaces, it needs to be wrapped in double quotes to be interpreted as one string.

Thanks for the PR @FH-Inway!  This PR is under review -- it initially looks good!


optimize: 2.x add config type,registry type,store mode log info

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
fixes #5209
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #5344 (997170a) into 2.x (a20ad42) will increase coverage by 0.00%.
The diff coverage is 60.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff            @@
##                2.x    #5344   +/-   ##
=========================================
  Coverage     48.86%   48.86%
- Complexity     4312     4314    +2
=========================================
  Files           771      771
  Lines         27290    27299    +9
  Branches       3396     3396
=========================================
+ Hits          13334    13339    +5
- Misses        12554    12558    +4
  Partials       1402     1402

Impacted Files
Coverage Δ

...seata/discovery/registry/MultiRegistryFactory.java
0.00% <0.00%> (ø)

...a/io/seata/discovery/registry/RegistryFactory.java
0.00% <0.00%> (ø)

...ain/java/io/seata/config/ConfigurationFactory.java
57.35% <100.00%> (+0.63%)
:arrow_up:

...o/seata/server/coordinator/DefaultCoordinator.java
45.45% <100.00%> (+0.64%)
:arrow_up:

...ava/io/seata/server/lock/LockerManagerFactory.java
61.11% <100.00%> (+4.86%)
:arrow_up:

...in/java/io/seata/server/session/SessionHolder.java
58.76% <100.00%> (+0.42%)
:arrow_up:

...in/java/io/seata/server/session/GlobalSession.java
70.30% <0.00%> (-0.76%)
:arrow_down:

...erver/storage/file/session/FileSessionManager.java
49.04% <0.00%> (-0.64%)
:arrow_down:

...rage/redis/store/RedisTransactionStoreManager.java
76.34% <0.00%> (+0.25%)
:arrow_up:

... and 1 more


BUG: Channel-Adjacency for Neuromag122

Describe the bug
When I try to get the channel-adjacency-matrix for Gradiometer from Neuromag122-Data with find_ch_adjacency
, a matrix with dimensions 306x306 is given (when read from ./mne/channels/data/neighbors/neuromag122_neighb.mat).
And when I try to compute the adjacency with mne.channels.channels._compute_ch_adjacency, I get the following result:
File "C:\Users\marti\anaconda3\envs\mne\lib\site-packages\mne\channels\channels.py", line 1426, in _compute_ch_adjacency
xy = _find_topomap_coords(info, picks, sphere=HEAD_SIZE_DEFAULT)
File "C:\Users\marti\anaconda3\envs\mne\lib\site-packages\mne\channels\layout.py", line 624, in _find_topomap_coords
pos = _auto_topomap_coords(
File "C:\Users\marti\anaconda3\envs\mne\lib\site-packages\mne\channels\layout.py", line 723, in _auto_topomap_coords
raise ValueError('The following electrodes have overlapping positions,'
ValueError: The following electrodes have overlapping positions, which causes problems during visualization:
MEG 001, MEG 002, MEG 003, [...]

Steps to reproduce
import mne
data_path = "./neuromag122_test_file-raw.fif"
info = mne.io.read_info(data_path)
# Trying the load the channel-adjacency
ch_adjacency = mne.channels.find_ch_adjacency(info, 'grad')
print(ch_adjacency[0].shape)
# Trying to bypass mne.channels.read_ch_adjacency()
mne.channels.channels._compute_ch_adjacency(info, 'grad')

Expected results
A 122x122 Matrix
Actual results
A 306x306 Matrix and a Value Error
Additional information
I uploaded an anonymized raw-file from our Neuromag122 for reproduction of the issue.

I dug into the problem and other functions like plots are working so probably the problem lies with how the paired gradiometer-channels in Neuromag122 are handled by _compute_ch_adjacency. For the info of my test-file, line 1412 in mne/channels/channels doesn't get True. The Coil-Type of Neuromag122 is FIFFV_COIL_NM_122

I will open a draft PR with a possible fix, but I still don't know why the saved matrix from fieldtrip in /channels/data/neighbours is 306x306

When I try to get the channel-adjacency-matrix for Gradiometer from Neuromag122-Data with find_ch_adjacency, a matrix with dimensions 306x306 is given (when read from ./mne/channels/data/neighbors/neuromag122_neighb.mat).

I think we got this file from FieldTrip -- looks like there was a similar error about 8 years ago. I've opened and upstream issue: https://github.com/fieldtrip/fieldtrip/issues/1676

Sound like the correct fix is to kill the .mat file, can you remove it from the repo using git rm and remove the commented-out lines in this PR?

Sorry meant for that comment to land in #8891


[terra-avatar] Add Accessibility Guide

Summary

Adding accessibility guide for terra-avatar.

Deployment Link

https://terra-core-deployed-pr-#.herokuapp.com/
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

I'm not sure why these CI and CD pipelines are failing, haven't changed any other files apart from these.


testing out coderabbit ai

Summary by CodeRabbit

New Features

Introduced a new section in the About page that displays various services through ServiceCard components.

Bug Fixes

Corrected a typo in the implementation of the ServiceCard component to ensure proper data display.

I love coderabbit


Redirect after deleting an item

if a item is deleted with the delete modal, the user should be redirected to the next possible item. This should normally have a lower id then the deleted one.
-> add a little nothing found image to display if no images are available or the last one is deleted.

added a redirect to the previous item after a deletion. Maybe adding later a better solution.


Increase go version to 1.12

Motivation
Increase go version to 1.12.X
Pull Request status

[x] Initial implementation
[ ] Refactoring
[ ] Unit tests
[ ] Integration tests

Coverage remained the same at 89.509% when pulling 4d1691665f8eade6c2274d06fb5ffcc5f676b4c3 on increasegover into 27af564c846de652d952197c3f2e48708be09786 on master.


cli tool doesn't respect $KUBECONFIG file location

For example:
$ conduit stat pod
stat .kube/config: no such file or directory

Thanks for reporting @DylanGraham. That was an oversight on our part. To work around this for now, you can explicitly include the --kubeconfig flag, but we'll update to support the env variable as well.


Update finder-test.sh

Modify WRITEDIR to ensure writer.sh creates missing directories

@lnxblog I think this could have a similar problem in that the student could create once by hand and then all future runs would succeed.
The way I was suggesting to change this is to modify https://github.com/cu-ecen-aeld/assignment-autotest/blob/3d8e2463334d7cd004a8f558a0846969689a3ba8/test/assignment1/assignment-test.sh#L21 to add logic which verifies writer.sh completes with success (and actually writes the file with the correct content) when we specify an arbitrary path like /tmp/aeld-data/$RANDOM/testfile.txt

Looking great @lnxblog , thanks!
Next steps:

Rebase this branch into a single commit.
Add a pull request at https://github.com/cu-ecen-aeld/assignments-complete-private/tree/assignment-1-complete with a merge of this branch content and showing a passing unit test.
Add a pull request at https://github.com/cu-ecen-aeld/assignments-complete-private/tree/assignment-2-complete merging this branch as well, prove the assignment 2 test still completes successfully as well.


Add more audio data to DALI_Extra

Signed-off-by: szalpal<EMAIL_ADDRESS>

I see that libri speech is on CC BY 4.0 and you need to (according to) -
You must give appropriate credit, provide a link to the license, and indicate if changes were made. You may do so in any reasonable manner, but not in any way that suggests the licensor endorses you or your use.

Please stick to the license.

Done. Accidentally removed it before

Please stick to the license.

Done. Accidentally removed it before

Please update README as well. We have an exception for MNIST there.


BUG: Fix the Paloma alert flapping from unresponsive webserver

added nginx caching for wcheck

Closing as fixed.


Update sites.yml

added fem.lol

This PR is:

[X] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[X] I used the uncompressed size of the site
[X] I have included a link to the GTMetrix report
[X] The domain is in the correct alphabetical order
[X] This site is not an ultra lightweight site
[X] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[X] Check to confirm

- domain: fem.lol
  url: https://fem.lol/
  size: 302
  last_checked: 2023-9-22

GTMetrix Report: https://gtmetrix.com/reports/fem.lol/TTSEEuqF/

Thanks!


in SSMS, Web Browser: Need instructions on how to configure Microsoft Edge instead of Internet Explorer

in SSMS, when I hit Ctrl+Alt+R to open the Web Browser window, I get the message:

This browser is no longer supported.
Upgrade to Microsoft Edge to take advantage of the latest features, security updates and technical support.

I have MS Edge installed.  Please add instructions to this page to show how to change the web browser used to MS Edge.

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

ID: c86abd88-6f91-9854-dec2-35137539023d
Version Independent ID: 117f69ca-6645-34a9-01c6-9180eadf8b14
Content: Web Browser - SQL Server Management Studio (SSMS)
Content Source: docs/ssms/sql-server-management-studio-web-browser.md
Product: sql
Technology: ssms
GitHub Login: @markingmyname
Microsoft Alias: maghan

@gbritton1 -- Gerald, thank you for your feedback.  Please try setting Edge as your default browser by going to Windows > Settings > Default apps, and let me know if this solves your issue.

Changing the default on windows to edge had no effect.  It seems that SSMS
doesn't respect that setting.  So, we need instructions as to how to do
this in SSMS
On Fri, Feb 11, 2022 at 3:54 PM William Anton Rohm @.***>
wrote:

@gbritton1 https://github.com/gbritton1 -- Gerald, thank you for your
feedback. Please try setting Edge as your default browser by going to
Windows > Settings > Default apps, and let me know if this solves your
issue.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/7299#issuecomment-1036614043,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC5R66HN7NFYI2XKI77FMYTU2VZPRANCNFSM5OENHSUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Gerald Britton, Pluralsight Author
LinkedIn Profile: http://ca.linkedin.com/in/geraldbritton
Pluralsight Courses: https://bit.ly/2WB2TdX https://bit.ly/2WB2TdX

@gbritton1 -- Gerald, thank you for your follow-up.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@markingmyname -- Mark, please look into this issue.

I got this response from DBA Exchange:
https://dba.stackexchange.com/questions/307496/how-to-change-the-web-browser-used-by-ssms-when-you-hit-ctrlaltr
[image: image.png]
Please add this option to SSMS
On Mon, Feb 14, 2022 at 6:13 PM William Anton Rohm @.***>
wrote:

@gbritton1 https://github.com/gbritton1 -- Gerald, thank you for your
follow-up. Please consider these resources:

SQL Server on Microsoft Q&A
https://docs.microsoft.com/answers/products/sql-server
DBA Stack Exchange
https://dba.stackexchange.com/questions/tagged/sql-server
Stack Overflow
https://stackoverflow.com/questions/tagged/sql-server

@markingmyname https://github.com/markingmyname -- Mark, please look
into this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/7299#issuecomment-1039668152,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC5R66EIFK254FPDIL73CZ3U3GEBLANCNFSM5OENHSUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Gerald Britton, Pluralsight Author
LinkedIn Profile: http://ca.linkedin.com/in/geraldbritton
Pluralsight Courses: https://bit.ly/2WB2TdX https://bit.ly/2WB2TdX

#assign: @erinstellato-ms

@erinstellato-ms, let's discuss this one in more detail.
I was able to try something.

@markingmyname we still need to discuss :)

Apologies if this isn't the right place, but I ran into a problem where I couldn't log in to my organisation's Azure databases through SSMS v18.12.1 because the "keep your account secure" page now doesn't support the Internet Explorer browser which is embedded in SSMS. This is the page which asks you to verify your phone number/authenticator app etc.

Clicking the mysecurityinfo link at the bottom opened up a normal Internet Explorer window, which I could sign in with and I can use SSMS normally now. Since this workaround isn't obvious, I think it would be better to have an option to configure an external browser for SSMS to use. Or perhaps SSMS could be updated to use an embedded Edge browser, which could be updated with each new release of SSMS.

As per @erinstellato-ms response,
We will now proceed to close this thread. If there are further questions, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue. To see what the community wants, please visit the SQL Server User Feedback site or the Q&A site.

@markingmyname The "workaround" @ajf2 suggested does not address the original problem, which is opening the web browser using the Ctrl-Alt-R keyboard shortcut in SSMS.  That problem still persists.  Worse yet, the workaround is only for those signing in to Azure databases if I understand correctly.  Many do not have access to Azure databases.  Worst of all, Internet Explorer is not installed on my up-to-date copy of Windows 11.
Visiting the page Disable and enable Internet Explorer on Windows I see the warning:

Warning
The retired, out-of-support Internet Explorer 11 desktop application has been permanently disabled through a Microsoft Edge update on certain versions of Windows 10. For more information, see Internet Explorer 11 desktop app retirement FAQ.

Few would install an out-of-support package just for this purpose, since it may well expose worse problems.
tldr: 1. The workaround is not usable by all since it requires Azure. 2. Even if you have Azure, you may have to install an obsolete unmaintained program (IE).  3. The workaround may not fix the original problem (I cannot test it since I have no Azure database to log in to).
Please reopen the issue and help us find a proper solution.


interpolation of imagenet

Hi, sorry to trouble you again. I have successfully your results on cifar10, and I plan to reproduce the results on ImageNet.
I follow your configs for imagenet and my result is lower than yours about 2%.
I would like to know the interpolation's type used in your paper for imagenet.
As most paper's only use Bicubic, and your paper use "random" for imagenet.
Thanks very much!

Hi,
I'm not sure if that's the issue here. Can you share your logs?
Also, can you verify your ImageNet copy's MD5?

https://drive.google.com/file/d/1XuQTQtFKY2qZfMWJxzoPBOKZNYoqezAF/view?usp=sharing
Hi, I upload my log to google drive, I hope I can compare it with your log.
Thanks very much!

I'm sorry, this log looks very different from the logs our training script provides. Are you using the same training script? Can you share the full command as well?

Thanks, I have found the difference, for I make lr = batch_size *device_num / 512 * lr,
I will fix it and try again. Thanks for your help!


Card with only one paragraph element in the card-body has unnecessary margin at the bottom

When a bootstrap card only contains one paragraph element
 in its card-body, then the paragraph still has its default margin-bottom: 1rem;     This should be removed for the last(!) paragraph in a card.
Simple example HTML

This paragraph has a default margin-bottom. It shouldn't

Solution
Add the following CSS
.card-body p:last-child {
  margin-bottom: 0;
}

JSFiddle
https://jsfiddle.net/qhv016wa/

Pull request: https://github.com/twbs/bootstrap/pull/31349

There is already a class card-text for it. Or you can use utility class mb-0.
https://github.com/twbs/bootstrap/blob/c7bc7a31bbc11c3f9af3ad317581cbd13c7bb5f1/scss/_card.scss#L62-L64

There is the exact same issue with form inside card, last .form-group having an unnecessary margin-bottom.
You can see it in the doc on this specific form: https://getbootstrap.com/docs/4.5/components/forms/#horizontal-form
It's probably trickier tho, since it's not mandatory to wrap submit button inside a .form-group.

In v5 .form-group has been dropped for margin utilities

And what if the last item is an .alert for instance? Since .alert has a bottom margin it will be the same issue. Same for many more components. IMHO, you should use margin utilities, as @rohit2sharma95 said.

Ok. Thank you for the card-text hint. Didn't see that one.
Of course manually setting a (non-)margin is always possible.

An alert has a default bottom margin. A bootstrap card doesn't.   => Should at least be mentioned in the docs.

This should ideally be addressed with the .card-text class, or with utilities IMO. People customize Bootstrap's HTML all the time, and we never really know what folks will do with it that could break our CSS.
For example, this code affects every
 within nested content, not just the immediate paragraphs.
.card-body p:last-child {
  margin-bottom: 0;
}

Of course we could limit it with a child selector >, but then that locks the HTML and creates a more specific selector.

Closing as a won’t fix per the above comment.


_AtomicShims: simplify double word setup

Unify the double word path by using a bit more macro expansion.  The two
differences between 32-bit and 64-bit is:

__uint128_t vs uint64_t
64-bit shifts vs 32-bit shifts

We can simply use a macro to expand out the underlying C type rather
than bifurcating the definitions.  For the second difference, we can
simply use the __INTPTR_WIDTH__ for the shifts.  We explicitly use
macros for the type-definitions to hide the typedef from Swift.
Use the C11 aggregate constructor to further reduce the code.

While trying to process the structure here which causes problems for Windows, this duplication was really bothering me.  This reduces the amount of code by about 2/3rds with the same interface set and does so semi-portably (it depends on clang still and doesn't work sufficiently for Windows yet for the same reason as before via the expansion of the prepare/load/store functions.

@swift-ci please test

CC: @lorentey

@swift-ci test


can't download dataset

hi! Very much appreciate your work.
However, I can't download the dataset through one-drive. I have tried using the web browser and other software to help download, but it didn't work :( .
Can u provide other ways to download the data, such as the baidu disc?
Thank you very much!!!!

Hi, I am uploading the data to the Baidu disk. You can check it later today. The share link is here https://pan.baidu.com/s/1U48TK1FdNU4jZuNShJPGdA?pwd=gg65.

Hi, I am uploading the data to the Baidu disk. You can check it later today. The share link is here https://pan.baidu.com/s/1K9t_atenvjuntcYTp4a_8A?pwd=ciyy.

thank you very much!


Launch contracts-helpers package

Launch Typescript library with helper functions for interacting with contracts.
Things like species.ts should be there to help encoding and decoding genes for breeding.

/cib


ffmpeg/ffprobe path in autoprocess.ini changed

I had an open issue last week related to files being created without sound. I solved this by updating the paths for ffmpeg and ffprobe. Today I found another file created without sound and noticed that the paths for ffmpeg and ffprobe had reverted. Any idea why this is happening or how to prevent it from happening in the future?

Unless you're using docker the script won't be modifying your FFMPEG paths?
I'm not even sure what this means

What was your before/after?

I modified the paths to: "/volume1/@appstore/ffmpeg/bin/ffmpeg" and "/volume1/@appstore/ffmpeg/bin/ffprobe" the paths reverted to "ffmpeg" and "ffprobe"
I am running sonarr in docker. I am testing right now by removing my /usr/local/sma path from docker that pointed to sma directory. Does that make sense?

Not sure why you would approach it that way.
Again operating on limited information here since you posted so little and dodged the templated info on all previous issues but assuming you're using one of the official docker containers, the container automatically makes configuration changes to autoProcess.ini via the included update.py file (part of the docker containers, not part of SMA). Seems like you're trying to get some alternate FFMPEG binary into the container in some fashion or another (I'm guessing this is where the issue with your previous FFMPEG build stemmed from since the included builds should support all those libraries that were disabled on your previous issues)
If you want to sub in an alternate version of FFMPEG, you can either adjust the FFMPEG URL via the SMA_FFMPEG_URL environment variable or if you just have alternate binaries you want to use, just replace the existing ones by mounting them using docker volumes, the setup portion of the script only pulls/downloads FFMPEG if it's not found
The SMA_FFMPEG_URL will only change things on first launch though, you can reference the startup script to see how that's handled (\etc\cont-init.d\90-sma-config). This is also where the update.py file is called that makes the configuration changes
If you're using a custom made docker container or a different docker container other than sonarr-sma/radarr-sma/sma-mod then none of  this applies


Test not found when rerun in VsTest on VSTS

Environment: VSTS
Task: Visual Studio Test
When using the "Rerun failed tests" option the rerun can't find the failed test.
The logs look like this:
**************** Rerunning failed tests for Test run 4951 *********************
Max attempts: 3; Current attempt: 1; Failed test cases threshold: 30; Failed test cases max limit: 0
[command]C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe @C:\TestAgents\1\_work\_temp\tmpF076.tmp
Microsoft (R) Test Execution Command Line Tool Version 15.0.26929.2
Copyright (c) Microsoft Corporation.  All rights reserved.
vstest.console.exe /TestCaseFilter:"FullyQualifiedName=Easyfy.Commerce.Web.TestsUI.CheckoutTests.Easyfy.Commerce.Web.TestsUI.CheckoutTests.AddToBasket"
"C:\TestAgents\1\_work\r4\a\uitests\Easyfy.Commerce.WebTestsUI.dll"
/logger:"trx"
/TestAdapterPath:"C:\TestAgents\1\_work\r4\a"
Starting test execution, please wait...
Information: [xUnit.net 00:00:00.6904083]   Discovering: Easyfy.Commerce.WebTestsUI
Information: [xUnit.net 00:00:01.1349037]   Discovered:  Easyfy.Commerce.WebTestsUI
Information: [xUnit.net 00:00:01.1473315]   Starting:    Easyfy.Commerce.WebTestsUI
Information: [xUnit.net 00:00:01.1998463]   Finished:    Easyfy.Commerce.WebTestsUI
Warning: No test is available in C:\TestAgents\1\_work\r4\a\uitests\Easyfy.Commerce.WebTestsUI.dll. Make sure that installed test discoverers & executors, platform & framework version settings are appropriate and try again.
Information: Additionally, you can try specifying '/UseVsixExtensions' command if the test discoverer & executor is installed on the machine as vsix extensions and your installation supports vsix extensions. Example: vstest.console.exe myTests.dll /UseVsixExtensions:true
**************** Completed test execution *********************

All tests are found in the initial run and all pass except AddToBasket in this example.
We suspect that the problem is the /TestCaseFilter which seems to have the namespace (Easyfy.Commerce.Web.TestsUI.CheckoutTests) added twice.
We tested this by running the test locally with /TestCaseFilter:"FullyQualifiedName=Easyfy.Commerce.Web.TestsUI.CheckoutTests.AddToBasket" and the test was found.
The tests are Selenium tests run with XUnit on a local server.
Complete test run log: rerun-test-not-found.txt

@holgoloid sorry for delayed response. Can you please let us know what type of tests are these (datadriven?)

No worries!
The tests are simple Selenium C# tests without any data inputs or connections.
AddToBasket from the example above looks like this:
namespace Easyfy.Commerce.Web.TestsUI {
  public class CheckoutTests : UITestClass {
    [Fact, UITest]
    public async Task AddToBasket() {
      // Go to some pages, click some buttons..
    }
  }
}

UITest is a custom attribute.

@holgoloid can you please share the TRX after first run?

Sure, the trx doesn't seem to be saved from the run in the logs above but I ran a new deployment, deliberately failed the same test and got the same behaviour.
Here is the trx:
AgentRunner_EASYBUILD 2018-05-07 15_48_19.zip

@holgoloid thanks for sharing information. It's confirmed as bug, fix is committed and we will roll out this fix within 2-3 weeks.


loading RData objects fails

HI Vladimir,
I copied locally the two RData objects found at
04.PooledResults/OTU_table_long.RData
04.PooledResults/OTU_table_wide.RData

but RStudio does not like them
> load("~/work/NGS-Analyses/NextITS_analysis/NextITS_Results_HiFi_reads/OTU_table_long.RData")
Error in load("~/work/NGS-Analyses/NextITS_analysis/NextITS_Results_HiFi_reads/OTU_table_long.RData") :
  bad restore file magic number (file may be corrupted) -- no data loaded
In addition: Warning messages:
1: In readChar(con, 5L, useBytes = TRUE) :
  truncating string with embedded nuls
2: file ‘OTU_table_long.RData’ has magic number 'X'
  Use of save versions prior to 2 is deprecated
> load("~/work/NGS-Analyses/NextITS_analysis/NextITS_Results_HiFi_reads/OTU_table_wide.RData")
Error in load("~/work/NGS-Analyses/NextITS_analysis/NextITS_Results_HiFi_reads/OTU_table_wide.RData") :
  bad restore file magic number (file may be corrupted) -- no data loaded
In addition: Warning messages:
1: In readChar(con, 5L, useBytes = TRUE) :
  truncating string with embedded nuls
2: file ‘OTU_table_wide.RData’ has magic number 'X'
  Use of save versions prior to 2 is deprecated

Hello Stephane,
Please use the readRDS function. E.g.,
OTU_table_long <- readRDS("~/work/NGS-Analyses/NextITS_analysis/NextITS_Results_HiFi_reads/OTU_table_long.RData")

Thanks for reporting! I should add this information to documentation.


Playing gifs ?

Sorry to ask here, but it doesn't seem you have a community forum or server.
It says in the description "it can be used for UI purposes such as loading screen or even gifs", does that mean it supports .gif files ? Because I can't get it to load a gif file, I know Godot doesn't normally uses these files.
Godot version : 3.5 | Addon version : last I assume, the one on the asset library

Sorry to say but it does not directly handle GIFs. It was mostly hinting out that it's possible to recreate GIFs through sprite sheets. I'll be re-editing that part, since it might cause more confusion. As of current, I have yet to see direct GIF implementation in Godot. I can only hope they include it in Godot 4.


Wrong Return type for useStoryblok()

Describe the issue you're facing
When using the new useStoryblok() wrapper to use the live preview in Storyblok, the function only returns the story object. This prevents us from using the assets array returned by setting resolve_assets: 1, because the assets array is on the same level as the story in the original response.
Reproduction
https://stackblitz.com/edit/astro-sdk-demo-s3f9zj?file=src%2Fpages%2Findex.astro
Steps to reproduce
const story = await useStoryblok(`cdn/stories/${slug}`, getFetchParams(version, language), {}, Astro)

// there is no way to access the resolved assets array here

System Info
astro: 4.13.1,
@storyblok/astro: 5.0.0,

Used Package Manager
npm
Error logs (Optional)
No response
Validations

[X] Follow our Code of Conduct

Hi @samuelemde,
Thank you for raising this issue. The Astro live preview feature is currently experimental, but we have an upcoming release that will make this process simpler and address the concerns you've mentioned.


chore: update mainnet acp hash_type

Codecov Report

Merging #25 into develop will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop      #25   +/-   ##
========================================
  Coverage    43.98%   43.98%
========================================
  Files           37       37
  Lines          948      948
========================================
  Hits           417      417
  Misses         531      531

Impacted Files
Coverage Δ

app/services/send_capacity_service.rb
0.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 e88df11...6edc227. Read the comment docs.


Virtual cards: rename "expire date" -> "expiry date"

That might require a migration and a strategy for reverse compatibility since we stored the field as expireDate in the DB. We can also start with just the labels if we want a quick fix.

This was mostly done in https://github.com/opencollective/opencollective-api/pull/7439 + https://github.com/opencollective/opencollective-frontend/pull/7723, but we still need a small cleanup to finish
opencollective-frontend/components/edit-collective/AssignVirtualCardModal.js
  95,9:         expireDate: values.expiryDate,

opencollective-frontend/components/edit-collective/VirtualCard.js
  324,24:                     // expireDate should be removed once https://github.com/opencollective/opencollective-api/pull/7307 is deployed to production
  325,39:                     props.privateData.expireDate || props.privateData.expiryDate
  332,26:                       // expireDate should be removed once https://github.com/opencollective/opencollective-api/pull/7307 is deployed to production
  333,52:                       handleCopy(props.privateData.expireDate || props.privateData.expiryDate)


XamlCompiler improperly finds attached properties/methods

Create a new C# app and put this in MainPage.xaml.cs:
public static async Task SetImageSourceAsync(Windows.UI.Xaml.Controls.Image imageControl, SoftwareBitmap bitmap)
{
}

This causes the following incorrect code to be generated:
switch (longMemberName)
{
     case "App3.MainPage.ImageSourceAsync":
     userType = (global::App3.App3_XamlTypeInfo.XamlUserType)GetXamlTypeByName("App3.MainPage");
      xamlMember = new global::App3.App3_XamlTypeInfo.XamlMember(this, "ImageSourceAsync", "System.Threading.Tasks.Task");
      xamlMember.SetTargetTypeName("Windows.UI.Xaml.Controls.Image");
      xamlMember.SetIsAttachable();
      xamlMember.Setter = set_0_MainPage_ImageSourceAsync;
      break;
     }
    return xamlMember;
}

Which leads to this error:
global::App3.MainPage.SetImageSourceAsync((global::Windows.UI.Xaml.Controls.Image)instance, (global::System.Threading.Tasks.Task)Value);

Note that this is because the Xaml Compiler looks for static method that start with the Set. So if you hit this issue, you can workaround it by changing the method from SetImageSourceAsync to something like DoSetImageSourceAsync


Fix example in schema README

We should only download dependence without installing,
otherwise we'll probably get error:
go install: no install location for directory /home/qhuang/specs/schema outside GOPATH
For more details see: go help gopath
Signed-off-by: Qiang Huang<EMAIL_ADDRESS>

Append a patch to modify README to keep it consistent with Makefile.

LGTM

Through aaf05c4 looks good to me.

LGTM


Connection timeout

I turn on the server with this plugin, when I c

onnect to the server from a mobile device I get this error.

Assuming you're hosting locally, there's a high chance you haven't port forwarded. Other than that you may be using the wrong IP

Is it possible to open ports if I have mobile Internet?


Dropbox api(NodeJS) filesDownload issue

hi, I am exploring the dropbox api and trying to write the code for downloading files from Dropbox into a local path using nodejs. Seeing the documentation it only tells me to provide the path of the file in dropbox in the args object. Does anyone know where to provide the local path.

There's an example using of a download-style call in Node here:
https://github.com/dropbox/dropbox-sdk-js/blob/master/examples/javascript/node/download.js#L19
You don't directly supply a local file path for the resulting file data in the SDK method argument. Instead, the callback gives you the file data, which you can use as desired. (E.g., save it to a file using fs.writeFile, as shown in this example.)