Question about Intallation

Prerequisites

[X] I have read the ServerlessLLM documentation.
[X] I have searched the Issue Tracker to ensure this hasn't been reported before.

System Information
OS: windows 11, using Ubuntu desktop, downloaded from MS store. Python 3.10, GPU 4070s
Problem Description
After setting the head and worker nodes, I'm following the QuickStart guide. And encounter this error message:
(sllm-worker) s2644572@DESKTOP-0417LDT:~$ sllm-store-server
Run server...
/home/s2644572/miniconda3/envs/sllm-worker/lib/python3.10/site-packages/sllm_store/sllm_store_server: error while loading shared libraries: libcudart.so.12: cannot open shared object file: No such file or directory
And the folder information is below:

Could you help, please? Thank you for your time in advance!
Steps to Reproduce
Code snippets:

Steps to reproduce:
1.conda activate sllm
ray start --head --port=6379 --num-cpus=4 --num-gpus=0
--resources='{"control_node": 1}' --block
2.conda activate sllm-worker
ray start --address=<IP_ADDRESS>:6379 --num-cpus=4 --num-gpus=2
--resources='{"worker_node": 1, "worker_id_0": 1}' --block
3.conda activate sllm-worker
sllm-store-server
Expected Behavior
$ sllm-store-server
TODO Run server...
WARNING: Logging before InitGoogleLogging() is written to STDERR
I20240720 08:40:59.634253 141776 server.cpp:307] Log directory already exists.
I20240720 08:40:59.671192 141776 checkpoint_store.cpp:46] Number of GPUs: 2
I20240720 08:40:59.671768 141776 checkpoint_store.cpp:48] I/O threads: 4, chunk size: 32MB
I20240720 08:40:59.797175 141776 checkpoint_store.cpp:69] GPU 0 UUID: cef23f2a-71f7-44f3-8246-5ebd870755e7
I20240720 08:40:59.951408 141776 checkpoint_store.cpp:69] GPU 1 UUID: bbd10d20-aed8-4324-8b2e-7b6e54aaca0e
I20240720 08:41:00.759124 141776 pinned_memory_pool.cpp:29] Creating PinnedMemoryPool with 1024 buffers of 33554432 bytes
I20240720 08:41:24.315564 141776 checkpoint_store.cpp:80] Memory pool created with 32GB
I20240720 08:41:24.318261 141776 server.cpp:279] Server listening on <IP_ADDRESS>:8073
Additional Context
Because I'm using a Windows machine, I just started getting familiar with Linux with the Ubuntu app. I'm not sure if my problem is related to the WSL connection. Also, could the SLLM run on Windows?
Ubuntu App description:
Install a complete Ubuntu terminal environment in minutes with Windows Subsystem for Linux (WSL). Develop cross-platform applications, improve your data science or web development workflows and manage IT infrastructure without leaving Windows.
Key features:

Efficient command line utilities including bash, ssh, git, apt, npm, pip and many more
Manage Docker containers with improved performance and startup times
Leverage GPU acceleration for AI/ML workloads with NVIDIA CUDA
A consistent development to deployment workflow when using Ubuntu in the cloud
5 years of security patching with Ubuntu Long Term Support (LTS) releases

Ubuntu currently provides the 24.04.1 LTS release. When new LTS versions are released, Ubuntu can be upgraded once the first point release is available. This can be done from the command line by using:
sudo do-release-upgrade

Installation tips:

Search for "Turn Windows features on or off" in the Windows search bar and ensure that "Windows Subsystem for Linux" is turned on before restarting your machine.
To launch, use "ubuntu" on the command-line prompt or Windows Terminal, or click on the Ubuntu tile in the Start Menu.

For more information about Ubuntu WSL and how Canonical supports developers please visit:
https://ubuntu.com/wsl

Usage Statistics (Optional)
No response

Hi,
It appears that CUDA 12 may not be installed correctly, or you might be using an older CUDA version. Could you check your CUDA version and switch to CUDA 12? If the issue continues, please let us know.

Hi,
Thank you for your help. I fixed the CUDA issue above but ran into a new one as below. I'm not sure if this is caused by the wsl connection. Given the configuration of my PC, it should not run out of memory. Any advice is really appreciated, thank you!

Hi,
Thank you for reporting this issue. It may be due to insufficient memory allocated to WSL, which by default uses half of the system's memory. Could you please run free -h to check if you have enough available memory? Our store-server creates a 4GB memory pool by default. If the memory is insufficient, you can either reduce the size of the created memory pool by setting the --mem_pool_size or increase WSL's memory allocation by modifying its configuration settings. We recommend choosing the latter option.
Additionally, once you have resolved all issues and successfully run ServerlessLLM on your machine, could you please document an example of your setup process on WSL? This would be highly valuable for Windows users. In your example, please cover steps from installing WSL and Ubuntu, any issues encountered and their solutions, and how you ultimately got ServerlessLLM to operate successfully. Write this example in markdown format and save it under the examples/ folder, then submit a pull request.
Thank you very much! We look forward to your contribution.

Another issue when deploying models:
When I try to deploy the example facebook model,
(sllm) s2644572@DESKTOP-0417LDT:~$ sllm-cli deploy --model facebook/opt-1.3b
INFO 10-30 22:09:40 deploy.py:160] Deploying model facebook/opt-1.3b.

I encounter this error message:
INFO 10-30 22:20:19 deploy.py:160] Deploying model facebook/opt-1.3b.
ERROR 10-30 22:26:50 deploy.py:172] Failed to register model. Status code: 500. Response: {"detail":"Cannot register model, please contact the administrator"}

The details while fetching the model are below:

I checked my storage and cude settings below, which looks fine to me. Any direction to further fix this is appreciated. Thank you very much!

Hi,
We couldn’t reproduce the error on our end. Could you please verify if you've applied the vllm patch located in sllm_store/vllm_patch? If the issue persists, try updating your sllm version. Alternatively, you can run:
pytest -s sllm_store/tests/python/test_save_vllm_model.py

to see if the test passes. Ensure you're in the ServerlessLLM/ directory when executing this test. If the test fails, please check the error log and share it here.

Thank you for your patience again. I updated the packages and locate the sllm_load.patch file under /sllm_store/vllm_path. Still the error message is below:
(VllmModelDownloader pid=1157) An error occurred while saving the model: [Errno 2] No such file or directory: './models/vllm'
And below is my error message after running test_save_vllm_model.py, I checked my torch is installed and importable the python terminal. Could you please let me know where to further check? Thank you very much!

Hi,
Did you run the script file patch.sh under the same directory as sllm_load.path?

Hi,
It seems that the pytest you called is the system's pytest, not inside the environment. Could you try pip install -r requirements-test.txt and execute the pytest command again?

Dear both, Thank you for your comment! Now, the pytest can run, and the test passed, as shown below.

After running the patch.sh file under the same dir as sllm_load.patch. I tried to deploy a model again following your Quickstart Guide. And the result is like below.

I'm not sure if the wsl connection stops the

I'm not sure if the wsl connection stops the storage-aware scheduling as error messages below. I'll keep looking up the WSL connection things. At the same time, I would appreciate any suggestions regarding the deploying issues! Thank you very much!
(StoreManager pid=6997) WARNING 11-03 16:36:28 store_manager.py:208] Hardware info not provided. Storage-aware scheduling might not work properly
(SllmController pid=6966) INFO 11-03 16:36:28 controller.py:73] Starting scheduler
(StoreManager pid=6997) INFO 11-03 16:36:28 store_manager.py:196] Initializing store manager
(StoreManager pid=6997) WARNING 11-03 16:36:28 store_manager.py:208] Hardware info not provided. Storage-aware scheduling might not work properly

It looks like this warning message isn't actually an error and shouldn’t impact the deployment. It’s more of an informational message, so it shouldn’t be causing the issue.

Hi, there have been some reported issues with WSL related to creating directories due to permission settings. This is often caused by user configuration and mounting issues. Could you please check if you can successfully create directories in this folder? Alternatively, you can run
import os
os.makedirs("./example")

to see whether the example/ folder is successfully created.

Hi,
I followed the QuickStart on WSL and everything looks good so far. Here are my configurations:
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.65
MSRDC version: 1.2.5620
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26100.1-240331-1435.ge-release
Windows version: 10.0.26100.2033
Initially, I tried running with 16GB of memory, but it failed. I then extended it to 28GB with an 8GB swap.
The model deployment works as shown in the following screenshot:

Thank you all for the support. It turns out to be there is a models folder created but it's inaccessible (cannot write or read inside). I deleted it and re-created it. Now, it's all good. I'll write a markdown file to record how it sets all of these up and commit it to the example/ folder. Thank you again!

Thank you all for the support. It turns out to be there is a models folder created but it's inaccessible (cannot write or read inside). I deleted it and re-created it. Now, it's all good. I'll write a markdown file to record how it sets all of these up and commit it to the example/ folder, if that's still the location you prefer. Thank you again!

Hi, we also notice that your machine only contains 1 GPU but you allocated 2 on your worker node. Please restart your worker node by:
conda activate sllm-worker
ray start --address=<IP_ADDRESS>:6379 --num-cpus=4 --num-gpus=1
--resources='{"worker_node": 1, "worker_id_0": 1}' --block


Bump setup-java actions to v3

Description
https://github.com/actions/setup-java/releases/tag/v3.0.0
Checklist

[ ] tests are added
[ ] CHANGELOG.md is updated

@Goooler Tnx for your contribution. Can you please chekc why the "Validation" step is not completed? It is still Waiting for status to be reported.

Seems like this issue: https://github.community/t/expected-waiting-for-status-to-be-reported/16727

