Skip to content

[Add] add Rocm ep doc - #12971

Merged
Faith Xu (faxu) merged 33 commits into
microsoft:gh-pagesfrom
ytaous:rocm-ep
Sep 20, 2022
Merged

[Add] add Rocm ep doc#12971
Faith Xu (faxu) merged 33 commits into
microsoft:gh-pagesfrom
ytaous:rocm-ep

Conversation

@PeixuanZuo

@PeixuanZuo PeixuanZuo commented Sep 15, 2022

Copy link
Copy Markdown
Contributor

Description: Describe your changes.
add rocm ep doc.
Perview Github page : https://ytaous.github.io/onnxruntime/

Added items
Home page

  • Update matrix on home page: https://ytaous.github.io/onnxruntime/
  • Link "Installation Instruction Follow build instructions from here" to onnxruntime build with ROCm(inference) page.(This link now point to preview page, will change later)

Installation

Build onnxruntime

ExecutionProvider

Performance

Comment thread docs/execution-providers/ROCm-ExecutionProvider.md Outdated
Comment thread docs/execution-providers/index.md Outdated
Comment thread docs/performance/graph-optimizations.md Outdated
Comment thread css/custom.css Outdated
Comment thread docs/build/training.md Outdated
Comment thread docs/execution-providers/ROCm-ExecutionProvider.md Outdated
Comment thread docs/execution-providers/ROCm-ExecutionProvider.md Outdated
Comment thread docs/execution-providers/ROCm-ExecutionProvider.md
Comment thread docs/execution-providers/index.md Outdated
Comment thread docs/execution-providers/ROCm-ExecutionProvider.md Outdated

@ytaous ytaous left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM - perhaps address the last comment and wait for approval from others before merge.
Thanks.

@faxu

Copy link
Copy Markdown
Contributor

Thanks for the fixes. Let me know when it's ready to merge.

@ytaous

ytaous commented Sep 20, 2022

Copy link
Copy Markdown
Contributor

Thanks for the fixes. Let me know when it's ready to merge.

Hi, Faith Xu (@faxu) - can u pls merge it now, thx a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants