webml-polyfill icon indicating copy to clipboard operation
webml-polyfill copied to clipboard

[NNAPI] Some fully connected tests failed after installing the latest software update on Android 10

Open Christywl opened this issue 5 years ago • 2 comments

Test Env: Chromium Version: nightly build 79.0.3917.0(d370896) Platform: Android 10

Expected Result: Tests should pass

Actual Result: Some fully connected tests failed after installing the latest software update on Android 10: image

Installed the latest software update on Pixel 3 and samsung Galaxy Note10+, this issue happened. Before software update, all fully connected test cases passed on Android 10.

How to Reproduce: Visit: https://brucedai.github.io/webnnt/test/issue_fully_connected_4.html?prefer=sustained

Christywl avatar Apr 15 '20 02:04 Christywl

After referring to the codes from android/platform , I think we should modify model.identifyInputsAndOutputs([op1], [op3]) to model.identifyInputsAndOutputs([op1,op2,b0], [op3]). And we'd better use

execution.setInput(1, op2_input);
execution.setInput(2, b0_input);

not

 model.setOperandValue(op2, new Float32Array([1, 0, 1, 1, 0, 0, 0, 1, 0, 0, 1, 1]));
 model.setOperandValue(b0, new Float32Array([0, 4]));

And I tried to do so to get correct result.

mingmingtasd avatar Apr 16 '20 08:04 mingmingtasd

@mingmingtasd I think this isn't real root cause, we may investigate the updated commits of NNAPI to find root cause.

BruceDai avatar Apr 16 '20 10:04 BruceDai