-
-
Notifications
You must be signed in to change notification settings - Fork 26.2k
[MRG+1] split installation into simple and advanced part #5257
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
[MRG+1] split installation into simple and advanced part #5257
Conversation
|
||
pip install -U scikit-learn | ||
|
||
If there are no binary packages matching your Python version you might |
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.
version, you
A comma here?
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.
Fixed (and the other ones). This is just copy and paste of what was under "installation" btw, so these are not really related to the move of the docs.
Should we also update |
yeah I think we should point there from the readme. |
ab7fa5b
to
f0852e2
Compare
Ok, addressed all comments, I think. ping @ogrisel and @GaelVaroquaux maybe? |
|
||
The `Python(x,y) <https://code.google.com/p/pythonxy/>`_ project distributes |
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.
It looks like this project has moved to http://python-xy.github.io/
looks ok to me, though I don't know the windows changes. |
----- | ||
|
||
At this time scikit-learn does not provide official binary packages for Linux | ||
so you have to build from source. |
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.
You should mention here that various distributions have scikit-learn packages, although they might not be up to date with the latest version of scikit-learn.
I don't think I made any windows changes @MechCoder. which do you mean? |
Never mind, I did not see the diff properly. |
f0852e2
to
41d0f1f
Compare
Added a remark about using the package manager on linux. |
+1 for merge on my side. As a side note, in terms of simplified install instruction, I am pretty happy with the ones that will ship with the next version of nilearn: |
yeah they are short and sweet. I wasn't sure what you though of a "first class conda" way. And if someone installed anaconda, they already have scikit-learn (and can use conda in the newest version). As you know, I'm a fan of iterative improvement. I feel this improves upon the past and we can improve upon it in the future ;) |
yeah they are short and sweet. I wasn't sure what you though of a
"first class conda" way.
In nilearn we target very non-technical users. This should be adapted for
scikit-learn.
As you know, I'm a fan of iterative improvement. I feel this improves
upon the past and we can improve upon it in the future ;)
Absolutely. It's the right way to do it.
What I really like in the nilearn install docs is the tab thingy, that
automatically focuses to the OS it detects you are running.
|
ohhhh I didn't realize. Awesome. #5737 ;) Merging this as doc improvement so I can tag ^^ |
[MRG+1] split installation into simple and advanced part
backported as 37d18ce |
Fixes #4742