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

[webgpu-native] Add where op #22014

Merged
merged 4 commits into from
Sep 20, 2024

Conversation

axinging
Copy link
Contributor

@axinging axinging commented Sep 6, 2024

Two failed cases caused by int64:
onnxruntime\js\test\data\node\opset11\test_where_long_example
onnxruntime\js\test\data\node\opset17\test_where_long_example

@axinging axinging force-pushed the where_webgpunative branch 6 times, most recently from 2ff6754 to f85b60f Compare September 13, 2024 01:33
@axinging axinging marked this pull request as ready for review September 13, 2024 01:35
@fs-eire fs-eire merged commit dfab322 into microsoft:fs-eire/webgpu-ep Sep 20, 2024
11 checks passed
@fs-eire
Copy link
Contributor

fs-eire commented Sep 20, 2024

Two failed cases caused by int64:
onnxruntime\js\test\data\node\opset11\test_where_long_example
onnxruntime\js\test\data\node\opset17\test_where_long_example

Should be OK for now. currently JSEP also does not register Where op for (u)int64. we can always come back to take a look for this when we see a model uses it.

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.

2 participants