I am not sure what you expect to happen? Do you think that a manual action is needed? If so, what action? I can not find anything about protected branches as is suggested in https://github.community/t/github-actions-pull-requests-always-include-some-outdated-checks/16157. Maybe, I am not authorized for that.

Alright, just be housekeeping.


request feature of comment for ended testrun

Hello.
Is it possible to add additional information related to the results or the ability to input data such as comments to the finished test run?
since there is no function to terminate the testrun and add any function to that test run,
it is inconvenient because I have to take notes on the test run or record it separately from the casebook.
thank you.

layout of the report detail page has been changed and we added a feature that allows to view and create comments on the report detail page. this feature will be added to the v1.3.4(#130) release.
thank you.

@dreamiiizer
v1.3.5 has been released.

thanks! 👍 👍


rounded props for a card throws an error

This error pops up when I use rounded props for cards.

The rounded prop was added last week, and not released yet. If you want to use this make sure to patch the dependency in your Cargo.toml file and point it to the repo main branch .
That's probably why you have this issue.
Also, for the future : please don't post screen captures of code, simply copy and paste the text, it's easier to read :)

You can use the latest version from git using:
[patch.crates-io]
patternfly-yew = { git = "https://github.com/ctron/patternfly-yew" }

Just be sure that the version requirement in your main dependencies section matches the version in the main branch.
For more information, see: https://doc.rust-lang.org/cargo/reference/overriding-dependencies.html


Added support for newest tensorflow

closes #8 , closes  #11 , closes  #12

I have changed both main.py and test.py to support latest tensorflow.
In main.py, I have implemented more elegant way to save model rather than saving architecture and weights separately.
Unfortunately I was not able to load earlier saved model in latest versions. I have added code to load model for models saved with updated main.py.
requirements.txt is also changed with versions on which I have tested this PR.

Please do review PR and tell me if there are any changes 😄 .
Thanks.

Thanks very much for this! :) It works great. I've actually managed to train the WRN-28-10 architecture so have push these to GitLFS also in https://github.com/asmith26/wide_resnets_keras/pull/14


bundle.js not loading in the browser: mixed-content error.

I cant get bundle.js to load in the browser from an https:// page due to "mixed-content" error.  I can load the bundle.js code via http:// in the address bar.
I dont really understand why the bundle is being pulled from the server after page load in an SSR setup... Isn't Razzle entirely SSR?
Three questions arise:

Is there a recommended configuration for working with a non-localhost https:// server?
Why is the browser trying to access an http:// address on port 3001 for a server side rendered page?
Are there hard-coded http links anywhere in the software?

thanks

ive raised a new more specific issue that should resolve this.


2024.2 abd dev main branch grive error while attaching session

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
I have one EDB already open when I try to attach the session before opening a second EDB I got this error:

Seems the attach command is trying to start a new session according to the log
📝 Steps to reproduce
load 2 EDB files
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
No response
🐍 Which Python version are you using?
3.9
📦 Installed packages
NA

error is expected. instead of opening multiple EDBs under multiple sessions, open them in a single session


Fix github fields

Relatedly, the nameservice/simple-storage apps' github field also trip up hpack, but I think it's a bug in the latter: https://github.com/sol/hpack/pull/399


Zero counts for C26060T and aa:orf6:D61L for BA.2 samples

Both sites have substantial coverage levels, but neither mutation is being reported at a non-zero count for samples where we otherwise have evidence of high frequencies of BA.2.  This is skewing our estimates of variant frequency and CIs.

viralrecon pipeline seems to be picking up C26060T.
Could this be caused by primers?

Looks like this is a constellation misannotation problem.

nuc:C26060T is a non-synonymous substitution in ORF3a T223I.  aa:orf3a:T223I occurs at high frequencies in these samples, but does not appear in the BA2 JSON.
D61L is a triple substitution (GAT to CTC).  Since our pipeline is handling nucleotides individually, we are picking this up as three separate features:

~27382C,aa:orf6:D61H
~27383T,aa:orf6:D61V
~27384C,T27384C

Action items @GopiGugan @Abayomi-Olabode

nuc:C26060T should be replaced with aa:orf3a:T223I in our BA.2 constellation
Since we don't presently have the ability to combine substitutions, my suggestion is to omit the D61L entry from the BA.2 constellation in our fork for now.

Alternatively, we can just count one of the three mutations for D61L, e.g., T27384C.


Use wcwidth to compute len of printable string

Hello,
When we combine PrettyPrintTree with other package like climage the len is miscalculate and the tree is Broken.
Do you thinks that it possible to use wcswidth ?
Thanks for this package ?

Yup, I just pushed a new version which should work now.
(Although the whole thing is a mess and I'd like to do a complete rewrite soon)

Thanks you very much !


Fix: Update the condition to display deleted attachment on LHN

Details

The title of thread in LHN still be attachment after deleting this attachment. It should be [Deleted message]
Fixed Issues

$ https://github.com/Expensify/App/issues/20580
PROPOSAL: https://github.com/Expensify/App/issues/20580#issuecomment-1587215594
Tests

Go to any chat
Send an attachment
Reply in the above attachment
Observe that new reply chat is created in LHN with title [Attachment]
Delete the above attachment
Verify that title in LHN still be [Deleted message]

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Go to any chat
Send an attachment
Reply in the above attachment
Observe that new reply chat is created in LHN with title [Attachment]
Delete the above attachment
Verify that title in LHN still be [Deleted message]

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/129500732/25a21c54-44ca-4c93-9ab7-d60a95b3782b

Mobile Web - Chrome
https://github.com/Expensify/App/assets/129500732/8e65b868-007c-4a9d-88a5-01938425b900

Mobile Web - Safari

https://github.com/Expensify/App/assets/129500732/44e50a02-f567-4204-8f70-5a20c5162bdc

Desktop

https://github.com/Expensify/App/assets/129500732/a4027996-0d13-4d93-a272-cbaf87a5aad3

iOS

https://github.com/Expensify/App/assets/129500732/48d3cd0a-a7ef-4737-a7db-16e05bb95468

Android

https://github.com/Expensify/App/assets/129500732/578a2754-9496-44e1-b2d6-973c9fd6d946

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

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

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/25520267/d46e656c-a0e0-48fd-98de-952d6a806905

Mobile Web - Chrome

mweb:chrome.webm

Mobile Web - Safari

https://github.com/Expensify/App/assets/25520267/cb4cdcc3-17fd-42c8-827b-160295603dc6

Desktop

https://github.com/Expensify/App/assets/25520267/98600a49-1412-4243-aa15-dc8590dba19a

iOS

https://github.com/Expensify/App/assets/25520267/4f9022ad-9132-48ca-95d9-0f07110cd6a3

Android

android.webm

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/cristipaval in version: 1.3.33-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


AWS distribution version and links update

https://github.com/kubeflow/blog/pull/119
Updated generalized Kubeflow on AWS links for 1.5.1 release
@PatrickXYS is no longer owning AWS docs

/lgtm

@Arhell can you please add approve label as well?

@Arhell can you please add approve label as well?

https://github.com/kubeflow/website/blob/master/OWNERS#L1

@chensun can you please add approve label as well?

/approve


Manage SSL certs

Imported from https://github.com/ho-ansible/ansible/issues/21

If we want the keys to be completely managed by ansible for both client and server, follow the burp docs on SSL certs. Store all certs on the ansible server (like for tinc) and push to client/server. Then disable auto-generation of certs on both client and server.


example channel and language updates

not sure if you want this to go to master directly?
I have adjusted the GPIO numbers in the example to suit most PI boards and improved some of the language.
thanks

Thanks a lot for this great contibution!


Missing works during import and the possible problems

"undefined method `file_name=' for nil:NilClass"
This is occurring because there is no version of the file.  Checked the first two in production and no file exists. It looks like an error occurred during upload and the content never got set.
7d278x872*
 "tmp/import/generic_file_7d278x872.json","undefined method `file_name=' for nil:NilClass"

8623j023c  File Set id: sn009w3415
 "tmp/import/generic_file_8623j023c.json","undefined method `file_name=' for nil:NilClass"

8623j317v  File Set id: 5hq37vn00r
"tmp/import/generic_file_8623j317v.json","undefined method `file_name=' for nil:NilClass"

8623j3850 File Set Id: vt435gd81h
"tmp/import/generic_file_8623j3850.json","undefined method `file_name=' for nil:NilClass"

vm40xr584 File Set Id: cvd66vz285
"tmp/import/generic_file_vm40xr584.json","undefined method `file_name=' for nil:NilClass"

x920fw89s File Set Id: d2514nm63r
"tmp/import/generic_file_x920fw89s.json","undefined method `file_name=' for nil:NilClass"

zc77sq19f File Set Id: kkw52j858k
"tmp/import/generic_file_zc77sq19f.json","undefined method `file_name=' for nil:NilClass"

Undefined method to_datetime
The files in this case have null date uploaded and date modified in the json file.  Both files were deposited by Linda Musser and do not have content or dates.  The were created on 2015-11-02T16:19:47.226Z  & 2015-11-02T16:33:47.014Z and have no content.
fx719m462*
"tmp/import/generic_file_fx719m462.json","undefined method `to_datetime' for nil:NilClass"

9s1616164*
"tmp/import/generic_file_9s1616164.json","undefined method `to_datetime' for nil:NilClass"

RDF was not parsable
This looks to be related to crazy characters in the description.  Need to figure out what character is causing the issue.
7p88cg655 File Set Id: jsx61dm39c
(0x0D)"ort/generic_file_7p88cg655.json","RDF was not parsable: [line: 18, col: 74] illegal escape sequence value:

Timeout
These are my large more than 10G files.
n009w2390 File Set Id: 4xg94hn09v
 "tmp/import/generic_file_n009w2390.json","STATUS: 500 org.modeshape.jcr.value.binary.BinaryStoreException: java.net.SocketTimeoutException

tt44pn00t File Set Id: mgq67jr99p
"tmp/import/generic_file_tt44pn00t.json","STATUS: 500 org.modeshape.jcr.value.binary.BinaryStoreException: java.net.SocketTimeoutException

No Error in log
empty title
No FileSet was created
5712mv79g

not in staging or prod with no json file FileSet may exists from a previous import
4t64gn298 File Set Id: pc821gk990

I reprocessed one of the timeout files and was able to get it to ingest successfully.
mv tmp/import/generic_file_n009w2390.json tmp/import2/
time RAILS_ENV=production bundle exec 'sufia_import -v -d tmp/import2 --json_mapping Import::GenericFileTranslator=generic_file_'


hyprland-session.service broken

Already reported ? *

[X] I have searched the existing open and closed issues.

Regression?
Yes
System Info and Version
 System/Version info
<Paste the output of the command here>

Description
Just installed yay -S hyprland to 0.45.1
Hyprland isn't starting anymore.
The reason is the broken /usr/lib/systemd/usr/hyprland-session.service
[Service]
Type=notify
ExecStart=/usr//usr/bin/Hyprland
ExecStop=/usr//usr/bin/hyprctl dispatch exit
ExecStopPost=systemctl --user unset-environment DISPLAY WAYLAND_DISPLAY HYPRLAND_INSTANCE_SIGNATURE XDG_CURRENT_DESKTOP
Restart=on-failure
Slice=session.slice
The ExecStart and ExecStop paths are wrong.
How to reproduce
Install 0.45.1 and try to start Hyprland with sddm or Hyprland command.
Crash reports, logs, images, videos
No response

Changed the paths to /usr/bin/ but Hyprland isn't starting anymore. Works perfectly with 045.1-1. Any ideas?

Duplicate of https://github.com/hyprwm/Hyprland/issues/8340. Use UWSM instead.
Also 0.45.1 no longer contains the service file. That may be a leftover from previous versions.

Duplicate of #8340. Use UWSM instead.
Also 0.45.1 no longer contains the service file. That may be a leftover from previous versions.

Thanks. I will test it. But this has a really big impact. Especially if only Hyprland is installed as the one DE.
I think, a super easy step by step guide is required to explain this new setup. I am receiving many requests from users with completely broken systems.

Especially for previous install using sddm
And they are a lot.

Followed your guide in UWSM but cannot find the uwsm session for Hyprland in /usr/share/wayland-sessions
How to start Hyprland after installing uwsm with sddm?

Make sure Hyprland has been built with systemd support. For some reason it's not automatically detected by meson (iirc the Arch package uses meson), so the flag has to be passed explicitly -Dsystemd=enabled, and also -Duwsm=enabled. The latter should be auto-enabled if systemd is also enabled though.

Make sure Hyprland has been built with systemd support. For some reason it's not automatically detected by meson (iirc the Arch package uses meson), so the flag has to be passed explicitly -Dsystemd=enabled, and also -Duwsm=enabled. The latter should be auto-enabled if systemd is also enabled though.

This is not for beginners. How does existing Hyprland users know that? Arch users are installing or updating Hyprland with sudo pacman -S hyprland resulting in a broken system.
All of my test systems are broken after running just yay.
Is there a way to make the update to 0.45.1 more user friendly fro Arch and Endeavour users?

Just restarted my Laptop with 0.45.1 with no issues.
Maybe it's only a KVM VM issue?

Just restarted my Laptop with 0.45.1 with no issues.
Maybe it's only a KVM VM issue?

Could be.
I'm on kvm too.
Something to investigate.

Is there a way to make the update to 0.45.1 more user friendly fro Arch and Endeavour users?

Check whether the PKGBUILDS are wrong, and if so, contact the package maintainers. However the systemd meson feature has been available for a long time now, shouldn't cause issues. The uwsm feature was added and enabled by default, so that also shouldn't cause issues.
I have no experience with KVM VMs, so I can't assist there.

Thank you for your feedback. I will create a new Arch Linux VM with Hyprland profile in a VM and will try again.

There is a similar report in Nixpkgs https://github.com/hyprwm/aquamarine/issues/109
This seems to be a Hyprland issue related to VMs rather than a service issue.
@gulafaran @UjinT34 can you take a look at the above? Seems it references your commits.

Looks more likely to be a VM issue. I am running @mylinuxforwork dotfiles and install script, with fully updated Arch OS on bare metal without any problem. Was trying to fix a problem that wasn't on my system for the past hours. Here's what I've got.

Hyprland Version I am running:

uwsm was not installed
installing uwsm did nothing.
tried 'Hyprland (systemd)` option in sddm, did not load (black screen).
however, the normal Hyprland option in sddm properly started as usual.

update:
added an entry into sddm .desktop file like so:

and started with the new option in sddm without an issue too.


html_str parameter not working

I am using Django to render an HTML file, then feed that file into html2image.
I am using the latest release: html2image==2.0.1
On Linux, using Chromium 93.0.4577.82 snap (Ubuntu 20.04)
To replicate:
hti = Html2Image()
hti.screenshot(html_str="
Hey there
")

This outputs a screenshot.png in my current working directory, however this screenshot is just a transparent image file, with no html content in it.
I tracked down the source of this issue to be the fact that on new versions of Ubuntu, Chromium is delivered only as a snap package, even if you use apt to install it (in this case, apt just uses snap under the hood). The problem with snap-installed packages is that they don't have access to the filesystem outside of the user's home directory. This includes the /tmp folder.
This took me a while to hunt down, and I feel like other less-python-inclined developers may also run into this issue, especially if they don't know a lot about Ubuntu. My fix was to pass the temp_path parameter, which I was only able to find by going through your source.
This parameter is only alluded to here in your CLI docs:
https://github.com/vgalin/html2image#using-the-cli
Once I found this, it was a simple fix:
hti = Html2Image(temp_path="/home/myuser")
hti.screenshot(html_str="
Hey there
")

And it worked. My humble recommendation would be to add to the docs to explain this parameter, and maybe a blurb about the limitations of Chromium installed via snap.
Thanks for your work on this project. It's really useful.

Thank you for opening this very detailed issue.
I took the (bad) habit to only answer issues after fixing them, but as you posted it more than a month ago I wanted to inform you that it didn't went unnoticed.
As for the temp_path parameter, it is mentionned in the usage section of the readme, but you have to click on the text reading "Multiple arguments can be passed to the constructor (click to expand)". It may be better to make this part visible by default ...
TODO:

Add a warning when a snap-based version of Chromium is used.
Make changes to the readme regarding the constructor parameters'.

Hi @vgalin, I got blank files when used the lib, and it looks like a folder permissions problem.
I think use tempfile.TemporaryDirectory instead of use a static /tmp/html2image folder, can solve this problem.
May I open a pull request doing this?


Fundamentals TOC

Fixes #20342

Thanks for the review @gewarren, great catches!

Thanks for the review, @IEvangelist !


Pipeline defintion is broken

./create-pipeline.sh concourse.suse.dev prabal-kubecf
apply configuration? [yN]: y
error: invalid pipeline config:
invalid jobs:
	jobs.upgrade-test-master.plan[1].put.status-master.src refers to a resource that does not exist
	jobs.upgrade-test-master.abort.in_parallel[0].try.put.status-master.src refers to a resource that does not exist
	jobs.upgrade-test-master.error.in_parallel[0].try.put.status-master.src refers to a resource that does not exist
	jobs.upgrade-test-master.failure.in_parallel[0].try.put.status-master.src refers to a resource that does not exist
	jobs.upgrade-test-master.success.put.status-master.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.2.plan[1].put.status-release-2.2.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.2.abort.in_parallel[0].try.put.status-release-2.2.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.2.error.in_parallel[0].try.put.status-release-2.2.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.2.failure.in_parallel[0].try.put.status-release-2.2.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.2.success.put.status-release-2.2.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.3.plan[1].put.status-release-2.3.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.3.abort.in_parallel[0].try.put.status-release-2.3.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.3.error.in_parallel[0].try.put.status-release-2.3.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.3.failure.in_parallel[0].try.put.status-release-2.3.src refers to a resource that does not exist
	jobs.upgrade-test-release-2.3.success.put.status-release-2.3.src refers to a resource that does not exist
	jobs.upgrade-test-pr.plan[1].put.status-pr.src refers to a resource that does not exist
	jobs.upgrade-test-pr.abort.in_parallel[0].try.put.status-pr.src refers to a resource that does not exist
	jobs.upgrade-test-pr.error.in_parallel[0].try.put.status-pr.src refers to a resource that does not exist
	jobs.upgrade-test-pr.failure.in_parallel[0].try.put.status-pr.src refers to a resource that does not exist
	jobs.upgrade-test-pr.success.put.status-pr.src refers to a resource that does not exist

diff prabal-kubecf.yaml kubecf.yaml
4,5c4,5
< s3_bucket: prabal-kubecf-ci
< s3_bucket_region: us-west-2
---
> s3_bucket: kubecf-ci
> s3_bucket_region: eu-central-1
8c8
< s3_final_bucket: prabal-kubecf
---
> s3_final_bucket: kubecf
21,22c21,22
< trigger_publish: false
< github_status: false
---
> trigger_publish: true
> github_status: true
36d35

@prabalsharma the PR is merged. I tested it locally and I could reproduce your issue and verify the fix. Check it too so we can close this issue.

Thanks @jimmykarily


Which heat pumps (or series) are supported?

Can you provide a list of Nibe heat pumps that are supported by your plugin?
It looks like your plugin would support only the F-series. I tried to follow your guide with my S-series heat pump (which I can control) via https://myuplink.com/, but failed to connect it via the URL in your setup guide.

Hi, I think that your question is related to https://github.com/hp-net/homebridge-nibe/discussions/114. Long story short: plug-in supports all devices that return data via old uplink api. If new my uplink api will support older devices I will consider switch.

MyUplink is now supported.


prototype approach to multiple data columns

The approach is (1) use Columns for both indexes and data, (2) bring in NamedTuples. Example:
julia> NDSparse(rand(3), Columns(open = rand(3), close = rand(3)))
NDSparse{NDSparseData._NT_openclose{Float64,Float64},Tuple{Float64}}:
 (0.07459099981805961,) => (open => 0.39695970629326216, close => 0.8768181865079363)
 (0.23068405270544878,) => (open => 0.21901570490914923, close => 0.9516909318576521)
 (0.8329699883751216,) => (open => 0.23058501858024938, close => 0.8031837612378654)

Extracting a vector of a particular data field will still be type unstable, right? The challenge here is also not being able to use our vectorization operators effectively. @tanmaykm experimented with this, I believe - so would be good to hear what he may have to say.

Not necessarily --- we could use a named tuple for the tuple of columns, and then you can get a whole column with data.col which is type-inferrable.
A larger problem here is that the mental model of arrays requires that the array be of a particular thing. If you have an array of things of type X, operations that make sense on X can be lifted to the array. It also allows names of data to be first-class variable names in the language. Together this enables vectorized statements like price1 .- price2. But if the price1 object contains multiple data columns, then the actual data is no longer first-class --- you have no way of naming the specific data you want within the language --- and many problems ensue.

Viral is referring to was what we saw here: https://github.com/tanmaykm/notebooks/blob/master/stocks/compare_df_like.ipynb
But it was not using named tuples then.

Yes, there is always potential for type-unstable code whenever you index into a structure containing more than one type. True of simple tuples as well.
Actually, this PR is mostly independent of those issues, because all it really does is generalize the Columns structure so it can produce NamedTuples as well as Tuples.
What I'm working on now is allowing the tuple of columns to itself be a NamedTuple, which naturally provides for named dimensions. Almost all the code can stay the same, since the only difference is that columns can be indexed by symbol as well as integer index. So far it feels like the right approach.

Added named columns. Demo:
julia> hi = NDSparse(Columns(city=[fill("New York",3); fill("Boston",3)],
                             date=repmat(Date(2016,7,6):Date(2016,7,8), 2)),
                             [91,89,91,95,83,76])
NDSparse{Int64,Tuple{String,Date}}:
 (:city,:date)
 ("Boston",2016-07-06) => 95
 ("Boston",2016-07-07) => 83
 ("Boston",2016-07-08) => 76
 ("New York",2016-07-06) => 91
 ("New York",2016-07-07) => 89
 ("New York",2016-07-08) => 91

julia> select(hi, :city => x->x=="Boston")
NDSparse{Int64,Tuple{Array{String,1},Array{Date,1}}}:
 (:city,:date)
 ("Boston",2016-07-06) => 95
 ("Boston",2016-07-07) => 83
 ("Boston",2016-07-08) => 76

Currently this requires my fork of NamedTuples.

Current coverage is 86.79% (diff: 78.08%)

Merging #24 into master will decrease coverage by 2.79%

@@             master        #24   diff @@
==========================================
  Files             4          4
  Lines           394        409    +15
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            353        355     +2
- Misses           41         54    +13
  Partials          0          0

Powered by Codecov. Last update 1df9be4...df7881c

Updates to NamedTuples are now on master there, so I'll merge this.


NPM scripts

Move from Gulp to NPM Scripts

Depends on #90 to be completed.

Let me suggest the this scripts, in particular the postversion hook and use of pre-commit and npm-watch.

We could also ensure the build step is run pre-commit to be sure any changes make it into /dist/

NPM Scripts related to #90 #93  as part of an over-arching "build flow revision".
suggest checking out: https://github.com/kenwheeler/cash/tree/npm-unit-tests branch as a base for this improvement.

@kenwheeler , could you have a look at https://github.com/kenwheeler/cash/tree/npm-scripts ... I've got babel running there, and npm scripts with my unit tests... one thing that's annoying though:
Adding a "header" banner to the file...
I am completely lost how to grab the package contents from NPM scripts... it'll have top be a js/bash script, right? -- and then... how is that different to gulp?

Nevermind. I sorted it.
LOL. :joy:

Switching to NPM scripts may make it difficult to implement custom build modularity ( issue #94 ) to Cash. Any thoughts on how we could implement that in the new build system?

Actually, Preprocess should be able to run from NPM Scripts. I even tried it... however; the preprocess-cli wrapper for preprocess seems broken/out of date... and just plain fails to install. If that was working there's not to use it.
Currently I've just got a single script concatenating all the files.
I, Personally, don't want to get in to complicated piping and argv handling, as I really think anything that complicated should be in a .js (gulp or otherwise) or .sh file

Agreed. I'd like it to be as simple as possible, but would still like to see custom build functionality. Any other libraries out there that do something similar?

You can do a lot of things with npm scripts like using glob, launch other npm scripts in sequence, even for loops like this.
They are easier than .js or .sh or gulp/grunt, cause they are just commands that you launch and store in a single row.
Concat files should be as easy as cat src/*js > build/${npm_package_name}.js however there can be some use cases where it is better to use scripts. In that case, usually I create a src/npm-scripts/ folder.

As of v2.1.2 Gulp tasks have been replaced with NPM scripts + pacco.


Dimension mismatch errors when running the Wasserstein attack

Describe the bug
Dimension mismatch errors occur when running the Wasserstein attack.
To Reproduce
import numpy as np
from art.estimators.classification.classifier import Classifier, ClassifierNeuralNetwork, ClassifierGradients
class DummyClassifier(ClassifierNeuralNetwork, ClassifierGradients, Classifier):
     def __init__(self):
            super(DummyClassifier, self).__init__()
     def class_gradient(self):
            return None
     def fit(self):
            return None
     def loss_gradient(self):
            return None
     def predict(self):
            return None
     def get_activations(self):
            return None
     def save(self):
            return None
     def set_learning_phase(self):
            return None
attack = Wasserstein(DummyClassifier())
# this will generate an error:
x = np.random.np.random.normal(size=(1, 32, 32, 3))
attack._unfold(x, 5, 5//2)
# this won't:
x = np.random.np.random.normal(size=(1, 3, 32, 32))
attack._unfold(x, 5, 5//2)
# but this will again:
x = np.random.np.random.normal(size=(1, 3, 33, 32))
attack._unfold(x, 5, 5//2)

The _unfold function is designed for channel_first use, so the first and second usecases are expected. The third usecase for unsquared images is unexpected and therefore will be fixed.


PIP and Shell should be working, for some reason I am getting this error now though.

The script loads all the way up to
 File "C:\Users\wyatt's\Desktop\reddit-karma-farming-bot-master\src\config\reddit\config_gen.py", line 2, in <module>     import praw
However, after that when it searches for praw, it gives the error of
ModuleNotFoundError: No module named 'praw'
After this, it then exits. Any solutions or fixes on this? I would appreciate it, thanks!

are you running the bash script in the repo? you should be running run_windows.bat and it will install all modules into a virtual env then ask for all the credentials and info

Went back through to see if I could find anything in the console that showed an error. Turns out this was towards the top. Locking [dev-packages] dependencies... Locking [packages] dependencies...            Building requirements... Resolving dependencies... Locking Failed!
This is most likely what is cause the issue.

hmmm try upgrading python - then clone a new folder and try it from scratch. i've never seen this issue before but it's likely related to the environment

Haven't hear back so i'm going to close this out. typically these are issues related to the environment and not the bot itself


"Rewrite DNS" example may mislead customer of using it

[Enter feedback here]
By default, If we use the example to rewrite an external domain name to an internal service domain name, it will cause the rewritten domain name can't be resolved.(Because of "/etc/resolv.conf" specify the DNS server to be the host configured DNS server which can't resovle k8s internal service domain name)
We'd better to put the note "# you can redirect this to a specific DNS server such as <IP_ADDRESS>, but that server must be able to resolve the rewritten domain name" to a more noticeable place.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2f95537e-770a-739c-0175-4447a995f8b2
Version Independent ID: 6c9a1132-05af-70f1-5a2b-8dad56de9cc3
Content: Customize CoreDNS for Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/coredns-custom.md
Service: container-service
GitHub Login: @jnoller
Microsoft Alias: jenoller

@superstandy
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@jnoller
Can you please check and add your comments on this doc update request as applicable.

Yo folks! I was the one who opened the ticket with Standy, and IMHO, it should mention that a common case is to use forward . <CoreDNS Service IP> which is the majority of cases on Azure.

thank you @superstandy and @galvesribeiro for the feedback.
I've updated the article to address your feedback and the changes should be available soon/=.


add “Clear All” button (medium)

@amyngai @tuckerbuchy is working on putting all the search filtering into the pills so once this is done they should be able to use the x's to remove filters. I think that's good for now and we could add a clear all in v2?

agreed ^


rank, quantile

{
  const randomNormal = d3.randomNormal.source(d3.randomLcg(42))();
  const randoms = Array.from({length: 300}, randomNormal);
  return Plot.plot({
    marks: [
      Plot.dotX(randoms, Plot.mapY("quantile", {y: randoms}))
    ]
  });
}

Cool :)
We already had
Plot.dotX(d3.sort(randoms), { y: (d, i) => i / (randoms.length - 1) }).plot()
but this is obviously nicer since it doesn't need to re-order the data (facets!), and knows what to do with NaNs.


Importing from Google Analytics doesn't work

Steps to reproduce:

Open a new clean incognito Google Chrome tab with no extensions

Go to https://caniuse.com/

Click «Settings», click «Import...» under «From Google Analytics:»

An import window appears and nothing happens further:

Only an error appears in the browser console:
Uncaught (in promise) Error: Cannot find module './retrieve_ga2'
    at webpackMissingModule (stats-import.module.js:62)

When I click «Access Google Analytics», nothing happens too.

I tried to allow pop-ups for the website and to clear cache in the browser settings, it hasn't helped. I use no VPN and no ad blocker. Chrome version: 80.0.3987.132.
The same happens in Safari, but the console messages are different:

Ugh, looks like I broke something in my last update. Should be fixed now, thanks much for reporting!

@Fyrd Thank you! It works now, I've checked in Chrome and Safari.
P.S. For googlers who experience broken import in Safari: try to change the website setting «Pop-up Windows» to «Block and Notify» and then, after you click «Import», click the pop-up icon in the browser address field.


PRelu,Slice

Thanks for your code,but my code crash when running the folowing layer:
slice,prelu,is mdl not support it now?by the way, the converting code is ok,it does not crash!

now, slice and prelu are not supported, if you are familiar with these, welcome to contribute and support.


Update docs for Vita Firmware 3.74

https://www.playstation.com/en-ca/support/hardware/psvita/system-software/

This has already been done.


Is it possible to show more than suggestion from history?

Like in title, sometimes I want to see more than one previous history entry to choose from.
Any suggestions what I need to change to achieve it?

See https://github.com/zsh-users/zsh-autosuggestions/issues/232#issuecomment-294860042
Maybe you want ctrl-r -> history-incremental-search-backward?  That is what I use when the autosuggestion isn't what I need.

I have installed .oh-my-zsh, so "ctrl-r"/"ctrl-s" functionality known from bash is working for me. But it shows only one suggestions.
Working #232 case is exactly what I'm looking for!
But partial solution proposed by OP in #232 is not working anymore. Code was refactored soon after.
Could you provide me some clue what should I change to mimic OP (partial) solution?

Alright, just ignore this if you already know: press ctrl-r again after it's showing a match, and it will keep going backwards to another match until you've reached the end.

Sorry @YASoftwareDev, but as stated in #232, the goal of this project is to emulate in zsh the Fish autosuggestions feature, which I don't think has support for multiple suggestions.


Fix failing tests

There are three issues that I fixed  :
1 : pytest rootdir
The first issue was a quick one, as the rootdir of pytest was set as manim/, the .npy files could not be found as their path was set to tests_data/etc and not tests/tests_data/etc.
I think that this was an inteniontal move by @leotrs, who wanted the tests to use a custom manim.cfg. Sadly, we can't currently use one when not rendering a scene not from command lines. See #184.
2 : skip_animation not enabled for tests : a t values issue.
The second issue the these failing tests undercovered was an issue with t values.
@leotrs changed a little bit the config that the tests use when they are running, he basically didn't add -s flag. I agree, this should not be a problem as the last frame should be the same if we skip the rendering process of the animations or not, but they are different. See #183 TLDR : This is a problem with the t values, when doing -s, there is only one t value generated by manim wich is 1.0 (the time when the animation ends), and when the skip_animations is False, there are, let's say, 15 values of t generated (as 15 fps) which go from 0 to not 1 but .93333. In other terms, the animation stops at 93%). I fixed THE TESTS  (not this issue) by simply adding to testing_utils.py file_write_config['skip_animations'] = True.
3 : frame_width slightly changed
I didn't have to fix this, I will explain.
frame_width is computed l62 config.py :
    config['frame_width'] = (config['frame_height']
                             * config['pixel_width']
                             / config['pixel_height'])

Because of how Python handles float, this value may differ depending how what is the value of  config['pixel_width'] and config['pixel_height']
If config['pixel_width'] is not set in a manim.cfg, (as it is done in the tests, see testing_utils.py, l.48), default value of pixel_width will be taken.
There is the problem,  config['frame_width'] should be the same in both cases (set in manim.cdg and not set) (as default values and values used in the tests have the same ratio),  they slightly differ (something like 14.211111111111 and 14.23333333).
That's why there is a slightly pixel related difference.
As we don't use a manim.cfg for the tests rn, it is ok. But when we will do so, we will have to render a new set of tests_data.
Issues to be fixed :

NameError: name 'FRAME_X_RADIUS' is not defined
Not sure how to fix that. The king @leotrs surely knows.

UnicodeEncodeError: 'charmap' codec can't encode character '\u2764' in position 2905: character maps to <undefined>
This has been recently added in the -h flag  : "Made with ❤ by the manim community dev".
Sadly, '\u2764' (the heart) is seems to not be suported by the decoding of subprocess.
To quicly fix this major issue, we could do the biggest change in manim's history : changing ❤ by <3.

I think I meant to reverse this change at some point but failed to. My apologies!

During the development of #98, skip_animations was definitely a sticking point. There is some logic in config.py to set it, but it's not as general as it should be. For example, it seems it's not being set correctly when ran from the testing suite. The main problem with this is that skip_animations is fixed by the values of other config options, and not by a config option itself (that is, it cannot be set with its own config option from default.cfg currently). I'm all ears for suggestions here.

Excellent catch here! What was the fix?

Standing issues.

Change every occurrence of FRAME_X_RADIUS to config['frame_x_radius']. Make sure to from manim.config import config

It originally said <3, and then someone changed it to the heart emoji/unicode. I will ping @eulertour here for no particular reason whatsoever ;)

Yes, I saw that. I would suggest adding skip_animation to the argument parser thing and doing something like this :

    config['skip_animations'] = any([config['save_last_frame'],
                                     config['from_animation_number'],
                                     default.get_boolean('skip_animations')])

I don't know if it makes sense, but it could work.

I don't think there is a fix lol. The solution would be to re-render the pre-rendered frames.

I hesitate to keep adding more CLI arguments, especially since the point is that there is already some logic for the option otherwise. Unless we add a new option that just overrides that default logic. (Man this friggin config system is so convoluted..)

No but I mean, we shouldn't have fractional frame widths anywhere, right? We have to round somewhere.

You're right, skip_animation does not need to be a CLI argument, but need IMO to be available in manim.cfg We can do this and keep the logic, I think.

Man this friggin config system is so convoluted..)

Arh, mathematicians, always complicated ..

I see you're point, but tbh I don't have an opinion on that. I think rounding could cause problems when one wants a precision, but I don't know enough about that to really say something constructive lol.
I don't even know why is frame_height set to 8 by default, as it is modified after in camera.py ..
Seems to be a little messy over there :/

Ok, let's do that. skip_animations wouldn't be the only config option that is on default.cfg but not available through CLI flags.

That's fair. Let's open another issue on this, since it's separate from "fix failing tests". Also, rounding that may lead to unexpected bugs, as you say.

Add encoding='utf-8' here to fix the encoding issue.
https://github.com/ManimCommunity/manim/blob/7a088cfd6df8438c33322f4d431cd3136ed29301/tests/test_cli/test_cli.py#L8-L11
It's a bit worrying that 1/14th of 2/100ths of a difference, which appears to within the margin of error of float operations in python, would cause a test failure. Still, can see if the tests are robust enough to handle modifications later on.

I think we have tests again ;)


Rebase

@ihmc3jn09hk Do you have a telegram account?

@an-tao Yes. Will seek for your help unless the new PR not working.


chore(cmd-api-server): fix linter warning in CreateCorsMiddleware

Description

I'd like to fix this linter warning

@bmvd70 Cheers! Assigned!


Multiple node-version-file (string[])

Description:
The idea would be to propose having an array of string when it comes to node-version-file so that it would have fallbacks if the file was not found
Justification:
Like many i'm using this action in a monorepo, where sometimes services/packages are not on par. Some have a .node-version file, some a .nvmrc file, and some a package.json file. I wished we could provide the 3 so that the first found gets picked.
Are you willing to submit a PR?
Yes, I could work on it.

I'm thinking there will be an issue with having an array/list as param with the toolkit (https://github.com/actions/toolkit/issues/184).

Hello @y-nk ,
Thank you for creating this issue, we will investigate it and come back to you as soon as we have some feedback.

hello

As far as I know, GitHub actions definitions do not allow string[] as inputs, so we'd have to serialize the list. I've posted a similar suggestion over here in issue 939.

@mk-pmb i'm good with that. i'll close this and we can continue arguing about it (while nothing is decided or done) in #939


feature: auto embed all links

Hello, thank you for the plugin. Is it possible to make it so that all links are automatically made in embed?

I didn't see such functionality in other plugins. I don't know whether it is possible to hook the action of pasting links in Obsidian. If you can provide an example, I can try to implement this feature.

Probably something like this - https://github.com/samwarnick/obsidian-simple-embeds

The plugin just changes the preview of the link. It didn't change the way Obisidian handles when the user pastes a link.

I guess you want the feature that Notion has, right?

I guess you want the feature that Notion has, right?

Yeah

In Obsidian is it possible to implement the same using this plugin or through other plugin?

I think there might be a way to implement this feature. But the thing is Obsidian didn't provide any sophisticated example to demonstrate their APIs, nor any detailed Offical document about their APIs. It's hard for me to find the right solution in this situation. If there is an example of this, it will save a lot of time.

It seems to me it is possible to implement this functionality as a markdown block. See this - https://github.com/MSzturc/obsidian-better-pdf-plugin

If you need any help (to test, for example) - just ping me

This example is just like the last one. It only changes the preview look of these links. 😂

I was checking this plugin. I don't know if EditorSuggestion is the right way to do this. I might have to check more plugins for references.
https://github.com/argenos/nldates-obsidian/blob/master/src/suggest/date-suggest.ts

Check out new 1.4.0 release

Thank you very much!

This is great! Is it possible to avoid the popup altogether and just "whenever I paste a link, default it to embedded"?

..err, actually, if you could just switch around the order of the 2 popup items, that would actually be ideal. So, the flow is like:

paste link
link-embed popup comes up
Create Embed is first selected option (instead of Dismiss)

That way, I can just press "return/enter" instead of down arrow and enter....yes, I'm lazy. 😝

I created a PR: https://github.com/Seraphli/obsidian-link-embed/pull/8


Add entry to changelist for RedirectPolicy API addition

Tracking Core API change in changelist for <IP_ADDRESS> release.

API change check
API changes are not detected in this pull request.


Naive arithmetic parser

Introduction
This PR adds a naive parser built on rust-peg and adds some demonstrative arithmetic that's evaluated through an interpreter.
Linked Issues
#4
Dependencies
Test

[ ] Tested Locally
[ ] Documented

Review

[ ] Ready for review
[ ] Ready to merge

Deployment

Still needs test coverage


add all-contributorsrc config

@all-contributors
please add @igtm  for code
please add @t-gyo  for code
please add @ymdarake  for code

@all-contributors
please add @ymdarake for code

@all-contributors
please add @t-gyo for code


Simplify errors

Some refactoring, have a look @crhym3

Thanks for refactoring errorNames! I never liked that slice at all.
LGTM

Wait...

LGTM


webrtc: support publishing with OBS and WebRTC

Codecov Report

Merging #1998 (15abf1d) into main (f1b9757) will increase coverage by 0.00%.
The diff coverage is 20.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main    #1998   +/-   ##
=======================================
  Coverage   62.06%   62.07%
=======================================
  Files         108      108
  Lines       11328    11332    +4
=======================================
+ Hits         7031     7034    +3
+ Misses       3797     3794    -3
- Partials      500      504    +4

Impacted Files
Coverage Δ

internal/core/webrtc_incoming_track.go
61.62% <20.00%> (-3.01%)
:arrow_down:

... and 6 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


How to pub/sub binary data?

The pub/sub api seems support string only.

@kozlovic is there any way to receive binary data? No matter what I do, the message received is always NULL when subscribing to a subject that broadcasts binary data.
@metalwood did you find a way to subscribe to binary data?
sorry if this is obvious, I can't seem to find any documentation or tests as examples.

@elimist You simply call natsConnection_Publish with data pointing to your byte array. You need to specify the length with parameter dataLen.
Check the doc for this API here
Let me know if you still have issues.

@kozlovic thanks for the reply. I'm not sure I understand what you mean by using natsConnection_Publish to subscribe to binary data.
I'm only using the cnats client to receive incoming data, and on the publishing side, I'm using the nats go client with default encoding (which supposedly sends byte arrays untouched).
When I telnet into my gnatsd server and subscribe to the binary channel, I can receive the binary data sent from my go client, so I don't think it's an issue with the sending side.

Then can you show code for your subscriber? That would be the easiest way to see what's wrong.

The problem can be demonstrated with the simple onMsg callback from the example code. When receiving binary data, both natsMsg_GetDataLength and natsMsg_GetData return nothing.

Which version of cnats are you using? I just tested and works fine for me (using a go client with EncodedConn to send a byte array). If the callback is invoked, I would think that Data and DataLength should return something, unless you are sending an empty array. Can you show me how you are sending (the go client side)?

For example, this works:
ec, _ := nats.NewEncodedConn(nc, "default")
defer ec.Close()

msg := []byte{1, 2, 3, 4}
ec.Publish(subj, msg)

With the above sender, my C callback could look something like:
if (natsMsg_GetDataLength(msg) != 4)
{
    printf("Wrong size\n");
    abort();
}
else
{
    char *pbytes = (char*) natsMsg_GetData(msg);
    int i, b;

    for (i=0; i<4; i++)
    {
        b = (int)(*pbytes);
        if (b != (i+1))
        {
            printf("Expected %d, got %d\n", (i+1), b);
            abort();
        }
        pbytes++;
    }
}

and it will work.

hm, that's very interesting. This is the code i've been testing with:
package main

import (
	"fmt"

	"github.com/elimist/flatbuffers-pub-test/fbs"
	flatbuffers "github.com/google/flatbuffers/go"
	"github.com/nats-io/nats"
)

func main() {
	nc, _ := nats.Connect("nats://localhost:4222")
	c, _ := nats.NewEncodedConn(nc, nats.DEFAULT_ENCODER)
	defer c.Close()

	oLat := float64(7.419505)
	oLng := float64(43.736825)

	dLat := float64(7.419758)
	dLng := float64(43.731142)

	b := flatbuffers.NewBuilder(0)

	aid := b.CreateString("test0")
	uid := b.CreateString("test1")

	fbs.LocationUpdateStart(b)
	fbs.LocationUpdateAddOrig(b, fbs.CreateCoords(b, oLat, oLng))
	fbs.LocationUpdateAddOrig(b, fbs.CreateCoords(b, dLat, dLng))
	fbs.LocationUpdateAddApptID(b, aid)
	fbs.LocationUpdateAddUserID(b, uid)
	fbs.LocationUpdateAddSpeed(b, float64(123.321))
	u := fbs.LocationUpdateEnd(b)
	b.Finish(u)
	bts := b.Bytes

	c.Publish("foo", bts)
	c.Close()
}

I'll try the above cb code and see if it works

The doc about EncodedConn that you pointed out specified this:
// DefaultEncoder implementation for EncodedConn.
// This encoder will leave []byte and string untouched, but will attempt to
// turn numbers into appropriate strings that can be decoded. It will also
// propely encoded and decode bools. If will encode a struct, but if you want
// to properly handle structures you should use JsonEncoder.

So only []byte and string are left untouched. That being said, could you print bts content before the call to publish?
fmt.Printf("bts=%v\n", bts)

Thanks!

With the above cb code, get the wrong size message. Also, I'm using the latest cnats.
When I print the byte slice, I can see that it's non-zero.
Well, since you are obviously able to receive the binary data it seems this is most likely a problem on my end and has nothing to do with cnats. I'm calling cnats from my c++ code so it might have something to do with that, I'm not sure... time to do more investigating!

Well, the code I showed was assuming I was sending a byte array of length 4. But clearly, that shows that I was able to send from EncodedConn a byte array and receive it (and verify the content) in the C client.

right, sorry I just blindly c/p'd that code into my callback hah

I hope I can get this resolved, but regardless, thanks for all your help!

I hope too. I know that the C client can receive bytes. Actually, this is the normal behavior. The String APIs are just a convenience, but really, you are sending a byte array of a given length. Sending is the same. When receiving a message on a given subject, the message has a pointer to a byte array of a given length. I have tested that you can send byte arrays from any client and receive them from the C client.
Anything I can do to help, let me know.


v2: Overtime as income

Reported by user that overtime seems to increase profit

Fixed in https://github.com/patsonluk/airline/commit/1c6f143a252cab577207fc48c2db1bcf20ba1b5b


[SourceKit] Add test case for crash triggered in swift::Identifier::isOperatorSlow()

Stack trace:
found code completion token A at offset 107
swift-ide-test: /path/to/swift/lib/AST/Identifier.cpp:63: bool swift::Identifier::isOperatorSlow() const: Assertion `res == conversionOK && "invalid UTF-8 in identifier?!"' failed.
8  swift-ide-test  0x0000000000b5b276 swift::Identifier::isOperatorSlow() const + 406
10 swift-ide-test  0x0000000000b7a164 swift::UnqualifiedLookup::UnqualifiedLookup(swift::DeclName, swift::DeclContext*, swift::LazyResolver*, bool, swift::SourceLoc, bool, bool) + 196
11 swift-ide-test  0x000000000095c6ab swift::TypeChecker::lookupUnqualified(swift::DeclContext*, swift::DeclName, swift::SourceLoc, swift::OptionSet<swift::NameLookupFlags, unsigned int>) + 187
12 swift-ide-test  0x0000000000917959 swift::TypeChecker::resolveDeclRefExpr(swift::UnresolvedDeclRefExpr*, swift::DeclContext*) + 121
14 swift-ide-test  0x0000000000ae6353 swift::Expr::walk(swift::ASTWalker&) + 19
15 swift-ide-test  0x0000000000918537 swift::TypeChecker::solveForExpression(swift::Expr*&, swift::DeclContext*, swift::Type, swift::FreeTypeVariableBinding, swift::ExprTypeCheckListener*, swift::constraints::ConstraintSystem&, llvm::SmallVectorImpl<swift::constraints::Solution>&, swift::OptionSet<swift::TypeCheckExprFlags, unsigned int>) + 119
16 swift-ide-test  0x000000000091efed swift::TypeChecker::getTypeOfExpressionWithoutApplying(swift::Expr*&, swift::DeclContext*, swift::FreeTypeVariableBinding, swift::ExprTypeCheckListener*) + 221
18 swift-ide-test  0x0000000000907c7e swift::getTypeOfCompletionContextExpr(swift::ASTContext&, swift::DeclContext*, swift::Expr*&) + 1150
20 swift-ide-test  0x0000000000865a46 swift::performDelayedParsing(swift::DeclContext*, swift::PersistentParserState&, swift::CodeCompletionCallbacksFactory*) + 230
21 swift-ide-test  0x0000000000774304 swift::CompilerInstance::performSema() + 3316
22 swift-ide-test  0x000000000071cc33 main + 35011
Stack dump:
0.	Program arguments: swift-ide-test -code-completion -code-completion-token=A -source-filename=<INPUT-FILE>
1.	While type-checking expression at [<INPUT-FILE>:2:1 - line:2:1] RangeText="Á"

This is hopefully the last SourceKit crash case in a while :-)
In order to avoid duplicates I've limited my submissions to max one crash case per unique function (swift::Identifier::isOperatorSlow() in this case). I've minimized all crash cases to the minimum length still triggering the original code path. In the case of duplicate I've submitted the shortest crash case.
Judging from code paths I've encountered while fuzzing I believe that fixing these 33 unique crash cases submitted during the last four days will increase the robustness of SourceKit in a very noticeable way.
I have one currently unsubmitted SourceKit hang which isn't submitted yet due to uncertainty about the correct RUN: line for hangs. It hogs 100 % CPU until killed. What is the appropriate RUN: line for such a case?

I don't think we want to add a test that hangs (until it has been fixed of course).  Could you file a bug for the hang and attach your test case?

Thanks for all the great tests!  I've merged all the most recent ones. In the future if you're adding a large series of related test cases like this, feel free to combine into a single pull request.  I ended up reviewing these by fetching all of the pull requests into git and reading them all at once anyway.


chore: remove style setting on node warning

Why does this PR exist?
N/A - discovered while testing #2991
Removes warning messages when applying styling to nodes:

Calling setBoundVariable with a variable id is deprecated. Please pass the variable node instead.

What does this pull request do?
Passes an instance of the variable instead of a variable id to the setBoundVariable() method.
See Figma Plugin API update docs: https://www.figma.com/plugin-docs/updates/page/2/#methodproperty-usage-changes
Testing this change

Open the plugin
Apply any styling to nodes
See there are no warnings on the console!

@cuserox tests are failing here - i guess its just expecting a different value to be called with

@cuserox tests are failing here - i guess its just expecting a different value to be called with

My bad 🙈 Fixed in this commit


Error when going through training tutorial

Hi!
I have installed the new version with Pytorch and was trying to go through the training demo that you did on Colab (https://colab.research.google.com/drive/1AJUcw5dEpXeDPHd0sotAz5lpWedFYSIL#sandboxMode=true&scrollTo=uraEWunn9VSz). I copied and pasted the code in my editor, and am training on a GPU. I get this error:
IndexError: tensors used as indices must be long, byte or bool tensors
which is raised when trainer.fit() is called. It seems to be a problem in how the files are passed? But I didn't change anything to your code, so it seems really bizarre.
I'm on Windows 10. Thank you in advance for your help!
ps: I was able to run code with the previous version fine. The new pytorch version has been installed in a separate environment

Thanks, I’ll try to reproduce today.
On Tue, Jun 15, 2021 at 6:59 AM SailorUrano @.***>
wrote:

Hi!
I have installed the new version with Pytorch and was trying to go through
the training demo that you did on Colab (
https://colab.research.google.com/drive/1AJUcw5dEpXeDPHd0sotAz5lpWedFYSIL#sandboxMode=true&scrollTo=uraEWunn9VSz).
I copied and pasted the code in my editor, and am training on a GPU. I get
this error:
IndexError: tensors used as indices must be long, byte or bool tensors
which is raised when trainer.fit() is called. It seems to be a problem in
how the files are passed? But I didn't change anything to your code, so it
seems really bizarre.
I'm on Windows 10. Thank you in advance for your help!
ps: I was able to run code with the previous version fine. The new pytorch
version has been installed in a separate environment
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/weecology/DeepForest/issues/201, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHBLB23HB23I4AYVVKIULTS5MC3ANCNFSM46XKWMHA
.

--
Ben Weinstein, Ph.D.
Postdoctoral Fellow
University of Florida
http://benweinstein.weebly.com/

Thank you very much for the quick response! So I got my mentor to do the same thing on her machine, and the code was running fine for her, but she's on Mac (she's training on a CPU). I get the error whether I train on the CPU or the GPU. So the problem seems to be about me using Windows? Do you have any idea how I could fix it?

I cleaned up the tutorial, in colab there need to upgrade. package in the env. Adding to docs now.
https://colab.research.google.com/drive/1gKUiocwfCvcvVfiKzAaf6voiUVL2KK_r?usp=sharing
I'm unclear from your message if your error was within collab or locally on a windows computer. If locally, can you paste the entire code to make it reproducible, not just the error.

Sorry if it was unclear, I meant I'm running it on my personal computer, which has Windows. I had to put the whole tutorial within a main2() in order to use if name == 'main': . Not doing that raised another error which seemed to refer to issues with parallel computation. The stuff within main2() should (hopefully!) be identical to the Colab demo. This is the code:
def main2():
#load the modules
import os
import time
import numpy as np
from deepforest import main
from deepforest import get_data
from deepforest import utilities
from deepforest import preprocess
#convert hand annotations from xml into retinanet format
#The get_data function is only needed when fetching sample package data
YELL_xml = get_data("2019_YELL_2_528000_4978000_image_crop2.xml")
annotation = utilities.xml_to_annotations(YELL_xml)
print(annotation.head())
#load the image file corresponding to the annotaion file
YELL_train = get_data("2019_YELL_2_528000_4978000_image_crop2.png")
image_path = os.path.dirname(YELL_train)
#Write converted dataframe to file. Saved alongside the images
annotation.to_csv(os.path.join(image_path,"train_example.csv"), index=False)

#Find annotation path
annotation_path = os.path.join(image_path,"train_example.csv")
#crop images will save in a newly created directory
#os.mkdir(os.getcwd(),'train_data_folder')
crop_dir = os.path.join(os.getcwd(),'train_data_folder')
train_annotations= preprocess.split_raster(path_to_raster=YELL_train,
                                annotations_file=annotation_path,
                                base_dir=crop_dir,
                                patch_size=400,
                                patch_overlap=0.05)

#Split image crops into training and test. Normally these would be different tiles! Just as an example.
image_paths = train_annotations.image_path.unique()
#split 25% validation annotation
valid_paths = np.random.choice(image_paths, int(len(image_paths)*0.25) )
valid_annotations = train_annotations.loc[train_annotations.image_path.isin(valid_paths)]
train_annotations = train_annotations.loc[~train_annotations.image_path.isin(valid_paths)]

#View output
print(train_annotations.head())
print("There are {} training crown annotations".format(train_annotations.shape[0]))
#print("There are {} training crown annotations".format(train_annotations.shape[1]))
print("There are {} test crown annotations".format(valid_annotations.shape[0]))

#save to file and create the file dir
annotations_file= os.path.join(crop_dir,"train.csv")
validation_file= os.path.join(crop_dir,"valid.csv")
#Write window annotations file without a header row, same location as the "base_dir" above.
train_annotations.to_csv(annotations_file,index=False)
valid_annotations.to_csv(validation_file,index=False)

#initial the model and change the corresponding config file
m = main.deepforest()
m.config['gpus'] = '-1' #move to GPU and use all the GPU resources
m.config["train"]["csv_file"] = annotations_file
m.config["train"]["root_dir"] = os.path.dirname(annotations_file)
m.config["score_thresh"] = 0.4
m.config["train"]['epochs'] = 2
m.config["validation"]["csv_file"] = validation_file
m.config["validation"]["root_dir"] = os.path.dirname(validation_file)
#create a pytorch lighting trainer used to training
m.create_trainer()
#m.config["train"]["fast_dev_run"] = True
#load the lastest release model
m.use_release()

start_time = time.time()
m.trainer.fit(m)
print(f"--- Training on GPU: {(time.time() - start_time):.2f} seconds ---")

#save the prediction result to a prediction folder
save_dir = os.path.join(os.getcwd(),'pred_result')
os.mkdir(save_dir)
results = m.evaluate(annotations_file, os.path.dirname(annotations_file), iou_threshold = 0.4, show_plot = False, savedir= save_dir)

if name == 'main':
main2()

Hi again!
I was wondering if you perhaps had the time to test my problem on a Windows machine? I'm still unable to train locally on my laptop, I can only do things on Colab (where things work fine). I have tried to look into the issue myself to see if I could fix it, but I wasn't very successful :/ Thank you in advance for your help!

Hey
Similar issue on my end, downloaded the repo and tried to run the tests, any of the tests that involve a training step are failing due to the same "IndexError: tensors used as indices must be long, byte or bool tensors"
The offending line of code comes from retinanet.py
line 117:
# create the target classification
gt_classes_target = torch.zeros_like(cls_logits_per_image)
gt_classes_target[
    foreground_idxs_per_image,
    targets_per_image['labels'][matched_idxs_per_image[foreground_idxs_per_image]]
] = 1.0

the reason this is failing is because targets_per_image['labels'] is of type torch.int32, and indexing tensors needs to be "long, byte or bool", i.e. torch.int64
The cause of this being the wrong int type seems to originate from here:
dataset.py, line 81:
# Labels need to be encoded
targets["labels"] = image_annotations.label.apply(
    lambda x: self.label_dict[x]).values.astype(int)

while it should be:
# Labels need to be encoded
targets["labels"] = image_annotations.label.apply(
    lambda x: self.label_dict[x]).values.astype(np.int64)

Seeing as this wasn't an issue on some peoples machines, perhaps it has something to do with whether running python 32 or 64 bit? and what the int defaults to.
But we should be explicitly setting the correct bit size here

Thanks, i'm trying to decide if there is any downside here.

Closed from the PR, new pypi version has better windows support, thanks @ElliotSalisbury


changes to accommodate old cogshop model and improved scenario concept

Description

PreProcessor now takes in a Scenario + start/endtime to produce a Case
Scenario(s) are now linked to the bidmethod for a shop based process
Added names to more of the config types (for resync purposed)
Removed alerts on PartialInput function

Checklist:

[ ] Tests added/updated.
[ ] Documentation updated.
[ ] Changelog updated in CHANGELOG.md.
[ ] Version bumped. If triggering a new release is desired, bump the version number in pyproject.toml per semantic versioning.

Output from resync init

@Juliamg Can't see the picture you posted with the resync output. Can you post it again? And then I can review

Can you see it now?

Having some issues with the ShopResult view - instances are not written correctly. Might be an issue with the filters


Add collector attribute and visibility reporting

This audit cookbook PR adds a new method for collecting inspec reports via data_collector(Chef Visibility)
After the code review, these are the remaining tasks

add unit tests for the classes
document the new 'collector' attribute

breaking change for those using the compliance token path since the 'collector' attribute defaults to 'chef-server'

need to test the other two collecting methods, chef-server and chef-compliance

This work was done by @alexpop

Current enriched data added to the report:
          "event_action": "exec",
          "event_type": "inspec",
          "compliance_summary": {
            "total": 11,
            "end_time": "2016-07-22T19:14:08+01:00",
            "node_name": "chef-client.solo",
            "passed": {
              "total": 3
            },
            "failed": {
              "total": 6,
              "minor": 3,
              "major": 2,
              "critical": 1
            },
            "skipped": {
              "total": 2
            },
            "status": "failed",
            "duration": 0.063081,
            "inspec_version": "0.28.0"
          }

When multiple profiles are executed, these only one report that aggregates profile results and summary information.
Hashes like profiles and controls that have user definable keys are also converted into arrays of hashes.

@alexpop Kudos for this huge change!!
@chris-rock @alexpop just one more thing in this review: could we extract the blob from the actual test:
https://github.com/chef-cookbooks/audit/pull/72/files#diff-9a9a9e0554d7eea4f1d1744ab333cfdeR27
I had a hard time reading the actual test with the huge blob in the middle ;)
In either case, this is :+1: from my side.

Appreciate the valid feedback Dom, I pushed a new commit.

Thanks @alexpop


Cannot setup Unity Container 5 (an issue with Unity.Configuration 5.11.1)

I'm trying to setup a simple .NET Core 3.1 project that uses Unity Container 5 (https://github.com/unitycontainer/unity)
I've added a reference to latest Unity Configuration 5.11.1 package (https://www.nuget.org/packages/Unity.Configuration/).
Then I've created interface, implementation and test app with a simpliest config file:
    <?xml version="1.0" encoding="utf-8"?>
    <configuration>
      <configSections>
        <section name="unity" type="Microsoft.Practices.Unity.Configuration.UnityConfigurationSection, Unity.Configuration" />
      </configSections>
      <unity xmlns="http://schemas.microsoft.com/practices/2010/unity">
        <container>
          <register
            type="Interfaces.InterfaceN, Interfaces"
            mapTo="Implementations.ImplementationN, Implementations">
          </register>
        </container>
      </unity>
    </configuration>

But i'm getting System.InvalidOperationException:

{"The type name or alias Implementations.ImplementationN,
Implementations could not be resolved. Please check your configuration
file and verify this type name."}

I've uploaded my code to GitHub.
Same code compiled for .NET Framework 4.8 works fine - GitHub
Any help is appreciated.

From Autofac IoC examples repository:
// THIS IS THE MAGIC!
// .NET Core assembly loading is confusing. Things that happen to be in your bin folder don't just suddenly
// qualify with the assembly loader. If the assembly isn't specifically referenced by your app, you need to
// tell .NET Core where to get it EVEN IF IT'S IN YOUR BIN FOLDER.
// https://stackoverflow.com/questions/43918837/net-core-1-1-type-gettype-from-external-assembly-returns-null
//
// The documentation says that any .dll in the application base folder should work, but that doesn't seem
// to be entirely true. You always have to set up additional handlers if you AREN'T referencing the plugin assembly.
// https://github.com/dotnet/core-setup/blob/master/Documentation/design-docs/corehost.md
//
// To verify, try commenting this out and you'll see that the config system can't load the external plugin type.
var executionFolder = Path.GetDirectoryName(typeof(Program).Assembly.Location);
AssemblyLoadContext.Default.Resolving += (AssemblyLoadContext context, AssemblyName assembly) =>
{
     // DISCLAIMER: NO PROMISES THIS IS SECURE. You may or may not want this strategy. It's up to
     // you to determine if allowing any assembly in the directory to be loaded is acceptable. This
     // is for demo purposes only.
     return context.LoadFromAssemblyPath(Path.Combine(executionFolder, $"{assembly.Name}.dll"));
 };

And after that the following code
var container = new UnityContainer().LoadConfiguration();

var type = container.Resolve<InterfaceN>();

is working like a charm.


Update port_scan engine.py #83

Describe your changes or your new module and let us know which OS and Python version you tested on.

OS: Linux
OS Version: Ubuntu
Python Version: 2.7.3

Hello,
Sorry I did not see this PR before the last commit, Merged.
Thank you.

No issues I was just going to close the PR after reviewing your changes.
Best Regards
Pradeep Jairamani


Script events when replaying the chain without validation

When replaying the chain without validation, scripts do not get evaluated and therefore script events are not emitted.
It would be helpful, however, for folks to have the ability to turn on all events regardless of the validation flag. See #3052

Where is the validation flag? I wonder if this is something that can be set when constructing ChainSyncClient - if so that's probably all we need?

This is the main ledger API function, the one that applies a block:
https://github.com/input-output-hk/cardano-ledger/blob/db8bedc57b8e55857271a8449c9b6231f0346e5a/eras/shelley/impl/src/Cardano/Ledger/Shelley/API/Validation.hs#L87-L95
The ApplySTSOpts allows for some configuration:
https://github.com/input-output-hk/cardano-ledger/blob/db8bedc57b8e55857271a8449c9b6231f0346e5a/libs/small-steps/src/Control/State/Transition/Extended.hs#L453-L462
and in particular, the validation policy is set by:
https://github.com/input-output-hk/cardano-ledger/blob/db8bedc57b8e55857271a8449c9b6231f0346e5a/libs/small-steps/src/Control/State/Transition/Extended.hs#L447-L451
Indeed I think you just need to use ValidateAll with chain sync (I forget if there is a Label that would work for script execution, so that you could use ValidateSuchThat and speed up the sync).

Thanks for the pointer. I'll look into it.

@JaredCorduan @TimSheard It turns out I simply need to use Cardano.Api.FullValidation instead of QuickValidation for the Plutus events to show up. I think this issue can be closed but you may want to expand the documentation on ValidationPolicy, especially since it didn't behave this way not long ago.


Adding support for management operation of webjobs

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

Will add the tests for these.

Merge once CI passes

Will take a look in a bit & fix CI


Update BingVoiceOutput.md for wrong text of Service name mapping

Motivations:

Can not play voice with the Korean of Service name mapping in [Cognitive-Speech-TTS](sample link : https://github.com/Azure-Samples/Cognitive-Speech-TTS) Android sample

Modifications:

Add space to the string for the Korean of Service name mapping

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@kimkevin : Thanks for your contribution to the Azure documentation! The author, @priyaravi20, has been notified to review your proposed change.

@kimkevin : Thanks for your contribution to the Azure documentation! The author, @priyaravi20, has been notified to review your proposed change.

#sign-off


Moving cursor one line down after toggling line comment

This is feature request.
It would be nice if one could configure cursor to move one line down after Toggle Line Comment action  (editor.action.commentLine).
Not all people might have such preference of course, so this should be configurable.

please, adding this feature wd be very much appreciated.


⚠ Bofrost.at has degraded performance

In a40b5a7, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1824 ms

Resolved: Bofrost.at performance has improved in 6b4d3db after 16 minutes.


Add a signup and login page to your project

I will add a sign up and login page for the website matching the color theme and fonts. I will also try to embed some cool animations.
Checklist
✔I have read the Contributing Guidelines
✔I have read the Code Of Conduct
✔The changes don't break the code
Assign the task to me @SaranshBangar

@manya1632 Alright, you can start working on it, but share some pictures of the work that you will do before raising a PR.
Moreover, make sure to follow the proper file structure

Sir, there will be some delay in completing this task due to my exams. So, I am expecting a delay of about 8 days . I want to do this task but if there's any urgency feel free to assign this task to someone else.
Thank you.

@manya1632

No worries, please take your time
No need for 'sir' please, it makes me feel old 😅

Haha, Thanks !

I am really sorry @iamsohelx, this issue is already assigned, you can take over some other issue or raise an issue of your own

@SaranshBangar No problem brother.

@SaranshBangar can i work on this issue. Will do it by tomorrow.

I am really sorry @aasiflm10, this issue is already assigned, you can take over some other issue or raise an issue of your own

No worries @SaranshBangar . If for some reason the asignee is not able to complete this issue, i would like to work on it.

No worries @SaranshBangar . If for some reason the asignee is not able to complete this issue, i would like to work on it.

Sure 👍🏼

@SaranshBangar is the issue is still open to work on

@SaranshBangar if this issue is still open I would like to work on this and would complete it by today.

working Sir
give me a day or 2

should i proceed with the task ?

Is this issue still available @SaranshBangar ? I can work on it.

The frontend is ready. Backend will be implemented in a day

@SaranshBangar should i just work more on the frontend ( add more pages to get details of the user and authenticate it ) or implement the backend only  for signup and  login page?

I am facing some issue in developing the backend for this particular part, so i'll complete the frontend make a PR .

I have completed my task and raised a PR. Please look at PR #166  !! Let me know if any changes are to be done . Thank You.

Sir please look at the PR and approve further actions

Approved #166