onnxruntime icon indicating copy to clipboard operation
onnxruntime copied to clipboard
verified publisher icon microsoft

update for ORT Web build guide

Open shalvamist opened this issue 3 years ago • 5 comments

I reordered a few things and added some more context. I think we can also add more information on converting ONNX to ORT format fairly easily. maybe in the doc update next iteration :)

Description: Describe your changes.

Motivation and Context

  • Why is this change required? What problem does it solve?
  • If it fixes an open issue, please link to the issue here.

shalvamist avatar Jul 20 '22 05:07 shalvamist

Could you share a link to the website for us to review?

fs-eire avatar Jul 26 '22 23:07 fs-eire

The new Web page is ready - https://shalvamist.github.io/onnxruntime/ Please let me know if you see any new issues

shalvamist avatar Aug 02 '22 19:08 shalvamist

Thank you @shalvamist. I will take a look

natke avatar Aug 04 '22 20:08 natke

The following is a screenshot of the section "Build Instructions" in the generated page from https://shalvamist.github.io/onnxruntime/docs/build/web.html

image

the section "Install NPM packages" and "Prepare ONNX Runtime WebAssembly Artifacts" are not obvious enough for users to recognize they are sections. and there are no indicators that shows they are step-1, step-2 and so on.

fs-eire avatar Aug 04 '22 22:08 fs-eire

Updated the docs with the latest comments - https://shalvamist.github.io/onnxruntime/docs/build/web.html

shalvamist avatar Aug 09 '22 23:08 shalvamist