Skip to content

Commit 18f4cf8

Browse files
luci-bisection@appspot.gserviceaccount.commoz-wptsync-bot
luci-bisection@appspot.gserviceaccount.com
authored andcommitted
Bug 1895867 [wpt PR 46183] - Revert "webnn: Enforce input data type constraints for gather indices", a=testonly
Automatic update from web-platform-tests Revert "webnn: Enforce input data type constraints for gather indices" This reverts commit ef29e11bfb1f1f4eae0935fb58f0e4129620837b. Reason for revert: LUCI Bisection has identified this change as the cause of a test failure. See the analysis: https://ci.chromium.org/ui/p/chromium/bisection/test-analysis/b/5728899092709376 Sample build with failed test: https://ci.chromium.org/b/8748452022241283393 Affected test(s): [ninja://services:services_unittests/WebNNGraphImplBackendTest.BuildAndComputeSingleOperatorGather](https://ci.chromium.org/ui/test/chromium/ninja:%2F%2Fservices:services_unittests%2FWebNNGraphImplBackendTest.BuildAndComputeSingleOperatorGather?q=VHash%3A3db3852ce13cc70a) If this is a false positive, please report it at http://b.corp.google.com/createIssue?component=1199205&description=Analysis%3A+https%3A%2F%2Fci.chromium.org%2Fui%2Fp%2Fchromium%2Fbisection%2Ftest-analysis%2Fb%2F5728899092709376&format=PLAIN&priority=P3&title=Wrongly+blamed+https%3A%2F%2Fchromium-review.googlesource.com%2Fc%2Fchromium%2Fsrc%2F%2B%2F5520313&type=BUG Original change's description: > webnn: Enforce input data type constraints for gather indices > > As specified in webmachinelearning/webnn#646 > > Bug: 328567884 > Change-Id: I33eba7e1def430b1cb94e3e7a4868e82c5bbd9a3 > Cq-Include-Trybots: luci.chromium.try​:mac14-blink-rel,mac14.arm64-blink-rel > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5520313 > Reviewed-by: Austin Sullivan <asully@chromium.org> > Commit-Queue: Lisha Guo <lisha.guo@intel.com> > Reviewed-by: ningxin hu <ningxin.hu@intel.com> > Cr-Commit-Position: refs/heads/main@{#1298443} > Bug: 328567884 Change-Id: Iab874e915f5603861929cdf07ba4c09698b76503 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5528819 Reviewed-by: Austin Sullivan <asully@chromium.org> Reviewed-by: Phillis Tang <phillis@chromium.org> Commit-Queue: Austin Sullivan <asully@chromium.org> Cr-Commit-Position: refs/heads/main@{#1298718} -- wpt-commits: 61d40184e4b3d7aa0cca8c16cf3488e248c3ff04 wpt-pr: 46183
1 parent cbb3dfc commit 18f4cf8

File tree

1 file changed

+4
-12
lines changed

1 file changed

+4
-12
lines changed

testing/web-platform/tests/webnn/validation_tests/gather.https.any.js

+4-12
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ const tests = [
88
{
99
name: '[gather] Test gather with default options and 0-D indices',
1010
input: {dataType: 'int32', dimensions: [3]},
11-
indices: {dataType: 'int64', dimensions: []},
11+
indices: {dataType: 'uint64', dimensions: []},
1212
output: {dataType: 'int32', dimensions: []}
1313
},
1414
{
@@ -24,23 +24,15 @@ const tests = [
2424
indices: {dataType: 'int64', dimensions: [1]}
2525
},
2626
{
27-
name:
28-
'[gather] TypeError is expected if the axis is greater than the rank of input',
27+
name: '[gather] TypeError is expected if the axis is greater than the rank of input',
2928
input: {dataType: 'float16', dimensions: [1, 2, 3]},
30-
indices: {dataType: 'uint32', dimensions: [5, 6]},
29+
indices: {dataType: 'int32', dimensions: [5, 6]},
3130
axis: 4
3231
},
3332
{
34-
name:
35-
'[gather] TypeError is expected if the data type of indices is float32 which is invalid',
33+
name: '[gather] TypeError is expected if the data type of indices is invalid',
3634
input: {dataType: 'float16', dimensions: [1, 2, 3, 4]},
3735
indices: {dataType: 'float32', dimensions: [5, 6]}
38-
},
39-
{
40-
name:
41-
'[gather] TypeError is expected if the data type of indices is int32 which is invalid',
42-
input: {dataType: 'float16', dimensions: [1, 2, 3, 4]},
43-
indices: {dataType: 'int32', dimensions: [5, 6]}
4436
}
4537
];
4638

0 commit comments

Comments
 (0)