more little bugs

Switch from "problem 1" to "Blank model". Enter editor mode and switch to "Solve for outputs". You'll see the problem 1 hints are displayed. They should have been cleared when we switched to a blank model.
Also in "Solve for outputs" mode the word "Input" is not centered over the inputs.
Make none of the weights/threshold be user-modifiable. Then the instruction string says "Adjust and undefined to solve the problem". This is a bug. It should be blank if there is nothing the user can adjust.

All fixed.


Update 'No file chosen' to 'No file selected' on Create Disk dialog

Storage Explorer Version: 1.10.0
Build: 20190906.4
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand Disks -> Select a resource group.
Click 'Create Disk' -> Observe the description under 'Source VHD' on Create Disk dialog.

Expect Experience:
Show 'No file selected'.
Actual Experience:
Show 'No file chosen'.

Fix checked into master


Question: health check says  "GET / HTTP/1.1" 302 237 "-" "-"

Question
health check says  "GET / HTTP/1.1" 302 237 "-" "-"
is this normal ?
To Reproduce
Steps to reproduce the behavior:
Ingress Controller details

Output of kubectl describe pod <ingress controller> . The  pod name can be obtained by running helm list.
Output of `kubectl logs .
Any Azure support tickets associated with this issue.

@niteenkole I assume this log is from the backend. Backend health, if status code is not specified, considers 200-399 as a healthy status code.

ok thanks
From backend health check it says "Success. Received 302 status code"


Minor typos fixing in the README.md.

There were a few typos in the README which now has been fixed.
Moreover, for additional coherence, the shell symbol has been modified from the dollar sign ($) to the florin sign (ƒ).

The florin sign is very classy. Thank you.


feat: session-based fingerprint cache

as mentioned in https://github.com/apify/browser-pool/issues/64
(some points in #137 too)
Allows browser fingerprints to be based on Session (instead of current proxy URL). If SessionPool is turned off, the original (proxyURL) behaviour is used.

cc @vladfrangu, this test seems to be quite flaky, it crashed on it several times for me as well
Summary of all failing tests
FAIL packages/memory-storage/test/write-metadata.test.ts
  ● writeMetadata option › when false › creating a data store should not write __metadata__.json file
    ENOENT: no such file or directory, scandir '/home/runner/work/apify-ts/apify-ts/packages/memory-storage/test/tmp/write-metadata-tests/no-metadata/key_value_stores/e5b92a37-47b3-4afd-a0a8-0e619b1413cc'


Flip two items

It would be nice to have a function to flip two items in cache. So they switch data with each other.

how would you use this?
I think probably not; memcached is a distributed system, so for most cases two keys may not even reside on the same server.


Storage space of ARM self-hosted runner is probably small

https://github.com/autowarefoundation/autoware/runs/4749425172?check_suite_focus=true
2022-01-08T18:56:43.4386749Z TASK [autoware.dev_env.ros2 : Install ros-galactic-desktop] ********************
2022-01-08T18:59:11.7105352Z ##[warning]You are running out of disk space. The runner will stop working when the machine runs out of disk space. Free space left: 15 MB

Or, the capacity allocated for Docker images is too small.

@xmfcx Have you said that you'll change the AWS machine settings in the last meeting? I don't remember well.

@xmfcx Have you said that you'll change the AWS machine settings in the last meeting? I don't remember well.

Yes, I'll look into it right away!

@kenji-miyake
Here is the report:
autowarefoundation organization has a GitHub Actions Runner registered with a name AutowareFoundationGithubActionsRunner.
This runner runs on an AWS t4g.xlarge instance. Amazon EC2 T4g instances are powered by Arm-based AWS Graviton2 processors.
It has 4 vCPUs and 16GiB RAM.
This binary was installed on it: https://github.com/actions/runner/releases/download/v2.276.1/actions-runner-linux-arm64-2.276.1.tar.gz
ubuntu@ip-192-168-22-5:~$ df -h /
Filesystem      Size  Used Avail Use% Mounted on
/dev/root        29G   25G  4.4G  85% /

Having fun with ncdu inside the runner
ncdu 1.14.1 ~ Use the arrow keys to navigate, press ? for help
--- / -------------------------------------------------------------------------------------------------------------------
   17.4 GiB [##########] /var
    4.5 GiB [##        ] /usr
    2.0 GiB [#         ] /snap
    1.9 GiB [#         ] /home
  273.0 MiB [          ] /opt
  191.8 MiB [          ] /boot
   69.0 MiB [          ] /root
    5.9 MiB [          ] /etc
    1.0 MiB [          ] /run
  600.0 KiB [          ] /tmp
...

--- /var ----------------------------------------------------------------------------------------------------------------
                         /..
   16.5 GiB [##########] /lib
  838.5 MiB [          ] /log
   91.6 MiB [          ] /cache
    3.8 MiB [          ] /backups
   92.0 KiB [          ] /snap
   32.0 KiB [          ] /tmp
   28.0 KiB [          ] /spool
e   4.0 KiB [          ] /opt
...

--- /var/lib ------------------------------------------------------------------------------------------------------------
                         /..
   15.4 GiB [##########] /docker
  929.7 MiB [          ] /snapd
  122.9 MiB [          ] /apt
   71.5 MiB [          ] /dpkg
    2.8 MiB [          ] /command-not-found
  872.0 KiB [          ] /fwupd
  608.0 KiB [          ] /usbutils
  532.0 KiB [          ] /systemd
...

--- /var/lib/docker/overlay2 --------------------------------------------------------------------------------------------
                         /..
    2.0 GiB [##########] /3f789e1ccb84eced273bbd5ec673d26bf2ba73badea2366fb5ce7c1e455deb3c
    1.7 GiB [########  ] /860d068134bb56e762666ace5f4f4e8776a54d2a4bd4dadaa213903dd36e1e8a
    1.6 GiB [#######   ] /961ad94a1a2d8bde97c3e7a4fd920b32dea466dee7a2e798cbd0cd9de3b9579c
    1.2 GiB [######    ] /oz9h7z6ksun9pzg6s52do34bs
    1.2 GiB [######    ] /igc3ivk7fko8yv0dy5f3ix6es
    1.1 GiB [#####     ] /acf011033a08512ba500bb9cb491c8fe2b85800fd51f5940688dacaae258b347
    1.0 GiB [####      ] /k39z3vtovy72t3rd9kul7ylts
    1.0 GiB [####      ] /ocnfwy30dy9qn4iv197j3wx32
  971.0 MiB [####      ] /ozlaq5qbqyl4vrp20vymjja8v
  764.8 MiB [###       ] /mk3870wqhylwvxem7ortmtvfm
  555.1 MiB [##        ] /75e02708a769eec5b19605086f65f8db2bad1cd2cc855134b7d7a5228e33f8ab
  379.1 MiB [#         ] /b466a752974568564c50ef30c00e5aa5c481241400098aec28cb965b4367d66e
  360.7 MiB [#         ] /7df0aabea828669c65dfc3d6c5fa1b2512eeb4beb2d7b985043a7fca7c5a90f9
  360.6 MiB [#         ] /19f37636febdd3ad4e677c761f54a18bcc406ef569a56c915a462b492a5b00d9
  170.9 MiB [          ] /8bfa3d5126374b0ff2bf1b5dbbd2b9b2c7250c85f8b0110c4e2cab8c3aee7acc
  170.9 MiB [          ] /c3ca31b9a7ce3931b2f9887e59374a97f5261c794a8f3c2c5eb0d091f30ed8e3
  121.5 MiB [          ] /28e5f0a7ef59844e7d1303e8207c0fe4f1d6b999e25ef0457e63d5634c2ee0d2
  108.3 MiB [          ] /b3e692e697ed02bc6c1f220ef9f222e6fd31f3b703f33787ea236671249ebdbe
  103.1 MiB [          ] /669a56810f2f0b641ee12f378861f98bb2e95a3feddeed03fff75a219e234ef8
   71.1 MiB [          ] /2698239224b14e3b7c4b8ae829c555b9baa308379f372d0834fe61b0b723d3f6
   71.1 MiB [          ] /98f54927abe28c6547dc7d3dc0796adfb3f2a8605b3c95fb9dbcf552f32cb071
   71.1 MiB [          ] /df580563836e90306194f98a714a4a3fa843731b35da05fe125d55e6ea714c4e
   71.1 MiB [          ] /03e58f58a62ce5c1b373555c209ad5484763751c6acad9363c85762e12d9116f
   67.2 MiB [          ] /d16b246099269bcd9dc49a24bd702f8a390070c47ddf063dd0a6b722e94f62cb
...

Then I ran:

ubuntu@ip-192-168-22-5:~$ docker images
REPOSITORY                   TAG                       IMAGE ID       CREATED        SIZE
ros                          galactic                  775c4b4329a0   7 days ago     625MB
ubuntu                       20.04                     9f4877540c73   7 days ago     65.6MB
autoware/model-zoo-tvm-cli   bleedingedge              2c1d92d7cf06   6 weeks ago    3.32GB
autoware/model-zoo-tvm-cli   bleedingedge-arm64        2c1d92d7cf06   6 weeks ago    3.32GB
autoware/model-zoo-tvm-cli   bleedingedge-cuda         051076320a77   6 weeks ago    7.1GB
autoware/model-zoo-tvm-cli   bleedingedge-cuda-arm64   051076320a77   6 weeks ago    7.1GB
moby/buildkit                buildx-stable-1           cb9b18fdb708   8 weeks ago    134MB
ros                          <none>                    d1dc7d553def   3 months ago   624MB
autoware/model-zoo-tvm-cli   <none>                    23e209b2f626   3 months ago   4GB

These docker images are occupying a lot of space.
They are generated by the runner probably, there are nothing related to them in the .bash_history file.
Then I've proceed to clean them a bit.
ubuntu@ip-192-168-22-5:~$ docker image prune
WARNING! This will remove all dangling images.
Are you sure you want to continue? [y/N] y
Deleted Images:
untagged: autoware/model-zoo-tvm-cli@sha256:a7831a40332f75ae98fb62a04f75ad2db6a6fbf8fafc409d0f69e95abc41dcb9
deleted: sha256:23e209b2f626500be6d8854fd1010229fca6f51876db4c63d3fd4851a536b7c0
deleted: sha256:dbafe75a3ee1ece980197b9886402ce3e1524b29bfbc1963690305488f81a0d7
deleted: sha256:7619c374c847a8893d3c85d6dca6202a90beb171237b3451ee0b5181c5a59b75
deleted: sha256:77ff28323bd0da84f6229138b3307e1ecfde62ebde42918feb7a8b52b95d5122
deleted: sha256:77fee4372c0621a932345c0288736a71ea21caf9a383c29aa017d137e2e07bc5
deleted: sha256:08e85464003280159ce7fd6cf708da6b67223505ff318307928375684e4dee3c
deleted: sha256:df4dcf3f56b841655e03092b79a66b996bbb9335b4151f55e9a1a73c10f18d71
deleted: sha256:1f4e146407f3bb98c8b661348e00af043449d5ad10242ccedde70d711badabbb
deleted: sha256:35acadea3b5a2a96f4b2f4c74ed894b3ba914a3f4d5d3c7a229791494e49bedc
deleted: sha256:7b8728eb6a07380fce7ee9581ed43d58d7f0f3250c50a44c38c7982536236e28
deleted: sha256:c9115e738038f017bc9cbaea0a25b42921263d79199b1133b964633162b0d843
deleted: sha256:1093f8b99677a049c1627342e2632c7d060f4cd058fec56a22c52ac358887570
deleted: sha256:bed5eaa83dce8f654fd931ac6b1c4de67fef4482b3270a5d98ae901a4b48d35e
deleted: sha256:4378b6ba3aa966c9d3d45efb1c97a87a79e2ccae62acf898e20ab7712b254726
deleted: sha256:cf1297c9d105cb8049c0f02f12a3c5a1c7c679f30a1de64c5f97b4c006cdc564
deleted: sha256:7a0d5c3393a6fb45b7436dee209a214e1d7d3497b4ed11c963c0de19c2173b3d
deleted: sha256:68d7ba65f4836161cd7e568374ad6eb90a1fbdbd807691f47b55dfd85222910e
deleted: sha256:942b063c301eb3a3581299227847df70c896991a18216541c4bafb41a4263ffb
untagged: ros@sha256:65beaaee344167f62b4ff87373d84faa27e1ea73b5de15f1a59646bea87de4d2
deleted: sha256:d1dc7d553defc3476d6d036001a2ebef1c69198b71864342ba12919988328c4a
deleted: sha256:fcb03701ef491fd30f750ea7c0697219e7efb79499394da89af966fb7f973991
deleted: sha256:108a51d95fa73e3c488d196b6c0e05e502746f88fd19a9f01f417044673f5e89
deleted: sha256:30dfd34b65d0c2cdf59cce36ff380adf6e01d41441d39824cea4b1407e060169
deleted: sha256:c23e51d5095f6c9180648792b4036e4538b51eeee815628885a23ca76ed04195
deleted: sha256:77f7733c23042efa6d253941dc786e0fe7b641e7622d1ec9b215e9ef8abcd743
deleted: sha256:e52184f9949df47092392cb61725debef580d6d20cb86518da7a98b946967c8a
deleted: sha256:19553373bc3b429fbfedb19ee24f9335856b7f0ed8beea7baaed31767a784bb9
deleted: sha256:d4ebf66ca37317b8fcd9ca3450e2707674b947c5f1751f62894e6869c0120343
deleted: sha256:0d590595a73a77a32952a1ac40fc91c8a0c7b965059e22f89deaac010c951d4b
deleted: sha256:12ebadd02095a97092b2327a681c7beffc761f73c544e077967f88f8388807f4

Total reclaimed space: 4.557GB

ubuntu@ip-192-168-22-5:~$ docker images
REPOSITORY                   TAG                       IMAGE ID       CREATED       SIZE
ros                          galactic                  775c4b4329a0   7 days ago    625MB
ubuntu                       20.04                     9f4877540c73   7 days ago    65.6MB
autoware/model-zoo-tvm-cli   bleedingedge              2c1d92d7cf06   6 weeks ago   3.32GB
autoware/model-zoo-tvm-cli   bleedingedge-arm64        2c1d92d7cf06   6 weeks ago   3.32GB
autoware/model-zoo-tvm-cli   bleedingedge-cuda         051076320a77   6 weeks ago   7.1GB
autoware/model-zoo-tvm-cli   bleedingedge-cuda-arm64   051076320a77   6 weeks ago   7.1GB
moby/buildkit                buildx-stable-1           cb9b18fdb708   8 weeks ago   134MB

ubuntu@ip-192-168-22-5:~$ df -h /
Filesystem      Size  Used Avail Use% Mounted on
/dev/root        29G   20G  9.0G  70% /

Now we have 9.0G space left. I could remove these images too but they would get redownloaded anyway probably.
We probably look into https://docs.github.com/en/actions/hosting-your-own-runners/autoscaling-with-self-hosted-runners for autoscaling since right now all arm jobs go into this specific runner.

Also for x86_64 jobs we are using GitHub-hosted runners:

Hardware specification for Windows and Linux virtual machines:
2-core CPU
7 GB of RAM memory
14 GB of SSD disk space
And I think these machines probably won't be enough for building the Autoware and we will need to look into self-hosted ephemeral runners for these too.

@xmfcx Thank you for your detailed report!

Now we have 9.0G space left. I could remove these images too but they would get redownloaded anyway probably.

Hmm, it's too small... :cry:
I want at least 30GB of free space, is it possible to extend the storage size of the VM?

Hardware specification for Windows and Linux virtual machines:

Regarding GitHub-hosted runners, I guess it's minimum requirements.
The actual storage space can be seen here.

And I think these machines probably won't be enough for building the Autoware and we will need to look into self-hosted ephemeral runners for these too.

Anyway, if there's no problem with the cost, it's nice.

@xmfcx Thank you for your detailed report!

Now we have 9.0G space left. I could remove these images too but they would get redownloaded anyway probably.

Hmm, it's too small... 😢 I want at least 30GB of free space, is it possible to extend the storage size of the VM?

I've increased the storage by 30GB, now we have 39GB empty space.
ubuntu@ip-192-168-22-5:~$ df -h
Filesystem       Size  Used Avail Use% Mounted on
/dev/root         58G   20G   39G  35% /

I confirmed that the CI is fixed in https://github.com/autowarefoundation/autoware/pull/8#issuecomment-1015132292.
Thank you @xmfcx !

@xmfcx I'm sorry but it seems we need a bit more space. :cry:
https://github.com/autowarefoundation/autoware/actions/runs/1712036308
Seeing this result, we need 55GB+ free space? :thinking:
https://github.com/autowarefoundation/autoware/actions/runs/1683139813
https://github.com/autowarefoundation/autoware/runs/4777125097?check_suite_focus=true#step:3:1110

docker-build-and-push-amd64
You are running out of disk space. The runner will stop working when the machine runs out of disk space. Free space left: 36 MB

Filesystem      Size  Used Avail Use% Mounted on
/dev/root        84G   30G   55G  35% /

I'll investigate more.

I measured the storage usage before/after building Autoware Docker images. It used 44GB for amd64.
https://github.com/autowarefoundation/autoware/runs/4857291024?check_suite_focus=true

@xmfcx Is it possible to add more 10~20GB space...?

@xmfcx Is it possible to add more 10~20GB space...? Or if we drop building prebuilt images, probably the current space is enough.

We could add but I think we should first try to reduce the space we are using. This is ok for single runner but for parallel runners it'll cost too much, not scalable. In autoware.auto 30gb total space for entire machine was enough.

We could add but I think we should first try to reduce the space we are using.

What do you think we can specifically do in order to save the space?

In autoware.auto 30gb total space for entire machine was enough.

Yes, but the current autoware.universe depends on CUDA, which uses additional 10GB+. :cry:

We won't use cuda on CI, we should make it optional.

If so, we can't check the build of some perception modules in autoware.universe, is that okay?

If so, we can't check the build of some perception modules in autoware.universe, is that okay? Also, could you tell me why do you think making it optional is good?

The CI machines don't have Nvidia gpus and CUDA only works on nvidia gpus so we couldn't check them anyway.
Instances with gpus are much more expensive, we could set them to be checked with lower frequency if needed.

@xmfcx

The CI machines don't have Nvidia gpus and CUDA only works on nvidia gpus so we couldn't check them anyway.

Yes, we can't check the runtime behavior, but I think we can check the build.

Instances with gpus are much more expensive, we could set them to be checked with lower frequency if needed.

We don't need GPUs.

@xmfcx So considering that, can we increase the space or cannot in any case?

@kenji-miyake right now it has 60GB size, will increase it to 80GB. Since it is only a single machine, it shouldn't be a problem hopefully.

@kenji-miyake Increased it, current configuration:
Filesystem       Size  Used Avail Use% Mounted on
/dev/root         78G   46G   33G  59% /

Thank you! But it seems there is another issue... :cry: I'll investigate and fix it.
https://github.com/autowarefoundation/autoware/actions/runs/1714127510

root@ip-192-168-22-5:/var/lib/docker# du -sh *
1.6M    buildkit
244K    containers
26M     image
76K     network
24G     overlay2
16K     plugins
4.0K    runtimes
4.0K    swarm
6.8G    tmp
4.0K    trust
38G     volumes

root@ip-192-168-22-5:/var/lib/docker# du -sh *
1.6M    buildkit
244K    containers
26M     image
76K     network
24G     overlay2
16K     plugins
4.0K    runtimes
4.0K    swarm
6.8G    tmp
4.0K    trust
38G     volumes

Yeah, I've also stated it in https://github.com/autowarefoundation/autoware/issues/13#issuecomment-1013343067 in collapsed logs.

Oh, yes it was folded, sorry.
And after docker system prune --all
root@ip-192-168-22-5:/var/lib/docker# du -sh *
1.6M    buildkit
244K    containers
3.1M    image
76K     network
906M    overlay2
16K     plugins
4.0K    runtimes
4.0K    swarm
4.0K    tmp
4.0K    trust
24G     volumes

@xmfcx Succeeded!
https://github.com/autowarefoundation/autoware/runs/4870569220?check_suite_focus=true
I'll try some more times to confirm there isn't an old and big cache left.


Support configuring the browser binary download path

By adding support for a variable PUPPETEER_DOWNLOAD_PATH it is possible
to support downloading the browser binaries into a folder outside the
node_modules this will in a way allow to preserve previous downloaded
binaries and not required to download then again
This PR is very similar to https://github.com/puppeteer/puppeteer/pull/5105 but with the addition that the puppeteer runner also makes use of the PUPPETEER_DOWNLOAD_PATH without additional configuration

@lcabral37 thank you for the PR, appreciate it.
My worry with this change is that it's becoming a bit complex to understand which download variables do what and how many you may or may not need to set. I'm wondering if there's a nicer way of doing this or we could take an opportunity to tidy up the ENV vars we support in a breaking change release.
I'd also be interested to know the situation that means you need to install the binaries outside of node_modules - most CI systems will cache this I think?
@mathiasbynens what do you think?

Thanks for the quick feedback.
I certainly do not have an overview as the maintainers of the project regarding its increasing complexity use of  environment variables but I  can  at least explain the reason behind this PR.
This was a result from a hackday in my organization that was a reflection of the issues I've notice our systems and colleagues dependency on downloading the chromium binaries.
We build our product with npm ci, (whether we are in our development environment or in our build CI service) which will result in purging the node_modules/ folder so it results in downloading the binaries every time we run this command (which might occur also very often in a developers daily workflow, when changing branches, starting a new task, etc.)
This behavior has resulted in slowing down the build process. At some point in time, we even had  3 different version dependencies  which would result in requiring to download 3 different chromium binaries( I know ! I know! insane but it has been fixed now).
I've also seen  issues  with storage.googleapis blocking our IPs (due rate limit), effectively stopping our build servers from building.
Another scenario I found out recently is that some  colleagues are permanently blocked unless they are working within the VPN...  storage.googleapis has permanently blocked their country (a quiet country in Asia)
Anyhow this is the short story of the pain I've seen with having to download the chromium binaries and I understand that this might not event be the best resolution for many of  theses issue (using a mirror host would be better in some of these scenarios for sure) but I honestly think this  approach would also be a major gain in this architecture. I've researched a bit through the documentation and could not find any combination that would mach this workflow. And it would make it possible to persist the same configuration for the download path  while upgrading puppeteer  (it would just download and run the newer revision without additional changes)
To answer your question  regarding CI systems, I can be sure but our experience is that we use npm ci  which will build the project with a clean slate  while respecting the package-lock. npm  itself will maintain a local cache of all npm modules downloaded so that running npm ci is very fast but it will still run every package postinstall  script and in puppeteer case, will perform the download. I mean... this is what we do. I definitely can't say for sure this is the best workflow.
Having bothered you with a 5m read already, I can finish with  saying that if you can identify any changes that could make this PR more future prof or at least less complex, I'm willing to make it so.

@lcabral37 thanks for explaining your situation, that makes sense. I can see the benefit of being able to manage where the binary is put yourselves.
@mathiasbynens what do you think to this as a new feature? I think we could rework the ENV vars potentially but at a high level do you think this is a feature worth adding?

Any status update on this @jackfranklin @mathiasbynens?
Is this worth of adding in a coming release or  just not interesting enough now?

Still no status update on this @jackfranklin @mathiasbynens?

@lcabral37 We're planning to work on this soon. Please don't ping issues repeatedly asking for updates. If there is an update, you'll see it here.

Longer-term we'll want to change the default download dir to ~/.puppeteer. The ENV var could then be used to restore the current functionality of download it it a local subdirectory.
I wonder if we should make this change all at once (to reduce confusion), or land this as-is first, and then change the default later. WDYT?

I think this can be landed as-is first because it does not block changing the default in the future.

I think this can be landed as-is first because it does not block changing the default in the future.

To clarify, my worry is that people will learn about the new env var, and by the time they start to use it, we'll have changed its meaning again. We could avoid this confusion by ensuring the two changes (this + a future patch that changes the default) make it into the same release, but we'd have to decide soonish.

I think this can be landed as-is first because it does not block changing the default in the future.

To clarify, my worry is that people will learn about the new env var, and by the time they start to use it, we'll have changed its meaning again. We could avoid this confusion by ensuring the two changes (this + a future patch that changes the default) make it into the same release, but we'd have to decide soonish.

By changing the meaning of the env var, do you mean the change of the default value? I think in this case the meaning is still the same: your custom download path would be used. Btw, I am not sure how much effort is it to change the default to something like ~/.puppeteer?

I think 99% of people won't have a strong opinion or care about where Puppeteer downloads things, so I think we can land this and then make a new change in the future and clearly document it as breaking. So I'm happy to ship this as is.

I definitely think it makes sense to allow configuring the download path. I see the point in downloading to the home folder to enable sharing binaries across projects or (as here) including the binaries in a CI image, but as a user I would be slightly annoyed if ~/.puppeteer was the default. Maybe that is just me though, but I would prefer if I had to explicitly enable downloading large binaries to a hidden folder in my home directory 🙂

By changing the meaning of the env var, do you mean the change of the default value? I think in this case the meaning is still the same: your custom download path would be used.

When the var is set, that path is used, sure. The difference we'd get when changing the default is when it's not set — the "meaning" changes in the sense that you now need to use the var in different cases.
Sounds like there's appetite for not putting large binaries in the home folder by default. Thanks for chiming in, everyone! Let's go with this as-is then.

@lcabral37 Would you be up for rebasing this patch?

@lcabral37 Would you be up for rebasing this patch?

Most certainly.
Rebase done with minimal manual changes to move introduced logic from install.js to src/install.js

Thanks @lcabral37! I think we can land this soon — I'll just wait for @jschfflr to chime in on the above discussion as well.

Just my 2 cents: I think, having puppeteer download files to ./.puppeteer instead of node_modules by default would be a better solution because it would minimise the amount of downloads a user would have to do by default. To make sure, people are aware of if it, I'd maybe print a line like: Downloading chromium to ~/.puppeteer or Found an already downloaded chromium version in ~/.puppetter.
By having it as the default, tools using puppeteer via npx like for example https://github.com/puppeteer/recorder would also benefit from this.

@lcabral37 Looks like npm run lint is unhappy, could you please take a look?

I suggest removing the changes to the puppeteer-firefox directory, since it's deprecated and not tested.

@lcabral37 Looks like npm run lint is unhappy, could you please take a look?
Sorry for that... when the lint tool is unhappy I'm unhappy :(

@mjzffr if you insist I can remove it, but for the time being I would prefer to leave it there for consistency. Test wise, I agree it should be tested if it is to be there. so I've performed a quick & dirty manual test to validate firefox downloads to the specified folder (if specified) and that it runs form the specified (if specified)
@jschfflr It would certainly resolve the problem I have but after thinking for a bit I  would dislike this approach as a user as I see that npm modules should be self contained in its ecosystem (node_modules, cache folders, etc ) and not leak out artifacts through the system... Unless I specify otherwise (not trying to defend my PR in this manner), I would find it very annoying to find a ~/.puppetter or other npm  artifacts (imagine several npm packages start to implement this approach)  that I might not even have installed directly but is instead a dependency of another package.
Thank you all for your patience


Feature request: showsMyLocationButton location

Perhaps a dumb feature to request, but for the app I'm working on the MyLocationButton is hidden a lot of the time. I could write my own, of course, but if this is easy to implement it would make for a pretty nice feature.

There are few issues about this already. Also not respecting the template. Closing.


Bump Dependencies #12012829193

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/12012829193

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add Golioth

Please add golioth to this west file.    See if you can get a demo project working with FRDM-MCXN947
Important note:
To get the FRDM-MCXN947 to build w/ golioth, you need to add these to the board kconfig overlay:
CONFIG_ENTROPY_GENERATOR=y
CONFIG_TEST_RANDOM_GENERATOR=y
ie.e.   create an board overlay in the example simliar to these:
https://github.com/golioth/golioth-firmware-sdk/tree/main/examples/zephyr/hello/boards
You can move a copy of the hello sample in this repository (mcxn-iot).

@jerpa77
I know we were having troubles with this project last week.      I want to post information to the golioth website.  However,  I would like to you quickly try something
In the main west.yml,    please reference  golioth v0.14 instead of "main".
This is what they mention on their github..

1.)  Please repeat the hello test with native sim.    If it fails, please capture a .gif
2.)  Please try using the MCXN947 board as well.  If it fails/crashes,  capture an animated gif.
Please spend an hour or two on this.  I just want to see of v0.14 behaves differently.

Ok, that's what I got with Golioth revision V0.14.0
native_sim
DHCPv4_client sample

Hello sample

FRDM-MCXN947
Hello sample

The program crashes after the net dns command execution

DHCPV4_client + Golioth Sample

The program crashes after the net dns command execution
If the golioth client is not created, the net dns command works well

I also tried moving changing the Zephyr version to 3.6.0
But it didn't work as well...

I'm not pretty sure If I'm doing something wrong


124 param test missing functionality

The following methods are added:

replaceMaskWithQueryParams method replaces {} mask to params
checkQueryParamsInUrl method checks that query params are set in url
setPath method in RequestData class to set path
Now it is impossible to set method's annotation as "/status/%s", only via {}

Tests were updated for these changes

[419] - https://transfer.sh/x54Jp/jdi-dark-tests.tar.gz

[419] - Allure report on Netlify: https://5e45dbc242faba839492f888--jdi-dark.netlify.com

[430] - https://transfer.sh/gIoW6/jdi-dark-tests.tar.gz

[430] - Allure report on Netlify: https://5e46714021deabe2c96294e3--jdi-dark.netlify.com


Create Campaign Class

Please provide specs for the class, makes it easier to impl

basically:

2D array (or whatever you feel is better) to define where maps are relative to each other.
CampaignMap -> a struct that has the map UUID (or a pointer to a map), the x and y coordinates of the map within the 2D array
Helper functions

get_north(int UUID), get_south(int UUID), get_east(int UUID), get_west(int UUID) to get neighbouring maps

A very rough UML for the class

Looking into the specs for the class, we need a unique identifier mechanism for maps too. This can affect how maps are serialized. @Polomich Since you are handling the serialization for a map, this means that you may need to update your work once the id system is in.
Some tasks that need to be done:

Create the campaign class and CampaignMap struct
Implement the map id mechanism
Serialize the class

Created a task branch for the map IDs: Task/COMP345-A2-37_--_Implement_Map_IDs

Changes for map id have been put in, see Feat[COMP345-A2-37]: Add in map ids

Created a branch for Campaign classes: [Story/COMP345-A2-37]_--Make_Campaign_Class](https://github.com/Trim0500/DnD-Character-Creation/tree/Story/COMP345-A2-37]--_Make_Campaign_Class)

Campaign class created, see Story/comp345 a2 37 make campaign class

Note that the branch for creating serializable functions for the class are in the same branch for making the class

Serialization of the campaign class is done, see Story/comp345 a2 37 make campaign class


suggestion: align terminology "staking token" => "native token"

"native token" more accurately identifies the asset as the native token of the chain, used for many things, not just stake

I think "native token" could also apply to tokens like gm/gn as they are native to the penumbra chain. The term native usually distinguishes an asset that originates on the host chain versus one that is ibc'd in.

is that useful? a chain can define many tokens but only one is favored as the 'default' for gas fees, delegation, &c
i think also staked tokens (delegation, unbonding), the various position NFTs, &c could also be described as "native" but it also seems clear that "the native token" would refer to the default token

i think we still do need some kind of terminology refinement
"staking token" could plausibly refer to delegation tokens, or something else.
"staking" is very literally a thing that hasn't happened to "staking tokens" yet.


How do we introduce constraints?

There are a lot of force fields that represent molecules as semi-flexible or even rigid. To have that functionality in a simulation, constraints have to be introduced.
The complexity that comes along with constraints is very different for MC and MD. While it is trivial to implement e.g. constant bonding lengths in MC, we need to have specific algorithms to do that in MD.
We should collect ideas how we can implement constraints so that they can be used conveniently for both propagators.
Some questions that pop into my mind:

Where/how to keep track of degrees of freedom?
Where would we implement information about constraints? Could they be part of Interactions?
Which functionalities do we want/need to provide? Constant bonding lengths, -angles, completely rigid molecules?
Where would constraint algorithms be implemented for MD?

Where/how to keep track of degrees of freedom?
Where would we implement information about constraints? Could they be part of Interactions?

We could store both of them in the Molecule struct. Something like:
struct Molecule {
    bonds: Vec<Bond>,
    angles: Vec<Angle>,
    rigid_bonds: Vec<Bond>,
    rigid_angles: Vec<Angle>,
}

And then it would be up to the algorithms to obey the constraints. A rigid molecule would have only rigid bonds. I still have to check the MD constraints algorithms to see if this could work.

Which functionalities do we want/need to provide? Constant bonding lengths, -angles, completely rigid molecules?

All of them?

Where would constraint algorithms be implemented for MD?

RATTLE and SHAKE algorithm can be implemented as part of a new Integrator

We could store both of them in the Molecule struct.

It is not clear to me: Where would we get e.g the bonding length from? Would we check the bonding potential's equilibrium distance?

This could be a solution. Or we could store them aside with the Bond, or in a RigidBond struct:
struct Molecule {
    bonds: Vec<Bond>,
    angles: Vec<Angle>,
    rigid_bonds: Vec<Bond>,
    rigid_angles: Vec<Angle>,
}

###########################

struct RigidBond {
    usize: i,
    usize: i,
    length: f64
}

struct Molecule {
    bonds: Vec<Bond>,
    angles: Vec<Angle>,
    rigid_bonds: Vec<RigidBond>,
    rigid_angles: Vec<RigidAngle>,
}

And then we can get the bong length from:

the initial configuration. This might be bad as we might not get the same bond length for every bond;
a parameter in the forcefield; This might be harder to parse and a bit ackward: why an Harmonic potential specifically?
a parameter in the [molecules] section.

(I am mainly throwing ideas as they come, most of them could work here 😃).

the initial configuration. This might be bad as we might not get the same bond length for every bond;

Yepp. Also, if we want to insert a molecule, we would need a separate configuration.

a parameter in the forcefield; This might be harder to parse and a bit ackward: why an Harmonic potential specifically?

I'll have to read a bit more about the MD constraint algorithms, but I think they need the potential function to compute the Lagrange multipliers that have to be introduced. For MC the potential function does not matter; we only need the equilibrium distance. In MC codes you'll often find the option to apply a harmonic potential that either takes an force constant or a flag fixed as input.
In any case, we need both information since we can only get the underlying potentials of bonds, angles, etc from the mapping via Molecule. It's hard to decide where to put the functionality before we sketch the constraint algorithms.


Filter ReportStream sending logic to only include COVID-19 results

Background
For now, public health departments have not requested flu data from SimpleReport. While we want to empower users to store their flu results in SimpleReport, we don't want to send that data along to ReportStream.
Action requested
Filter out any non-covid results from TestEvents before they're sent to ReportStream. This will likely need to be done within TestEventExport.java
Acceptance Criteria

[ ] Only COVID-19 results are being passed to ReportStream

This may also be dependent on #3663 completing.

After going over the current multiplex implementation with @emmastephenson  it was decided that the reporting to ReportStream will not be impacted with the current multiplex changes and that we will continue to report only covid results.
Moving this ticket back to the backlog so it gets worked on in the second round of multiplex changes. The scope of this ticket has increased to include some refactoring on the TestOrderService as well. Below is a summary of the refactoring needed:
.

A transactional method that accepts a list of results and handles all the storing of the information in the database should be created and this method should be called for all types of tests events.

Methods addTestResultMultiplex and addTestResult should call the transitional method and handle the necessary conversions to pass a List of results.

The method that sends the event information to ReportStream needs to be called after all results(multiplex and non multiplex) are processes and store in the database so it needs to be moved to the transactional method mentioned above.

the current implementation to get the information to ReportStream will get impacted with ticket #3757 (which removes the result field on the TestEvent object) as it depends on the result field.


Programming Languages - Python Project section

Programming Languages - Python Project section
Description:
This pull request adds python projects to build up a resume. It has projects at beginner, intermediate and advanced level. It provides all resources including source code, documentation and tutorials
Resource URL:

These are only some of the URLS to the source code I have provided in the python project section:
-https://github.com/topics/python-blockchain
-https://pythongeeks.org/python-library-management-system-project/
-https://www.geeksforgeeks.org/real-time-translation-app-using-python/
-https://www.geeksforgeeks.org/gui-chat-application-using-tkinter-in-python/
-https://realpython.com/build-a-chatbot-python-chatterbot/
-https://www.geeksforgeeks.org/data-visualization-with-python/
-https://pythongeeks.org/python-music-player/
-https://cointrader.readthedocs.io/en/latest/_modules/cointrader/chart.html

Checklist:
Before submitting your pull request, ensure that you have completed the following tasks:

[x] I have carefully reviewed and adhered to the contributing guidelines before creating this pull request.
[x] I followed the prescribed PR title template. (Check this if you're adding a resource)

@jfmartinz Kindly label this PR under GSSoC.

Yes, I have made the requested changes kindly review it. @jfmartinz

Still small. Can you check at your end?

Can you please solve the conflict

The changes mentioned are already done. Could you please tell if any other changes required.

Can please raise another PR, the PR is already messy and there is a conflict.


[noetic-devel][Windows] Fix some devel space is not captured in the context of catkin_make run_tests

The Problem
This is manifesting when running catkin_make run_tests and I saw some shared libraries are not found when rostest runs the ROS nodes in my workspace. It tuned out that <catkin_ws>\devel\bin (where the DLL lives) should be captured by catkin/python/catkin/environment_cache.py in setup_cached.bat and later pass to the processes running rostest, but it didn't.
So, as I trace upward, I found when generate_cached_setup.py gets invoked, at that moment devel space is empty but only contains the common files (env.* and setup.*) added by catkin. Therefore, as this particular logic gets exercised inside the call stack from generate_cached_setup.py, it will skip adding the bin and lib subfolder to the %PATH%, because those subfolders doesn't yet exist. As a result, the <catkin_ws>\devel\bin doesn't get added and passed to rostest later.
Why is it not a problem to Linux? On Linux, there is a concept or RPATH\RUNPATH ,which is backed inside the binaries, for the loader to know where to search the shared objects. On Windows, the shared library search-order system is different and by the convention we used in ROS, the search path needs to be added to %PATH%.
Proposal Solution
I believe there is a reason to keep this check, so I go another route to force the basic layout to be created ahead in the devel space, so later they will be added by _setup_util.py.
And please consider to back-port to kinetic-devel. Thanks!

This change would add each of the directories to the environment variables even if they aren't being populated. I don't think that is desired.
Did you run catkin_make run_tests as the first command without a normal build before?

Did you run catkin_make run_tests as the first command without a normal build before?

I am running catkin_make first and then catkin_make run_tests.
The thing I observed is that the setup_cached.bat gets generated fairly earlier even before any files get copied to devel\bin when catkin_make is invoked. And the later catkin_make run_tests won't trigger setup_cached.bat to be regenerated neither.
Btw, removing the check added in https://github.com/ros/catkin/pull/777 helps fix this problem, but it is probably going to have some side-effects (roswtf tests?).

The goal is to only add paths to environment variable where necessary. So removing the previous check wouldn't be good since it add non-existing directories and this patch only creates the directories unconditionally (basically the check) and adding unnecessary paths in some cases.
I would prefer a solution which still maintain the original goal to only add paths to environment variable when necessary.


Display test's time in ms

One killer feature of qunit, that prevents me from switching to mocha completely for my browser tests, is the display of required time in milliseconds. This allows me to abuse my unit tests for simple benchmarking/profiling and iterate quickly to tweek performance.
Mocha currently only displays the time for longer tests: http://whiteout-io.github.io/crypto-lib/test/index.html
What would be awesome if there was an option to always display runtime like so:

@tj what would you recommend in place of test cases to provide a consistent platform for benchmarking?
Execution time acceptability is extremely relative to platform and use case, the testing framework shouldn't choose when and when not to display time, this option should be left up to the users of the framework.

+1, I agree with @tj that this does not have any real benchmark application, but it's an interesting metric all the same.

This is already possible using the -s or --slow flag to set "'slow' test threshold in milliseconds", for example:
mocha test -s 0

Why does timing appear only sometimes?

@rash240719
It probably happened because those tests finished under 1ms and their durations were considered 0ms, which are not slow when -s 0.
I believe that this is the only case not being shown by the default reporter.
ref: https://stackoverflow.com/a/32055699


Support TypeLists in the tf-gen to enable integration of the Kubernetes provider

Is your feature request related to a problem? Please describe.
The TF Kubernetes provider uses a nested schema with TypeLists, which are not currently supported in tf-gen. While other providers remain partly functional without this support, the Kube provider does not.
Describe the solution you'd like
Support for nested schemas using TypeList.

TypeLists and TypeSets are now supported.


DOCUP - add a 'Usage' section to the README.md

The following would be helpful if added to the README.md
Usage
This node will allow you to LIST, GET, PUT and DELETE files on a remote FTP server. The GET and PUT options use msg.filename (the path/name of the file on the remote machine) and msg.localFilename (the path/name of the file on the local machine - i.e. the one running NR).
The DELETE option only specifies msg.filename (the path/name of the file on the remote machine)
The LIST option returns an array of the file/folders in the user's default ftp folder, containing
type, name, sticky, rights(an object), acl, owner, group, size and date.

I added the Usage section to the README. This issue will be closed.


".ui.form .divider" rule breaks ".ui.vertical.divider"

As the subject line says.  There is a rule in form.css that adds a margin to all .divider inside .ui.form which messes up .ui.vertical divider alignment.  Consider using :not(.ui) or :not(.vertical) in the form.css rule.

Please create a test case so I can evaluate the bug. There is a jsfiddle to fork in the readme

Here's the fiddle.

Thanks, this is clearly a bug

Btw, it's not only about "vertical" rule. E.g. "fitted" rule is also broken.
http://jsfiddle.net/q39snkqw/1/
Guess there is more there.

Should be solved in 2.1.6 (see commits) this was accidentally mistagged.


Cannot read property 'state' of undefined

Issue Id: ecbd7e50-c928-ce36-5c47-2ced492afdb1Versions - 1.38.1-  3db7e09f3b61f915d03bbfa58e258d6eee843f35-  dcc01858d3f75696568eb751238d01b30a1e7d33-  5ab84ce18fe7dee304325952f2849264dd4a48c1Stack TypeError: Cannot read property 'state' of undefined/vs/workbench/contrib/extensions/browser/extensionsList.ts#147:16 (renderElement)/vs/base/browser/ui/list/listPaging.ts#58:38

Closing because there is no enough info to fix it.


Find a way to indicate if the code example is editable

what if the code editor had a color theme applied and editable code has a matching theme?

Ended up doing it this way:


Add net-gateway-api to release list

As per title, this patch adds net-gateway-api to the list.
The preview is here https://github.com/nak3/release-1/tree/ad-gateway-api#day-of-the-release
/cc @markusthoemmes @dprotaso @carlisia

/lgtm
/approve


bug/refactor: Options on the right take up most of the screen on mobile

Refactor Request
-- fill in the line below:
** discord username: notDepresseDeveloper#0919 **
Is your refactor request pertaining to a problem? Please elaborate.
The options on the right cover a large part of the map on mobile devices, and even when the language option on the test side is added, almost a quarter of the screen is becoming unusable depending on the device. This can cause problems, especially on devices with small screens.
Suggested solution
This can cause problems, especially on devices with small screens. We think it would be more beneficial to add a dropdown-like feature for that part.
Extra context
Problem test envirorment:
Operating System: Android
Browser : Chrome Version 110.0.5481.64
Device : Samsung SM-J33F
Here is couple screenshots for the problem I've mentioned:

@sevilayerkan I think this is a UX improvement but not a bug and should be addressed as such. Marking this as approved and should be assigned to dev for an update.

I will start doing it. Do we have a UI? Are we going to do something ourselves?


Break out stroke and border color

Works for me, wanted to get it online for other people to review.

Looks like a lot of the indenting got messed up with this commit. Perhaps try again?


problems with mdsrv.dev.html and loading of xtc files

since the last 2-3(?) commits, I have now problems to use the mdsrv.dev.html. I fixed it by copying the script part from the ngl.dev.html.
but I still have problems to import .xtc trajectories. the error I receive:

trajectory.js:721 Uncaught TypeError: Expecting a function in instanceof check, but got undefined

also the gro_trajectory example (executed on the github.io) gives an error:

ngl.full.min.js:1929 Uncaught TypeError: a.eachSelectedAtom is not a function

I will fix that for now, but in the long run I will remove mdsrv.dev.html and just have it in the mdsrv repository.

also the gro_trajectory example (executed on the github.io) gives an error:

that was fixed in 7151fa62bc350afbf0043d861df8ac6b70551690

I will fix that for now, but in the long run I will remove mdsrv.dev.html and just have it in the mdsrv repository.

The idea being that you take a release from NGL to use with MDsrv instead of trying to keep up with the development versions of NGL. There also will be more frequent releases of NGL.

jupp, this makes way more sense. I will simply go back to the latest released version. I agree, that mdsrv.dev.html is also not needed. Thanks :-)

went ahead and removed it (576bc1a2e4176bbc841173161e617e6fcc2c1b94) and added an issue to mdsrv


Update markpart swagger.

Codecov Report
Attention: 35 lines in your changes are missing coverage. Please review.

Comparison is base (220add7) 12.16% compared to head (bf66b88) 18.82%.
Report is 2 commits behind head on main.

Files
Patch %
Lines

...ts/MarketParticipant/V1/MarketParticipantClient.cs
0.00%
32 Missing :warning:

...aHub.WebApi/Controllers/EsettExchangeController.cs
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2380      +/-   ##
==========================================
+ Coverage   12.16%   18.82%   +6.65%
==========================================
  Files          81       95      +14
  Lines        4388     4510     +122
  Branches      836        0     -836
==========================================
+ Hits          534      849     +315
+ Misses       3844     3643     -201
- Partials       10       18       +8

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


Domain joined devices, device tunnel and windows enterprise

A couple of questions on this:
"Device Tunnel can only be configured on domain-joined devices running Windows 10 Enterprise or Education version 1709 or later"

If the machine is only Windows Enterprise after the user logs in (i.e. using a Microsoft 365 licence that dynamically changes to Enterprise based on the users licence) will the device tunnel still function when the user isn't logged in?

If the Device tunnel only works on domain joined machines then how are we supposed to join the machine to the domain when the VPN is the only way of getting to the domain controller?

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

ID: d3e10ece-cd5d-0080-a1d3-df64466d1a0b
Version Independent ID: 5b0e8894-8d85-7d93-9dc7-d79fd15504d3
Content: Always On VPN features
Content Source: WindowsServerDocs/remote/remote-access/vpn/vpn-map-da.md
Product: windows-server
Technology: networking
GitHub Login: @Teresa-Motiv
Microsoft Alias: v-tea

@OffColour Please ask your question at Microsoft Q & A. https://docs.microsoft.com/en-us/answers/products/


Links for Bulma extensions

Apparently the URLs for the documentation of Bulma extensions have changed, for example Switch.

Looking at the code, it seems this has been addressed already? Can you confirm?

It is indeed! Thanks for the confirmation!


Guidebook UI close for items

About the PR
fixes #29459
Why / Balance
QoL
Technical details
52e80502c4406002ddb054a54e542cfda07e3b56:

Removed ToggleGuidebook method in GuidebookUIController class and embedded it's logic into OpenGuidebook method.
I suppose it was meant to be so because it's summary is "Opens or closes the guidebook" while old code only opened it.

52e80502c4406002ddb054a54e542cfda07e3b56:

Renamed GuidebookWindow _entries property to Entries and made it public so I can check it value in GuidebookUIController class.

Media

https://github.com/space-wizards/space-station-14/assets/78957156/e9e1fc30-72b3-41f4-bd57-beca03a586cc

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: Activating guidebook book with opened guidebook UI now closes it

Can you make it keep this functionality?
Content.Client_1uHbeBSFDm.mp4

I'll try. I'll take a look soon

Can you make it keep this functionality?
Content.Client_1uHbeBSFDm.mp4

Done


cargo says "Fetch" when checking out cloned repos

When cargo checks out git repos from the bare repos (~/.cargo/git/db) into ~/.cargo/git/checkouts, it says Fetch [==>  although there is actually no web traffic being done.
This is slightly confusing imho.
When extracting .crate archives, it seems that Checkout [ => is used correctly.
Can we use "Checkout" for checking out git repos as well?

I tried to dive a bit into the code and tried to find out what is actually happening.
It looks like here
https://github.com/rust-lang/cargo/blob/7ff1bdbcc38b1295c51306dbab2fd9c7b3581e0f/src/cargo/sources/git/utils.rs#L709-L743
the same progress indicator is used for different things sometimes (which partly involves web traffic but also processing some cpu-heavy things)

I think that the high traffic was caused by cargo downloading a lists of clones/pull requests from github for the specific git dependency and the cpu load by extracting the git log data and feeding it into the in-cache repo, however I failed to get some isolated example of this and the debug mode was still not very clear about what was going on under the hood unfortunately.


YANP Fansub (New link)

Source information
Yanp Fansub 1.4.37
Source language
Portuguese
Source new URL
https://trisalyanp.com
Other details
No response
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

old domain is still working for me

old domain is still working for me

the admins are currently deleting the old domain stuff (chapters/manga) and passing them to the new one
it will only work until they finish moving everything


Dependency issues when adding angularfire2 to an angular-cli generated project

Adding angularfire2 to a band new angular-cli generated angular 4 project causes dependency issues:
$ ng new foo
[ ... Project 'foo' successfully created.]
$ cd foo
$ npm install firebase angularfire2 --save
<EMAIL_ADDRESS>/xxx/foo
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| >=4.0.0-beta <5.0.0 but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| >=4.0.0-beta <5.0.0 but none was installed.

For completeness, here is an extract from package.json as generated by the angluar-cli (version 1.5.3).
"dependencies": {
    "@angular/animations": "^5.0.0",
    "@angular/common": "^5.0.0",
    "@angular/compiler": "^5.0.0",
    "@angular/core": "^5.0.0",
    "@angular/forms": "^5.0.0",
    "@angular/http": "^5.0.0",
    "@angular/platform-browser": "^5.0.0",
    "@angular/platform-browser-dynamic": "^5.0.0",
    "@angular/router": "^5.0.0",
    "angularfire2": "^5.0.0-rc.4",
    "core-js": "^2.4.1",
    "firebase": "^4.6.2",
    "rxjs": "^5.5.2",
    "zone.js": "^0.8.14"
  },

Finally, the 'npm outdated' list. (BTW, I am surprised to see 5.0.0-rc.0-next in the angularfire2 wanted column even though package.json specifies the latest version 5.0.0-rc.4 but I don't think that is related to the dependency issues.)
$ npm outdated
Package                   Current           Wanted      Latest  Location
@angular/cli                1.5.3            1.5.3       1.5.4  foo
@types/jasmine             2.5.54           2.5.54       2.8.2  foo
@types/node                6.0.92           6.0.92      8.0.53  foo
angularfire2           5.0.0-rc.4  5.0.0-rc.0-next  5.0.0-rc.4  foo
codelyzer                   3.2.2            3.2.2       4.0.1  foo
jasmine-core                2.6.4            2.6.4       2.8.0  foo
jasmine-spec-reporter       4.1.1            4.1.1       4.2.1  foo
karma-chrome-launcher       2.1.1            2.1.1       2.2.0  foo
protractor                  5.1.2            5.1.2       5.2.0  foo
ts-node                     3.2.2            3.2.2       3.3.0  foo
tslint                      5.7.0            5.7.0       5.8.0  foo
typescript                  2.4.2            2.4.2       2.6.1  foo

I'm having the same issue

same problem

Same problem any luck?

Closing, outdated. Feel free to open a new issue if you're having trouble with more recent releases.


Don't use in ios mobile system,can use in Android;

in the  ios  browser ，the api have no effect

Not able to reproduce. Please provide a minimal reproduction repo to test on 🙏


adding .editorconfig

to avoid roughly formating errors maybe adding a .editorconfig file with the PSR2 standards would be nice

sure, but .editorconfig doesn't cover all of PSR2


Support for RHEL7

I've been using MUPX for a while.  My current environment standardized on RHEL7 and there where some problems that needed to be ironed out on the platform.
These changes allow MUPX to work on RHEL7.

I've been able to get mupx setup, deploy and logs tested. These operations run without error but i'm finding a problem when running the meteord image with the deployed app. docker ps shows both mongodb and the app image, but it can not be connected to on the network (locally on the server or remotely)
telnet <IP_ADDRESS> 80
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
Connection closed by foreign host.
This may be an issue with docker network that i'm still verifying. Not sure if anybody has any suggestions on this? Still investigating.

Ok ... so the networking issues that I was experiencing where due to our internal proxy server. I was able to resolve them by setting http_proxy to the mupx.json for this environment. There are no issues with any mupx operation on a RHEL7 server. I do not have ability to test CentOS.

Is there any interest in merging RHEL support?


noise2self: threads and processes

📚 Documentation

The DelayQueue fixes the Jupyter/Torch example.
Processes are not needed, threads work.
Threads work in Jupyter itself or in the stand-alone script.

Processes did not work inside Jupyter

Details
Previously we concluded that processes (rather than threads) were needed for the noise2self Jupyter notebook. This actually not true. I ran noise2self example on latest async PR #1625 and it worked fine with threads.
On a hunch I disabled the DelayQueue and performance tanked. With no DelayQueue we end up continuously running 6 copies of the Torch computation in the background. With a DelayQueue when scrolling around we run nothing in the background. Back when we decided processes were the answer we did not yet have a DelayQueue. We added it later to prevent spamming remote servers with request for slices we don't actually need, but it works strongly to our benefit here as well.
I believe that there is contention with running Torch in the background. But if you run zero copies instead of 6 that contention is zero!
Since it works with threads, we can run this in Jupyter itself. The version using processes did not run in Jupyter, only in the stand-alone script version. You would need to modify the the Jupyter version to load the tiny MNIST datasets fully into memory.
If you don't do that it still lets you scroll freely, which is nice, but you can't see any of the data until you stop. If you loads those two into memory it works identically to the old version with processes. You see the two all the time, and then the Torch results pops in later. Actually the threads version with DelayQueue should perform better, because a worker is always available.
This means we have not yet encountered a case where we need to use processes. We could leave process support out entirely. Or we could include it, it's only a few lines, in the experimental part, and decide about exposing it later.
Traces
Here it is with threads with no DelayQueue, it's running 6 copies of the Torch computation in the background at all times. The draws are happening only every 250ms or so:

Here it is with threads and a 100ms DelayQueue, nothing is running in the background so the frame rate is 30Hz or so, then it does the one load (Torch computation) at the end:

Conclusion
When we originally saw this problem we said "there is contention, how can we eliminate that contention?" and we eliminated it by switching from threads to processes. Instead the DelayQueue solution is to eliminate the contention by running nothing in the background.
It might seem like we did the wrong thing at first, but actually the delay queue was a bit tricky, while switching to processes was mostly a matter of just calling ProcessPoolExecutor instead of ThreadPoolExecutor. At any rate we have both solutions now and they both work.

This was implemented in #1704 and the situation resolved and so can be closed for now. If the need for further discussion arises then we can reopen


soc: xtensa: keep the CAVS legacy mode still works

Keep the cavstwist.sh it is still working by specifying an
environment variable when CAVS_LEGACY_MODE is set. Provide
a fallback once the client-server-based tool does not work
well in some conditions.
Signed-off-by: Enjia Mai<EMAIL_ADDRESS>

Set up my old environment (+ CAVS_LEGACY_MODE=y) and tested this out with cavstwist.sh + samples/hello_world and Twister + tests/kernel/fatal/common and it worked a-OK. :smile:

I also think we should make a note somewhere in the public documentation on how to turn on legacy mode. Aside from those two comments, this looks good to go.

Yes, I think this is necessary. Will add them ASAP.

Maybe call this something more descriptive than "legacy"? Even something like "OLD_FLASHER" would provide better detail.

Thanks for the review! Let's call it old flasher.

Maybe add this info to cavstwist.sh? I'm imagining people will try to use cavstwist.sh when the new cavstool.py fails, but then cavstwist.sh will fail and there's no info about this new env var anywhere.

Sorry, @edersondisouza I missing this msg,  I will add it right away, thanks!


tensorflow.python.framework.errors_impl.NotFoundError:  Resource localhost/_AnonymousVar330/N10tensorflow3VarE does not exist.

I tried to run train.py on a tensorflow2.0 env in conda but get this error. I really dont know  what to make out of it , really appreaciate your help.
2020-05-09 11:23:38.277906: W tensorflow/core/common_runtime/bfc_allocator.cc:424] _***********************
2020-05-09 11:23:38.308158: W tensorflow/core/framework/op_kernel.cc:1622] OP_REQUIRES failed at conv_ops.cc:501 : Resource exhausted: OOM when allocating tensor with shape[4,32,416,416] and type float on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
Traceback (most recent call last):
File "train.py", line 299, in
main(args)
File "train.py", line 276, in main
max_queue_size   = 8
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/keras/legacy/interfaces.py", line 91, in wrapper
return func(*args, **kwargs)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/keras/engine/training.py", line 1732, in fit_generator
initial_epoch=initial_epoch)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/keras/engine/training_generator.py", line 220, in fit_generator
reset_metrics=False)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/keras/engine/training.py", line 1514, in train_on_batch
outputs = self.train_function(ins)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/tensorflow_core/python/keras/backend.py", line 3740, in call
outputs = self._graph_fn(*converted_inputs)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/tensorflow_core/python/eager/function.py", line 1081, in call
return self._call_impl(args, kwargs)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/tensorflow_core/python/eager/function.py", line 1121, in _call_impl
return self._call_flat(args, self.captured_inputs, cancellation_manager)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/tensorflow_core/python/eager/function.py", line 1224, in _call_flat
ctx, args, cancellation_manager=cancellation_manager)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/tensorflow_core/python/eager/function.py", line 511, in call
ctx=ctx)
File "/home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/tensorflow_core/python/eager/execute.py", line 67, in quick_execute
six.raise_from(core._status_to_exception(e.code, message), None)
File "", line 3, in raise_from
tensorflow.python.framework.errors_impl.NotFoundError:  Resource localhost/_AnonymousVar330/N10tensorflow3VarE does not exist.
[[node yolo_layer_2/AssignAddVariableOp (defined at /home/joseph/anaconda3/envs/ML/lib/python3.6/site-packages/tensorflow_core/python/framework/ops.py:1751) ]] [Op:__inference_keras_scratch_graph_77400]
Function call stack:
keras_scratch_graph

@joseph-jomon Did you figure out a solution to this problem?

same issue

same issue

same issue


ingress.md follows v1.19

Close #26890

/assign @inductor

/lgtm
Thank you for quick update! :smile:

/approve


Custom formatter

This PR implements the feature requested in issue #7 .

The method LoggerConfig.set_formatter has been added. It allows users to pass any logging.Formatter they need.
The default formatter object has been renamed from Formatter to DefaultFormatter to avoid any confusion.

TODO :

Documentation update
Version bump (0.2.2 ?)

Supporting Python 2.6 makes little sense nowadays, don't worry about failing 2.6 tests (they can be disabled by removing relevant lines from appveyor.yml and .travis.yml).

Documentation update would be needed, it's a public API extension, documentation should mention it.
Version bump: I'll version bump (I think minor - it's a new feature) and release after pulling, but you can extend CHANGELOG and add yourself to AUTHORS.
Tests: a new test checking if LoggerConfig-provided formatter is actually used would be in place.

Do you intend to supplement documentation for formatter + add testcase for it? I wouldn't like to take in PR and release new version without them, and I'm - unfortunately - too busy this week and next to add these myself.

Requested changes have been done. Let me know if you have any further comments.

Thank you! I'll remove python26 jobs from CI and release new pytest-logger.


Add support for Pester tags

Pester supports adding tags to Describe blocks. I propose extending OVF to filter for tests with matching tags and only return the matching tests. Similar support should be added for excluding tags as well.
Example of retrieving all tests from the WindowsServerOvf module with a tag of disk
$tests = Get-OperationValidation -ModuleName 'WindowsServerOvf` -Tag 'disk'

Example of running all OVF tests except for any with a tag of memory
Invoke-OperationValidation -ExcludeTag `memory` -IncludePesterOutput

Strongly desire this; we have a basic set of tests and want to run them both during deployment-artifact-creation and post-deployment; there are some tests that should not run during artifact-creation (domain checks, etc).

please feel free to create a PR, I'm hoping that the folks that need this will make it better!

A PR will be coming in the near future for this. I'm currently working on merging the changes I've made in my internal OVF implementation in another project. That PR will add the ability to send script parameters to the Pester tests referenced in #6.


Comply with EFF's DNT policy v0.1

https://www.eff.org/dnt-policy
Seems pretty comprehensive, and might be a bit of work to get up to code. Just creating a stub for now so we don't lose it. [IRC]
cc: @xnyhps @Changaco @ivanmushketyk

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Closing in light of our decision to shut down Gratipay.
Thank you all for a great run, and I'm sorry it didn't work out! 😞  💃


MonadFail instance for Shell

Should Shell adopt MonadFail?

Yes!  Shell already implements an explicit fail method for Monad, so it's only natural to use MonadFail now that it's available


Reverse-mode differentiation rule for 'sub' not implemented

I get an error with the gradients of multivariate_normal.logpdf wrt the covariance (input and mean are fine). For example, this
import jax.numpy as np
from jax import grad
from jax.scipy.stats import multivariate_normal

f = lambda x: multivariate_normal.logpdf(np.zeros(1), np.zeros(1), np.ones((1, 1))*x)
grad(f)(1.)  # fails

gives the error:
Traceback (most recent call last):
  File "[...]/jax/jax/interpreters/ad.py", line 149, in get_primitive_transpose
    return primitive_transposes[p]
KeyError: sub

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "[...]", line 43, in <module>
    print(grad(f)(1.))  # fails
  File "[...]/jax/jax/api.py", line 149, in grad_f
    ans, g = value_and_grad_f(*args, **kwargs)
  File "[...]/jax/jax/api.py", line 186, in value_and_grad_f
    g = vjp_py(onp.ones((), onp.result_type(ans)))
  File "[...]/jax/jax/api_util.py", line 56, in apply_jaxtree_fun
    ans = fun(*args)
  File "[...]/jax/jax/api.py", line 457, in out_vjp_packed
    return out_vjp(cotangent_in)
  File "[...]/jax/jax/interpreters/ad.py", line 78, in vjp_
    _, arg_cts = backward_pass(jaxpr, consts, (), dummy_primal_and_ct)
  File "[...]/jax/jax/interpreters/ad.py", line 135, in backward_pass
    cts_out = get_primitive_transpose(eqn.primitive)(ct_in, *invals, **eqn.params)
  File "[...]/jax/jax/interpreters/ad.py", line 152, in get_primitive_transpose
    "Reverse-mode differentiation rule for '{}' not implemented".format(p))
NotImplementedError: Reverse-mode differentiation rule for 'sub' not implemented

Also, I notice that the multivariate_normal density is implemented with matrix inverses and determinant rather than the usual more efficient formulation in terms of the square root cov. Is there a reason for this?

PR #410 fixes your immediate issue.
As for the less efficient formulation, I suspect the reason is that the formulation in scipy uses a symmetric eigendecomposition, which we only added to JAX relatively recently (https://jax.readthedocs.io/en/latest/jax.scipy.html#jax.scipy.linalg.
PRs to use the more efficient formulation are welcome!
(The only caveat is that eigh is only implemented on CPU at the moment, but we should probably just add it on GPU rather than worrying about that.)

I would guess the Cholesky square root (rather than the symmetric square root) is often more efficient, though doesn't have some other nice properties of the symmetric square root solution.


Label Dataset

Describe
Model I am using (LayoutLM ...):
Can I label my dataset something other than (0-15)
I tried labeling it 16 and above I get an error on in convert_examples_to_features KeyError: '17'

@sarahbacha I don't quite understand your question. Can you show more details?

If I want to classify more than 16 types of forms

I think you need to delete the cached files and try again.


AzureCliCredential does not work as intended when using a Chinese version of Windows

Azure Identity:
1.12:
Windows 11:
Python 3.7:

Describe the bug
AzureCliCredential returns an ClientAuthenticationError when azure cli is not installed, on a Chinese windows installation.
The correct error to return is a CredentialUnavailableError, which is returned if using English language version of Windows. Returning the wrong error, makes it not possible to use AzureCliCredential in a ChainedTokenCredential chain.
I belive this bug is because AzureCliCredential is hardcoded to look for the English text:
ex.stderr.startswith("'az' is not recognized")
To Reproduce
Steps to reproduce the behavior:

Use a Chinese version of Windows 11.
Attempt to use AzureCliCredential, without having azure cli installed

Expected behavior
I expect the CredentialUnavailableError to be returned, not ClientAuthenticationError.
Screenshots

Hi @tboerstad thanks for your issue, we'll take a look and get back to you as soon as possible.

@pvaneck @xiangyan99 This is a real issue, we must not assume the OS is in English to work properly. This is an example what you would get on a French machine:
>az
'az' n’est pas reconnu en tant que commande interne
ou externe, un programme exécutable ou un fichier de commandes.

Let's make sure we're not coupling our implementation with a given OS configuration.

@tboerstad Thanks for the issue!
@lmazuel Yea, definitely an oversight in implementation. I'll look into resolving this.


3.0 RC: PostgreSQL connection fails when supplying password and always fails in release builds

Connecting to PostgreSQL always fails in release builds, and it always fails in debug builds when you supply a password.
Steps to reproduce
1. Create user + database:
createuser bertandernie
createdb bigbird
psql
alter user bertandernie with encrypted password 'sesamestreet';
grant all privileges on database bigbird to bertandernie;
\q

2. Update configuration
Update configure.swift to have the following configuration:
    // Configure PostgreSQL Database
    let postgresConfig: PostgreSQLDatabaseConfig
    if env.name == "production" {
        // here you would probably have production credentials
        print("Using PostgreSQLDatabaseConfig as database user with supplying password")
        postgresConfig = PostgreSQLDatabaseConfig(hostname: "localhost", port: 5432, username: "bertandernie", database: "bigbird", password: "sesamestreet")
    } else {
        // here you would have development credentials
        print("Using PostgreSQLDatabaseConfig as database user without supplying password")
        postgresConfig = PostgreSQLDatabaseConfig(hostname: "localhost", port: 5432, username: "bertandernie", database: "bigbird")
    }
    let postgresDatabase = PostgreSQLDatabase(config: postgresConfig)
    databases.add(database: postgresDatabase, as: .psql)
    services.register(databases)

3. Build and run
$ swift build
Compile Swift Module 'App' (15 sources)
Linking ./.build/x86_64-apple-macosx10.10/debug/Run
 $ .build/x86_64-apple-macosx10.10/debug/Run
Running development build on macOS (non-release, debug mode)
Using PostgreSQLDatabaseConfig as database user without supplying password
Migrating psql DB
Migrations complete
Server starting on http://<IP_ADDRESS>:8080
^C $ .build/x86_64-apple-macosx10.10/debug/Run --env prod
Running production build on macOS (release, debug mode)
Using PostgreSQLDatabaseConfig as database user with supplying password
Migrating psql DB
Fatal error: Error raised at top level: ⚠ PostgreSQL Error: No password is required
- id: PostgreSQLError.trust
: file /BuildRoot/Library/Caches/com.apple.xbs/Sources/swiftlang/swiftlang-902.0.41/src/swift/stdlib/public/core/ErrorType.swift, line 191
Illegal instruction: 4
$

Observe that running the development build will connect properly when you don't supply a password, but it fails with PostgreSQL Error: No password is required when you do supply a password.
4. Build and run for release
$ swift build --configuration release
...
$ .build/x86_64-apple-macosx10.10/release/Run
Running development build on macOS (non-release, release mode)
Using PostgreSQLDatabaseConfig as database user without supplying password
Migrating psql DB
[PostgreSQL] dataCorrupted(Swift.DecodingError.Context(codingPath: [], debugDescription: "Cannot initialize PostgreSQLAuthenticationType from invalid Int32 value 8", underlyingError: nil))
^C $ .build/x86_64-apple-macosx10.10/release/Run --env prod
Running production build on macOS (release, release mode)
Using PostgreSQLDatabaseConfig as database user with supplying password
Migrating psql DB
[PostgreSQL] dataCorrupted(Swift.DecodingError.Context(codingPath: [], debugDescription: "Cannot initialize PostgreSQLAuthenticationType from invalid Int32 value 8", underlyingError: nil))
^C $

Observe that the release build will always fails with Cannot initialize PostgreSQLAuthenticationType from invalid Int32 value 8, regardless of whether or not you supply a password.

On Ubuntu Linux (16.04 LTS) I see different behaviour:
1. Build and run
$ swift build
...
$ .build/x86_64-unknown-linux/debug/Run
Running development build on linux (non-release, debug mode)
Using PostgreSQLDatabaseConfig without password
Migrating psql DB
Fatal error: Error raised at top level: ⚠ PostgreSQL Error: Password is required
- id: PostgreSQLError.password
: file /home/buildnode/jenkins/workspace/oss-swift-4.1-package-linux-ubuntu-16_04/swift/stdlib/public/core/ErrorType.swift, line 191
Current stack trace:
0    libswiftCore.so                    0x00007fb657da7080 _swift_stdlib_reportFatalErrorInFile + 221
1    libswiftCore.so                    0x00007fb657b1634c <unavailable> + 1368908
2    libswiftCore.so                    0x00007fb657d4fd32 <unavailable> + 3702066
3    libswiftCore.so                    0x00007fb657b15a46 <unavailable> + 1366598
4    libswiftCore.so                    0x00007fb657d4fc37 <unavailable> + 3701815
5    libswiftCore.so                    0x00007fb657b15a46 <unavailable> + 1366598
6    libswiftCore.so                    0x00007fb657c82c89 <unavailable> + 2862217
7    libswiftCore.so                    0x00007fb657b53850 swift_errorInMain + 318
8    Run                                0x00005589f548c531 <unavailable> + 4400433
9    libc.so.6                          0x00007fb65558e740 __libc_start_main + 240
10   Run                                0x00005589f50cc239 <unavailable> + 467513
Illegal instruction (core dumped)
$ .build/x86_64-unknown-linux/debug/Run --env prod
Running production build on linux (release, debug mode)
Using PostgreSQLDatabaseConfig with password
Migrating psql DB
Migrations complete
[ WARNING ] Debug build mode detected while configured for release environment: production. (Application.swift:60)
[ INFO ] Compile your application with `-c release` to enable code optimizations. (Application.swift:61)
Server starting on http://<IP_ADDRESS>:8080
^C $

Observe that when building on Ubuntu, the connection only succeeds if you do supply a password...
2. Build for release and run
$ swift build --configuration release
...
$ .build/x86_64-unknown-linux/release/Run
Running development build on linux (non-release, release mode)
Using PostgreSQLDatabaseConfig without password
Migrating psql DB
[PostgreSQL] dataCorrupted(Swift.DecodingError.Context(codingPath: [], debugDescription: "Cannot initialize PostgreSQLAuthenticationType from invalid Int32 value 12", underlyingError: nil))
^C $ .build/x86_64-unknown-linux/release/Run --env prod
Running production build on linux (release, release mode)
Using PostgreSQLDatabaseConfig with password
Migrating psql DB
[PostgreSQL] dataCorrupted(Swift.DecodingError.Context(codingPath: [], debugDescription: "Cannot initialize PostgreSQLAuthenticationType from invalid Int32 value 12", underlyingError: nil))
^C $

Observe that on Ubuntu Linux the release build will also always fail, like on macOS


SmallMoleculeComponent._rdkit not equal upon roundtrip with to_json, from_json

Serializing and deserializing a SmallMoleculeComponent sourced from the tyk2 system in openfe-benchmarks doesn't appear to cleanly roundtrip:
from importlib import resources
from gufe.tokenization import TOKENIZABLE_REGISTRY

# get a `SmallMoleculeComponent` from openfe-benchmarks
with resources.path('openfe_benchmarks.data',
                    'tyk2_ligands.sdf') as fn:
    ligands_sdf = Chem.SDMolSupplier(str(fn), removeHs=False)
    lig  = [SmallMoleculeComponent(rdkit_ligand) for rdkit_ligand in ligands_sdf][0]

# serialize
lig_json = lig.to_json()

# drop this `SmallMoleculeComponent` from the registry so we ensure we get a new object upon deserialization
TOKENIZABLE_REGISTRY.pop(lig.key)

# deserialize
lig2 = SmallMoleculeComponent.from_json(lig_json)

# yields True
lig2 == lig

# yields False!
lig2._rdkit == lig._rdkit

@richardjgowers can I get your eyes on this one? I think you had recently spent time working on the SmallMoleculeComponent serialization, so you might be the most knowledgeable on what could be happening here.

@dotsdl re: the rdkit inequality, that's a red herring, see:
m = Chem.MolFromSmiles('CC')
m2 = Chem.MolFromSmiles('CC')

m == m2
>>> False

I think what's happening here is this line in the error: getNumImplicitHs() called without preceding call to calcImplicitValence().  I think I'm keeping the "public" Properties of the rdkit Molecule when we serialise (user defined tags) but not bookkeeping Props.  If I do lig2._rdkit.UpdatePropertyCache() then everything starts working again.
I think the change to fix this will be to do the cache update in the from_dict() but I'll double check it in the morning


Absolute paths

This should fix all issues with cron.

The only conflict is this, which one is the correct?
<<<<<<< absolute-paths
            handleClaim(packpub, args, config, dir_path)
=======
            handleClaim(packpub, args, config)
        except NoBookException as e:
            log_info('[*] ' + e.message)
>>>>>>> master

I think is the one with 4 params, should I keep the NoBookException ?
try:
            packpub.runDaily()
<<<<<<< absolute-paths
            handleClaim(packpub, args, config, dir_path)
=======
            handleClaim(packpub, args, config)
        except NoBookException as e:
            log_info('[*] ' + e.message)
>>>>>>> master
        except Exception as e:
            log_debug(e)
            if args.notify:
                Notify(config, None, None, args.notify).sendError(e, 'daily')
            return

I think is like this, can you just confirm?
try:
            packpub.runDaily()
            handleClaim(packpub, args, config, dir_path)
        except NoBookException as e:
            log_info('[*] ' + e.message)
        except Exception as e:
            log_debug(e)
            if args.notify:
                Notify(config, None, None, args.notify).sendError(e, 'daily')
            return

Perfect :)
niqdev<EMAIL_ADDRESS>schrieb am Fr., 16. Dez. 2016, 22:59:

I think is like this, can you just confirm?
try:
packpub.runDaily()
handleClaim(packpub, args, config, dir_path)
except NoBookException as e:
log_info('[*] ' + e.message)
except Exception as e:
log_debug(e)
if args.notify:
Notify(config, None, None, args.notify).sendError(e, 'daily')
return
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/niqdev/packtpub-crawler/pull/36#issuecomment-267705979,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEmPPDsjm76EmcL7S4Gmy95qPxtmZYkYks5rIwmigaJpZM4LPJKR
.

Do you have any idea why I can't see your name @juzim among the contributors?

Maybe the rebase messed it up. I tried to keep all three features in sync,
I guess that's how I pay for it.
niqdev<EMAIL_ADDRESS>schrieb am Fr., 16. Dez. 2016, 23:19:

Do you have any idea why I can't see your name @juzim
https://github.com/juzim among the contributors?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/niqdev/packtpub-crawler/pull/36#issuecomment-267709943,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEmPPFk5XGihOmn2wr3Fr2-3f1pswRyxks5rIw6LgaJpZM4LPJKR
.

About the merge, when you have time, just check if everything works fine (at least the configuration that you have added). I'll do the same.
About the contributions, I think you have something in your settings that block your visibility (I can't see your picture in the commits also)

Oh my god, thanks for telling me about contributors! I never bothered to look and was still using my work email in the config. So many contributions lost…

:+1:


Please help a quadriplegic

I’m a quadriplegic who can’t use my hands at all.
I used SmartNav by naturalpoint for about 20 years. It was great. But they discontinued the product a couple of years ago and my last unit finally died on me about a month back. I have been looking for a good alternative with no good solutions presenting themselves so far. I tried an eye tracker from Tobii/dynavox and it didn’t really work well. I was intrigued by the video of the Precision Gaze Mouse, particularly since it could be used with NaturalPoint’s TrackIR. I ordered a TrackIR and it arrived today.
The user manual for Precision Gaze Mouse says that I have to choose “GazePlusMouse” in the “titles tab” of the TrackIR but I find no entry for anything like that, nor anything that could be confused with the Precision Gaze Mouse in TrackIR. Nor was there an option for me to choose a new title to enter manually.
Please advise. I’m at my wit’s end. I had great computer facility for about 20 years with the SmartNav and right now I feel dead in the water. I’ve tried multiple options already and nothing else gets me more than about 1/3 as effective as I was with the SmartNav.
Please help. I want my computer capability back :-(

Sorry to hear you lost your computer capability! I'd be really frustrated if that happened. Did you try refreshing the titles list in the TrackIR program?

Sorry to hear you lost your computer capability! I'd be really frustrated if that happened. Did you try refreshing the titles list in the TrackIR program?

How do I do that?
Basically, to be clear, this is what I’m attempting to do: use a TrackIR ONLY (no eye-tracking) with the Precision Gaze Mouse software (and a 3rd-party dwell-clicking app) to control my mouse in a Windows desktop environment (browsing, clicking around, etc.). This is what I used to do with the SmartNav from naturalpoint (a product that has now, VERY unfortunately, been discontinued).
The demo video regarding the Precision Gaze Mouse looked closest to replicating the performance of the SmartNav, compared to everything else I’ve seen out there. And I have now tried some of the alternatives, all of which are very poor substitutes and extremely unappealing (so far. I have one more coming in tomorrow).
Please tell me EXACTLY what I have to do to make this work. Do I open Precision Gaze Mouse first? Does the order in which I open the applications matter?
How do I refresh the software list in TrackIR? All I see there are games (none of which I own). Is there a way to manually add an entry there?
I have the Trackclip Pro as well but I don’t care for it much. The TrackIR seems to pick things up fine if I put a large reflective dot on my nose (which I’m used to from the SmartNav), so I would prefer to keep doing that.
Please advise. This is one of my last 2 chances of regaining something close to normal mouse control. My quality-of-life will be undeniably worse if I can’t get back to something similar to what I enjoyed for about 20 years with the SmartNav.
Thank you in advance.

Great let me know how it goes

Closing due to no response, hoping you got it working


Support passing options into the create call

I noticed that in order to turn auto conversion on for data you need to pass typecast: truein the request. In order to do that through the airrecord wrapper we'd need a way to send it to the create call.
From my base's API documentation:

Airtable API performs automatic data conversion from string values if typecast parameter is passed in (click for example). Automatic conversion is disabled by default to ensure data integrity, but it may be helpful for integrating with 3rd party data sources.

Do you think it'd make sense to enable that by default?

I would probably say no. Since you have to be explicit about it in the POST request, should probably likewise be explicit about it in the Ruby wrapper.

👍 would accept a PR that does this


Plugin not being transpiled

I have been trying to use systemjs-builder to bundle the typescript with plugin-typescript but I am getting an error [Unexpected reserved word] when trying to load the plugin. This indicates that plugin-typescript is not getting transpiled by traceur.
I have tried updating plugin-typescript to explicitly set its format to 'es6' and that changes the error but it is still not getting put through traceur. The strange thing is that I had it working at some point yesterday.
Can you think of anything I may be doing wrong? I have updated the example project in plugin-typescript to have a 'gulp bundle' task which exhibits the error.

The issue is that setting registry: 'npm' in the package.json makes jspm think that the package is CommonJS.
I've added a new option called jspmNodeConversion: false which can be used alongside registry: 'npm' to ensure that this doesn't happen. It is coming in the next version of jspm to be posted very soon.
In the mean time I would suggest removing the registry: 'npm' part for now.
Just let me know if you need help further.

Thanks for the response - I've removed the 'registry: npm' setting and updated to systemjs-builder 0.10.0. I am still getting an error though.
Cannot call method 'then' of undefined
    at Loader.pluginLoader.instantiate (C:\Work\bitbucket\plugin-typescript\node_modules\systemjs-builder\lib\builder.js:57:6)
    at Promise.resolve.then.then.load.address (C:\Work\bitbucket\plugin-typescript\node_modules\systemjs-builder\node_modules\systemjs\node_modules\es6-module-loader\dist\es6-module-loader.src.js:1546:33)
    at tryCatchReject (C:\Work\bitbucket\plugin-typescript\node_modules\systemjs-builder\node_modules\systemjs\node_modules\es6-module-loader\dist\es6-module-loader.src.js:1183:30)
    at runContinuation1 (C:\Work\bitbucket\plugin-typescript\node_modules\systemjs-builder\node_modules\systemjs\node_modules\es6-module-loader\dist\es6-module-loader.src.js:1142:4)
    at Fulfilled.when (C:\Work\bitbucket\plugin-typescript\node_modules\systemjs-builder\node_modules\systemjs\node_modules\es6-module-loader\dist\es6-module-loader.src.js:930:4)
    at Pending.run (C:\Work\bitbucket\plugin-typescript\node_modules\systemjs-builder\node_modules\systemjs\node_modules\es6-module-loader\dist\es6-module-loader.src.js:821:13)
    at Scheduler._drain (C:\Work\bitbucket\plugin-typescript\node_modules\systemjs-builder\node_modules\systemjs\node_modules\es6-module-loader\dist\es6-module-loader.src.js:97:19)
    at Scheduler.drain (C:\Work\bitbucket\plugin-typescript\node_modules\systemjs-builder\node_modules\systemjs\node_modules\es6-module-loader\dist\es6-module-loader.src.js:62:9)
    at process._tickCallback (node.js:419:13)

The error happens here:
  // <IP_ADDRESS>
  function proceedToTranslate(loader, load, p) {
    p
    // <IP_ADDRESS>.1 CallTranslate
    .then(function(source) {
      if (load.status != 'loading')
        return;

      return Promise.resolve(loader.loaderObj.translate({ name: load.name, metadata: load.metadata, address: load.address, source: source }))

      // <IP_ADDRESS>.2 CallInstantiate
      .then(function(source) {
        load.source = source;
			console.log('here I am');
        return loader.loaderObj.instantiate({ name: load.name, metadata: load.metadata, address: load.address, source: source });
      })

      // <IP_ADDRESS>.3 InstantiateSucceeded
      .then(function(instantiateResult) {

        if (instantiateResult === undefined) {
          load.address = load.address || '<Anonymous Module ' + ++anonCnt + '>';

So loader.loaderObj.instantiate is calling 'then' on an undefined.
Loader.prototype.instantiate = function(load) {
    // load transpiler as a global (avoiding System clobbering)
    if (load.name === this.transpiler) {
      var self = this;
      return {
        deps: [],
        execute: function() {
          var curSystem = g.System;
          var curLoader = g.Reflect.Loader;
          __eval('(function(require,exports,module){' + load.source + '})();', g, load);
          g.System = curSystem;
          g.Reflect.Loader = curLoader;
          return getTranspilerModule(self, load.name);
        }
      };
    }
// FW - added this
		else {
			return Promise.resolve(undefined);
		}
  };

Adding the condition at the end to return a valid promise allows everything to continue and the bundling now works fine. This is probably not the right fix but it keeps me going for now.
As before you can recreate this by running gulp bundle in plugin-typescript (after running jspm install in the example folder). Thanks.

Great to hear that - thanks, I added a fix in https://github.com/systemjs/builder/commit/6d277bd7e94f827c8dc5f0844bd2e912913a98b6.


Fix(core): remove sorting on recommendations retrieval

Recommendations were being sorted and this affects the behaviour of Recommend rules (order is given from the API).
Removed filtering based on 'objectID' as it is no longer needed.

This was necessary when providing multiple objectIDs like recommendations in a cart. Doesn't this break this behavior?

HI @francoischalifour,
@jojva confirmed that they don't return duplicated objectIDs in a previous PR, so I think we can safely remove this logic.

Hi @francoischalifour,

Right, but do they sort the merged hits now?

That's the part I don't understand. When we say "multiple indices", does this mean a recommendation request with several sub-queries, each targeting an index? And if so, I wonder why the library needs to re-sort those, discarding the origin of the index?
Results do not necessarily return a recommendation score, as we now have recommend rules, that are able to pin items. And secondly, recommendation scores are not necessarily the only basis for the ranking. It can also be affected by boost and bury (new recommend rules options), as well as the rest of the ranking formula.
I don't think we can or should mix hits coming from several indices. I'd like to understand the idea behind this and how we can solve the issue.

@jojva I'm out of the loop with new Recommend features like rules and pins. How about we sync together this week? DMing you.

Closing as this issue needs to be discussed further given its impact on the recommend flows.


docs: Refactor ECHO architecture diagram

🤖 Generated by Copilot at 6bd1b41
Summary
Walkthrough

Current dependencies on/for this PR:

main

PR #3187   👈

This comment was auto-generated by Graphite.


feat(example): add README on examples/discounted-cartrade

Signed-off-by: Yasushi Takahashi<EMAIL_ADDRESS>

This PR is a starting point to resolve #1148

Codecov Report

Merging #1155 (feace2e) into main (291dd3b) will decrease coverage by 0.06%.
The diff coverage is n/a.

:exclamation: Current head feace2e differs from pull request most recent head 696599d. Consider uploading reports for the commit 696599d to get more accurate results

@@            Coverage Diff             @@
##             main    #1155      +/-   ##
==========================================
- Coverage   73.19%   73.13%   -0.07%
==========================================
  Files         246      246
  Lines        8619     8617       -2
  Branches      996      996
==========================================
- Hits         6309     6302       -7
- Misses       1779     1784       +5
  Partials      531      531

Impacted Files
Coverage Δ

...main/typescript/rustc-container/rustc-container.ts
66.29% <0.00%> (-2.25%)
:arrow_down:

...t-tooling/src/main/typescript/common/containers.ts
74.07% <0.00%> (-1.24%)
:arrow_down:

...rc/main/typescript/fabric/fabric-test-ledger-v1.ts
83.42% <0.00%> (-0.18%)
:arrow_down:

.../main/typescript/plugin-ledger-connector-fabric.ts
82.80% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 291dd3b...696599d. Read the comment docs.


Add cloud shadow projection notebook

Can you move this to the AgriDataValue folder?

Can you move this to the AgriDataValue folder?

done


Docker-compose uses unofficial v1r3n/dynomite image

The Conductor docker-compose.yml uses the unofficial (-looking) v1r3n/dynomite image instead of Netflix/dynomite:
https://github.com/Netflix/conductor/blob/e100b98451ece8fec1bc6b4e2af6e5d5a8f9fb90/docker/docker-compose.yaml#L44
Is there a reason for this? It'd be good to know why Conductor recommends this, if it does.
Thanks!

@robertlagrant Thanks for reporting this.
This docker image belongs to the original author of Conductor and Dyno-Queues (a queueing recipe built on top of Dynomite). We will work with the dynomite team for an official docker image that can be used here.


Actors API typo

URL of the docs page

How is it currently worded?
You can provided the
How should it be worded?
You can provide the
Screenshots

Additional context

@derrickburns - which page is this? And it would be great if you could submit a PR for these that you are reviewing

@derrickburns - Ping?

/assign

@derrickburns - which page is this? And it would be great if you could submit a PR for these that you are reviewing

FYI @msfussell page: https://docs.dapr.io/reference/api/actors_api/