Investigate and fix pthreads-related hang on argon/profile1_core/streams/test6075.obu

Running the argon test vector in test6075.obu hangs rav1d (at least on donna). This happens ~48% into the test suite.
$ target/release/dav1d -i tests/argon/profile1_core/streams/test6075.obu --verify 9e0549e41d672ab97e9b99181556b14c
dav1d 1.0.0-130-g26eca15 - by VideoLAN
Unknown Metadata OBU type 0
Decoded 2/4 frames (50.0%) - 3409.25/25.00 fps (136.37x)

output from dav1d:
dav1d 1.2.0-1-gf89dbc0 - by VideoLAN
Unknown Metadata OBU type 0
Decoded 3/4 frames (75.0%) - 3842.95/25.00 fps (153.72x)

NOTE: Adding --threads 1 to the rav1d invocation fixes the problem. gdb backtrace:
#0  __futex_abstimed_wait_common64 (private=0, cancel=true, abstime=0x0, op=393, expected=0, futex_word=0x56381524a340) at ./nptl/futex-internal.c:57
#1  __futex_abstimed_wait_common (cancel=true, private=0, abstime=0x0, clockid=0, expected=0, futex_word=0x56381524a340) at ./nptl/futex-internal.c:87
#2  __GI___futex_abstimed_wait_cancelable64 (futex_word=futex_word@entry=0x56381524a340, expected=expected@entry=0, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=private@entry=0)
    at ./nptl/futex-internal.c:139
#3  0x00007fb737807ac1 in __pthread_cond_wait_common (abstime=0x0, clockid=0, mutex=0x56381522d100, cond=0x56381524a318) at ./nptl/pthread_cond_wait.c:503
#4  ___pthread_cond_wait (cond=0x56381524a318, mutex=0x56381522d100) at ./nptl/pthread_cond_wait.c:627
#5  0x0000563813ab1ebd in dav1d_get_picture ()
#6  0x0000563813a62b43 in dav1d::main ()
#7  0x0000563813a63093 in std::sys_common::backtrace::__rust_begin_short_backtrace ()
#8  0x0000563813a63069 in _ZN3std2rt10lang_start28_$u7b$$u7b$closure$u7d$$u7d$17h3352d299af5ab469E.llvm.9145856085723143525 ()
#9  0x0000563813c11eba in core::ops::function::impls::{impl#2}::call_once<(), (dyn core::ops::function::Fn<(), Output=i32> + core::marker::Sync + core::panic::unwind_safe::RefUnwindSafe)> ()
    at library/core/src/ops/function.rs:280
#10 std::panicking::try::do_call<&(dyn core::ops::function::Fn<(), Output=i32> + core::marker::Sync + core::panic::unwind_safe::RefUnwindSafe), i32> () at library/std/src/panicking.rs:492
#11 std::panicking::try<i32, &(dyn core::ops::function::Fn<(), Output=i32> + core::marker::Sync + core::panic::unwind_safe::RefUnwindSafe)> () at library/std/src/panicking.rs:456
#12 std::panic::catch_unwind<&(dyn core::ops::function::Fn<(), Output=i32> + core::marker::Sync + core::panic::unwind_safe::RefUnwindSafe), i32> () at library/std/src/panic.rs:137
#13 std::rt::lang_start_internal::{closure#2} () at library/std/src/rt.rs:128
#14 std::panicking::try::do_call<std::rt::lang_start_internal::{closure_env#2}, isize> () at library/std/src/panicking.rs:492
#15 std::panicking::try<isize, std::rt::lang_start_internal::{closure_env#2}> () at library/std/src/panicking.rs:456
#16 std::panic::catch_unwind<std::rt::lang_start_internal::{closure_env#2}, isize> () at library/std/src/panic.rs:137
#17 std::rt::lang_start_internal () at library/std/src/rt.rs:128
#18 0x0000563813a62ed2 in main ()

Building the C version of dav1d that rav1d is based on also exhibits this issue so the fix to this was likely made after we transpiled the code. Pretty sure the fix is to migrate these two patches which would catch us up with dav1d 1.1.0:

https://code.videolan.org/videolan/dav1d/-/commit/3f19ece69ff5dcddf3791025dfaf5ab15196c899
https://code.videolan.org/videolan/dav1d/-/commit/9b4b2448106f863bfa4789ba6e86555ad57c838f


OpenCV bindings for Python 3 not installed

I installed OpenCV 4.5.2 via this script on a fresh Jetson Nano and checked the installation with the procedure described in this issue. This was working properly, Cuda was enabled and OpenCV version was 4.5.2 both using command opencv_version and in a Python 3 interpreter.
However, I restarted the Jetson Nano and now in a Python 3 interpreter I got version 4.1.1. If I open an interpreter in sudo mode it still shows version 4.5.2.
Do you know the cause of this issue? However, solving the issue does not seem hard, I guess that copying the system Python cv2 shared library to the local Python dist-package folder should do the trick.

Ok, this did the trick:
sudo cp /usr/local/lib/python3.6/dist-packages/cv2/python-3.6/*.so /usr/lib/python3.6/dist-packages/cv2/python-3.6/

unfortunately this does not the trick for me.
build_opencv.sh runs without errors (4.4 and 4.5.5), but cuda-support is only available in python2.7 not in python3.6.

sudo cp /usr/local/lib/python3.6/dist-packages/cv2/python-3.6/*.so /usr/lib/python3.6/dist-packages/cv2/python-3.6/

@Randy37773
I was able to resolve the issue by running the script below.
sudo cp /usr/local/lib/python3.6/site-packages/cv2/python-3.8/*.so /usr/lib/python3.8/dist-packages/cv2/python-3.8/

Solve it for me:
ln -s /usr/local/lib/python3.6/site-packages/cv2/python-3.6/cv2.cpython-36m-aarch64-linux-gnu.so cv2.so


[Bug][Dashboard] Engineering Throughput and Cycle Time - Team View shows no data

Search before asking

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

What happened

It seems to be cause by this:

What do you expect to happen
Show data
How to reproduce
Install v0.18.0-beta1 and configure teams
Anything else
No response
Version
v0.18.0-beta1
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This is not a bug caused by data collection, I have turned it off.


Kap offers unsupported update

Platform: OS X 10.11.6 (El Capitan)
Kap Version: 1.0.1 (<IP_ADDRESS>)
Kap offers me to update to a newer version that isn't supported by El Capitan.
.
Expected behavior
Kap shouldn't offer users to update to a not compatible version.
Workaround
Check if the current OS version is compatible with the newer version of Kap

Thanks for reporting this! Unfortunately there's no way to fix this without requiring significant effort, that effort is better focused on supporting the latest OS and working on fixing features that will benefit our upgrading users. Not worth the effort needed to fix this IMHO. Upgrade your OS. 👍

@albinekb Haha ok. I understand.

Upgrade your OS. 👍

naaaah. I'm fine with el capitan ;)

Thank you for understanding 🙏😊 I urge you to upgrade, the new Kap is amazing. 👌😉

Is there an option to disable checking for updates in v1?
I'm running El Capitan v10.11.6 (15G17023) and it does the forced upgrade - which breaks Kap every time I reboot.
It's a pain in the ass to re-install the old version so often. 😿
I promise I'm not just a complainer, if someone can point me in the right direction, I'd love to submit a PR.

@justsml thanks for that, but we do not want to add an option to stop updates, we don't want people to stay on old versions, because it will be hell to maintain.
What I suggest you doing, I can help you if you want - is to build your own old version from source.  Would that be an ok workaround for you? I can even build and upload it for you if you want. Thanks for understanding 🙏♥

@albinekb may you consider adding an option to stop updates /only/ on release 1.0.1? (<IP_ADDRESS> maybe)
Some of us are blocked on El Captain (mid 2011 macs, for exemple — which still works amazingly well), each time we use kap, we need to delete the forced update to 2.0 and download again 1.0.
Offering an autoupdate is nice, forcing users to get non compatible updates can be disappointing in the long run. Thanks a lot.

We don't have the resources to support old releases @yamsellem, which means we won't be releasing any patches for v2.2.0 or before. Additionally, official support for Kap is limited to macOS 10.12 or never. This isn't the answer you were hoping for, but I hope you can understand our need to ruthlessly prioritise in order to get the best Kap to as many people as possible.
Your best best is to apply the workaround @albinekb suggested above:

What I suggest you to do is to block kap-updates.now.sh in your firewall/hosts file, here's how to do it https://www.wikihow.com/Block-a-Website-on-Mac

@skllcrn I completely understand your priorities & editing /etc/hosts is perfectly fine to me.
Thanks for your quick solution to the issue — autoupdates are not fetched anymore on 1.0.1, perfect!

Glad that it helped @yamsellem! I think you can get Mojave running on your 2011 mac if you'd like, see this video: https://www.youtube.com/watch?v=ZLVvP3WCCI0


End to end testing from RawSign

Following up for the breakage fixed by https://github.com/MystenLabs/sui/pull/4911. original issue caused by the shift in underlying library on Ed25519 secretKey representation (32 vs 64 bytes).

added


[Feature Request]: implement bolt12 offfers/static invoices via boltz.exchange integration

bolt12 static invoices are also called offers. helps to improve the user experience for lightning payments.
https://bolt12.org/
these are being worked on in the boltz repo
https://github.com/BoltzExchange/boltz-client/issues/304

https://x.com/Boltzhq/status/1853834574914113720


Can't override location cache

I changed location property in json
added location_cache and then passed along -lc false and then --location_false
6-07-23 21:42:37 [x] PokemonGO Bot v1.0
2016-07-23 21:42:37 [x] Configuration initialized
[2016-07-23 21:42:37]
[2016-07-23 21:42:37] [x] Last location flag used. Overriding passed in location
Other Information
OS: osx yosemite
Git Commit: 5cd992f7677e9e2c6f2a67c773d752d8469f54ad

Can confirm that location flag is being ignored even in the config.json


komorebi integration causes a console window to spawn when using pythonw

When running yasb through pythonw with komorebi connected a komorebic.exe console window will temporarily spawn. This appears to be on a set interval and at the same time the state command is used. Running yasb alone through pythonw or running it normally with komorebi does not cause this behavior.
Windows 11 22000.348
python - 3.10.0
yasb - 277ee711ee7bb609c472f5e4d4657c86bbe8109f
komorebi - 5e3f1cb
Not a huge deal for me personally but might bother some.
(also thanks for the CPU widget, works great so far.)

I have also dealt with this issue. I fixed this by using Start-Process -WindowStyle hidden -FilePath python ./main.py    which achieves the same effect.

@Vernetzt @rwendell
Thanks for reporting this issue, I have just pushed a fix to main which should resolve this issue: https://github.com/denBot/yasb/commit/6fd575371e9ab6336ea9716ada46a0c98a2937ce
The various subprocess commands executed by yasb require the shell argument to be True in order to prevent console windows from spawning when running the application via pythonw. Custom widgets have also been changed to execute passed arguments via the shell.
Closing for now. Please re-open this issue if you identify any more issues.


feat(webhook-stages): add docs on preconf webhooks

adds documentation on preconfigured webhook stages with examples
@lwander @dorbin PTAL

@dorbin thanks for the feedback. i'll try to get this addressed soon!

@dorbin feedback addressed!


MSID: 85795 Version: 2 DOI: 2<IP_ADDRESS>4625

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=85795)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/85795/v2)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: https://github.com/elifesciences/enhanced-preprints-data/pull/104
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews
Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>4625)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[x] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/85795/v2/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/85795)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/85795

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/85795

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/85795/v2/85795-v2.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/85795)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Corresponding authors both have the same email, but this is an issue in the source preprint, so OK to proceed.


race_ghost: possible fix for peds

Don't send ped and vehicle arguments sometimes on MapStarting to change dimension of them

Other latent arguments are optional and aren't needed, but since latent events are slower this might help reducing the chance for the ped bug anyway


Doubts about the multiprocessing start method and behavior

First of all, thanks for developers' time in creating such a useful tool.
I am trying to modify the text_dedup/minhash.py and use it for deduplication of a ~800GB dataset with each document size varing from 5GB to 50GB. The detailed code running environment is as follows.
CPU: 128-cores
Memory: 2TB
OS: Ubuntu 20.04.5 LTS
Kernel: 3.10.0-1160.el7.x86_64
User: root
Python version: 3.8.10

And I run the code with following command (I modify the data-loading code so that it can handle json files directly):
python -m text_dedup.custom_minhash --path /path/to/src --output /path/to/dst --column text --ngram 7 --min_length 7 --num_perm 64 --threshold 0.85 --local

I found that when the code ran into some specific steps, such as fingerprinting, it seemed that only 1 process was working as I could only see 1 core has 100% load at the same time. And this step could take extremely long time, namely about 3 hours. I tried to search in the multiprocessing documentation and other related sources, but I cannot figure out why. I tried to change the multiprocessing method multiprocessing.set_start_method, but this does not work too.
So I am wondering, is this an expected situation or is there something going wrong? And why did you use fork in the multiprocessing start method? Any special consideration?
Appreciate your time and patience!

When the code is running, I try to use htop to inspect the process status, it looks like:

Something is definitely wrong here. Could you share your modified code here?
If map function is not parallelising properly, it might be a datasets issue.
Also, just to make sure I am reading your message correctly, are you using the script to dedup less than 160 files? The file size seems too big to be suitable for minhash.

Thanks for replying! The modified parts are mainly about dataset loading and saving. The other modified parts would only be performed when --debug is specified. As I am not using --debug here and saving is functioning properly, I would like to only explain the loading part. It is like:
if args.local:
    # ds = load_from_disk(args.path)
    ds = load_dataset(
        "json",
        data_files=get_all_files(
            args.path,
            legal_file_type=('.json','.jsonl'),
            need_abspath=True,
        )[0]
    )

I modify the code behavior when loading from local. Here get_all_files would return a tuple (paths: List[str], n_files: int), where paths is the file paths of the json files. I am not sure if this would cause trouble but I see similar usage in official documentation, about how to load local json files. I will search in the documentation and community to find out if there are similar problems about datasets.
Another thing you mention, sorry for the mistakes. Originally the files were actually so big, but I've run some processing so now each file is 50~100M, and the ~800GB dataset consists of ~9900 files. I am not sure if this file size is suitable for minhash. If you would spare your time, could you please explain why file sizes would affect the usage of minhash or simply provide some supplemental materials for me to learn? I thought that the program would always read the data from all the files then process them together as ONE dataset, and the only problem of too big file size might be the slow reading speed.

I retry the 3 types of multiprocessing start methods this morning, and I find that when I use spawn as the start method, finally I can see TWO processes running together. But it is still weird enough, because:

I manually specify n_proc=120 but there are ONLY 2 processes working. Yet there are still other processes just like the screenshot above, which never run and consume exactly the same VIRTUAL memory as the running one.
The TWO running processes do not double the efficiency; actually both of them have only ~50% cpu loads, so in total they exactly work at almost the same speed or even slower than the fork one (one process running but 100% load), as told by the iter speed (~30000 examples/s) in tqdm.
I further sort the related processes by the TIME+, and I find that the running processes are actually the earliest one(main process, or the one exactly runs in fork) and the latest one(has shortest TIME+ and biggest PID).

I don't know if the information could help. Feel free to inform me if any further information is needed.

Hey, I debug the code and finally find the cause. I modified the ngram part in embed_func but I really forgot about it. I modify the ngram code for it to properly adjust to multiprocessing. Now the program runs very well. Sorry to bother you and really thankful for all the things you've done!
Still I would like to enquire you about the file size problems. I still cannot find out the exact influences of too big file sizes. Anyway I appreciate for your time and patience.

No worries. Glad you fixed the problem. Here is my reasoning behind the big size, assuming it is natural language text data: minhash is a way of approximating jaccard similarities with sets of ngrams. If two files are sufficient large, their ngram/vocabulary differences will be very small to be discernible by the approximation. If you are comparing DNA sequences with a bigram, any type of DNA sequence will likely be clustered as similar, in which case you need to have a relatively large ngram size. If you are trying to deduplicate different datasets, there is another way to do it.

Thanks for replying! I may apologize for the ambiguity. I actually include many pieces of jsonlines within one .json file. For example, one json file may contain 10k pieces of texts, each of which is approximately hundreds of chars length. In such cases I think minhash is actually excellently suitable, in accordance with the benchmark you've given.

That makes a lot of sense then.
And for the fork settings you asked, originally it is used for reduce memory usage in macOS. Fork in this case means that the shared object uf is not copied to child processes as long as it is not modified (it is like a read reference). And the surrounding code that stops the garbage collector is also used for this reason. This behaviour is also dependent on the system implementation, e.g. Windows might behave very differently in this case. Additionally, memory usage reported using top or htop might not reflect the real usage.
In short, fork should work reasonably well on macOS and Unix systems, but feel free to test different settings if they work better on your system.

Closing it for now. Feel free to open another one if you have more issues.


MLP example doc does not include custom mnist module

The MLP example documentation makes use of a custom MNIST loader (https://github.com/ml-explore/mlx-examples/blob/main/mnist/mnist.py), and does not run as written. This is not obvious from following along with the doc. It is particularly confusing because there is a PyPI package of the same name.

Thanks for the input, will update it.

Addressed in #198


implement refresh_tokens

should be very easy if we base it on an authorization, i am not sure if we need to have an expiry for a refresh token though... maybe 1Y?

should be very easy if we base it on an authorization, i am not sure if we need to have an expiry for a refresh token though... maybe 1Y?

done.


Add push to Docker hub with TAG on image

When we are about to build the memgraph-mage:latest?

When we are about to build the memgraph-mage:latest?

This was problem, now it is fixed.


Warnings for unused bindings not appearing

Steps to reproduce:

Create queue trigger function
Go to integrate tab, add blob output and save
Go back to develop tab.
Drop message in queue and see log update with function execution (this step should be optional I think)

Expected result: Logs show warning indicating that output binding argument is missing
Actual result: No warning appears in logs.

What language are you using? In the case of C#, I could see us logging an error if you haven't included the binding in your function signature.
For all the other loosely typed languages, we don't really know if you've used it in your code. Since actually writing to the output binding isn't required (e.g. scenarios where you only conditionally produce output), we can't use that as an indicator.

Yes this was C#

This should work. If it repros, we should fix it.


can not start this project with webstorm IDE

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

download the source cirrocumulus-main.zip

unzip the package and load into the webstorm IDE

install the dependent with command "install i"

npm start the project

Expected behavior
I want to download the development src code into webstorm to run the single cell data,
but  it get error. lying below picture
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]   windows11
Browser [e.g. chrome, safari] chrome 113.0.5672.129
Version [e.g. 22] latest

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

You also need to start the cirrocumulus server using either the launch or serve command. Please see https://cirrocumulus.readthedocs.io/en/latest/installation.html#developer-instructions


Addressing linux-related issues and simplifying instructions display

As per #256 , there were some permissions related issues when attempting to run the software on Linux.
Using the excellently detailed issue write up by @benizar as a starting point, I developed the changes in this commit that i have tested now without issue running on Windows and Ubuntu computers (with details to be posted below).
Changes:

instead of display instructions from echo commands within instructions.sh (which causes permissions issues as an executable file in some Linux environments), these same instructions are instead stored in an environment variable and echo-ed in the start up command
UID and GID are defined as empty string environment variables and set to define a user in the docker compose file, which (suprisingly to me) avoids permissions issues while avoiding warning that the UID and GID were not set (occurs if they aren't already set as blank; Docker does this otherwise and sets them)
to do the above, there is a new .env file in the project root folder containing the new environment variables: INSTRUCTIONS, UID and GID
updated readme.md in process folder to advise that Linux users may need to run 'bash ./global-indicators.sh' prefixed by 'sudo' (as per Docker advice on Linux post-installation)

I have successfully run the above on the following two systems without issue/warning/error to configure, analyse and generate resources for the provided example city of Las Palmas:

Ubuntu 20.04.4 LTS

Processor: Intel Xeon W-2145 CPU @ 3.70GHz x 16
RAM: 62.5GiB
Analysis step took about 0.5 minute (according to time print out from analysis)

Windows 11 Pro for Workstations 10.0.22624

Processor: Intel(R) Xeon(R) E-2236 CPU @ 3.40GHz, 3408 Mhz, 6 Core(s), 12 Logical Processor(s)
RAM: 32 GB
Analysis step took about 1 minute (according to time print out from analysis)
I also repeated the analysis using Windows Subsystem for Linux on the same computer (didn't require sudo)

It would be great if some MacOS users could test this out; thanks!

ps.  I've added a few people as reviewers to cast the net widely as I know everyone is busy --- hopefully someone can look at this sooner than later and set it to run to confirm it works on

someone else's MacOS computer (priority; not tested)
someone else's Ubuntu computer
someone else's Windows computer

I think once we have confirmed this works on each of these platforms when not run by me, it should be good to merge (ie. don't need to have all reviewers review - but if people can, so we can get this sorted sooner than later, I'd appreciate it!).
Perhaps we could consider having this as a merge rule in future (run on 3x platforms, by persons other than the pull requester).

Perhaps we could consider having this as a merge rule in future (run on 3x platforms, by persons other than the pull requester).

We really should just set up a simple CI for any PRs or pushes to main to run the script on windows, mac, linux on github actions.

That was a fast!!
I just cloned and tested the linux branch and everything worked perfectly with the example data (approximately took 3.1 minutes, but I think it was faster than that). I'm in a different Ubuntu 22.04, with the latest docker installation.
The .envfile is much more elegant that the fast-and-dirty patch I proposed XD
I'm afraid I don't have access to a Win or iOS computer but I think that these commits should not affect any other system. On Monday, we will test it on windows just to be sure.

We really should just set up a simple CI for any PRs or pushes to main to run the script on windows, mac, linux on github actions.

@carlhiggs here's a simple example to copy from if you agree: https://github.com/gboeing/osmnx/blob/main/.github/workflows/tests.yml

@gboeing sounds like an ideal solution; thank you for sharing the example workflow!  I could try to look at this later today or tomorrow, but happy if you want/had time to set that up.  Thanks all!

We really should just set up a simple CI for any PRs or pushes to main to run the script on windows, mac, linux on github actions.

@carlhiggs here's a simple example to copy from if you agree: https://github.com/gboeing/osmnx/blob/main/.github/workflows/tests.yml

I just had a look at the example, and thinking about our use case with the way things currently work (software to be run/tested using a docker compose environment with ghsci and postgis containers), I'm not immediately sure how/if it's possible to do this exactly using the GitHub actions approach.  Any thoughts/experience?
Bundling our software within the docker image itself would be another approach (e.g. like you do with the osmnx image), but that's a big change (not necessarily bad), requiring  re-thinking the whole architecture and workflow for users. (not sure that I'm up for that big a rewrite at this point... Perhaps something to think about for down the track)
Maybe I'm just not familiar with with the way things would work too see the obvious way to do this.
Any thoughts appreciated!

Maybe this StackExchange post describes what I think we would want to do for using a Docker compose file to define environment: https://stackoverflow.com/a/64373702/4636357  ... perhaps this could work combining your example with this approach...

I think perhaps --- given we have tested this on Linux and Windows, and have no reason to expect issues on MacOS (previously worked there), on this occasion I will merge this, but created a new issue to capture the great idea to move more towards having a CI/CD workflow


Pipeline mode support

I found that torch2.4 officially supports Pipeline Parallelism, the package name is torch.distributed.pipelining。 It was migrated from the PiPPy project.
Will exllamav2 add support for Pipeline Parallelism sometime?

Possibly, but not before fully exploring tensor parallelism which potentially makes PP totally redundant. I.e. why run two staggered batches at 1x speed when you can run a single batch at 2x speed?

TP have high communication requirements, so the performance improvement for PCIe devices might not be as significant as expected. On the other hand, PP has lower communication requirements, theoretically leading to a larger performance boost.
However, PP doesn't seem to be very useful for single-batch inference. 😂😂😂


Doughnut chart reponsiveness with bootstrap

Hi @gevgeny ,
Can you please help with snippet to render doughnut chart reponsiveness with bootstrap class 'col-md-4'.
Thanks
Ashutosh

Please you stackoverflow for questions


Update the wording of the bulk_execution_affinity properties in P1436

Rename bulk_execution_affinity_t::scatter_t to bulk_execution_affinity_t::spread_t.
Rename bulk_execution_affinity_t::compact_t to bulk_execution_affinity_t::close_t.
Refine the wording of the bulk_execution_affinity_t properties to clarify the requirements on binding and chunking, based on feedback from SG1.

@mhoemmen thanks for the feedback.
I have updated the MR to apply the suggested changes, and also added some additional wording to describe the terms of art used in the definition of the bulk_execution_affinity_t properties and a note to state that we intend to propose a follow-up property that specifies how larger iteration spaces are subdivided into chunks.

@AerialMantis Thanks! :-D  Sorry I've been quiet lately -- I've been working on P1673 pretty hard :-) .


[discussion] Add option to collaborate

It would be great to be able to work on the same board with multiple users. It's more an idea placeholder for now, rather than actual task. Would it be inline with Excalidraw development plan? :)

Agreed, that would be great!

JSFiddle uses TogetherJS.

@j-f1 interesting project. I tried it on jsfiddle and it sometimes works, sometimes not, and that's just textareas, so I'd be surprised if it worked in our project.
It also seems pretty dead:

How can we restart / expand TogetherJS development?

Done!!


Setup Postgres Backups

Need to figure out how to grant appropriate permissions in Postgres or modify my backup script to do daily Postgres backups to S3.

This is all set. Fixed the permissions and added to the crontab!


Support set global policies in xmake.lua

Apis

[x] set_policy

v2.3.4

[x] set_policy("check.auto_ignore_flags", false), @see https://github.com/xmake-io/xmake/issues/782
[x] set_policy("check.auto_map_flags", false)
[x] set_policy("build.across_targets_in_parallel", false)

I have finished it and we can run xmake l core.project.policy.policies to show all builtin policies.
$ xmake l core.project.policy.policies
{
  "check.auto_map_flags" = {
    type = "boolean",
    description = "Enable map gcc flags to the current compiler and linker automatically.",
    default = true
  },
  "build.across_targets_in_parallel" = {
    type = "boolean",
    description = "Enable compile the source files for each target in parallel.",
    default = true
  },
  "check.auto_ignore_flags" = {
    type = "boolean",
    description = "Enable check and ignore unsupported flags automatically.",
    default = true
  }
}


bug: Effects Context Menu missing

The effects context menu is not appearing when right-clicking on effects.
Reference
https://discord.com/channels/732325252788387980/1116078321067892796/1219639830594457671

Hey @kgar did you remove context menu from effects tab, or is a bug? I'm using v11.315, with 5e 3.0.4 in a world only with tidy and context menu don't work only in effects tab.

Closing as completed.


use latest collection date as in configs as default

Description
Used the latest_collection_date stored in the config file as the default date used in public_discourse_getter() function
Fixes #48
Instructions for Reviewer
In order to test the code in this PR you need to ...
Please pay special attention to ...
Checklist:

[ ] I have refactored my code out from notebooks/
[ ] I have checked the code runs
[ ] I have tested the code
[ ] I have run pre-commit and addressed any issues not automatically fixed
[ ] I have merged any new changes from dev
[ ] I have documented the code

[ ] Major functions have docstrings
[ ] Appropriate information has been added to READMEs

[ ] I have explained this PR above
[ ] I have requested a code review

no review needed, small change


how to upgrade to business edition...

i have a code but system say error...
any suggestion?

failed to deploy upgrade stack: Network portainer-upgrade-1689353906-2-18-4_default Creating Network portainer-upgrade-1689353906-2-18-4_default Created Container portainer-upgrade-1689353906-2-18-4-updater-1 Creating Container portainer-upgrade-1689353906-2-18-4-updater-1 Created portainer-upgrade-1689353906-2-18-4-updater-1 | {"level":"info","image":"portainer/portainer-ee:2.18.4","time":1689353908,"caller":"/home/runner/work/portainer-updater/portainer-updater/portainer/portainer.go:76","message":"Updating Portainer on standalone environment"} portainer-upgrade-1689353906-2-18-4-updater-1 | {"level":"info","containerId":"09603e5f79d38ab2ef21a7f329e96e1497b4fec69cddcd66171f103e42ee4e9d","image":"portainer/portainer-ee:2.18.4","time":1689353908,"caller":"/home/runner/work/portainer-updater/portainer-updater/dockerstandalone/update.go:28","message":"Starting update process"} Aborting on container exit... Container portainer-upgrade-1689353906-2-18-4-updater-1 Stopping Container portainer-upgrade-1689353906-2-18-4-updater-1-update Stopping Container portainer-upgrade-1689353906-2-18-4-updater-1-update Stopped Container portainer-upgrade-1689353906-2-18-4-updater-1 Stopped
and after:
upgrade failed: server should have been restarted by the updater

You can't update. Add-ons' code are not modifiable by the users, only by the add-on's developer. They are not general docker containers. Please read more about add-ons in HA documentation.


Bug: On-Demand video treated as live on Safari with Media Chrome

Is there an existing issue for this?

[X] I have searched the existing issues

Which Mux Elements/Packages does this apply to? Select all that apply
mux-video
Which browsers are you using?
Safari
Which operating systems are you using?
macOS
Description
When using <mux-video> in conjunction with Media Chrome, since version v0.16.5, when you try to play a HLS (.m3u8) video in macOS/iOS Safari, it treats it as mediastreamtype="live" on the Media Chrome <media-controller>. This causes the video to start from the beginning on every play/pause, and other weird behavior:
https://github.com/muxinc/elements/assets/63538910/517172bf-96f6-4369-9c71-dddffb049f7d

versions <0.16.5 (0.16.4) and non-hsl streams work fine
https://www.media-chrome.org/docs/en/stream-type
Reduced test case
https://codesandbox.io/p/devbox/media-stream-type-qmjqmn?file=%2Fsrc%2FApp.vue
Steps to reproduce
reproduction sandbox is provided; remember to use Safari where this issue is present
Current Behavior
<media-controller> is mediastreamtype="live", which is incorrect
Expected Behavior
<media-controller> is mediastreamtype="on-demand"
Errors
No response
What version of the package are you using?
^0.16.5

Thanks @jakubslambor for the code sandbox, that's very helpful.
I noticed with <mux-video> you're using the src="https://test-streams.mux.dev/x36xhzz/x36xhzz.m3u8" ...
Although that should work, the API for using <mux-video> is with videos hosted by Mux Video, and you initialize it with playback-id for example <mux-video playback-id="fujsyrk0014ec2lpnm11bzc2maysaqpqa"></mux-video>. Setting arbitrary src= is not something we test or guarantee support for with <mux-video>.
(Incidentally, the URL you used is hosted by mux.dev, but that's a test stream that we happen to host for Hls.js demo purposes, and it isn't hosted by our "normal" Mux Video hosting product.)
This is still probably a bug that we should look into, but to get unblocked I think you have 2 options here:

If your video is a Mux Asset, use <mux-video playback-id="{PLAYBACK_ID}">
If your video is not a Mux Asset use <video> or use <hls-video>: https://github.com/muxinc/hls-video-element

Would either of those be possible for you to try?

@dylanjha Oh cool, I didn't know about hls-video-element, I'll give that a shot! Thanks 🙏


INSTALL.md: Add the missing bpftrace build dependency libs/pkgs for Ubuntu

Following CMake errors are seen when compiling bpftrace on Ubuntu 20.04,   so add the missing libraries libgtest-dev and libgmock-dev to the list of libs/pkgs that needs to be installed.
CMake Error at /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146 (message):
  Could NOT find GTest (missing: GTEST_LIBRARY GTEST_INCLUDE_DIR
  GTEST_MAIN_LIBRARY)

CMake Error at /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146 (message):
  Could NOT find GMock (missing: GMOCK_LIBRARY GMOCK_INCLUDE_DIR
  GMOCK_MAIN_LIBRARY)

Following build error is seen when finishing the compilation of bpftrace, so add the missing asciidoctor pkg to the list of libs/pkgs that needs to be installed.
[100%] Generating bpftrace.8
/bin/sh: 1: ASCIIDOCTOR-NOTFOUND: not found
make[2]: *** [man/adoc/CMakeFiles/adoc_man.dir/build.make:65: man/adoc/bpftrace.8] Error 127
make[2]: *** Deleting file 'man/adoc/bpftrace.8'
make[1]: *** [CMakeFiles/Makefile2:2392: man/adoc/CMakeFiles/adoc_man.dir/all] Error 2
make: *** [Makefile:141: all] Error 2

Checklist

[ ] Language changes are updated in man/adoc/bpftrace.adoc and if needed in docs/reference_guide.md
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

Thanks!


Paradrop Reinforcements

Add an option to reinforcements module to allow paradropping rather than moving in on foot.

Maybe soon, we'll see.


cannot get ip from libvirt bridge

I try to test the default bridge virbr0 of libvirt. It functions ok the VMs can get the dhcp ips. But if I try to use this tool, it said did not receive offer or lease.
dora.py -a 52:54:00:b1:55:66 -s <IP_ADDRESS> -d
2020-10-20 22:25:53,194 - MainThread - DEBUG: Attempt 9 - No sockets available to read from... sleeping for 10 ms
2020-10-20 22:25:53,204 - MainThread - DEBUG: Did not receive offer, retrying
2020-10-20 22:25:53,204 - MainThread - DEBUG: Sleeping 100 ms then retrying discover
...
2020-10-20 22:31:12,268 - MainThread - DEBUG: Attempt 11 - Sending discover packet to <IP_ADDRESS> with tx_id=2227595823
2020-10-20 22:31:12,268 - MainThread - DEBUG: Select: ([<socket.socket fd=4, family=AddressFamily.AF_INET, type=SocketKind.SOCK_DGRAM, proto=0, laddr=('<IP_ADDRESS>', 67)>], [<socket.socket fd=5, family=AddressFamily.AF_INET, type=SocketKind.SOCK_DGRAM, proto=0, laddr=('<IP_ADDRESS>', 68)>], [])
2020-10-20 22:31:12,268 - MainThread - DEBUG: Connecting to <IP_ADDRESS>:67
...
2020-10-20 22:31:12,268 - MainThread - DEBUG: Packet Sent
2020-10-20 22:31:12,269 - MainThread - DEBUG: Unable to obtain offer run client with -d for debug info
2020-10-20 22:31:12,269 - MainThread - ERROR: Unable to obtain lease, run with -d to debug

The dnsmasq conf is:
bind-dynamic
interface=virbr0
dhcp-range=<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>
dhcp-no-override
dhcp-authoritative
dhcp-lease-max=253

any idea, how to make it work?

hm... it seems do not work at all. I also try to obtain ip from actual router, I get same problem cannot get and offer or lease.
dora.py -s <IP_ADDRESS> -a  52:54:00:b1:55:66 -p 688

Hey @braindevices been a while since I've looked at this code, could you try running dora.py -a 52:54:00:b1:55:66 -@ <IP_ADDRESS> -d and letting me know what the output is?

Alternatively if you specify an interface with -i and remove the -s option it should try to obtain a lease over broadcast on that interface.


Add explicit test for HABTM associations

When reading #572, I realised no explicit test for this case was present, although the system is supposed to work. This test proves the compatibility of cancancan with HABTM associations.

Hi i asked the opened the issue #572, Does it mean that HABTM association is not already part of this gem? @coorasse


BASE_URL for non-root URL paths

Version and OS
v0.2.9, macOS
It would be great if BASE_URL worked to run the app on alternative paths. For example, https://example.com/adr/. Right now, every link on the website is relative to /, but when running multiple web apps on a single machine, they can't all run on the main path. I use a nginx reverse proxy to separate the apps, which works somewhat for ADRFinder, but breaks all links when clicked and css / images.

Check the instructions for ChangeDetection (from which this is based) for non-root URL in nginx.
https://github.com/dgtlmoon/changedetection.io/wiki/Running-changedetection.io-behind-a-reverse-proxy
Haven't tried it myself so if that doesn't work let me know.

Hm so I have this in my nginx config
location /adr/ {
	proxy_pass http://localhost:5500/;
        proxy_set_header Host "localhost";
        proxy_set_header X-Forwarded-Prefix /adr;
        proxy_set_header X-Forwarded-Proto $scheme;
}

but it's still linking to root URL paths, not /adr paths.

My bad. Needed to add USE_X_SETTINGS=1 to the docker-compose.
version: '2'
services:
    adrfinder:
      image: ghcr.io/chuckmac/adrfinder
      container_name: adrfinder
      hostname: adrfinder
      volumes:
        - ./datastore:/datastore
      ports:
        - 5500:5500
      restart: unless-stopped
      environment:
        - USE_X_SETTINGS=1


Font download task

This downloads the font files during pre-pop.

Both the  import_font function and the PDF config setting do already exist - you just redefine them, but for what purpose? If your actual intention was to improve them, you should modify the existing functions rather than redefining them.
However, I don't quite see what you wanted to improve - what effectively does it do differently, and why?

Hi nursix,
I was working on this during GCI, here is all the previous talk we did - https://github.com/edengci/eden/pull/80

That doesn't answer my question: the functionality already exists in trunk, and your PR just duplicates the existing functions, which is clearly wrong.
If you intended to improve it, then what exactly did you try to improve? If you can explain it, then I may be able to guide you to it, otherwise I'm afraid this PR is not acceptable.

Existing functionality is here:
https://github.com/flavour/eden/blob/master/modules/s3/s3import.py#L4084
So, I don't see the point with this pull request - especially since you're not modifying the existing functionality, but adding duplicates in the same places.
I've given a number of pointers for possible improvements, so if you wanted to action any of those and then re-submit your PR maybe it would make start to make sense.
But right now it doesn't really, sorry.

Hmm, I didn't knew someone implemented it already, must have been between the time period of GCI and now, since it's already their I am closing the PR.


Adds support for VS's telemetry reporting system.

Adds support for VS's telemetry reporting system.
Count numbers of environments, v2 vs v3, completion DB analysis time, and installed packages.
Ensures private data is protected before reporting.
All data that is collected at any point appears in Tools, Python Tools, Diagnostic Info. Users can opt-out from sending usage information under Help, Send Feedback, Settings.
When reviewing, please check that I've protected all telemetry-only code with exception handlers. It is totally unacceptable to crash the product at any point because of telemetry issues.

The rest LGTM.


float matchers

Add float matchers. Same implementation/API as jest/the internal FB jest wrapper

Targeting not-matcher-hint branch for snapshot compatibility

Cool, @bryphe was saying the only thing holding back using Rely on Revery was this feature!


Make  confirmText and cancelText a widgets

Text is the widget also but having a widget as param will add more flexibility

It doesn't make sense changing the type of the variable and not the name as now it is a thing completely different

I've got your point but this isn't a version great enough to be called a big release neither a code change that justify being classified as a version with "incompatible" API changes

The semversion is a little bit about different.

Yes, it is:
Item 4:
Major version zero (0.y.z) is for initial development. Anything MAY change at any time. The public API SHOULD NOT be considered stable.
I'm following it

I won't change to stable major 1.0.0 until rxdart is removed from the code (thinking about if the tests are fundamental to this one too)
If you can, please help me implementing the tests


Remove old kotlin docs link

Kotlin docs are converted and moved to kotlin-support repo now.

Note: the CircleCI tests will fail at the moment. There's an issue with payment and so the large machine is not allocated. We'll just need to ignore those tests for now.


Plain-text passwords

It's a big security risk storing passwords in plain text. In fact, my workplace forbids it, and rightly so.
Is it possible already to write a configuration which prompts for passwords? If not, I think this is an important feature which needs to be included.
The first step might be to use Get-Credential to prompt for missing passwords, or on some configuration flag, e.g. "PromptForPassword": true.
I'm led to believe that PSCredential objects will compile as into plain text in .MOF files unless there's a certificate on the target machine which the local machine has the public key for. So, the second step might be automating/documenting that process.
Happy to help contribute where I can.

I agree!!
For now, we have been scrubbing our .json files after execution.  Our passwords have to be updated/changed every 60 days, so leaving them behind in the .json files brings little value to us due to our organization policies.  Then, we have to manually add them back in before we complete an upgrade (and scrub again after complete).  And... I really don't like the idea of leaving behind clear text passwords in any files after the execution is complete.
We even dabbled with adding a pre/post-processing steps by:

trying to read in the .json file
$config = Get-Content -Raw -Path myfile.json | ConvertFrom-JSON

fetch passwords from our password management system and plug those values into the password fields.  EX:

$psa='mycoolpassword'
$config.ConfigData.Credentials.PrimarySiteAdmin.Password = $psa

write a new temporary .json file with the passwords embedded
$config | ConvertTo-Json -depth 100 | Out-File "temp.json"

then... run the Configure-ArcGIS command using the temp.json file.
Configure-ArcGIS temp.json InstallLicenseConfigure

Then... remove the temp.json file
rm temp.json

Although, we ran into some issues with the powershell ConvertTo-Json encoding certain characters and gave up.  We were not interested in adding another solution to resolve this issue (like python).  And... it seems that powershell added a new EscapeHandling parameter that might help with this - https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.utility/convertfrom-json?view=powershell-7#parameters
However... we are still on Powershell v5 (and this was introduced at 6.2).  Something to look into again
… back to topic...
I'm interested in a more secure capability to better protect these credentials.  Ideally something that is compatible with other 3rd party password management solutions.
Prompting for a password (and cleaning up any temporary files on the back-end that logged the credentials) is 1 big step forward, but may not be fully compatible with other 3rd party solutions.
I realize we cannot support all (or ANY) 3rd party password management solutions, but maybe having the option for generic integration would be best.  Something like:
a) PSDSC reading from an environment variable
This would be another good step forward - if the environment variable can be set (outside of the PSDSC execution), and PSDSC can read it, then we are no longer 'storing clear text passwords in a file', but we are still storing them in a session environment variable. I'm assuming that closing down the powershell session would purge all this information?
b) taking passwords in as command line arguments to the PSDSC execution
this could be a little hairy with the amount of credentials that may be needed.
c) Still taking a clear text password in the .json file, but then encrypting that content after execution This is similar to how some of the REST API functions work.
See WEBGISDR - https://enterprise.arcgis.com/en/portal/latest/administer/windows/create-web-gis-backup.htm

PORTAL_ADMIN_PASSWORD =
Specify the password of the portal administrator account.
PORTAL_ADMIN_PASSWORD_ENCRYPTED = <true | false>
Set this option to false the first time you populate the file with your administrator password. When you save the file, your password is encrypted and the value for PORTAL_ADMIN_PASSWORD_ENCRYPTED is set to true to indicate the password has been encrypted. If you need to change the password in the future, set PORTAL_ADMIN_PASSWORD_ENCRYPTED = false, provide your new administrator password, and save the file.

And Server/Portal identity store configurations
REF1 - https://developers.arcgis.com/rest/enterprise-administration/server/updateidentitystore.htm
REF2 - https://developers.arcgis.com/rest/enterprise-administration/server/userstore.htm
d) Other options not laid out here?!?!
Thanks for the consideration

Hello @tristanm and @pfoppe,
We now support the use of Encrypted Password Files and Encrypted MOF Files in the latest release, v3.0.0.
V3. Password Files and Encrypted Deployments:
https://github.com/Esri/arcgis-powershell-dsc/wiki/V3.-Password-Files-and-Encrypted-Deployments

@cameronkroeker, this is fantastic news!
For the curious, it would seem that the string generated using Read-Host -AsSecureString | ConvertFrom-SecureString is only decryptable by the same user on the same machine, which is pleasing.
It would be good to also document when the process for generating a certificate for encrypting MOFs is actually necessary. In PS >= 5.0 with DSC in push mode it would appear is it not. Looking at a MOF file on a target node, it is indeed encrypted. That said, I don't know how Invoke-ArcGISConfiguration behaves in v3.1.0 with encrypting MOFs on the local machine when PasswordFilePath is being used. Anyone able to confirm this behaviour?

@cameronkroeker, this is fantastic news!
For the curious, it would seem that the string generated using Read-Host -AsSecureString | ConvertFrom-SecureString is only decryptable by the same user on the same machine, which is pleasing.
It would be good to also document when the process for generating a certificate for encrypting MOFs is actually necessary. In PS >= 5.0 with DSC in push mode it would appear is it not. Looking at a MOF file on a target node, it is indeed encrypted. That said, I don't know how Invoke-ArcGISConfiguration behaves in v3.1.0 with encrypting MOFs on the local machine when PasswordFilePath is being used. Anyone able to confirm this behaviour?

Hi @tristanm ,
The password file path is encrypted independently of the MOF files, as the Invoke-ArcGISConfiguration won't encrypt the password file. Once you have generated the encrypted password file, you can provide the path to password file which is local to the target machine. I will look into clarifying this within the wiki page.
Thanks,
Cameron K.

Hi @cameronkroeker, sorry, I should have been clearer with my comment (it was approaching midnight at the time). I've answered by own question since then:
What I meant to say was, I understood from this doc that MOFs were always encrypted when they were sent to the target node, if they were sent in push mode (as opposed to from a pull server), and I had verified this. Given this fact, I thought that the process of setting up certificates is perhaps not necessary for all situations where password encryption is required, but I wasn't sure if MOFs on the authoring node were encrypted because they get deleted before they can be inspected.
Since then, I've tested this for myself by preventing the authoring node's MOFs from being deleted. I can confirm that neither the MOFs nor the passwords within them are encrypted if a certificate is not used, so yes, certificates are very much required.
Additionally, MOFs on target nodes are themselves encrypted, not just the passwords within. This is not the case with MOFs on the authoring node where just the passwords are encrypted.
All sorted now 👍


Edit button style for template cards

Currently, the buttons in the cards for our software templates don't match the buttons that we have elsewhere. It would amazing if someone could fix the styling to match the tertiary button style that you see in TechDocs and Explore.
Currently

Ideally

I can solve it

@rcoder23 Awesome 😎

Sorry ,sir I am removing my assignment .I am beginner now Thank you

@rcoder23 no problem, and good luck! :grin:

This issue seems to have already been fixed in this pull request: #1677
😊

@ellieseastream oh yeh, thanks! :grin:


Update Kubernetes dashboard URL

The Kubernetes dashboard URL returns 404:
https://github.com/aws-samples/eks-workshop/blame/master/content/dashboard/dashboard.md#L12
This is the new link:
https://raw.githubusercontent.com/kubernetes/dashboard/master/aio/deploy/recommended/kubernetes-dashboard.yaml

addressed in #218


Support entering/exiting alternate screen

To start with I think an entry-point boolean is enough. Things like the snake game can make use of it.
Still can't really think of a good design for dynamically switching in and out of the mode, though. It has all kinds of other implications on things like static logging, what happens to nodes that are "outside" the alternate screen, how to handle multiple and/or nesting, etc.


[Bug] This issue occurs when installing and using on Linux (CentOS and Ubuntu) according to the documentation.

🐛 Bug
I tried to deploy according to the documentation, but I always get an error. I don’t know how to resolve this error. The error message is as follows:
terminate called after throwing an instance of 'tvm::runtime::InternalError'
  what():  [07:03:10] /usr/share/miniconda/envs/mlc-llm-build/conda-bld/mlc-chat-cli-nightly-package_1692162405160/work/3rdparty/tvm/src/runtime/vulkan/vulkan_instance.cc:111: InternalError: Check failed: (__e == VK_SUCCESS) is false: Vulkan Error, code=-9: VK_ERROR_INCOMPATIBLE_DRIVER
Stack trace:
  [bt] (0) /usr/local/miniconda3/envs/mlc-chat-venv/bin/../lib/libtvm_runtime.so(tvm::runtime::Backtrace[abi:cxx11]()+0x27) [0x7efcd7cbed27]
  [bt] (1) /usr/local/miniconda3/envs/mlc-chat-venv/bin/../lib/libtvm_runtime.so(+0x680a8) [0x7efcd7c820a8]
  [bt] (2) /usr/local/miniconda3/envs/mlc-chat-venv/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanInstance::VulkanInstance()+0x15a6) [0x7efcd7db6626]
  [bt] (3) /usr/local/miniconda3/envs/mlc-chat-venv/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanDeviceAPI::VulkanDeviceAPI()+0x46) [0x7efcd7db2f36]
  [bt] (4) /usr/local/miniconda3/envs/mlc-chat-venv/bin/../lib/libtvm_runtime.so(tvm::runtime::vulkan::VulkanDeviceAPI::Global()+0x4c) [0x7efcd7db32cc]
  [bt] (5) /usr/local/miniconda3/envs/mlc-chat-venv/bin/../lib/libtvm_runtime.so(+0x19930d) [0x7efcd7db330d]
  [bt] (6) /usr/local/miniconda3/envs/mlc-chat-venv/bin/../lib/libtvm_runtime.so(+0x70c1c) [0x7efcd7c8ac1c]
  [bt] (7) /usr/local/miniconda3/envs/mlc-chat-venv/bin/../lib/libtvm_runtime.so(tvm::runtime::DeviceAPI::Get(DLDevice, bool)+0x299) [0x7efcd7c8c369]
  [bt] (8) mlc_chat_cli(+0xfb98) [0x560e90f12b98]

已放弃 (核心已转储)

VK_ERROR_INCOMPATIBLE_DRIVER

Looks like an issue of the Vulkan driver

The platform I am using is a virtual machine platform, which does not have GPU support. Is it because of the lack of GPU support?

All models of MLC-LLM are running with GPU (together with the quantization step), so a GPU is needed

@Hzfengsy 感谢大佬的指点，既然宣称能在浏览器和Iphone上运行，虚拟机上都运行不了，而且我一直没搞懂怎么在浏览器上运行的，


Phenopackify disease

The disease class was still missing a number of parameters from Phenopacketys v2 or represented them slightly different. This PR:

renames ageOfOnset to onset, since it could be another time element
adds resolution
adds excluded flag option (CAVE: should we add "default": false?)
adds clinicalTnmFinding
adds primarySite
adds laterality

The additional parameters which are not part of PXF Disease are left in place.

Is this a request from the PRC?

@jrambla Not anymore, though in the general remit of PXF alignment & there was the age… issue still open. However, while working on a Phenopackets … thing w/ Peter I realized that there will be some complex remapping, which can be avoided by just fixing this toward phenopackets style. And additional parameters were kept.

Also, clear problems like ageOfOnset being a TimeElement which now isn’t necessarily a kind of age .. being addressed.

@mbaudis I think the reason why we didn´t use timestamps in the Beacon model was because the reference birthdate timestamp is rarely collected in datasets other than EHR of course and we thought rarely shared as it is very identifiable. And at least at the beginning of v2 everything was instead referenced to age, as AgeAt elements in ISO duration format. Of course other than the problem of including birthdate, one way or the other will do other to have the cross-reference among all kinds of events: Exposures, Treatments, Medications ..>> Diseases, PhenotypicFeatures(symptoms, etc), but changing only a couple of elements Medications and Disease to timestamps won´t really do in my opinion. I guess your concern with having everything is as AgeAt in Beacon is the mapping from Beacon ageAt to Phenopackets timestamps won´t be possible?

@clauw87 You are exactly right in that the original timestamps etc. use in PXF wasn't sufficient, especially for cancer use cases. But now one can have any of the time elements represented as any format defined in TimeElement, depending on the use case; this can be a timepoint (timestamp), an age (age -> iso8601duration), a time interval for followup (also iso8601duration), a concept or even a fuzzy age range. This has potentially other problems (e.g. you are not forced to use the most practical format...) but it is "enabling".
There are always other ways to do it (and I have argued for some of those), but it seems to work when modeling the current way PXF is doing it... This is all about GA4GH standardization, where it doesn't degrade the system.

For 2.0.1(?)

Closing this here, to re-do it directly in the unified repo.


PropertyAccessMode enhancement

Could PropertyAccessMode have an additional option, possibly called IgnoreDuringConstruction, which would tell EF to create the property, use it's public getter when storing to the database for query reasons, but when instantiating from the database, just ignore it.  The use case for this would be having getter-only properties computed from other properties, which can be easily queried at the database with Linq, without the need to do this:
private int _myProperty; public int MyProperty { get { return Number1 + Number2; } private set { } }
The above works fine, but adds junk to the object that serves no useful purpose other than to trick EF into not doing anything to set the property.

@jdcrutchley Just want to clarify something. You said, "trick EF into not doing anything to set the property." I don't see anything in that property definition that will prevent EF from setting the property. Can you elaborate more?
That being said, the idea of computed properties that are persisted for query is interesting. We will discuss. 😸

I probably worded that badly.  My understanding was, it'll find the field _myProperty, which isn't actually used for anything, and set that.  I have the private set only so that EF will pick up the property in the first place without having to do anything special in the mapping.  However, now that I think about it more, I probably don't need the field since a do-nothing setter serves the same purpose. I'll try that out.
Maybe just close this, since the only way I can think of to make this nicer is to have EF automatically make a property for any public getters it finds, and if there's no setter just ignore it during construction.  My perfect scenario would be to have a public get-only method, EF picks that up as a property and stores the value in the database so it can be easily queried, and just ignores it when reconstructing the objects.  That may mess up other expected behaviors for other folks though.

We discussed this in triage and decided that as this time having no field and an empty setter is a reasonable solution for his mapping, which is not something commonly asked for.

Tried this in 3.1.1 and when adding a new entity to the DbContext, I got the exception "Unable to track an entity of type 'MyEntity.MyProperty#MyType' because primary key property 'KeyField1' is null." I should mention MyProperty is the calculated field and it is an Owned type and I got a composite key (KeyField1, KeyField2) on MyEntity. As soon as I make MyProperty a regular { get; set; } one, the problem goes away...

@daniel-jann Please open a new issue and post a small, runnable project or complete code listing that reproduces the behavior you are seeing so that we can investigate.

@daniel-jann Please open a new issue and post a small, runnable project or complete code listing that reproduces the behavior you are seeing so that we can investigate.

I did so in #19791


feat(docs): add right nav

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

@JeremyWuuuuu
Can I copy your code and change it a bit? If not, I will cancel. I hope to have this feature in version 2x
Below is your pr
https://github.com/element-plus/element-plus/pull/1341

@cs1707

@JeremyWuuuuu
Can I copy your code and change it a bit? If not, I will cancel. I hope to have this feature in version 2x
Below is your pr
element-plus/element-plus#1341

This work is done by @Ryan2128, you might mention the reference in the code. That'd be ok to us.

@JeremyWuuuuu
Can I copy your code and change it a bit? If not, I will cancel. I hope to have this feature in version 2x
Below is your pr
element-plus/element-plus#1341

This work is done by @Ryan2128, you might mention the reference in the code. That'd be ok to us.

OK,Thx.
@Ryan2128. Can I copy your code ?

@cs1707 Can the pr be merged?

Dear @liushuiyuan001 . Thanks for your contribution.
Some styles need to be modified.

@cs1707 Can the pr be merged?

Dear @liushuiyuan001 . Thanks for your contribution.
Some styles need to be modified.

OK.I will fix

@media (max-width: 1490px) {
 .right-nav {
   display: none;
 }
}

@cs1707  The whole style seems to have some problems. I can only fit the big screen.It's OK? Otherwise the content will be covered.

@media (max-width: 1490px) {
 .right-nav {
   display: none;
 }
}

@cs1707 The whole style seems to have some problems. I can only fit the big screen.It's OK? Otherwise the content will be covered.

@media (max-width: 1490px) {
 .right-nav {
   display: none;
 }
}

@cs1707 The whole style seems to have some problems. I can only fit the big screen.It's OK? Otherwise the content will be covered.

I think the style still needs to be optimized. eg: position, scrollbar. And the error reporting in the console needs to be fixed.

@cs1707
I just fix error reporting and fix style.It' OK ? What should I do ?


Detail view not displaying correctly for pdf file

Describe the bug
An uploaded pdf file displays on a resources detail page as "This image could not be found". "This image" is a link to the pdf file.
To Reproduce
Steps to reproduce the behavior:

Upload PDF file
Viiew detail page of resource

Expected behavior
Expected a download link similar the the standard Nova File Field
Screenshots

Desktop (please complete the following information):

Laravel Version: [ex: 8.62]
Browser and Version: [ex: Firefox 94.0.1]
Nova File Upload Field Package Version: [ex: 0.4.0]

I may be missing something!
The relevant fields section of the resource (standard Nova File Field just there for reference):
File::make('Nova File Field', 'file')->exceptOnForms(),

FileUpload::make("FileUpload Field", 'file')
    ->path("public")
    ->creationRules('required', 'mimes:pdf', 'max:5000')
    ->updateRules('sometimes', 'mimes:pdf', 'max:5000')
    ->hideFromIndex()
    ->prunable(),

@warksit Thanks for writing in. I believe this is because the file upload field does not extract thumbnails from PDFs. Only image files (likely bitmaps) would have thumbnails available.

Re-opening: I would be happy to consider a PR for this issue.

OK, thanks for the response. Let me look into the code and see what needs to be done


docs: Update Auroras(2024-08-17 G3 Storm)

Astronomy/

Auroras.md

2024SolarActivity

#4462

Closed by

#4468


Create termination test

Redesign the smart contract for the way to terminate it. Specifically, I introduced an isActive public variable and an onlyActive modifier to guard each functions.


new ban validator functionality

create a new contract and implement a ban function for the validators that will be banned;
create a new internal function _withdraw to handle the withdraw process and re-use;
write the tests for the new inspector contract;
fix some other tests;

@R-Santev some test files are modified because of tabulation and fixed small issues that caused tests to fail. Additionally, I have removed the whitelisting from the registerValidator, because it is a different purpose.


Converting image coordinate to vehicle coordinate

Hi,
How can I convert pedestrian box center pixel values (cx, cy)  to values which shows the distance from vehicle may be vehicle coordinate system?

Hi, to clarify a bit, are you trying to get the depth of a 2D box, or compute the distance from the vehicle for a 3D box?

Hi, I want to calculate the distance between agent 2D box and vehicle.

I don't believe that there are associations between vehicle 2d and 3d boxes, so you may have to use the lidar depths to complete an approximate depth for the box in this case.

What components can be used t

I don't believe that there are associations between vehicle 2d and 3d boxes, so you may have to use the lidar depths to complete an approximate depth for the box in this case.

What components can be used to find the lidar depth for the box?

There's not a canonical way, but one example would be to find the average depth of the lidar points that fall inside the box.

There's not a canonical way, but one example would be to find the average depth of the lidar points that fall inside the box.

Could you give me an example as to how to do that using the components given?
My requirement is that I have to find the depth of a vehicle from the camera across frames. These vehicles should also have a 2D bounding box for further detection.
What would be the general process of finding the depth using the components provided by the dataset?

We provide the laser points projected to each camera here: https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/dataset.proto#L211. Given a 2D bounding box, you can find which of these laser points falls inside the box, and find some summary statistic of these points that fits your use case (e.g. mean, min etc.).
This function maybe helpful to parse the projected points: https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/utils/frame_utils.py#L34

We provide the laser points projected to each camera here:
https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/dataset.proto#L211
. Given a 2D bounding box, you can find which of these laser points falls inside the box, and find some summary statistic of these points that fits your use case (e.g. mean, min etc.).
This function maybe helpful to parse the projected points:
https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/utils/frame_utils.py#L34

We provide the laser points projected to each camera here:
https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/dataset.proto#L211
. Given a 2D bounding box, you can find which of these laser points falls inside the box, and find some summary statistic of these points that fits your use case (e.g. mean, min etc.).
This function maybe helpful to parse the projected points:
https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/utils/frame_utils.py#L34

Thank you for the reply. I am using parquet files. If I am not wrong, one has to use v2 format to parse these files. Which of the functions in the v2 format can I use to perform the above mentioned methodology by you?

Ah you're right. The projected lidar points can be found here then: https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/v2/perception/lidar.py#L99

Ah you're right. The projected lidar points can be found here then:
https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/v2/perception/lidar.py#L99

Thank you so much Alex. I shall get back if I have any further questions. Appreciate your help :)

Ah you're right. The projected lidar points can be found here then:
https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/v2/perception/lidar.py#L99`

Could you tell me if this can be used in any way to find an association between vehicle 2D and Lidar3D box? Or do the parquet files for this component only for the pedestrian class?
https://github.com/waymo-research/waymo-open-dataset/blob/5f8a1cd42491210e7de629b6f8fc09b65e0cbe99/src/waymo_open_dataset/v2/perception/box.py#L81

I believe these only exist for pedestrian boxes.

Oh hmm I'm a little confused about your use case I think. I thought you needed to use the 2D bounding boxes, which were labeled for images only, and are separate from the 3D bounding boxes, which are labeled for the LiDAR. Between these two sets of boxes we only have correspondences for pedestrians. If you are using the 3D bounding boxes, then yes, there is natively depth encoded already. The projected boxes above correspond to the 3D boxes simply projected into the image plane.

Oh hmm I'm a little confused about your use case I think. I thought you needed to use the 2D bounding boxes, which were labeled for images only, and are separate from the 3D bounding boxes, which are labeled for the LiDAR. Between these two sets of boxes we only have correspondences for pedestrians. If you are using the 3D bounding boxes, then yes, there is natively depth encoded already. The projected boxes above correspond to the 3D boxes simply projected into the image plane.

Sorry for the confusion. I do require 2D boxes of those objects as well, whose 3D boxes are projected into the image plane for my use case. I would need an object_id that would correspond with both these boxes.
Question:

Can I use the 2D boxes created by the function show_projected_camera_synced_boxes for my use case? ( I have attached the code which was present in the tutorial). Since the boxes that are generated here is using  ""box = label.camera_synced_box"". In v2 format it should be "LiDARCameraSyncedBoxComponent" which also has key.laser_object_id. Then I would know that the 2D and 3D box created would be for that particular object. With the 3D box I would calculate the depth of the object and use the same 2D box which would have the same id key.laser_object_id.

`def show_projected_camera_synced_boxes(camera_image, ax, draw_3d_box=False):
"""Displays camera_synced_box 3D labels projected onto camera."""
Fetch matching camera calibration.
calibration = next(cc for cc in frame.context.camera_calibrations
if cc.name == camera_image.name)
for label in frame.laser_labels:
box = label.camera_synced_box
if not box.ByteSize():
  continue  # Filter out labels that do not have a camera_synced_box.
if (FILTER_AVAILABLE and not label.num_top_lidar_points_in_box) or (
    not FILTER_AVAILABLE and not label.num_lidar_points_in_box):
  continue  # Filter out likely occluded objects.

# Retrieve upright 3D box corners.
box_coords = np.array([[
    box.center_x, box.center_y, box.center_z, box.length, box.width,
    box.height, box.heading
]])
corners = box_utils.get_upright_3d_box_corners(
    box_coords)[0].numpy()  # [8, 3]

# Project box corners from vehicle coordinates onto the image.
projected_corners = project_vehicle_to_image(frame.pose, calibration,
                                             corners)
u, v, ok = projected_corners.transpose()
ok = ok.astype(bool)

# Skip object if any corner projection failed. Note that this is very
# strict and can lead to exclusion of some partially visible objects.
if not all(ok):
  continue
u = u[ok]
v = v[ok]

# Clip box to image bounds.
u = np.clip(u, 0, calibration.width)
v = np.clip(v, 0, calibration.height)

if u.max() - u.min() == 0 or v.max() - v.min() == 0:
  continue

if draw_3d_box:
  # Draw approximate 3D wireframe box onto the image. Occlusions are not
  # handled properly.
  draw_3d_wireframe_box(ax, u, v, (1.0, 1.0, 0.0))
else:
  # Draw projected 2D box onto the image.
  draw_2d_box(ax, u, v, (1.0, 1.0, 0.0))`

If it is possible, how do I find the native depth of the 3D bounding box that is already encoded?


script.py

A better way is to create our fancy button by extending the Button class.

leave it

I added you to the list of contributors. If you like my projects, you can follow me so we can work together in the future
Thank you!


Does Emmet still work?

This is awesome, but emmet (and normal HTML autocompletion, etc.) seems to have stopped working for me...I know so little about VS Code extensions to know how to help...

Yeah unfortunately it's something that needs to be inplemented manually.
Luckily it looks like it might be relatively easy to add in using this https://github.com/Microsoft/vscode-emmet-helper

Cool. So am I understanding you to say this can be done by your plugin? Or are you saying I need to implement this somehow? #confused

I'll be adding it, sorry for the confusion :)
In the mean time you could possibly try adding this to your vscode preferences:
"emmet.showExpandedAbbreviation": "always",
"emmet.includeLanguages": { "svelte" }

Not tested it but Microsoft/vscode#29532 suggests it might work

Released in v0.2.0

This needs to be reopened. Emmet abbreviations have started working again, but somehow single/short abbreviations.
This works:

This does not:

Here's a cheat sheet of different possibilities, for reference: https://docs.emmet.io/cheat-sheet/

hmm, I can get that second one to work, however it only shows up if I manually trigger autocompletion with ctrl+space. Also curiously when it does decide to open the auto complete box automatically, it disappears every other keystroke. I'll open a new issue for this


Error in the CycloneDX JSON report with ci/Dockerfile

Describe the bug
Asking Tern to generate the report in the cyclonedx json format make it bug. Other format such as json or yaml does not bug.
To Reproduce
The Docker is run in Gitlab CI.

Build and run the Dockerfile from the ci file
Push the Docker to the container and use it in a pipeline
Launch Tern on a Dockerfile and put the report format as cyclonedxjson : tern report -f cyclonedxjson -d Dockerfile -o output.json
Error when Tern try to generate the CycloneDX report

Error in terminal
2022-03-28 08:54:55,489 - DEBUG - generator - Generating CycloneDX JSON document...
Traceback (most recent call last):
  File "/usr/local/bin/tern", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/tern/__main__.py", line 309, in main
    do_main(args)
  File "/usr/local/lib/python3.9/site-packages/tern/__main__.py", line 115, in do_main
    drun.execute_dockerfile(args)
  File "/usr/local/lib/python3.9/site-packages/tern/analyze/default/dockerfile/run.py", line 217, in execute_dockerfile
    report.report_out(args, *image_list)
  File "/usr/local/lib/python3.9/site-packages/tern/report/report.py", line 86, in report_out
    report = generate_report(args, *images)
  File "/usr/local/lib/python3.9/site-packages/tern/report/report.py", line 41, in generate_report
    return generate_format(
  File "/usr/local/lib/python3.9/site-packages/tern/report/report.py", line 56, in generate_format
    return mgr.driver.generate(images, print_inclusive)
  File "/usr/local/lib/python3.9/site-packages/tern/formats/cyclonedx/cyclonedxjson/generator.py", line 60, in generate
    report = get_document_dict(image_obj_list)
  File "/usr/local/lib/python3.9/site-packages/tern/formats/cyclonedx/cyclonedxjson/generator.py", line 45, in get_document_dict
    image_componet = mhelpers.get_image_dict(image_obj)
  File "/usr/local/lib/python3.9/site-packages/tern/formats/cyclonedx/cyclonedxjson/image_helpers.py", line 30, in get_image_dict
    purl = PackageURL('docker', None, image_dict['name'], image_dict['version'])
  File "/usr/local/lib/python3.9/site-packages/packageurl/__init__.py", line 238, in __new__
    raise ValueError('Invalid purl: {} is a required argument.'
ValueError: Invalid purl: name is a required argument.

Expected behavior
As for the others format, it should generate the report.
Environment you are running Tern on

Docker image in the ci file
The file dist/tern-.tar.gz in the Docker file as been change to dist/tern-.whl because python setup.py bdist_wheel generate a .whl file

Please attach files if they exist

tern.log

Hi @ThiefaineM , what was the solution? I faced with the same issue


Graph Editor: Add OpenUSD export option to the menu

Add an option to the menu that allows the user to create a new OpenUSD stage that references the currently active MaterialX material.

Some info from Slack and additional suggestions.
It seems it would be easiest to write this in Python but don't have it.

Grab a prebuilt version of Usd (e.g. from  NVIDIA)
Suggested to create a stage file with a reference to the MaterialX document.

So that needs to be created and saved.
Can use some default geometry or reference the MaterialX shader ball or any other geometry (bit more work)

You can use Python to create the usda file and write the reference
Use usdcat to "flatten" the file to another usda file.
Load it into something like usdview  for comparison.
If you want to be "nice" hook this all up to a the editor and bring up usdview.

e.g. Here is the default marble in usdview

Notes: Make sure that your PYTHONPATH, PATH and PXR_MTLX_STDLIB_SEARCH_PATHS are set correctly. You need the latter so that MaterialX definitions are found.

@kwokcb Could we potentially simplify this further, and run usdcat directly on the MaterialX file that is saved from the Graph Editor?  This would reduce the process of USD file generation to a single step, and the Graph Editor would only need the ability to run an arbitrary "post-process hook" on a MaterialX file after it is saved, with the invocation of usdcat on the generated file being one option that the user can request.

Hi @jstone-lucasfilm ,
I think the simplest possible option is to pass the filename to a script / cmd (which is your command hook). The script needs to consume the file name as an argument and can then do whatever it want's with it. So in this case it would need to fill-in / create the usda file with the materialx reference such as shown below:
#usda 1.0
(
    defaultPrim = "mxCapsule"
)

def Xform "mxCapsule" (
    kind = "component"
)
{
    def Capsule "geom" (
        prepend apiSchemas = ["MaterialBindingAPI"]
    )
    {
        rel material:binding = </MaterialX/Materials/Marble_3D>
    }
}

over "MaterialX"
(
    references = [@  ...file string inserted here... @</MaterialX>]
)
{
}

The script can run usdcat for flattening or any command it wants, including also running usdview.
If you want to make it arbitrary a command line option could be added. e.g.
--<export / post save option> "<run my script %f>"

where %f is the name of the saved file, and "run my script %f" would be run after save.


Windows CLI does not accept single quote as string delimiter

I'm creating Actions on OpenWhisk via Windows CLI. When a string is passed with single quotes, the quotes are ignored and as delimiter and treated as part of the string. If a string has a blank, then it sees as two different values and so an error occurs.
When I use double quotes, it works fine.
Linux CLI is working fine.

Run 'wsk --help' for usage.
C:\Users\IBM_ADMIN\Desktop\OpenStack Files\Uses Case>wsk action invoke --blocking --result hello --param name 'Bernie_Sanders' --param place 'Vermont'
{
"payload": "Hello, 'Bernie_Sanders' from 'Vermont'"
}
C:\Users\IBM_ADMIN\Desktop\OpenStack Files\Uses Case>wsk action invoke --blocking --result hello --param name "Bernie Sanders" --param place 'Vermont'
{
"payload": "Hello, Bernie Sanders from 'Vermont'"
}

@dubeejw do you have any insight on dealing with single quotes using the CLI on windows

I think we should doc the samples using double quotes until we find a solution.

Since this affects the lab I plan to run in London, quick question:
May I assume that this will be solved before October, 26th?
I.e. may I assume single quotes will work again by then or do I really need to change the lab instructions?

This behavior is due to the Windows cmd command processor.  Single quotes are not treated like double quotes, but are instead treated pretty much like any other character.  So, on Windows systems, arguments requiring quoting should use double quotes.
http://stackoverflow.com/questions/24173825/what-does-single-quote-do-in-windows-batch-files
You can also validate this with a simple echoargs.bat file
@echo %0
@echo %1
@echo %2
@echo %3

>echoargs.bat '1 2 3'
echoargs
`1
2
3`

>echoargs "1 2 3"
echoargs
"1 2 3"
ECHO is on.
ECHO is on.


Download location Desktop

Description of the new feature / enhancement
To always download to a specific area when saving to the desktop. If i save to the desktop it is put in place of a missing icon and i have to look for it
Scenario when this would be used?
This would make work easier
Supporting information
No response

So could you specify your request a bit more. Do you talk about all Apps or just PowerToys? And what do you mean by area?
/needinfo

When you save to the desktop the file will save to a random spot on the desktop and you have to loo for it. Being able to designate an area would be great. Also the ability to save icon locations
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows
From: Aaron @.>
Sent: Wednesday, December 21, 2022 12:13 PM
To: @.>
Cc: @.>; @.>
Subject: Re: [microsoft/PowerToys] Download location Desktop (Issue #22920)
So could you specify your request a bit more. Do you talk about all Apps or just PowerToys? And what do you mean by area?
/needinfo
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/22920#issuecomment-1361687721, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWTBWX5HY45TAQEZPDCZ2QDWOM3EHANCNFSM6AAAAAATFWI4UQ.
You are receiving this because you authored the thread.Message ID: @.***>

Don't think it's that easy. Might need something like Stardock Fences.

There is a program called Icon Shepard that works but would rather have it in powertoys. I also sent a suggestion to be able to change the color of folders. When I first saw Windows 11 I thought it was a feature. I downloaded an icon pack to change the color of my folders but it doesn’t keep the Windows 11 style.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows
From: @.>
Sent: Friday, December 30, 2022 2:41 AM
To: @.>
Cc: @.>; @.>
Subject: Re: [microsoft/PowerToys] Download location Desktop (Issue #22920)
Don't think it's that easy. Might need something like Stardock Fences.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/22920#issuecomment-1367774491, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWTBWX73GLVYLXKW3TCSJLLWP2G3NANCNFSM6AAAAAATFWI4UQ.
You are receiving this because you authored the thread.Message ID: @.***>

Think this may be closer to /dup #1392

The ability to change the color of all folders. I have folder icons I use currently but would rather it be built in. But they are not the windows 11 folder icon
From: Aaron Junker @.>
Sent: Wednesday, December 21, 2022 12:14 PM
To: microsoft/PowerToys @.>
Cc: MZedalis @.>; Author @.>
Subject: Re: [microsoft/PowerToys] Download location Desktop (Issue #22920)
So could you specify your request a bit more. Do you talk about all Apps or just PowerToys? And what do you mean by area?
/needinfo
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/22920#issuecomment-1361687721, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWTBWX5HY45TAQEZPDCZ2QDWOM3EHANCNFSM6AAAAAATFWI4UQ.
You are receiving this because you authored the thread.Message ID: @.***>

Sorry responded to wrong email. When you have a lot of icons on your desktop and download or save something to it. The icon appears where it wants to. Would be great to specify an area
From: Aaron Junker @.>
Sent: Wednesday, December 21, 2022 12:14 PM
To: microsoft/PowerToys @.>
Cc: MZedalis @.>; Author @.>
Subject: Re: [microsoft/PowerToys] Download location Desktop (Issue #22920)
So could you specify your request a bit more. Do you talk about all Apps or just PowerToys? And what do you mean by area?
/needinfo
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/22920#issuecomment-1361687721, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWTBWX5HY45TAQEZPDCZ2QDWOM3EHANCNFSM6AAAAAATFWI4UQ.
You are receiving this because you authored the thread.Message ID: @.***>


Dart VM subtyping issue

Consider the following code

class T1 {}

abstract class S extends T1 {}

class T0 implements S  {}

X foo<X, Y extends X>(Y y) => y;

main() {
  foo<T0, T1>(new T1());
}

This code produces an error in dartanalyzer  --strong and --preview-dart-2 modes and no issue in VM
Expected: a compile error in Dart VM
Dart VM version: 2.0.0-dev.49.0 (Wed Apr 18 20:41:36 2018 +0200) on "windows_x64"

VM (CFE) now also issues a compile-time error for this example:
foo.dart:10:3: Error: Type argument 'T1' doesn't conform to the bound 'X' of the type variable 'Y' on 'foo'.
 - 'T1' is from 'foo.dart'.
Try changing type arguments so that they conform to the bounds.
  foo<T0, T1>(new T1());
  ^
foo.dart:7:10: Context: This is the type variable whose bound isn't conformed to.
X foo<X, Y extends X>(Y y) => y;
         ^


update readme doc

Coverage remained the same at 81.343% when pulling c2d4d94660b581b68797af8ec2ed4f93b2cc341c on rotembr-patch-7 into 39360fd36d8d15b9614f70a870975ef9e1ed1dde on development.


Compatible with Azure Openai's 2023-07-01-preview version API interface about the error information returned by the intercepted interface

Compatible with Azure Openai's 2023-07-01-preview version API interface about the error information returned by the intercepted interface
Describe the change
Compatible with Azure Openai's 2023-07-01-preview version API interface about the error information returned by the intercepted interface.
Describe your solution
Add the parsing of the innererror node in the error message, and it is compatible with the old version of the interface.
Tests
Related test cases have been added to error_test.go
Additional context
CURL request logging
curl "https://TEST.openai.azure.com/openai/deployments/gpt-35-turbo/chat/completions?api-version=2023-07-01-preview" \
>   -H "Content-Type: application/json" \
>   -H "api-key: MY-KEY" \
>   -d "{
>   \"messages\": [{\"role\":\"user\",\"content\":\"write an erotic novel\"}],
>   \"max_tokens\": 800,
>   \"temperature\": 0.7,
>   \"frequency_penalty\": 0,
>   \"presence_penalty\": 0,
top_p\">   \"top_p\": 0.95,
top\":>   \"stop\": null
> }" -v
*   Trying <IP_ADDRESS>:443...
* TCP_NODELAY set
* Connected to TEST.openai.azure.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/certs/ca-certificates.crt
  CApath: /etc/ssl/certs
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.2 (IN), TLS handshake, Certificate (11):
* TLSv1.2 (IN), TLS handshake, Server key exchange (12):
* TLSv1.2 (IN), TLS handshake, Server finished (14):
* TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
* TLSv1.2 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.2 (OUT), TLS handshake, Finished (20):
* TLSv1.2 (IN), TLS handshake, Finished (20):
* SSL connection using TLSv1.2 / ECDHE-RSA-AES256-GCM-SHA384
* ALPN, server accepted to use h2
* Server certificate:
*  subject: C=US; ST=WA; L=Redmond; O=Microsoft Corporation; CN=japaneast.api.cognitive.microsoft.com
*  start date: May  3 22:24:37 2023 GMT
*  expire date: Apr 27 22:24:37 2024 GMT
*  subjectAltName: host "TEST.openai.azure.com" matched cert's "*.openai.azure.com"
*  issuer: C=US; O=Microsoft Corporation; CN=Microsoft Azure TLS Issuing CA 05
*  SSL certificate verify ok.
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* Using Stream ID: 1 (easy handle 0x560147eb07c0)
> POST /openai/deployments/gpt-35-turbo/chat/completions?api-version=2023-07-01-preview HTTP/2
> Host: TEST.openai.azure.com
> user-agent: curl/7.68.0
> accept: */*
> content-type: application/json
> api-key: MY-KEY
> content-length: 196
>
* Connection state changed (MAX_CONCURRENT_STREAMS == 100)!
* We are completely uploaded and fine
< HTTP/2 400
< content-length: 622
< content-type: application/json
< x-request-id: 2c8f0db0-8a90-43d2-89db-17c66e374a68
< ms-azureml-model-error-reason: model_error
< ms-azureml-model-error-statuscode: 400
< x-ms-client-request-id: 267046f7-10ce-46cd-8e89-40b4e27eb61b
< apim-request-id: 267046f7-10ce-46cd-8e89-40b4e27eb61b
< azureml-model-session: dep-659513-1
< azureml-model-group: online
< openai-processing-ms: 185.3565
< strict-transport-security: max-age=31536000; includeSubDomains; preload
< x-content-type-options: nosniff
< x-ms-region: Japan East
< date: Sun, 23 Jul 2023 19:22:56 GMT
<
* Connection #0 to host TEST.openai.azure.com left intact
{"error":{"message":"The response was filtered due to the prompt triggering Azure OpenAI’s content management policy. Please modify your prompt and retry. To learn more about our content filtering policies please read our documentation: https://go.microsoft.com/fwlink/?linkid=2198766","type":null,"param":"prompt","code":"content_filter","status":400,"innererror":{"code":"ResponsibleAIPolicyViolation","content_filter_result":{"hate":{"filtered":false,"severity":"safe"},"self_harm":{"filtered":false,"severity":"safe"},"sexual":{"filtered":true,"severity":"medium"},"violence":{"filtered":false,"severity":"safe"}}}}}

@sashabaranov Hi, hope you review this PR. In addition, due to my limited technology, I am not sure whether these two checks of codecov need to be processed. THANKS!


[5.4] Can't Wrap DB Names Containing Spaces When Aliasing

Database names containing spaces are perfectly legal in Microsoft SQL Server.
Currently, when using SQL Server, if you have a database which includes spaces, the grammar handles wrapping it fine.
So this:
My Sample Database.dbo.my_table.my_column
Correctly becomes:
[My Sample Database].[dbo].[my_table].[my_column]
But if you also try to alias a column when the database contains spaces, it breaks.
This:
My Sample Database.dbo.my_table.my_column AS my_aliased_column
Becomes:
[My] AS [Sample]
I have traced the error to Database/Grammar/wrapAliasedValue() using explode( ' ' ) - the code thinks the spaces in the database name are the spaces around the "AS".
We can fix this by using regex instead of explode, splitting on the "AS" more accurately.
Just replace the explode with this:
$segments = preg_split( '/\s+as\s+/i', $value );
Note: When you have an Eloquent belongsToMany method that has to include the database name (eg. we need to join across DBs), this type of alias is created automatically, generating this error.
I originally committed this to 5.3 (https://github.com/laravel/framework/pull/17292) and was asked to commit here instead.

Please add tests to cover this.

I don't know how to edit a pull request, but this test works on the new code (and fails on the current code):
For /tests/Database/DatabaseQueryBuilderTest.php
public function testAliasWrappingWithSpacesInDatabaseName()
{
	$builder = $this->getBuilder();
	$builder->select('w x.y.z as foo.bar')->from('baz');
	$this->assertEquals('select "w x"."y"."z" as "foo.bar" from "baz"', $builder->toSql());
}

I don't know how to edit a pull request

Just push to the same branch.

Thanks Graham.
Unfortunately, I'm a Git newb and am really struggling to do this - just spent over 2 hours wrestling with it and have totally failed. I have added that test case and run it and it works but can't add it to the patch-2 branch.
Should I start again and make a PR containing both changes or could someone else add the test for me.
Thanks for your help!

@voidstate Just browse to https://github.com/voidstate/framework/tree/patch-2 and add browse the file you want and edit there online, since it's just an test you can do it like this...

@fernandobandeira Thanks. That was much easier than what I was trying to do. :)

What happens if the table name contains [space]as[space]? Like foo as bar?

"As" is a reserved word in most rdbms. http://www.petefreitag.com/tools/sql_reserved_words_checker/?word=As


Update readme with important info

Added FESTIM and COMSOL versions, fixed some links, added installation guide for FESTIM

@RemDelaporteMathurin I don't think MHIMS version is a necessary info for reproducibility. Do you have any other suggested change from your side?
Thank you for your contribute

I don't think MHIMS version is a necessary info for reproducibility

I'd beg to differ. The data that is shared in this repo has been produced with a specific MHIMS version, or at least a certain commit (since MHIMS is hosted on GitHub). Who knows, maybe in a while, MHIMS will be released open-source, or someone will have access to the repo (even if it remains private) and will want to reproduce this data. Plus, maybe in the future there will be a breaking change or a feature deprecation, knowing the right version is essential for reproducibility.


created a branch, created a file and made a commit, and opened a pull request

Add amyjanes20's file