Skip to content

A very critical error on the installation documentation (I think) #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

alvations
Copy link

On the installation page, it says:

"We don’t recommend installing scipy or numpy using pip on linux, as this will involve a lengthy build-process with many dependencies. Without careful configuration, building numpy yourself can lead to an installation that is much slower than it should be. If you are using Linux, consider using your package manager to install scikit-learn. It is usually the easiest way, but might not provide the newest version. If you haven’t already installed numpy and scipy and can’t install them via your operation system, it is recommended to use a third party distribution."

Isn't it suppose to be We recommend installing scipy or numpy using pip on linux?

@GaelVaroquaux
Copy link
Member

GaelVaroquaux commented Nov 6, 2015 via email

@alvations
Copy link
Author

Okay, so it's you recommend to use conda or other package manager. Thanks for the note. I thought I was seeing things =) I think it's good to refer to consider using your package manage, e.g. conda, to install scikit-learn

@alvations alvations closed this Nov 6, 2015
@GaelVaroquaux
Copy link
Member

GaelVaroquaux commented Nov 6, 2015 via email

@alvations
Copy link
Author

What if we change the wording to:

If you are using Linux, consider using your package manager, e.g. 
*yum, apt-get, dpkg, zypper, etc.*, to install scikit-learn. It is usually 
the easiest way, but might not provide the newest version.

Is that correct?

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.

2 participants