Skip to content

Conversation

amueller
Copy link
Member

Fixes #4742

@amueller amueller added this to the 0.17 milestone Sep 11, 2015

pip install -U scikit-learn

If there are no binary packages matching your Python version you might
Copy link
Member

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?

Copy link
Member Author

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.

@glouppe
Copy link
Contributor

glouppe commented Sep 28, 2015

Should we also update https://github.com/scikit-learn/scikit-learn/blob/master/README.rst where there are installation instructions?

@amueller
Copy link
Member Author

yeah I think we should point there from the readme.

@glouppe glouppe added Easy Well-defined and straightforward way to resolve Need Contributor Documentation Build / CI labels Oct 19, 2015
@amueller amueller force-pushed the installation_instructions branch from ab7fa5b to f0852e2 Compare November 2, 2015 02:24
@amueller
Copy link
Member Author

amueller commented Nov 2, 2015

Ok, addressed all comments, I think. ping @ogrisel and @GaelVaroquaux maybe?


The `Python(x,y) <https://code.google.com/p/pythonxy/>`_ project distributes
Copy link
Member

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/

@MechCoder
Copy link
Member

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.
Copy link
Member

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.

@amueller
Copy link
Member Author

amueller commented Nov 3, 2015

I don't think I made any windows changes @MechCoder. which do you mean?

@MechCoder
Copy link
Member

Never mind, I did not see the diff properly.

@amueller amueller force-pushed the installation_instructions branch from f0852e2 to 41d0f1f Compare November 4, 2015 19:28
@amueller
Copy link
Member Author

amueller commented Nov 4, 2015

Added a remark about using the package manager on linux.

@GaelVaroquaux GaelVaroquaux changed the title [MRG] split installation into simple and advanced part [MRG+1] split installation into simple and advanced part Nov 5, 2015
@GaelVaroquaux
Copy link
Member

+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:
https://circle-artifacts.com/gh/nilearn/nilearn/287/artifacts/0/home/ubuntu/nilearn/doc/_build/html/introduction.html#installation

@amueller
Copy link
Member Author

amueller commented Nov 5, 2015

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 ;)

@GaelVaroquaux
Copy link
Member

GaelVaroquaux commented Nov 5, 2015 via email

@amueller
Copy link
Member Author

amueller commented Nov 5, 2015

ohhhh I didn't realize. Awesome. #5737 ;)

Merging this as doc improvement so I can tag ^^

amueller added a commit that referenced this pull request Nov 5, 2015
[MRG+1] split installation into simple and advanced part
@amueller amueller merged commit 7c35640 into scikit-learn:master Nov 5, 2015
@amueller
Copy link
Member Author

amueller commented Nov 5, 2015

backported as 37d18ce

@amueller amueller deleted the installation_instructions branch May 19, 2017 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocker Build / CI Documentation Easy Well-defined and straightforward way to resolve
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants