Skip to content

0.2.2 wheel is forcing tqdm==4.19.9 #766

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

Closed
mitar opened this issue Feb 28, 2019 · 10 comments
Closed

0.2.2 wheel is forcing tqdm==4.19.9 #766

mitar opened this issue Feb 28, 2019 · 10 comments

Comments

@mitar
Copy link

mitar commented Feb 28, 2019

So while setup.py has code which forces tqdm==4.19.9 version only on Python 2 and leaves version unforced on Python 3, 0.2.2 package was published with a wheel which is the same for both Python 2 and Python 3 and have tqdm==4.19.9 version hard-coded. It seems when wheel was made Python 2 was used.

I think that if you have such condition in setup.py you have to publish a different wheel for Python 2 and Python 3 and not the same one.

@surgan12
Copy link
Contributor

@soumith , just a quick question !
Why are we forcing tqdm==4.19.9 in setup.py for python2 .
Simply adding 'tqdm' is using tqdm==4.28.1 for python3 and tqdm==4.31.1 for python2.

I think we can remove that line .

@mitar
Copy link
Author

mitar commented Feb 28, 2019

See: #542

@surgan12
Copy link
Contributor

@mitar I Tried using the versions mentioned above by me and no error is showing for me.

@mitar
Copy link
Author

mitar commented Feb 28, 2019

There is no error. It is just that it forces tqdm==4.19.9 on Python 3. Which is not what it should be doing. You cannot install any newer version then. So if you have another package which requires a newer version you have a problem (which is what happens in my case).

@surgan12
Copy link
Contributor

With error i meant , mentioned in #541 .
And yes you are right the old version will cause some issues.

@mitar
Copy link
Author

mitar commented Feb 28, 2019

But tqdm/tqdm#548 is not closed?

@surgan12
Copy link
Contributor

@mitar , i tested with tqdm==4.31.1 and everything mentioned in #541 was working fine.

@jgbustos
Copy link

I'd suggest using platform specific dependencies (see https://setuptools.readthedocs.io/en/latest/setuptools.html#declaring-platform-specific-dependencies and https://www.python.org/dev/peps/pep-0508/) so the downgrade is only forced for Python 2.

@soumith
Copy link
Member

soumith commented Feb 28, 2019

we'll fix it today.

@fmassa
Copy link
Member

fmassa commented Feb 28, 2019

I think I'll just remove the dependency on tqdm, and use instead the version from pytorch in https://github.com/pytorch/pytorch/blob/13e6326c07cd44ad52db32ac3918eeb79f8230bc/torch/utils/model_zoo.py#L126-L151

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants