Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix get_started pip instructions #17725

Merged
merged 1 commit into from
Feb 29, 2020
Merged

Fix get_started pip instructions #17725

merged 1 commit into from
Feb 29, 2020

Conversation

leezu
Copy link
Contributor

@leezu leezu commented Feb 29, 2020

The class attribute in the respective .md files must match one of the versions
listed in the version dropdown specified in get_started.html

Without this fix, the instructions for installing version 1.5 will always be shown, even if users select another version in the dropdown.

CC @TaoLv, as this error was introduced in #16442.

The class attribute in the respective .md files must match one of the versions
listed in the version dropdown specified in get_started.html
Copy link
Member

@TaoLv TaoLv left a comment

Choose a reason for hiding this comment

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

Thank you for the fix, @leezu. Is it possible to pre-view the website with this PR?

@@ -70,4 +70,4 @@ $ pip install --pre mxnet -f https://dist.mxnet.io/python/all
</div> <!-- End of master-->

{% include /get_started/pip_snippet.md %}
{% include /get_started/gpu_snippet.md %}
Copy link
Contributor

Choose a reason for hiding this comment

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

what has changed in all these gpu_snippets?

Copy link
Member

Choose a reason for hiding this comment

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

I bet it is eol character - those md files had no eol before

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. My editor ensures each file has eol by default

@leezu leezu merged commit 10a12d5 into apache:master Feb 29, 2020
@leezu leezu deleted the fixgetstarted branch February 29, 2020 21:09
MoisesHer pushed a commit to MoisesHer/incubator-mxnet that referenced this pull request Apr 10, 2020
The class attribute in the respective .md files should match one of the versions
listed in the version dropdown specified in get_started.html
anirudh2290 pushed a commit to anirudh2290/mxnet that referenced this pull request May 29, 2020
The class attribute in the respective .md files should match one of the versions
listed in the version dropdown specified in get_started.html
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants