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
Error installing skimage- #1506
Comments
That is correct and the intended behavior. Required dependencies must be installed first, then scikit-image... at least when one uses pip. pip itself is not a package manager. If you want that, a distribution like Anaconda or Canopy will "just work" by grabbing all necessary dependencies and then what you asked for. |
Well - I think pip is a package manager - and if you install scikit-image, pip will also install:
It's just that scipy is not among the declared dependencies for the install. Should it be? |
Looking a little further, scipy >= 0.9 is declared in requirements.txt so it should be brought along. |
I can't remember the exact source of the discussion, but I think it was deliberate that the requirements.txt requirements were different from the default pip / source install requirements. |
Here is the reason for it:
|
Hum - well - they do have 'eggs' available for OSX, but not for Windows or Linux. So.... ? |
/cc @blink1073 |
We could special case On Mon, May 11, 2015 at 3:25 PM, Stefan van der Walt <
|
Just to be clear - we mean 'wheels' right? I think eggs have definitely gone off. |
Is it important that eggs are available? Why can't we let pip
compile scipy from source (as long as it doesn't upgrade
unnecessarily)?
|
a) It will upgrade unnecessarily if someone upgrades scikit-image; |
I thought the upgrade happens only when you specified a version. |
It always upgrades: https://pip.pypa.io/en/latest/user_guide.html#only-if-needed-recursive-upgrade Yes, it's lousy. |
Yes, I meant to say |
I'm pretty sure this problem will go away (updating numpy, scipy, etc) if one does not define version (e.g. My take is that duplicating 7 requirements is worth it, and be rid of this installation trouble. To be clear, the proposal is replacing
with
|
From the https://pip.pypa.io/en/latest/reference/pip_install.html#cmdoption-U "Upgrade all specified packages to the newest available version. This process is recursive regardless of whether a dependency is already satisfied." |
@arve0, even if we do not specify the version, we will still have this mess. |
@blink1073 Sorry, I'm not following you. How will This implies that the user either will get compiled wheels for his operating system (true for all dependencies on Mac and windows I believe), or has a system with compiler (should be added to the docs how the most popular, read Ubuntu, get that installed). Are my assumptions wrong, or what have I missed? |
For people wanting to upgrade, that will be |
But when we add a new dependency (like dask), that would not work. — On Sat, May 23, 2015 at 2:18 AM, Arve Seljebu notifications@github.com
|
We could recommend — On Sat, May 23, 2015 at 2:18 AM, Arve Seljebu notifications@github.com
|
or |
So that should work today without any changes to our setup.py, no? |
All dependencies will have to be added (scipy and numpy), but versions should be stripped to avoid upgrading system-installed packages like scipy, matplotlib, etc. |
In other words; dependencies will be either soft dependencies or hard dependencies.
E.g.
Might add; there is nothing wrong with having (sane) hard dependencies in requirements.txt for developers. A developer should definitely have reasonable updated numpy/scipy/etc. |
@Arve, the listed versions are hard dependencies in that we rely on functions or an API provided by those versions or the version is the minimum we can test against. — On Sat, May 23, 2015 at 7:49 AM, Arve Seljebu notifications@github.com
|
Well, then one should just remove
EDITED |
Haha, sorry! I mean, remove it! 😆 |
Agreed, with the new suggested incantation we can certainly simplify setup.py, can to take a crack at the fix and the documentation update? — On Sat, May 23, 2015 at 8:06 AM, Arve Seljebu notifications@github.com
|
I can take a look at it in the evening (GMT+1). |
Thanks brother, keep keeping us honest. 😜 |
Hi!
I've install skimage on a MacOs 10.10, like it's said in the documentation:
pip install -U scikit-image
and it say that need scipy to work, I had to install it to work.
The text was updated successfully, but these errors were encountered: