Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[js/web] fix output type mapping #11049

Merged
merged 2 commits into from
Mar 30, 2022
Merged

[js/web] fix output type mapping #11049

merged 2 commits into from
Mar 30, 2022

Conversation

fs-eire
Copy link
Contributor

@fs-eire fs-eire commented Mar 30, 2022

Description: fix incorrect mapping for output type.

@fs-eire fs-eire merged commit 6c7090a into master Mar 30, 2022
@fs-eire fs-eire deleted the fs-eire/fix-output-type branch March 30, 2022 23:26
chilo-ms pushed a commit that referenced this pull request Apr 19, 2022
chilo-ms added a commit that referenced this pull request Apr 19, 2022
* Update tools/ci_build/upload_python_package_to_azure_storage.py to not use the azure blob storage python package (#11114)

* Fix the rocm packaging pipeline package upload problem (#11174)

In #11114 , I changed the script to use azcopy instead of azure blob storage's python APIs. However, it doesn't work for the AMD rocm pipeline, because:

1. The machines do not have azcopy installed
2. The machines are not in Azure, so they don't have Azure managed identity. So they still need to use SAS.

Therefore in this PR I get the old python file back, but only use it in the AMD pipeline.

* Scoped GIL release in run_with_iobinding (#11248)

* [js/web] disable test_tan temorarily (#11048)

* [js/web] fix output type mapping (#11049)

Co-authored-by: Changming Sun <chasun@microsoft.com>
Co-authored-by: Dmitri Smirnov <yuslepukhin@users.noreply.github.com>
Co-authored-by: Yulong Wang <7679871+fs-eire@users.noreply.github.com>
siweic0 pushed a commit to siweic0/onnxruntime-web that referenced this pull request May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants