Support proxying using vsock

Summary of the PR
Hi. I have been working on adding AWS Nitro Enclave emulation support in QEMU. Nitro enclave VMs are spawned from EC2 VMs and the enclave VMs always see the parent VM as CID 3 and the applications in the enclave VM talk to the parent VM using AF_VSOCK to CID 3. More details about this are in the cover letter for the patch posted in qemu-devel.
From the QEMU side, it would be cumbersome to run a separate VM with CID 3 with the necessary applications while running a nitro-enclave VM. So there was some discussion in the virtualization mailing list about how we could make the UX simpler for users without requiring to run  a separate VM with CID 3 and instead run the applications of the parent EC2 VM in the host machine. At the end, it was suggested by Stefano that we could use vhost-user-vsock for vsock emulation in QEMU for nitro-enclave and then in vhost-device-vsock we could add support for forwarding the vsock packets to the host machine. The discussion can be found in this thread.
This PR adds 2 new options to vhost-device-vsock: forward-cid and vsock-listen.
forward-cid: When this option is given (usually with the value 1), all connections from the guest VM will be forwarded to this CID to the same ports regardless of their destination CID.
forward-listen: This is a string of port numbers separated by '+'. The vhost-device-vsock will be listening to these ports in the host machine and the connections will be forwarded to the guest VM.
For testing, first run the vhost-device-vsock from this branch:
./target/release/vhost-device-vsock --vm guest-cid=4,forward-cid=1,forward-listen=9001,socket=/tmp/vhost4.socket

Then run a VM:
./qemu-system-x86_64 -machine q35,memory-backend=mem0 -enable-kvm -m 8G -nic user,model=virtio -drive file=fedora40.qcow2,media=disk,if=virtio --display sdl -object memory-backend-memfd,id=mem0,size=8G -chardev socket,id=char0,reconnect=0,path=/tmp/vhost4.socket -device vhost-user-vsock-pci,chardev=char0

Now you can run a AF_VSOCK server in the host machine and a client inside the VM that connects to the same port but any CID (e.g., CID 3) and verify that the server and client communication works. You can run a server inside the VM listening to port 9001 and run a client in host that connects to CID 1 port 9001 and verify that the communication works.
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR have Signed-Off-By trailers (with
git commit -s), and the commit message has max 60 characters for the
summary and max 75 characters for each description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[ ] Any newly added unsafe code is properly documented.

I see a few checks have failed with the same 4 tests that I added which require connecting/listening using AF_VSOCK with error "Operation not permitted" etc. The tests pass when I run them in a debian docker container. So I am guessing something needs to be updated about the VMs that run these tests.

I see a few checks have failed with the same 4 tests that I added which require connecting/listening using AF_VSOCK with error "Operation not permitted" etc. The tests pass when I run them in a debian docker container. So I am guessing something needs to be updated about the VMs that run these tests.

@andreeaflorescu do you know if our CI runners has vsock_loopback kernel module available?
It should be automatically loaded when opening an AF_VSOCK socket, and it provides the CID 1 handling, so it may be the issue if it's not available, since we can't bind CID 1 IIUC.

I see a few checks have failed with the same 4 tests that I added which require connecting/listening using AF_VSOCK with error "Operation not permitted" etc. The tests pass when I run them in a debian docker container. So I am guessing something needs to be updated about the VMs that run these tests.

@andreeaflorescu do you know if our CI runners has vsock_loopback kernel module available?
It should be automatically loaded when opening an AF_VSOCK socket, and it provides the CID 1 handling, so it may be the issue if it's not available, since we can't bind CID 1 IIUC.

Indeed on my laptop tests are working and I have this behaviour:
$ lsmod | grep vsock_loopback

$ cargo test -p vhost-device-vsock
...
test result: ok. 38 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.01s

$ lsmod | grep vsock_loopback
vsock_loopback         12288  0
vmw_vsock_virtio_transport_common    57344  1 vsock_loopback
vsock                  61440  3 vmw_vsock_virtio_transport_common,vsock_loopback,vmw_vsock_vmci_transport

If we can't enabled that kernel module, maybe we should just skip the test if we can't bind CID 1.

@stefano-garzarella I have now addressed some of the comments but I needed some input or had some questions about the others. I have not yet updated README and changelog but will do so after I get through all the reviews at the end. Can you please do another round of review and help me with the other reviews? Thanks!

I have now gone through the reviews and readme and changelog is updated as well. Thanks for the reviews @stefano-garzarella
Interestingly, the 'coverage-x86_64' run started failing but it didn't fail last time so to check if it was my latest push that did something, I reset the branch to the old commit (with a whitespace change to trigger the CI) and the 'coverage-x86_64' failed again which didn't fail before. So I am not sure why the new failure occurred. Did anything change on the CI side?

I have now gone through the reviews and readme and changelog is updated as well. Thanks for the reviews @stefano-garzarella
Interestingly, the 'coverage-x86_64' run started failing but it didn't fail last time so to check if it was my latest push that did something, I reset the branch to the old commit (with a whitespace change to trigger the CI) and the 'coverage-x86_64' failed again which didn't fail before. So I am not sure why the new failure occurred. Did anything change on the CI side?

We updated the CI, but we shouldn't touch anything about coverage, so this could be related to the test failure, since coverage runs the test, so if some tests are failing, I think also the coverage can't be calculated.
What about adding a third commit where we skip the new vsock tests if we can't create AF_VSOCK sockets?
I suggest a separate commit, so we can revert it when we fix our CI.

We updated the CI, but we shouldn't touch anything about coverage, so this could be related to the test failure, since coverage runs the test, so if some tests are failing, I think also the coverage can't be calculated.
What about adding a third commit where we skip the new vsock tests if we can't create AF_VSOCK sockets?

Yeah I can look into skipping the tests but did you get a chance to look into fixing the CI so that we are able to run them? Running them in a standard docker container works fine for me.

We updated the CI, but we shouldn't touch anything about coverage, so this could be related to the test failure, since coverage runs the test, so if some tests are failing, I think also the coverage can't be calculated.
What about adding a third commit where we skip the new vsock tests if we can't create AF_VSOCK sockets?

Yeah I can look into skipping the tests but did you get a chance to look into fixing the CI so that we are able to run them? Running them in a standard debian docker container works fine for me.

It depends on the host kernel as I mentioned here: https://github.com/rust-vmm/vhost-device/pull/706#issuecomment-2321312778
Our CI runners are provided by AWS, so we need some input from them. @roypat @andreeaflorescu do you have any advice?

@stefano-garzarella I have now introduced the "proxy" feature and made it default and "ignore"d the 4 tests that were failing in the CI. A lot of changes so please re-review all the commits.
I think in the CI for now we should make a change to run both "cargo build --release --no-default-features" "cargo test" and "cargo build --release" "cargo test" i.e., run without the feature and with the feature so that we cover everything. Please run locally with the feature turned off and on to verify that I haven't broken anything. Thanks!

@stefano-garzarella I have now introduced the "proxy" feature and made it default and "ignore"d the 4 tests that were failing in the CI. A lot of changes so please re-review all the commits.

Done, thanks!

I think in the CI for now we should make a change to run both "cargo build --release --no-default-features" "cargo test --no-default-features" and "cargo build --release" "cargo test" i.e., run without the feature and with the feature so that we cover everything. Please run locally with the feature turned off and on to verify that I haven't broken anything. Thanks!

Yeah, we have an issue opened in rust-vmm-ci: https://github.com/rust-vmm/rust-vmm-ci/issues/152
If you have time/desire, feel free to take a look at it.

We updated the CI, but we shouldn't touch anything about coverage, so this could be related to the test failure, since coverage runs the test, so if some tests are failing, I think also the coverage can't be calculated.
What about adding a third commit where we skip the new vsock tests if we can't create AF_VSOCK sockets?

Yeah I can look into skipping the tests but did you get a chance to look into fixing the CI so that we are able to run them? Running them in a standard debian docker container works fine for me.

It depends on the host kernel as I mentioned here: #706 (comment)
Our CI runners are provided by AWS, so we need some input from them. @roypat @andreeaflorescu do you have any advice?

Mh, I just connected to one of our CI runners, and it seems the module is available:
$ find /lib/modules/$(uname -r) -type f -name '*.ko' | grep vsock
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vmw_vsock_vmci_transport.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vmw_vsock_virtio_transport_common.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vsock_loopback.ko                       <------
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vmw_vsock_virtio_transport.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/hv_sock.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vsock.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vsock_diag.ko
/lib/modules/6.5.0-1024-aws/kernel/drivers/vhost/vhost_vsock.ko

maybe it's something to do with docker? :thinking:

Mh, I just connected to one of our CI runners, and it seems the module is available:

@roypat thanks for checking!

$ find /lib/modules/$(uname -r) -type f -name '*.ko' | grep vsock
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vmw_vsock_vmci_transport.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vmw_vsock_virtio_transport_common.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vsock_loopback.ko                       <------
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vmw_vsock_virtio_transport.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/hv_sock.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vsock.ko
/lib/modules/6.5.0-1024-aws/kernel/net/vmw_vsock/vsock_diag.ko
/lib/modules/6.5.0-1024-aws/kernel/drivers/vhost/vhost_vsock.ko

maybe it's something to do with docker? 🤔

yeah, it could be.
Can we make sure they are loaded?
Theoretically if they're not loaded, the first time an AF_VSOCK socket is created, the kernel loads af_vsock and vsock_loopback automatically, but it's possible that being inside a docker, this is not allowed.
One way to test it, is the following:
# start the container
# run this in the container
python3 -c "import socket; s = socket.socket(socket.AF_VSOCK, socket.SOCK_STREAM); s.bind((1, socket.VMADDR_PORT_ANY))"

Thanks. Good stuff!

python3 -c "import socket; s = socket.socket(socket.AF_VSOCK, socket.SOCK_STREAM); s.bind((1, socket.VMADDR_PORT_ANY))"

Sorry for the delay, finally got around to testing this. I got the following result:
$ lsmod | grep vsock
$ docker run --device=/dev/kvm -bla- rustvmm/dev:v44
root@05334d5ffd14:/workdir# python3 -c "import socket; s = socket.socket(socket.AF_VSOCK, socket.SOCK_STREAM); s.bind((1, socket.VMADDR_PORT_ANY))"
$ lsmod | grep vsock
vsock_loopback         12288  0
vmw_vsock_virtio_transport_common    61440  1 vsock_loopback
vmw_vsock_vmci_transport    40960  0
vmw_vmci              106496  1 vmw_vsock_vmci_transport
vsock                  69632  3 vmw_vsock_virtio_transport_common,vsock_loopback,vmw_vsock_vmci_transport
ssm-user@ip-172-31-68-60:/var/snap/amazon-ssm-agent/7994$

So it does seem to get loaded even from the docker container.

Yes, that sounds quite plausible. I'll make a note to look into having the CI runners just load the vsock module on startup! (although might not get to it this or next week :( )

Yes, that sounds quite plausible. I'll make a note to look into having the CI runners just load the vsock module on startup! (although might not get to it this or next week :( )

Great! BTW no rush at all, we already merged this PR, and we skipped the tests if we can't use AF_VSOCK.


Bug report：if the label in the current note is empty, this code will display all the notes with empty label

What happened?
The following code, I used to display notes with the same label, but after the plugin is upgraded to V0.5.24, if the label in the current note is empty, this code will display all the notes with empty label, in my view Come on, it's a bug.
If the current note label is empty, I expect the display result to be empty as well.
DQL
No response
JS
var tags = dv.current().file.tags.sort(t => t.name);
if (tags) {
  for (let tag of tags) {
	dv.paragraph(dv.pages(tag).where(p => p.file.name != dv.current().file.name).file.name.array().map(n => `[[${n}]]`).join("<br>")
	)
  }
}

Dataview Version
0.5.24
Obsidian Version
0.14.15
OS
Windows

Fixed in 0.4.25.


[ 🚧 WIP ] Added a program file for checking palindrome

Why:
Closes #1983
What's being changed:
Added ProgramFile For  Checking palindrome in C#

!Done


elan: wrap embedded linker

Motivation for this change
Lean 4 will soon start shipping its own toolchain including lld: https://github.com/leanprover/lean4/pull/795. This change is in preparation of that by wrapping the embedded ld.lld (Linux)/ld64.lld (macOS) in order to pass the correct interpreter. While we're at it, we drop the old GMP wrapper that has not been necessary for a long time, if at all.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Works for me.  If anybody wants to test this (it would be great if someone could test this on darwin), this is what you should try:
echo 'int main() { return 42; }' >main.c
nix shell .#elan -c leanc +Kha/lean4:4.0.0-bundle-rc main.c
./a.out


Request money - Delay when showing money request amount

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

Action Performed:
1-First login to different accounts which two different users, one as user A and the other as user B
2-[user A] Start chat with user B
3-[user A] Click in plus sign and select Request money
4-[user A] Write a number, click Next button and Request button
5-[user B] Enter in chat with user A
6-[user B] Open the message of request amount
Expected Result:
That user B see the amount of request money in the moment.
Actual Result:
The user B see by a moment the amount zero before the see amount correctly
Workaround:
Unknown
Platforms:

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

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

Version Number: v1.3.60-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/bf3b58a9-7f5d-4914-a2e6-92bf46de808c
https://github.com/Expensify/App/assets/115492554/8c891d0c-7349-43cc-a5a2-a7f9a9e587ea
Expensify/Expensify Issue URL:
Issue reported by: @Mauri-salazar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692743125769409
View all open jobs on GitHub

I think this is known and will be covered here - #26270

Closing in favor of https://github.com/Expensify/App/issues/26270


Is it possible to get value of $index similar to case without vs-repeat?

Since vs-repeat keeps only necessary elements in DOM it is expected that value of $index will be different from the case of without vs-repeat.
See the jsbin example here
But I have a peculiar use-case where in the rendered table is sorted and/or filtered and I need to get $index value as shown in without vs-repeat example.
To give you details about use-case, the check-boxes in example can be multi-selected. i.e. I select a particular check-box hold Shift and then scroll down and check another check-box. Doing so should select all the check-boxes in between rows. Now for this to be implemented, I will want to know actual index values as in case of without vs-repeat.
Is there a workaround to get $index values similar to without vs-repeat? I tried to derive it using totalHeight and scrollTo values, but is there a more reliable way?

Me too was silly :)


ERNN model  code does not follow the paper. could you help?

ξ(k+1) = ξ(k) + η(k)[F( τ; ξ(k) + hτ−1)−(ξ(k) + hτ−1)]; (10)  is the paper.
but in your code may follow as:
ξ(k+1) = ξ(k) + η(k)[F( τ; ξ(k) + hτ−1)−(ξ(k) + ξ(0)];

Thank you for your issue.
I uploaded wrong version by mistake.
The right code was already uploaded.


[docker] be more resilient to high system loads

What does this PR do?
When testing 6.0 on very high load, some asynchronous components might become unresponsive and be unsubscribed from docker event multiplexing. This mechanism was introduced to reduce the load on the docker daemon (avoiding 4 concurrent stream connections), while still ensuring the whole agent does not hang if one component fails.
This PR introduces the following changes:

raise the event processing timeout from 100ms to 500ms. Worst case scenario, 2sec delay on events is acceptable
homogenize error handling on all three consumers (AD listener, AD provider and tagger collector) to backoff for a second and try re-subscribing to events. If that happens, the following catchup is done:

AD listener: transform the init method building the initial state from the container list in a catchup method able to compare the current state with the container list and new/delete services for the diff. This makes sure we don't miss any service modification
AD provider: invalidate our cache, we'll pull all template in the next run
Tagger collector: continue on our merry way, if we missed containers, they'll be individually fetched and cached later

set the AD listener goroutines to be spawned by the main component instead of the listeners themselves, to improve consistency and make stacktraces more readable

Individual components might still temporarily miss their healthchecks, but they'll be able to recover once the spike is over.
Motivation
The agent should be able to recover, and keep monitoring the load to give insights on the source of the issue.

I'm a bit mixed about this, I'm confortable raising the timeout (https://github.com/DataDog/datadog-agent/pull/1446/commits/e8b66939055e92577b0c0f6e9e74b44414287292) but I'm not sure we need the sleep/catchup mechanism.
IMHO, we should rather investigate why the subscriber take more than 500ms to process the event, and if it's legit let the event streamer handle the back pressure on its subscriber (by buffering for example).
Also, we should probably send events asynchronously to avoid waiting the processing of all the subscribers https://github.com/DataDog/datadog-agent/blob/master/pkg/util/docker/event_stream.go#L171-L176

Superseeded by https://github.com/DataDog/datadog-agent/pull/1474


bpftrace: 0.16.0 -> 0.17.0

Description of changes

bpftrace v0.17.0 (release note). This includes BTF support for kernel modules.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I tested this on NixOS 22.11 with BTF support.

Thanks for the update!
I've tested bpftrace 0.17 manually as well, doesn't look like there is any problem.
It'd be great to add a test for BTF modules though now bpftrace supports it -- could you cherry-pick the top commit of https://github.com/martinetd/nixpkgs/commits/bpftrace_btf_module ?

Sure, I cherry-pick-ed and also the test passed on my environment.

The test didn't pass on aarch64 (not much details, "machine # ERROR: Error attaching probe: kfunc:nf_tables:nft_trans_alloc_gfp"...), so much for assuming the VM environment would be similar enough...
I don't have any powerful aarch64 machine so it'll take a while to reproduce (I don't even know if this board can run qemu in our test configuration... probably if I figure how to disable kvm...); might want to revert my extra check if it becomes a problem.

So I can run qemu (without kvm is a disaster of timeouts, but recompiling my kernel with kvm enabled and it's actually useable), struggling to upgrade bpftrace because barely 1GB of ram isn't enough when nix itself takes over 500MB... and could reproduce alright.
It looks like a problem with kfuncs in general ; I've opened https://github.com/iovisor/bpftrace/issues/2496
I've opened https://github.com/NixOS/nixpkgs/pull/214485 to disable the test on aarch64 until that is fixed.


RawValue vs DisplayValue

Hi,
What is the difference between RawValue and DisplayValue? Having a case where the DisplayValue gets correct result but RawValue shows correct result but with some additional incorrect characters.
Thanks!

On Android the difference is that raw value contains characters that are usually not displayed to the user, like ASCII 29 and 30; GS and RS. Display value doesn't have this characters.
On iOS/macOS they are the equal.


Branching Pagebuilder

When editing a branching page show the current page at the top of the screen and allow that branch to be deleted easily.

Fixed in [ab1c5d455493f5c70847259140dd294282b0cef3]


Update azure-pipelines-conditional-integration.yml for Azure Pipelines

Fix cron string

Waiting on https://dev.azure.com/dnceng-public/public/_build/results?buildId=433805&view=logs&j=1f132584-ab08-5fbf-e5d0-5685de454342 to finish, then this will be ready to merge


Unity 2018.2

Hello There.
The assetstore description says that it should be compatible with 2018.2, but there is only a version for 2018.3 is it possible to get a package for 2018.2?
Thanks!

Sorry about that. Didn't realise that you can only use the package on the same version or above than the one I published it on.
I've submitted an update, should be done within 24 hours (I hope) 😁


Allow users the ability to manage website access without using terminal commands

What are you trying to do?
Improve the user experience for allowing website access to Ollama models. Currently, users are required to run terminal commands to get this working. If the user restarts their computer they have to run these commands again, which is not ideal.
How should we solve this?
Add configurations directly within the Ollama app to manage which websites or Chrome extensions can have access to Ollama.
What is the impact of not solving this?
Makes it much harder for the average person to figure out how to run Ollama models with a website or Chrome extension.
Anything else?
I'm using Ollama with my web app and Chrome extension, JobJette (jobjette.com), an AI job search copilot. I have a dedicated page with setup instructions to get Ollama installed with an AI model and connected to both the website and Chrome extension. These required setup instructions are too complicated for the average user to accomplish because they require running terminal commands.

Is this related?
https://github.com/ollama/ollama/blob/main/docs/faq.md#how-do-i-configure-ollama-server

Is this related?
https://github.com/ollama/ollama/blob/main/docs/faq.md#how-do-i-configure-ollama-server

I'm already using these terminal commands, which is fine for developers, but for the average person I'd like to simplify the setup process for connecting to websites or Chrome extensions by including some configuration settings for this directly within the Ollama app.

Merging with https://github.com/ollama/ollama/issues/3639


Resource folder structure recreation

With these changes the resource folder structure is recreated automatically, even though everything is lower case due to the way Unity stores paths.

Hey. Thank you for PR. Since I'm a little busy with other things right now I can't check it normally so I will just point some issues in your code.

Because there is a mix of tabs and spaces I can't point all indentation issues in browser but you can check them in text editor.

Thank you for taking the time and pointing them all out and sorry about all that indentation, I had visual studio configured incorrectly. Anyways, I think this commit solves pretty much everything, but there are still a few minor issues that are not entirely related to my clumsiness.
By the way this project is amazing, keep it up!


(Feature) Remove footer and header in embedded mode

Problem:
When wizard invest page embedded in iframe there is header/footer of ICO Wizard inside the iframe. Users of ICO Wizard decide not embed the page because it's not convenient/not design friendly.
Example of an embedded invest page https://www.thebtcbtq.com/btq
https://cl.ly/143d3c0U2U1O
Solution:
When invest page is in an iframe, don't show footer and header. Should be configurable in a yml config file. By default header/footer should be disabled in the embedded mode.

@igorbarinov I was testing this in the BTQ page.
With the current iframe's 600px height and removing header/footer and paddings/margins, the page won't be fully visible.

In this scenario, the minimum height for the QR Code to be visible, is 900px, but with this height when QR is not being displayed there will be a big white space beneath the wizard.

Probably we should re-arrange the page elements when the height is ~600px or think in a responsive approach.

Just to make it clear, the solution will include adding a new attribute to ICOConfig (should we change the number of that variable too?)
The new attribute will be: showHeaderAndFooterInIframe, and it will be falsy by default. That means, no header nor footer.
@igorbarinov @vbaranov is it ok?

...ICOConfig (should we change the number of that variable too?)

What do you mean?

The new attribute will be: showHeaderAndFooterInIframe, and it will be falsy by default. That means, no header nor footer.

Sounds good to me

What do you mean?
Sorry, the "name" of the variable I meant

Oh, yes, maybe, rename to CrowdsaleConfig. Also, lets change example in wiki too after renaming: https://github.com/poanetwork/token-wizard/wiki/Token-Wizard-ICO-Configuration


fix: avoid instrumentation of project if none is loaded, fixes #5065

The Issue

#5065

How This PR Solves The Issue
Avoids instrumentation of the project if none is available.
Manual Testing Instructions
Run ddev --version in a folder without DDEV config.
Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

Shouldn't we also fix the place where it uses the invalid data? pkg/ddevapp/ddevapp.go:371
https://github.com/ddev/ddev/blob/9b3b0d9b241bbd32059a85ad57deb6312323796f/pkg/ddevapp/ddevapp.go#L371
Seems like that also should have been initialized? Or is it the app itself that doesn't exist here?

Yes, app does not exist, that's the problem here.

$ ddev version
 ITEM             VALUE
 DDEV version     v1.22.0-alpha3-30-gb57c5900
 architecture     amd64
 db               ddev/ddev-dbserver-mariadb-10.4:v1.22.0-beta1
 ddev-ssh-agent   ddev/ddev-ssh-agent:v1.22.0-beta1
 docker           24.0.2
 docker-compose   v2.18.1
 docker-platform  dev
 mutagen          0.17.1
 os               linux
 router           traefik:v2.10
 web              ddev/ddev-webserver:v1.22.0-beta1

Instrumentation is opted in, but AmplitudeAPIKey is not available. This usually means you have a locally-built ddev binary or one from a PR build. It's not an error. Please report it if you're using an official release build.

Looks like it is impossible to test this build from PR.
I tried my local build instead. I placed some dummy values in pkg/versionconstants/versionconstants.go and configured the Makefile.
I use OpenSnitch to inspect my internet traffic and I see api2.amplitude.com in my connections, and DDEV works if I disable the internet.
No more errors :+1:


Update and rename src/languages/tr.json to kaynak/Diller/en.json

Tr Türkiye

Hello sir,I submitted my KYC for the last 8 month to above' at that time I didn't update my number,email and Facebook,but now I did all necessary but still show me review in progress

Same here, I am trying 8+months" but KYC not completed.
On Sun, Mar 19, 2023, 4:32 AM Abdull216 @.***> wrote:

Hello sir,I submitted my KYC for the last 8 month to above' at that time I
didn't update my number,email and Facebook,but now I did all necessary but
still show me review in progress
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-1475024364,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FRNLMQEI5KPXSUWWTBU2LW4Y5GVANCNFSM6AAAAAAVWYEGHY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

https://pi-network-vn.business.site/?utm_source=gmb&utm_medium=referral

https://snssdk1233.onelink.me/bIdt?pid=tiktokweb_landing&c=video_7244210378433924358&af_adset_id=0_unknown&click=0&_d=0&embed_source=&ug_source=&ug_campaign=&wid=7253406480686450182&rid=11058833689987598226&vid=&vidab=70508271%2C71298620%2C71355676%2C71359500%2C71365375%2C71382220%2C71382235%2C71382253%2C71390783&lvid=&jump_time=1689987619548&af_adset=direct&af_ad_id=no_referrer&af_dp=snssdk1233%3A%2F%2Faweme%2Fdetail%2F7244210378433924358%3Frefer%3Dweb%26gd_label%3Dclick_wap_launch_popup%26page_name%3Dreflow_video-unifying%26params_url%3Dhttps%253A%252F%252Fwww.tiktok.com%252F%2540trbinance%252Fvideo%252F7244210378433924358%253Ftarget%253Dlaunch_popup%2526rid%253D11058833689987598226%2526vidab%253D70508271%25252C71298620%25252C71355676%25252C71359500%25252C71365375%25252C71382220%25252C71382235%25252C71382253%25252C71390783%2526click_item_id%253D7244210378433924358%26needlaunchlog%3D1%26referer%3Ddirect%26wid%3D7253406480686450182%26vidab%3D70508271%252C71298620%252C71355676%252C71359500%252C71365375%252C71382220%252C71382235%252C71382253%252C71390783%26jump_time%3D1689987619548%26is_sub_only_video%3D0%26ug_source%3D%26ug_campaign%3D

620ed609e12a9b21f72b9c45349637b2ba4e128deebe3b81563e25f277ed0a6699005a80e331d1a56285a0dd6f992649381c44c69116239838fd95af8235e3ba

https://glstrk.com/?offer_ids=Njc4&affiliate_id=Mjg4MTc%3D
On Sat, 14 Dec 2024 at 19:10, Badboy442010 @.***> wrote:

620ed609e12a9b21f72b9c45349637b2ba4e128deebe3b81563e25f277ed0a6699005a80e331d1a56285a0dd6f992649381c44c69116239838fd95af8235e3ba
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2543229055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD3OMLHA4L76CKTTPFJUTBL2FRX7ZAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBTGIZDSMBVGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

👍
On Wed, Dec 18, 2024, 5:18 AM EVANDRO FRANCISCO GARCIA <
@.***> wrote:

👍🤜🤛👏
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2549919003,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FRNLK4CCMPRKMCMLGDII32GCZ4VAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBZHEYTSMBQGM
.
You are receiving this because you commented.Message ID:
@.***>

👍 Thank You for your response
On Fri, Dec 20, 2024, 9:13 AM EVANDRO FRANCISCO GARCIA <
@.***> wrote:

Thank you
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2556226490,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FRNLMKPC23LQ656IIGIBL2GOG7DAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGIZDMNBZGA
.
You are receiving this because you commented.Message ID:
@.***>

Hi
في الجمعة، ٢٠ ديسمبر ٢٠٢٤, ٩:٠٣ ص Anil3781 @.***> كتب:

👍 Thank You for your response
On Fri, Dec 20, 2024, 9:13 AM EVANDRO FRANCISCO GARCIA <
@.***> wrote:

Thank you
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2556226490,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/A3FRNLMKPC23LQ656IIGIBL2GOG7DAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGIZDMNBZGA>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2556347152,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4SHWWD52BJXJ24T2XNBXCD2GOXM5AVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGM2DOMJVGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

https://clapper.vip/Btc1312
On Wed, 18 Dec 2024 at 10:42, Anil3781 @.***> wrote:

👍
On Wed, Dec 18, 2024, 5:18 AM EVANDRO FRANCISCO GARCIA <
@.***> wrote:

👍🤜🤛👏
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2549919003,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/A3FRNLK4CCMPRKMCMLGDII32GCZ4VAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBZHEYTSMBQGM>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2550848045,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD3OMLE4I6M6GPDU7MFC3FT2GE7P7AVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJQHA2DQMBUGU
.
You are receiving this because you commented.Message ID:
@.***>

https://clapper.vip/Btc1312
On Fri, 20 Dec 2024 at 07:05, ahmedaswad @.***> wrote:

Hi
في الجمعة، ٢٠ ديسمبر ٢٠٢٤, ٩:٠٣ ص Anil3781 @.***> كتب:

👍 Thank You for your response
On Fri, Dec 20, 2024, 9:13 AM EVANDRO FRANCISCO GARCIA <
@.***> wrote:

Thank you
—
Reply to this email directly, view it on GitHub
<
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2556226490>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/A3FRNLMKPC23LQ656IIGIBL2GOG7DAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGIZDMNBZGA>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2556347152,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/A4SHWWD52BJXJ24T2XNBXCD2GOXM5AVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGM2DOMJVGI>

.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2556350868,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD3OMLC6B5FZIXL44EFESIL2GOXSVAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGM2TAOBWHA
.
You are receiving this because you commented.Message ID:
@.***>

Thank you
On Thu, Dec 26, 2024, 10:23 PM EVANDRO FRANCISCO GARCIA <
@.***> wrote:

Very good
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2562948518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FRNLJS6EM47WHS3H7OEJT2HQX75AVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRSHE2DQNJRHA
.
You are receiving this because you commented.Message ID:
@.***>

https://bscminer.com/#/?invite=RGYP7
On Sat, 14 Dec 2024 at 19:10, Badboy442010 @.***> wrote:

620ed609e12a9b21f72b9c45349637b2ba4e128deebe3b81563e25f277ed0a6699005a80e331d1a56285a0dd6f992649381c44c69116239838fd95af8235e3ba
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2543229055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD3OMLHA4L76CKTTPFJUTBL2FRX7ZAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBTGIZDSMBVGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

https://bscminer.com/#/?invite=RGYP7
On Thu, 26 Dec 2024 at 17:53, EVANDRO FRANCISCO GARCIA <
@.***> wrote:

Very good
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2562948518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD3OMLD4R3VIB4U7KUITPGD2HQX75AVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRSHE2DQNJRHA
.
You are receiving this because you commented.Message ID:
@.***>

https://bscminer.com/#/?invite=RGYP7
On Thu, 26 Dec 2024 at 19:11, Anil3781 @.***> wrote:

Thank you
On Thu, Dec 26, 2024, 10:23 PM EVANDRO FRANCISCO GARCIA <
@.***> wrote:

Very good
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2562948518,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/A3FRNLJS6EM47WHS3H7OEJT2HQX75AVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRSHE2DQNJRHA>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2563000443,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD3OMLAVMNVKPR23ZY3XQQD2HRBGNAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRTGAYDANBUGM
.
You are receiving this because you commented.Message ID:
@.***>

https://app.glitchy.ai/sign-up?rid=28817
On Sat, 14 Dec 2024 at 19:10, Badboy442010 @.***> wrote:

620ed609e12a9b21f72b9c45349637b2ba4e128deebe3b81563e25f277ed0a6699005a80e331d1a56285a0dd6f992649381c44c69116239838fd95af8235e3ba
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2543229055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD3OMLHA4L76CKTTPFJUTBL2FRX7ZAVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBTGIZDSMBVGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

👍
On Sat, Dec 28, 2024, 11:03 PM EVANDRO FRANCISCO GARCIA <
@.***> wrote:

Incredible
—
Reply to this email directly, view it on GitHub
https://github.com/pi-apps/pi-explorer/pull/73#issuecomment-2564387066,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FRNLM5XZ2XZMHO7AR4NFD2H3OG3AVCNFSM6AAAAABTTXS7WCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRUGM4DOMBWGY
.
You are receiving this because you commented.Message ID:
@.***>


Cannot use live caption at 100th

could you solve this?
I run the /bin/chorme.exe, it cannot load live caption.

@wilson818 Do you not use the portable .bat? If you just run the exe, it will store your user profile data in %APPDATA%\Local\Chromium, rather than in the USER_DATA folder in the portable zip.
As far as live caption, I have not tested that with M100, but have used it with M98. None of my patches should mess with that, so this is likely an upstream thing. I will test with the next release, and get back to you here.
Otherwise, is Thorium working okay for you?

@wilson818 Try latest release. And if you are using portably, use the .bat. Otherwise use the installer.

@wilson818 hows is live caption coming?

@wilson818 Try latest release. And if you are using portably, use the .bat. Otherwise use the installer.

I can't use AVX2， I only can use AVX.
I have tried M100.0.4848.0 with installer, it is the same, Live Caption could not be used...

@wilson818 Ok. Next release coming tomorrow, and will be a normal AVX release.

@wilson818 Ok. Next release coming tomorrow, and will be a normal AVX release.

I am using 100th 4888 with AVX, all things are well but Live Caption still does not work.
I have tried it on another windows11 computer, but the same result.

@wilson818 I actually found in the code where it only will compile SODA (needed for live caption) if the CPU is detected as having sse4.1. I'm thinking since these builds are AVX or AVX2 that might have caused it. I made the commit to thorium, and next build I will test and you can test as well.
I knew that live caption worked on a vanilla build of Chromium, so I searched the entire Thorium codebase with a program that searches file contents, not just names, and searched for things like "live" "caption" "live caption" "soda" and "SODA" and read through them and that's how I found this. Can't guarantee it will fix it but it looks promising.

@wilson818 New release here and in the AVX2 repo.

@wilson818 How's it coming.

@wilson818 How's it coming.

I have tried 101st and 102nd which are AVX editions, but the live caption still could not be shown.

@wilson818 How's it coming.

I have good news to tell you, the live caption on 102.0.4973 may be working but not shown!
When I turn on the live caption on it, the CPU is using 30-40% on Thorium, but when I turn it off, it's 10-20%.
I think the floating windows of the live caption were not shown, maybe you just need to fix this little problem.

@wilson818 Are you using google sync?

@wilson818 Are you using google sync?

yes, I am using Google sync. but on another computer without using google sync, the effect of live caption in Thorium is the same so far.

@wilson818  Wdym you're using google sync, but on another computer without using it?
I think Live Caption uses online translation which would use the same API as "translate this page"
API Keys are needed for more than just sync, the API Keys bundled in Thorium have quotas for the sync api, translation api, geotagging api, google drive api, and others. You might try enabling sync in Thorium and see if it fixes it

@wilson818 Wdym you're using google sync, but on another computer without using it?
I think Live Caption uses online translation which would use the same API as "translate this page"
API Keys are needed for more than just sync, the API Keys bundled in Thorium have quotas for the sync api, translation api, geotagging api, google drive api, and others. You might try enabling sync in Thorium and see if it fixes it

I tried Google Chrome, it could use Live Caption when I don't log in a google account.
I had enabled sync in Thorium, and the live caption and translation could not be used.(in Google Chrome, the translation icon is on the address box when I am surfing a Spanish or French page)

@wilson818 Did you add yourself to that Google group?

@wilson818 Did you add yourself to that Google group?

https://groups.google.com/u/1/a/chromium.org/g/google-browser-signin-testaccounts
Is this group? I've been joined.
but it said: You don't have permission to access this content

@wilson818 Thats ok, as long as you're joined.
So sync is working for you, but NOT translate or live caption?

@wilson818 Thats ok, as long as you're joined. So sync is working for you, but NOT translate or live caption?

I've joined it last year (it showed me I joined it in 2021), now no translation, no live caption.   >_<

@wilson818 Try to unjoin, then re join.

@wilson818 Try to unjoin, then re join.

I've unjoined and then re-join, the live caption and translate are still no show...

I can translate pages just fine, is that what you're referring to?
Live Caption though, I can confirm this is an issue on Windows and Linux Thorium builds.

the previous post was talking about runing chrome.exe directly in the folder of your build.
but if I run Thorium.bat, it could not got translate pages.

@wilson818 thats interesting and strange. Glad you got it working though.

@wilson818 When you run chrome.exe directly, it uses the normal profile directory, similar to if you had installed it. The thorium.bat is designed to disable logging, disable default browser check, and to create and use a user profile dir in the same folder it is located in, in the USER_DATA folder. You might try deleting the contents of that folder, and then just copying the contents of C:\Users$USERNAME\AppData\Local\Chromium\User Data into it.

@wilson818 When you run chrome.exe directly, it uses the normal profile directory, similar to if you had installed it. The thorium.bat is designed to disable logging, disable default browser check, and to create and use a user profile dir in the same folder it is located in, in the USER_DATA folder. You might try deleting the contents of that folder, and then just copying the contents of C:\Users$USERNAME\AppData\Local\Chromium\User Data into it.

thank you .  and I can only use 100th with thorium.bat because 101st and 102nd are with bugs on Windows11

@wilson818 See note in latest releases. > https://github.com/Alex313031/Thorium-Win/releases/tag/M102.0.4973.0
"NOTE: There is an upstream bug in Chromium. It only appears on Windows 10/11. Windows 7/8/8.1 are fine.
Problem: You may get a condition when launching where extensions will not start and open tabs will say "RESULT_CODE_MISSING_DATA" or "STATUS_STACK_BUFFER_OVERRUN".
To fix this: Right click the Thorium shortcut, or the actual executable at C:\Users\USERNAME\AppData\Local\Chromium\chrome.exe , go to properties, and set the compatibility mode to windows 7 or 8."

@wilson818 See note in latest releases. > https://github.com/Alex313031/Thorium-Win/releases/tag/M102.0.4973.0
"NOTE: There is an upstream bug in Chromium. It only appears on Windows 10/11. Windows 7/8/8.1 are fine. Problem: You may get a condition when launching where extensions will not start and open tabs will say "RESULT_CODE_MISSING_DATA" or "STATUS_STACK_BUFFER_OVERRUN". To fix this: Right click the Thorium shortcut, or the actual executable at C:\Users\USERNAME\AppData\Local\Chromium\chrome.exe , go to properties, and set the compatibility mode to windows 7 or 8."

It could not be remembered by win11, it needs to be set the compatibility every time for running.

@wilson818 The thorium install dir has changed and i was slightly incorrect in my instructions.
Go to
C:\Users\USERNAME\AppData\Local\Thorium\chrome.exe
Right click, go to properties, then the compatibility tab, then click the "change for all users" button, and set to win 7 or 8 for all users.

@wilson818 Compatibility mode issue has been fixed for a while now, and live caption should also be working. Try latest release.

@wilson818 Compatibility mode issue has been fixed for a while now, and live caption should also be working. Try latest release.

yes, I can run your building without Compatibility mode now. And are you meaning the M106 is working with live caption?
I didn't see the release of M106 with AVX1. Could you release M106 ?

@wilson818 Oh sorry, I forgot to upload it yesterday lol.

@wilson818 Other NTP page and search settings bugs have been fixed! New bug free M106 release > https://github.com/Alex313031/Thorium-Win/releases/tag/M106.0.5219.0

@wilson818 Is your original issue fixed? This issue is getting stale, and if I don't get a reply in another ~month I will be closing it. Feel free to reopen if you see this and it is closed and it isn't fixed.

@wilson818 Try latest release > https://github.com/Alex313031/Thorium-Win/releases/tag/M109.0.5362.0 I am closing this issue though because it had become stale.


Output path

Hi Adam,
Working great so far!!! :) Apparently we have lots of temperate phage which is very exciting to see.
Quick feature request - flag for dumping output files into a given path?
Thanks!
-Stephanie

I second this request :)

me too!


Update sentiment.g.dart

Cast error fixed: if score is "null" it cannot be cast as a "num" type

Hi Nadia! Thanks for helping to fix this issue.
The only thing it's that sentiment.g.dart it's an autogenerated file and this issue will resurface when we generate the files again.
To solve this you need to add a @JsonKey(defaultValue: 0) line on top of score and magnitude in the file sentiment.dart and then run flutter run build_runner build --delete-conflicting-outputs to generate the files.
Let me know if you have any questions.

Hi Israel,
Thank you very much, I will do that !
Have a nice day,
Nadia Hilmi
Le 5 déc. 2022 à 15:03, Israel Ibarra @.***> a écrit :

Hi Nadia! Thanks for helping to fix this issue.
The only thing it's that sentiment.g.dart it's an autogenerated file and this issue will resurface when we generate the files again.
To solve this you need to add a @JsonKey(defaultValue: 0) line on top of score and magnitude in the file sentiment.dart and then run flutter run build_runner build --delete-conflicting-outputs to generate the files.
Let me know if you have any questions.
—
Reply to this email directly, view it on GitHubhttps://github.com/Deimos-Applications/dialog_flowtter/pull/22#issuecomment-1337422770, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AREAXCNUR5JP7UXE76QYPE3WLXYZVANCNFSM6AAAAAASUBCVSM.
You are receiving this because you authored the thread.Message ID: @.***>


chore: add github issue templates

Intent

Help members outside the authzed organization to contribute to the project by directing them to the right places
Help project maintainers to have more structured information and labeling

How-To Test

Visit https://github.com/yordis/spicedb/issues/new/choose
Verify you are happy with it

I have read the CLA Document and I hereby sign the CLA


No map size limit

Right now, when the map grows out of bounds of the generated map image, the map stops being able to be drawn. Ideally, the map image generator should be able to crop the parts of the map hat are outside of the image.
Changes will probably have to be made in OccupancyGridImageDrawer.cpp

The idea is to make a portion of the map (depending of the scale factor) available. The new implementation should act like a GPS navigation map, with the robot in the center of the map, map top in the robot's orientation. This would enable to see the surroundings of the robot and the laser scan to avoid nearby obstacles.


Open questions

Do we compute optical properties on grid interfaces or midpoints?
Does our linear interpolator recover linear and piecewise linear functions exactly?

Check interpolator logic

I think from a quick look at the CAM code that the optical properties are all at interfaces, and I think they are using the same wavelength grid in the aerosol code as is used in radiation, so there is no interpolation


modify layout of codenew shortcode

Update layout of copy-code block within codenew shortcode.
See issue #23020.

Page previews:
https://deploy-preview-23021--kubernetes-io-master-staging.netlify.app/docs/tasks/inject-data-application/define-interdependent-environment-variables/

preview looks good.
LGTM.

/area web-development

/cc @tengqm @sftim

/lgtm

/approve
Thanks @kbhawkey this is clearly better


OData CreateEntry

OpenUI5 version: 1.77.0
Browser/version (+device/version):81.0.4044.138
Any other tested browsers/devices(OK/FAIL):
URL (minimal example if possible):
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:
1.I am trying to create a new Entry using oDataModel.createEntry().
I am assigning the created Context to a SmartForm and opening it in a Dialog.
My CreatedContext path will be something like this "/000001(row_id='0000000001',rmprf='')".
when ever I am changing a value in the oSmartForm a Merge request will be sent with the value
rmprf="SOMEVALUE".
As a result I will get some data like this "__metadata": { "id":
"https:..../000001(row_id='0000000001',rmprf='SOMEVALUE')", "uri":
"https:../000001(row_id='0000000001',rmprf='SOMEVALUE')", "type": "000001Type" }, "row_id":
"0000000001", "rmprf": "SOMEVALUE".
But the problem here is the bindingContext path of my oSmartForm is still
"/000001(row_id='0000000001',rmprf='')", where as it should be
"/000001(row_id='0000000001',rmprf='SOMEVALUE')".
This problem is only there in versions 1.74 and above, in version 1.73 it is working good.
Can someone please give me the solution
What is the expected result?
Binding Context of SmartForm should be updated.
What happens instead?
Binding Context of SmartForm not updating.
Any other information? (attach screenshot if possible)
here is sample code:
var oModel = new sap.ui.model.odata.v2.ODataModel({ serviceUrl: serviceURL, loadAnnotationsJoined: true, defaultBindingMode: "TwoWay"});
createdContext = oModel.createEntry(sBindingPath, { groupId: "changes", success: fnSuccess, error: fnError }); oModel.submitChanges({ groupId: "changes", });
var fnSuccess = function(oData, oResponse) {
var oSmartForm = new sap.ui.comp.smartform.SmartForm({ entityType : oEntityType.name, }); oSmartForm.setBindingContext(createdContext);
oCreateDialog = new sap.m.Dialog({ content : oSmartForm });
oController._oCreateDialog.bindElement(sPath);
oController.getView().addDependent(oController._oCreateDialog);
oController._oCreateDialog.open();

Hi @Srikar1995,
the path /000001(row_id='0000000001',rmprf='') of the created entity indicates the rmprf is a key property of the corresponding entity type. While I found no place in the OData V2 specification https://help.sap.com/doc/5890d27be418427993fafa6722cdc03b/Cloud/en-US/OdataV2.pdf which explicitly disallows updating key properties of an entity, I am convinced such an update is not be possible. So the OData server will (likely) ignore the changed key property. And even if it would accept this, the key predicate in the path of the context for the created entity will not be changed by this.
I however do not understand why the behavior should be as described in 1.73...
Regards,
Patric


Slot props

What problem does this feature solve?
Currently, if you are using multiple slots within a component, there is no structured way to pass data through the props. Rather, you will need to pass it at the component level. This is not a limitation but does lead to disorganized code, and can complicate the DX of creating more advanced components.
What does the proposed API look like?
Overview

I propose having the ability to pass attributes such as class and href through slots.
In the case of attributes such as class, the passed string should be additive.
To give the dev full control, these attributes would be applied explicitly within the component using $slots['slotname'].attrs and optionally passing a specific key to attrs such as $slots['slotname'].attrs['class']
It should be possible to use v-bind or something similar to apply all of the attributes to a given element
(Optional) it would be ideal to be able to pass props and directives like v-if as well

Example
This example illustrates a possible implementation using a single component (MyComponent.vue) in a page (MyPage.vue) and what the rendered HTML output could look like.
MyComponent.vue
<template>

      <slot name="heading" />

      <slot name="button" />

</template>

MyPage.vue
<template>
  <MyComponent>
    <template #header class="font-bold">
      This is my heading
    </template>
    <template #button href="/about" class="bg-primary">
      About us
    </template>
  </MyComponent>
</template>

HTML Output

      This is my heading

      About Us

A duplicate of mine : https://github.com/vuejs/core/issues/7999.
Also I've seen your question on stackoverlfow, let's discuss there in the comments.

@shareefhadid check my solution https://stackoverflow.com/questions/76621595/vue-3-slots-attributes-can-you-pass-data-through-props/76627568#76627568

Your case is opposite of mine : #7999. Also I've seen your question on stackoverlfow, let's discuss there in the comments.

Thank you for sharing :) I am not sure if this is a duplicate exactly, I think we are talking about something very similar, but I would hope the Vue team would consider going beyond just classes and support directives and props as well.
If you agree, please upvote so we can get this feature some attention!

https://github.com/vuejs/core/issues/7999#issuecomment-1741706068


changing PIN doesn't change it globally

tested with v1.5.0

start from scratch. create account A and set PIN
create another account B and keep it selected
change PIN

account A will have old PIN
possibly related to https://github.com/encointer/encointer-wallet-flutter/issues/487
this can lead to an ugly state where different PINs coexist

Oh this is a regression, it used to work earlier...

not a problem anymore. tested with 1.8.0

oops, wrong. this is a terrible issue, actually. try creating multiple accounts and change pins when selecting changing accounts. big mess


Skip Navigation Links

Accessibility Feature Suggestion
Skip Navigation Links
On most pages, keyboard and screen reader users must navigate a long list of navigation links and other elements before arriving at the main content. This can be particularly difficult for users with some forms of motor disabilities. Skip navigation links can help improve accessibility.
A skip navigation link is helpful for users who don't want to work their way through all the links on the page before getting to the main content. This link does not have to be visible on the page, but it should be listed first in the tab order.
As substrate docs is navigation heavy it would be advisable to add in a skip navigation link.
Example of implementing a skip navigation that is hidden until user navigates with a keyboard:
https://webaim.org/techniques/skipnav/

Hi Carina,
With this one I think it is appropriate that the main navigation comes before the documentation navigation in the hierarchy. If it didn't where could it be placed?
The main issue here in my opinion is that the main (shared) navigation exists at all. The real solution to this issue would be to remove it all together, any other change would just be a workaround to accommodate for the keeping of this navigation that is not entirely relevant to the documentation site.
I hope that makes sense. Let me know if there is any further information I can provide.


New App Type - React App

Create a new app type for scaffolding React application

Good to have . I am waiting.......

I am working on this issue, #buildingReactGenerator-smacss

Awesome!


Validation Glitch?

So, I was fooling around with combining egg moves and rby moves and so i made a new team with Bulbasaur that had light screen and petal dance which is valid.
Meganium can pass light screen (lvl up move) and petal dance(an event move) to Bulbasaur
but when i add an RBY TM (body slam or swords dance etc.) which Bulbasaur can only learn in RBY which I don't think should be valid, the validator suddenly says

Bulbasaur's past gen egg moves Light Screen, Petal Dance do not have a valid father. (Is this incorrect? If so, post the chainbreeding instructions in Bug Reports)

So is this a bug? I believe the validator should be saying that body slam is incompatible with one or more moves because it can learn light screen and petal dance, but not body slam with those moves.
Sorry about me closing and reopening. That was an accident.
EDIT: Also, how can bulbasaur learn light screen and skull bash at the same time, there are no valid fathers for both of those moves.

This issue seems to have been fixed at some point, since:

Also re: getting a Bulbasaur with Light Screen and Skull Bash, pretty sure it's just
Get a male Ledyba which learns Light Screen by level up
Breed that with a female Paras
Trade male Paras with Light Screen to RBY and teach it the Skull Bash TM
Breed male Paras with Light Screen and Skull Bash with female Bulbasaur


margin and padding utilities not working?

It would seem that this theme includes the bootstrap 5 css, merging it with the theme's css. So I would expect the bootstrap utilities to work. However, in version 7.0.5 of the theme, it seems that margin and padding utilities (such as mr-1, ml-2, mx-2) are not working at all?

My bad, the problem was actually that margin and padding utilities have changed !
Margin-left is no longer ml, but ms (margin-start), and Margin-right is now me (margin-end).
Same goes for padding!
https://getbootstrap.com/docs/5.2/utilities/spacing/


Package registration

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/66847
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 5c9717369f6cfb86c871f4cbda225b0083592de6
git push origin v0.1.0


Simple Notification Service Cloud Formation Syntax

CloudFormation SNS resources:

AWS::SNS::Topic - http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-sns-topic.html
AWS::SNS::TopicPolicy - http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-sns-policy.html

LGTM


Add diagonal Coulomb Hamiltonian class

Fixes #117

This is a semi-draft pull request, i.e. I have a complete implementation, but I'm not sure if it's what was intended. Any guidance would be greatly appreciated! :smiley:


Window Heigth and Width properties are Nan on Linux in combination with  WindowState="FullScreen"

Describe the bug
A simple Window defined in MainWindow.axaml with the option  WindowState="FullScreen" works well on Windows 10 and the properties this.Height and this.Width, as seen in  MainWindow.axaml.cs, have correct values. However, when running the same application on a Linux system (Ubuntu 20.04 in my case) the Window becomes fullscreen which is good but the values of the properties this.Height and this.Width are Nan.
To Reproduce

create a basic non mvvm avalonia application with the option WindowState="FullScreen" added and with the option Opened="OnOpened".
make a method in MainWindow.axaml.cs, public void OnOpened(object sender, EventArgs e).
add a few debug statements like: Debug.WriteLine("this.Height=" + this.Height); Debug.WriteLine("this.Width=" + this.Width);
set a breakpoint (I use Visual Studio 2019 pro).
inspect the values of this.Height and this.Width.

Expected behavior
Correct values for this.Height and this.Width
Desktop (please complete the following information):
Developing on Windows 10 Pro (up to date) with Visual Studio 2019 targeting .NET 5 and Avalonia Version="0.10.0"
Tested on Linux Ubuntu 2.,04 (no values for this.Height and this.Width)
AvaCanvas2.zip

X11 is an asynchronous protocol, so resize event arrives way after than we've sent the open request. I'd suggest to override ArrangeCore if you want to get notified about resize events.

From your comment in gitter @aadslingerland I see you've solved your problem by subscribing to the LayoutUpdated event, so I will close this. Thanks!


What does STD and MEAN do?

Hi, I'm using your code as a jumping off point for learning about localiser development, and was just a bit confused as to what STD and MEAN are explicitly used for? I think it looks like some kind of normalisation operation, but not 100% sure. Any info you can give would be greatly appreciated.
Cheers.

Hi, yeah you're right. In machine learning, this is called feature scaling (https://en.wikipedia.org/wiki/Feature_scaling). It is mostly used to get a better convergence.
Btw instead of manually calculating std and mean, you can also use from keras.applications.... import preprocess_input. I had hoped for a faster convergence speed by calculating the exact std/mean, but it seems preprocess_input works just as well. Hope this helps. -- Lars


std::badalloc in sympify()

The following code throws a MemoryError: std::bad_alloc error:
import symengine as se
se.sympify("Equality(sp_0_0 * sp_1_0, 1) & Equality(10 / (sp_1_2 * sp_1_3) / sp_1_1 / sp_1_0, 1)")

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "symengine_wrapper.pyx", line 522, in symengine.lib.symengine_wrapper.sympify
MemoryError: std::bad_alloc

This also fails with the same error:
se.sympify("And(Equality(sp_0_0 * sp_1_0, 1), Equality(10 / (sp_1_2 * sp_1_3) / sp_1_1 / sp_1_0, 1)))")

Meanwhile, this doesn't throw but it does not recognize Equality as a symengine function, and returns undefined function symbol:
>>> e = se.sympify("Equality(sp_0_0 * sp_1_0, 1)")
>>> type(e)
<class 'symengine.lib.symengine_wrapper.FunctionSymbol'>

Therefore the following also fails, but with a much friendlier message:
>>> e = se.sympify("Equality(sp_0_0 * sp_1_0, 1)")
>>> se.And(e, e)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "symengine_wrapper.pyx", line 1480, in symengine.lib.symengine_wrapper.And.__new__
  File "symengine_wrapper.pyx", line 4287, in symengine.lib.symengine_wrapper.logical_and
TypeError: Cannot convert FunctionSymbol to symengine.lib.symengine_wrapper.Boolean

SymPy can parse all 3 expressions just fine.
It seems there are 2 issues: sympify not recognizing Equality as a built-in function, and constructors And, Or, etc. rejecting FunctionSymbol in a really user-unfriendly way.
I'm using symengine 0.92 with Python 3.9.

This is a bug in the C++ library. Until that is fixed, can you use Eq instead of Equality?


Using Internal Exchange Server

I followed this page and was able to create a Node server to read mail from an office 365 account.
I also have an internal Exchange account that I would like to read from. To do this I went into the Application Registration Portal and created a new application for an internal Exchange e-mail address.  I updated the .env file with the new access credentials.  Next, I re-ran the same server code.  The server received a successful connection but received a message that there are 0 mail messages to read.
It isn't clear how to use the information on this page to read from an internal Exchange server.  My Office365 account is<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 53b47c58-f96e-dca0-6ae6-0e22bda41a07
Version Independent ID: 760e409d-5f01-7bf2-0c8e-a05e75afa711
Content: How to use Outlook REST APIs in a Node.js app
Content Source: docs/rest/node-tutorial.md
Service: unspecified
Product: outlook
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

@MichaelWhite63: Generally, you cannot use the Microsoft Graph to access on-premises Exchange servers. There is preview support for a very specific setup if you're on-premises Exchange servers are Exchange 2016 and you have a hybrid deployment with Office 365. You can find the details here:
https://developer.microsoft.com/en-us/graph/docs/concepts/hybrid_rest_support

Jason,
Thank you for your response. The version of on-premise Exchange that my Node server needs to communicate with is Exchange 2013 CU19.
Can you direct me to a page that contains instructions on how to build an API to read from an Inbox on that version of Exchange.
Michael White

On May 18, 2018, at 11:32 AM, Jason Johnston<EMAIL_ADDRESS>wrote:
@MichaelWhite63: Generally, you cannot use the Microsoft Graph to access on-premises Exchange servers. There is preview support for a very specific setup if you're on-premises Exchange servers are Exchange 2016 and you have a hybrid deployment with Office 365. You can find the details here:
https://developer.microsoft.com/en-us/graph/docs/concepts/hybrid_rest_support
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Exchange 2013 is accessible via Exchange Web Services (EWS). You can find those docs here: https://msdn.microsoft.com/en-us/library/office/dd877012(v=exchg.150).aspx

Jason,
Do you have a Javascript example that I could use in Node.js?
Michael
On Fri, May 18, 2018 at 12:40 PM, Jason Johnston<EMAIL_ADDRESS>wrote:

Closed #234 https://github.com/OfficeDev/outlook-dev-docs/issues/234.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OfficeDev/outlook-dev-docs/issues/234#event-1634689264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHp9RegJK7EqwCDz5EWc57VvegS37T9Mks5tzvmFgaJpZM4UEjsh
.

No sorry, I haven't worked with EWS in years.


use isomorphic-fetch simplify http resolver

fix https://github.com/APIDevTools/json-schema-ref-parser/issues/107

Conceptually this seems like a good idea, but a few things:
Firstly, is there any documentation required to fully solve #107, does the user need to change anything or is it just magic?
Secondly. another attempt was made to fix #107 by using request in #108, and that was closed as it doesn't work in the browser. fetch() does work in the browser ofc, but we might need to do a little more work to make sure node-fetch is only loaded in node and normal fetch is available in the browser.
@P0lip I'd appreciate your input here on the second thing, because this will effect @stoplightio stuff, probably for the better.

Conceptually this seems like a good idea, but a few things:
Firstly, is there any documentation required to fully solve #107, does the user need to change anything or is it just magic?
Secondly. another attempt was made to fix #107 by using request in #108, and that was closed as it doesn't work in the browser. fetch() does work in the browser ofc, but we might need to do a little more work to make sure node-fetch is only loaded in node and normal fetch is available in the browser.
@P0lip I'd appreciate your input here on the second thing, because this will effect @stoplightio stuff, probably for the better.

Secondly. another attempt was made to fix #107 by using request in #108, and that was closed as it doesn't work in the browser. fetch() does work in the browser ofc, but we might need to do a little more work to make sure node-fetch is only loaded in node and normal fetch is available in the browser.

support via browser field in package.json on browser downgrade to native fetch
see https://github.com/node-fetch/node-fetch/blob/2.x/package.json#L6
see https://github.com/node-fetch/node-fetch/blob/2.x/package.json#L6
see https://github.com/defunctzombie/package-browser-field-spec

Secondly. another attempt was made to fix #107 by using request in #108, and that was closed as it doesn't work in the browser. fetch() does work in the browser ofc, but we might need to do a little more work to make sure node-fetch is only loaded in node and normal fetch is available in the browser.

support via browser field in package.json on browser downgrade to native fetch
see https://github.com/node-fetch/node-fetch/blob/2.x/package.json#L6
see https://github.com/node-fetch/node-fetch/blob/2.x/package.json#L6
see https://github.com/defunctzombie/package-browser-field-spec

I think removing timeouts and redirect logic is a little nuclear here, as there are alternative approaches.
To get timeouts working with abort controller: https://dmitripavlutin.com/timeout-fetch-request/
There's some stuff on google about follow redirects being built in https://stackoverflow.com/questions/39735496/redirect-after-a-fetch-post-call#39739894 but im not sure if thats going to work for this specific implementation.
Anyhow, if we remove these that's probably a major release, and it's not a great sales pitch to just say there's less functionality now.

I think removing timeouts and redirect logic is a little nuclear here, as there are alternative approaches.
To get timeouts working with abort controller: https://dmitripavlutin.com/timeout-fetch-request/
There's some stuff on google about follow redirects being built in https://stackoverflow.com/questions/39735496/redirect-after-a-fetch-post-call#39739894 but im not sure if thats going to work for this specific implementation.
Anyhow, if we remove these that's probably a major release, and it's not a great sales pitch to just say there's less functionality now.

Btw I appreciate your work on this! Sorry if that doesn't come across, I write a little too quickly sometimes, never mean to sound rude! 😅

Btw I appreciate your work on this! Sorry if that doesn't come across, I write a little too quickly sometimes, never mean to sound rude! 😅

i planning use typescript design a new library.

i planning use typescript design a new library.

@septs I don't understand. You had almost finished this feature and instead something something now you're doing a new one in TypeScript?
There's already a TypeScript rewrite of this library that's coming along quite nicely, and it does a lot more than just switch to TS. There's years of domain knowledge built up in this codebase and that one. Would you like to work together instead of duplicating efforts?

@septs I don't understand. You had almost finished this feature and instead something something now you're doing a new one in TypeScript?
There's already a TypeScript rewrite of this library that's coming along quite nicely, and it does a lot more than just switch to TS. There's years of domain knowledge built up in this codebase and that one. Would you like to work together instead of duplicating efforts?


Cisco router config syntax issue

On this step of the guide:
p-10-4-0-17# config t router bgp 65001 address-family-ipv4 maximum-paths 8 end
The correct syntax is address-family ipv4 (no hyphen)

made the correction. Thanks @givenand.


Corrupt sends with v1.0.4

I was trying to use this Android tool to send a file to my Linux laptop and I'm pretty sure it's corrupting the transfer.
I have a 2,693,769 byte binary file (tgz file) that on the other side of the wormhole ends up with an extra 375 bytes of seeming garbage at the end. I tried slicing that suffix off and passing it to the Linux "file" command and it doesn't have a detected structure.
I'm wondering if there's a uninitialized buffer read or incorrectly sized buffer involved?

I was able to reproduce this by generating a file of that length. I'll take a closer look this evening.

I believe this should be fixed in 233058dc. If you are able, can you try manually installing this build to confirm that it fixes the issue for you? https://github.com/psanford/wormhole-william-mobile/releases/tag/v1.0.5rc1

Works now on that previously bad file transfer
On Mon, Nov 8, 2021, 22:42 Peter Sanford @.***> wrote:

I believe this should be fixed in 233058d
https://github.com/psanford/wormhole-william-mobile/commit/233058dc00222a1adab36429598edf5334d57be5.
If you are able, can you try manually installing this build to confirm that
it fixes the issue for you?
https://github.com/psanford/wormhole-william-mobile/releases/tag/v1.0.5rc1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/psanford/wormhole-william-mobile/issues/15#issuecomment-963812431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABGJ5GKBECK2DXWQ2GE53ULCRFBANCNFSM5HTUK5IA
.
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.

1.0.5 is now available on the Play Store with this fix.
Thanks again!


Publish latest version on crates.io

I noticed latest changes are not pushed to crates.io. I am mainly interested in https://github.com/alexcrichton/ssh2-rs/commit/9eb9a5b85c359e1ee8aa8016eecb516039bf4069
Can you update the version in crates.io with latest on master?
Thanks

Done!


py38_images tests failing after recent merge in Main

🐞 Problem
The Main branch & subsequently all rebased branches on the current main are failing the py38_images test. The problem seems to be that the images from test_porkchop_plotting in test_porkchop.py are different from those in the assertion ... The can be seen in the code snippet below as well as this link
=================================== FAILURES ===================================
____________________________ test_porkchop_plotting ____________________________
Error: Image files did not match.
  RMS Value: 23.55042462462195
  Expected:
    /tmp/tmpi6yv37y_/test_porkchop.test_porkchop_plotting/baseline.png
  Actual:
    /tmp/tmpi6yv37y_/test_porkchop.test_porkchop_plotting/result.png
  Difference:
    /tmp/tmpi6yv37y_/test_porkchop.test_porkchop_plotting/result-failed-diff.png
  Tolerance:
    2

💡 Possible solutions
We need to check for the reason why the test is failing in the first place & find the commit that was recently merged leading to failure of the required test. Appropriate changes need to be done if the test has wrong assertion, upload the correct image or revert the commit so as to restructure it with all the tests passing

May be related: https://github.com/matplotlib/pytest-mpl/issues/91

We need to find out whether this was caused by our commit or by a new version of a dependency.

The test also fails in the local virtual environment run ... Seems to be something related to the libraries 🤔
============================================================== FAILURES ===============================================================
_______________________________________________________ test_porkchop_plotting ________________________________________________________
Error: Image files did not match.
  RMS Value: 23.55042462462195
  Expected:
    /tmp/tmp8c_srs2q/test_porkchop.test_porkchop_plotting/baseline.png
  Actual:
    /tmp/tmp8c_srs2q/test_porkchop.test_porkchop_plotting/result.png
  Difference:
    /tmp/tmp8c_srs2q/test_porkchop.test_porkchop_plotting/result-failed-diff.png
  Tolerance:
    2

I have created the images after running the invocation commands ... The images are as follows:
BASELINE

RESULT

RESULT_FAILED_DIFF

It seems so that the problem here is the font, as mentioned appropriately by @xlxs4, as the plots don't show any difference 🤔

matplotlib 3.4.0 was released four days ago https://pypi.org/project/matplotlib/3.4.0/ please check whether the correct builds were using 3.3.*

From the issue I linked (and mpl-compare's docs):

Different versions of Matplotlib and FreeType may result in slightly
different images. When testing on multiple platforms or as part of a
pipeline, it is important to ensure that the versions of these
packages match the versions used to generate the images used for
comparison. It can be useful to pin versions of Matplotlib and FreeType
so as to avoid automatic updates that fail tests.

regardless of what this turns out to be, let's avoid getting bitten by this in the future

Fair enough, but first we ought to understand what is the root cause. If we loosen up the tolerance, we might miss legitimate bugs.

Tests are failing again after downgrading Matplotlib to 3.3.4 😢

@astrojuanlu ... Yes the Build passes without any errors with Matplotlib==3.3.4 ... I have pinned the library version to 3.3.4 for the time being ... I will be confirming soon what the root cause may be :) & @xlxs4 is absolutely correct that for current builds, we can pin the library version of Matplotlib & FreeType
tests/tests_plotting/test_misc.py::test_plot_inner_solar_system_static PASSED                                                   [  8%]
tests/tests_plotting/test_misc.py::test_plot_outer_solar_system_static PASSED                                                   [ 16%]
tests/tests_plotting/test_porkchop.py::test_porkchop_plotting PASSED                                                            [ 25%]
tests/tests_plotting/test_static.py::test_basic_plotting PASSED                                                                 [ 33%]
tests/tests_plotting/test_static.py::test_basic_trajectory_plotting PASSED                                                      [ 41%]
tests/tests_plotting/test_static.py::test_basic_orbit_and_trajectory_plotting PASSED                                            [ 50%]
tests/tests_plotting/test_static.py::test_trail_plotting PASSED                                                                 [ 58%]
tests/tests_plotting/test_static.py::test_plot_different_planes PASSED                                                          [ 66%]
tests/tests_plotting/test_static.py::test_body_plotting PASSED                                                                  [ 75%]
tests/tests_plotting/test_static.py::test_plot_ephem_epoch SKIPPED (need --remote-data option to run)                           [ 83%]
tests/tests_plotting/test_static.py::test_plot_ephem_no_epoch SKIPPED (need --remote-data option to run)                        [ 91%]
tests/tests_plotting/test_tisserand.py::test_tisserand_plotting PASSED                                                          [100%]

These tests are just a way to check internal consistency. I propose that we regenerate the images with latest matplotlib so we can make the tests pass without changing anything else.
To regenerate the images: https://github.com/matplotlib/pytest-mpl/#with-baseline-images

Closed in #1144


fix(*): Typo

Description

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

@tony19 Thanks for the review, I have fixed all the issues you raised.


[Defect] Deletion of text & images partly working but not accurately

https://ckeditor.com/cke4/presets-all

Fixed


Endpoint is now alert.palette-software.com

@baspeti why change the end point? This would break the library completely

@StephenHynes7 Oh, I'm terribly sorry, I pushed this change to the wrong (original) repository.

This is one of the most amusing PRs I've seen in a long time. Set the endpoint to some random server? :)

Aha, hehe @baspeti saw your comment too late.

You guys are quick! :) I didn't have time to realize, that my PR was compared to the original master, and then I clicked. (But I guess you got notified.)
Anyway, I'm glad I could make you smile a bit.
Btw, LogEntries rocks! (But there are some typos in the NLog getting started page though... Are you interested about them?)

The title caught my eye. Sure if you have any thoughts on improving the doc here please let me know, I have become to used to it to notice these things :(

Helping out @baspeti.
We are talking about this page:
https://logentries.com/doc/dotnet/
This line has typos:
private static readonly Logger m_logger = LogManager.getCurrentClassLogger();
I think the correct one would be (capital L and capital G)
private static readonly Logger m_Logger = LogManager.GetCurrentClassLogger();
Also in here: ```private static readonly Logger m_Logger = LogManager.getCurrentClassLogger();

There are minor stuff but people like me who just copy-pastes lines from documentation would face compile errors immediately and that's a little scary as a first impression on a library.


Bugfix in gcd/lcm laws and Rational.gcd implementation

A corner case was handled incorrectly (detected by tests!)

Codecov Report

Merging #643 into master will increase coverage by 0.24%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #643      +/-   ##
==========================================
+ Coverage    50.1%   50.35%   +0.24%
==========================================
  Files         173      173
  Lines       11170    11167       -3
  Branches     1555     1566      +11
==========================================
+ Hits         5597     5623      +26
+ Misses       5573     5544      -29

Impacted Files
Coverage Δ

...ws/src/main/scala/spire/laws/CombinationLaws.scala
94.73% <ø> (-0.51%)
:arrow_down:

...re/shared/src/main/scala/spire/math/Rational.scala
96.52% <100%> (+0.23%)
:arrow_up:

...spire/optional/intervalGeometricPartialOrder.scala
93.75% <0%> (-6.25%)
:arrow_down:

extras/src/main/scala/spire/math/FixedPoint.scala
34.97% <0%> (-1.1%)
:arrow_down:

...ala/spire/math/poly/BigDecimalRootRefinement.scala
85.43% <0%> (ø)
:arrow_up:

...red/src/main/scala/spire/math/interval/Bound.scala
59.05% <0%> (ø)
:arrow_up:

...e/shared/src/main/scala/spire/math/Algebraic.scala
68.37% <0%> (+0.2%)
:arrow_up:

...ed/src/main/scala/spire/math/poly/PolySparse.scala
95.08% <0%> (+0.44%)
:arrow_up:

...ore/shared/src/main/scala/spire/math/Complex.scala
46.42% <0%> (+0.79%)
:arrow_up:

... and 4 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 6878c49...2d292be. Read the comment docs.


Segmentation fault when running gen_data_flow_ops.dynamic_stitch

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
2.10.0
Custom Code
Yes
OS Platform and Distribution
Ubuntu 22.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Segmentation fault probably due to dimension mismatch.

Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np
from tensorflow.python.ops import gen_data_flow_ops
try:
  try:
    with tf.device('/CPU'):
      arg_0_0_tensor = tf.random.uniform([], minval=-256, maxval=257, dtype=tf.int32)
      arg_0_0 = tf.identity(arg_0_0_tensor)
      arg_0_1_tensor = tf.random.uniform([1], minval=-256, maxval=257, dtype=tf.int32)
      arg_0_1 = tf.identity(arg_0_1_tensor)
      arg_0 = [arg_0_0,arg_0_1,]
      arg_1_0_tensor = tf.random.uniform([], minval=-256, maxval=257, dtype=tf.int32)
      arg_1_0 = tf.identity(arg_1_0_tensor)
      arg_1_1_tensor = tf.random.uniform([1], minval=-256, maxval=257, dtype=tf.int32)
      arg_1_1 = tf.identity(arg_1_1_tensor)
      arg_1 = [arg_1_0,arg_1_1,]
      out = gen_data_flow_ops.dynamic_stitch(arg_0,arg_1,)
  except Exception as e:
    print("Error:"+str(e))
  try:
    with tf.device('/GPU:0'):
      arg_0_0 = tf.identity(arg_0_0_tensor)
      arg_0_0 = tf.cast(arg_0_0, tf.int32)
      arg_0_1 = tf.identity(arg_0_1_tensor)
      arg_0_1 = tf.cast(arg_0_1, tf.int32)
      arg_0 = [arg_0_0,arg_0_1,]
      arg_1_0 = tf.identity(arg_1_0_tensor)
      arg_1_0 = tf.cast(arg_1_0, tf.int32)
      arg_1_1 = tf.identity(arg_1_1_tensor)
      arg_1_1 = tf.cast(arg_1_1, tf.int32)
      arg_1 = [arg_1_0,arg_1_1,]
      gen_data_flow_ops.dynamic_stitch(arg_0,arg_1,)
  except Exception as e:
    print("Error:"+str(e))
except Exception as e:
  print("Error:"+str(e))

### Relevant log output

```shell
2023-01-08 19:41:09.950654: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:09.955250: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:09.955367: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:09.955670: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-08 19:41:09.956097: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:09.956203: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:09.956296: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:10.309982: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:10.310131: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:10.310228: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-08 19:41:10.310311: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 4220 MB memory:  -> device: 0, name: NVIDIA GeForce GTX 1660 Ti, pci bus id: 0000:01:00.0, compute capability: 7.5
Error:{{function_node __wrapped__DynamicStitch_N_2_device_/job:localhost/replica:0/task:0/device:CPU:0}} indices[0] is out of range [Op:DynamicStitch]
Segmentation fault

</details>

@nimashiri,
I was able to execute the given code without any issue/crash on tensorflow v2.11. Kindly find the gist of it here and let us know if it serves the purpose. Thank you!

@nimashiri, I was able to execute the given code without any issue/crash on tensorflow v2.11. Kindly find the gist of it here and let us know if it serves the purpose. Thank you!

On 2.11 it should work fine, the issue is for 2.10.

@nimashiri,
As it was the bug on tensorflow v2.10, it has been resolved on the latest stable tensorflow v2.11 where we can observe that Segfault error has not been raised. Thank you!


[editor] Support Updating AIConfig State when aiconfig Prop Changes

[editor] Support Updating AIConfig State when aiconfig Prop Changes
Reviving the changes closed in https://github.com/lastmile-ai/aiconfig/pull/960 so that we can leverage the aiconfig prop to the AIConfigEditor component for updating the config in state. This will be useful for 2 reason in the vscode extension:

Currently, re-saturating the webview has no aiconfig in state. This causes an empty webview (with loading spinner) in the case where the server has issues and can't load the content (e.g. if the config JSON has invalid model parsers defined at the top). We can improve that UX to have a readonly copy of the config by saving the aiconfig if webview context to re-saturate with. The problem with doing that right now is that the /load call after the editor renders would not update the editor with the loaded config, which means it could be showing a stale config. This PR would fix that
Undo/redo. Similar to the above, in vscode extension undo/redo is controlled by the webview extension host and updates the server with the updated state. This then propagates the updated aiconfig to the webview client, which currently does nothing with the updated config. This PR would fix that as well

An important note here:
When an updated aiconfig prop is passed, we just replace the entire aiconfig state in the editor, instead of consolidating with existing state. This will ensure:

new prompt keys are set in the _ui, meaning the prompts are remounted (so, undo/redo will replace the input properly)
config passed through is the source of truth, instead of having a weird case where client state is persisted / overwriting the provided config, causing incorrect state

We can consider the scenario of passing updated aiconfig prop through as treating the editor as a controlled component, at least temporarily. As a 'user' of the component, one would expect passing a new aiconfig prop would make the editor have that exact contents.
Lastly, just noting we could do some smarter consolidation using lodash deep equality checks in order to retain existing prompts that are unchanged when passing a new aiconfig prop through (so, they'd keep their _ui.id and not be remounted). Just noting that there would probably be no perf benefit (since deep equality check may be just as expensive as rerendering the components, if not more). The only benefit could be in improving the UX if the current implementation causes weirdness. For example, in the test video there is some flickering of the audio input as it re-renders, but I'm not sure if that warrants the expensive equality checks on its own
Testing:
Updated LocalEditor to leverage state instead of callbacks for streaming:
output_chunk: (data) => {
            //onStream({ type: "output_chunk", data: data as Output });
            const output = data as Output;
            setAiConfig(
              (prev) =>
                ({
                  ...prev,
                  prompts: prev!.prompts.map((prompt) => {
                    if (prompt.name === "get_activities") {
                      return { ...prompt, outputs: [output] };
                    }
                    return prompt;
                  }),
                } as AIConfig)
            );
          },
          aiconfig_chunk: (data) => {
            //onStream({ type: "aiconfig_chunk", data: data as AIConfig });
            setAiConfig(data as AIConfig);
          },

Then make sure streaming works
https://github.com/lastmile-ai/aiconfig/assets/5060851/06d46a28-e589-42fe-805e-282614b48ffb
Note that in the video I'm also clicking into the input to show that it's continuously re-rendered, so user cannot actually click and edit it while the state is changing from the streaming

Incredible writeup of the various cases and options considered. This change looks good to me. Main question -- did you check this in the vscode editor? I tried this from #960 when I was building the VSCode editor, and it didn't seem to propagate updates properly. However, it's possible that some of the other fixes you made to re-render as state changes were the missing piece.

Ah, no I didn't try it out since I'm not sure the best way to do that besides pulling my aiconfig-editor package PR and rebasing this on top and linking it; how did you test out #960 before?
Fwiw, the change of overwriting the state (mainly, the fact that the ClientPrompts have new ids/keys) is what will 'fix' the issue you saw

how did you test out https://github.com/lastmile-ai/aiconfig/pull/960 before?

With difficulty. I copied the VSCodeEditor.tsx (and related files) into the aiconfig-editor folder, built the main.js bundle from there, and copied it into the vscode-extension folder lol. Probably overkill and better to just publish a package update.
One thing I'd suggest is we should have a test-aiconfig-editor package on npm that we update to test stuff out as needed. But also feel free to push an update to the main package directly right now to test it out.

how did you test out #960 before?

With difficulty. I copied the VSCodeEditor.tsx (and related files) into the aiconfig-editor folder, built the main.js bundle from there, and copied it into the vscode-extension folder lol. Probably overkill and better to just publish a package update.
One thing I'd suggest is we should have a test-aiconfig-editor package on npm that we update to test stuff out as needed. But also feel free to push an update to the main package directly right now to test it out.

Ah, ok, that's what I feared :s
I was planning to publish and bump the minor version with this since it does add some new behaviour that changes how the editor behaves.
I can publish to a test package and update test plan for vscode, but will land first so it's easier to do that without dealing with additional merge issues


Is it possible to drag item outside of RecyclerView?

Hi,
Is it also possible to drag item outside of RecyclerView's boundaries?

Drag item from one RecyclerView to another

Thank you for your response.

Hi. This library does not support such operation. I guess implementing that feature requires a lot of work to support.

@MartinKocour
Have you found any library that supports dragging between Layout?
Just like this dragula javascript library is good.

@Honghe
I've found this lib https://github.com/woxblom/DragListView

@MartinKocour
Thanks, even though the woxblom/DragListView has the limitation of only can be used as horizontal based on HorizontalScrollView