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

Installation instructions for windows need to be updated #1146

Closed
Garyfallidis opened this Issue Nov 5, 2016 · 5 comments

Comments

Projects
None yet
3 participants
@Garyfallidis
Member

Garyfallidis commented Nov 5, 2016

To say that you need to have visual studio together with anaconda to make everything work. Except if you use the dipy conda binaries. @arokem we need to add pytables on that conda right?

@Garyfallidis

This comment has been minimized.

Member

Garyfallidis commented Nov 5, 2016

Also we need to check if sklearn is available in anaconda by default and put a note for cvxopt (that is not available) but there is a work around. Using conda install -c clg_boar cvxopt

@arokem

This comment has been minimized.

Member

arokem commented Nov 7, 2016

You don't need Visual Studio if you install from conda forge. That's the whole idea - it gets compiled for you on Appveyor (assuming that works...).

It looks like cvxopt is also available through conda-forge: https://github.com/conda-forge/cvxopt-feedstock, so that's not a problem either.

As for whether some package (e.g. scikit-learn) is installed per default: I think it doesn't matter. Conda installs all the dependencies specified in the recipe, so it doesn't matter whether sklearn, cvxopt or any other dependency is already installed on the user's machine when they install dipy. The dependencies should get automatically installed.

For the time being, added cvxopt, pytables to the dependencies: conda-forge/dipy-feedstock#6

@Garyfallidis

This comment has been minimized.

Member

Garyfallidis commented Nov 7, 2016

Yes, I agree. But we can put something for the people developing. That they need visual studio.

@arokem

This comment has been minimized.

Member

arokem commented Nov 7, 2016

How's this?

96c2e41

On Sun, Nov 6, 2016 at 4:29 PM, Eleftherios Garyfallidis <
notifications@github.com> wrote:

Yes, I agree. But we can put something for the people developing. That
they need visual studio.


You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
#1146 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHPNjVZ695lySl-96RTGcMKMLHlCj1Lks5q7nD-gaJpZM4KqGhH
.

@skoudoro

This comment has been minimized.

Member

skoudoro commented Oct 23, 2017

closed by #1361

@skoudoro skoudoro closed this Oct 23, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment