Skip to content

Conversation

@BruceDai
Copy link
Contributor

@@ -0,0 +1,272 @@
{
"tests": [
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about one test case coming from or going to 1D? e.g. "newShape": [24],

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And/or "newShape: [-1] which would consolidate all elements into a single 1D tensor. I don't really care which, but I think we should have at least one 1D test case.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I've added such tests. Please take another look.

@BruceDai BruceDai force-pushed the add_webnn_reshape_tests branch from 29f21e0 to 481a16f Compare January 6, 2023 06:43
@BruceDai
Copy link
Contributor Author

BruceDai commented Jan 6, 2023

@Honry All checks passed, PTAL, thanks.

Copy link
Contributor

@Honry Honry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@Honry Honry merged commit d0bc4f3 into web-platform-tests:master Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants