Skip to content
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

Is requirement numpy==1.16.4 really needed ? #519

Closed
cedias opened this issue Nov 16, 2019 · 6 comments
Closed

Is requirement numpy==1.16.4 really needed ? #519

cedias opened this issue Nov 16, 2019 · 6 comments

Comments

@cedias
Copy link

cedias commented Nov 16, 2019

Hi,

While installing ligthning via pip I saw that the numpy requirements was fixed to version 1.16.4:

ERROR: pytorch-lightning 0.5.3.2 has requirement numpy==1.16.4, but you'll have numpy 1.17.4 which is incompatible.

After a quick scroll through the source code i'm wondering: is there a reason why this requirement is so specific ?

Also, wouldn't it make more sense to use pytorch tensors to try and drop the numpy requirements ? Or at least, drop the == 1.16.4

@williamFalcon
Copy link
Contributor

nah. it’s a typo. Mind submitting a PR?

great catch!

@cedias
Copy link
Author

cedias commented Nov 16, 2019

Here you go : #520

@Borda
Copy link
Member

Borda commented Nov 16, 2019

@williamFalcon we already have it here #500 waiting for you...
Btw, how it happened that this issue does not have any label?

@cedias
Copy link
Author

cedias commented Nov 16, 2019

wouldn't it make more sense to use pytorch tensors to try and drop the numpy requirements ?

It's probably worth noting that searching "numpy" within lightning codebase yields only 13 results...

@Borda
Copy link
Member

Borda commented Nov 16, 2019

Do you know a person using Python who doesn't have already installed numpy directly or as dependence? =)

@williamFalcon
Copy link
Contributor

Fixed in #500

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

3 participants