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

Incorrect installation instruction #17244

Closed
eric-haibin-lin opened this issue Jan 7, 2020 · 7 comments · Fixed by #18587
Closed

Incorrect installation instruction #17244

eric-haibin-lin opened this issue Jan 7, 2020 · 7 comments · Fixed by #18587

Comments

@eric-haibin-lin
Copy link
Member

The instructions for installing mxnet master branch is incorrect.
https://mxnet.apache.org/get_started?version=master&platform=linux&language=python&environ=pip&processor=cpu

Screen Shot 2020-01-07 at 3 38 52 PM

It should only show the ones with --pre suffix

@aaronmarkham

@leezu
Copy link
Contributor

leezu commented Jan 8, 2020

How did you arrive at the master version URL? It's not linked from the https://mxnet.apache.org/get_started page

@eric-haibin-lin
Copy link
Member Author

There is a drop-down list for "MXNet version", and i selected "Master"

@leezu
Copy link
Contributor

leezu commented Jan 8, 2020

Thanks. I didn't realize that this dropdown list exists. There seems to be no UI element indicating the existence of the dropdown prior to hovering over the version number.

@aaronmarkham
Copy link
Contributor

The content looks fine in here: https://github.com/apache/incubator-mxnet/blame/master/docs/static_site/src/_includes/get_started/linux/python/cpu/pip.md

This is probably some javascript bug in here: https://github.com/apache/incubator-mxnet/blob/master/docs/static_site/src/assets/js/options.js
Never been a fan of these conditional selectors... seems like they're always breaking.

@eric-haibin-lin
Copy link
Member Author

@aaronmarkham what's the alternative solution you're proposing?

@aaronmarkham
Copy link
Contributor

aaronmarkham commented Jan 15, 2020

@aaronmarkham what's the alternative solution you're proposing?

Maybe mxnet-oncall can spend time looking at the js function that runs that feature.
I looked it over and nothing pops out as why it isn't working. Happy to work with someone on it that maybe knows more js and how to use the debugger to experiment with the div states.

@ys2843
Copy link
Contributor

ys2843 commented Jun 18, 2020

@aaronmarkham @eric-haibin-lin Above PR fixed this issue, preview link attached, please feel free to verify.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants