-
Notifications
You must be signed in to change notification settings - Fork 33
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
Dependency version conflict with noise2void/keras~=2.3/tensorflow~=2.5 #100
Comments
Yeah, I came across that (rather ridiculous) pin in the past. I even looked through their source to see why, and there was no good reason for that pin. It's rather frustrating. You can see some discussion here: https://github.com/napari/superqt/issues/45 I'll see what I can do |
Side note, I know the Jug lab is also working on such a plugin... are you collaborating with them? |
while you're waiting on this, an even greater service to the community would be to open an issue with tensorflow. I would be really surprised if later versions of typing-extensions didn't also work for them. That pin is really troublesome and consequential for a lot of packages. |
In the recent version of tensorflow it's already fixed: |
i mean a backport ... eg 2.5.0-post1 |
Describe the bug
Hi @tlambert03 ,
I'm working on a napari-plugin for noise2void, which is based on a slightly outdated version of tensorflow. It is compatible with tensorflow versions about 2.5 which is just about 1 year old.
When installing it together with recent napari, I'm receiving a dependency error because of
superqt
. I'm wondering if it would be possible to relax the constrainttyping-extensions>=3.10.0.0
to something more compatible totyping-extensions~=3.7.4
:https://github.com/napari/superqt/blob/a3b0f1b1150b055ab25e1b462293368e68255f28/setup.cfg#L41
To Reproduce
In an empty environment
causes this error:
See also:
A bit more context:
https://github.com/juglab/n2v/blob/11669b7e09633f007c5920feb713fc955dbe4da8/setup.py#L49
https://github.com/keras-team/keras/blob/b5cb82c689eac0e50522be9d2f55093dadfba24c/setup.py#L35
https://github.com/tensorflow/tensorflow/blob/a4dfb8d1a71385bd6d122e4f27f86dcebb96712d/tensorflow/tools/pip_package/setup.py#L93
Thanks!
Best,
Robert
The text was updated successfully, but these errors were encountered: