-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Implement version selector install guides #18835
base: master
Are you sure you want to change the base?
Conversation
Hey @ys2843 , Thanks for submitting the PR
CI supported jobs: [centos-gpu, unix-cpu, edge, windows-gpu, windows-cpu, miscellaneous, sanity, clang, centos-cpu, unix-gpu, website] Note: |
@mxnet-label-bot remove [pr-awaiting-review] |
@mxnet-bot run ci [unix-cpu, windows-gpu] |
Jenkins CI successfully triggered : [unix-cpu, windows-gpu] |
7c0c47d
to
1ccfb68
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @ys2843. Let's remove the windows docker instructions, as they only exist in the codebase due to a symlink but are not supported and shouldn't be shown on the website.
I think the symlink would be removed as part of adding platform and version specific build from source instructions in #18666
Thanks @leezu good catch! |
@leezu Just to clarify, since we are going to have a unified installation widget that covers all versions. In which version we don't support |
7f9ad8c
to
71b4186
Compare
8c360cd
to
12dd397
Compare
Description
fix #18833 and #18666
This PR includes the changes to create a comprehensive installation widget that covers all the versions.
Changes
Comments