intel / webml-polyfill

Deprecated, the Web Neural Network Polyfill project has been moved to https://github.com/webmachinelearning/webnn-polyfill
Apache License 2.0
161 stars 42 forks source link

[Doc] Update native mapping doc for NNAPI and ONNX #1139

Closed Christywl closed 4 years ago

Christywl commented 4 years ago

https://github.com/intel/webml-polyfill/blob/master/docs/native_mapping.md

Need add the following ops into native mapping doc:

mingmingtasd commented 4 years ago

https://docs.google.com/spreadsheets/d/1PoS_lhr7wsO-jW0FYM9ecvAADPMfXxZJFu5oaIJOwCs/edit?usp=sharing I have updated mapping doc for NNAPI and ONNX, please help to review these highlighted contents, thanks! @huningxin

huningxin commented 4 years ago

Thanks @mingmingtasd . For argument mapping, please keep it simple. It is just needed to list the parameter name in native API with necessary information. It isn't needed to copy and paste all content from native API documentation. Please refer to previous ones as example.

mingmingtasd commented 4 years ago

Thanks @mingmingtasd . For argument mapping, please keep it simple. It is just needed to list the parameter name in native API with necessary information. It isn't needed to copy and paste all content from native API documentation. Please refer to previous ones as example.

I have updated this doc as you suggested. Thanks! @huningxin

huningxin commented 4 years ago

Thanks @mingmingtasd ! NNAPI column looks good and I merged it.

For ONNX part, we may need to remove that column as it is not used for implementation. @belem , any thoughts?

mingmingtasd commented 4 years ago

Merged, so close this issue.