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

numpy 1.20.0 and python 3.6.9 #4

Closed
Joeycho opened this issue Jun 14, 2021 · 2 comments
Closed

numpy 1.20.0 and python 3.6.9 #4

Joeycho opened this issue Jun 14, 2021 · 2 comments

Comments

@Joeycho
Copy link

Joeycho commented Jun 14, 2021

Hi, I just started to clone the project and start MXMNet architecture in my environment.

I wonder how you deal with python 3.6.9 and numpy 1.20.0 version conflict.

The current RDkit version relies on numpy 1.20.0, and numpy 1.20.0 does not support python 3.6.

https://numpy.org/doc/stable/release/1.20.0-notes.html

If you need further information, please let me know. I would be happy to update the info.

I am installing the packages in my virtual environment python=3.6.9 via Miniconda3.

@zetayue
Copy link
Owner

zetayue commented Jun 14, 2021

Thank you for pointing it out! Python version should not be 3.6.9. Sorry for any inconvenience caused.

Python 3.6.9 is only for the previous dependencies in requirements.txt that only needed Numpy 1.17.2. When I updated the dependencies in requirements.txt, Python version was not updated correspondingly. I just double-checked that my most recently tested environment has Python 3.7.10.

@Joeycho
Copy link
Author

Joeycho commented Jun 14, 2021

@zetayue Hi, thanks for quick reply!

MXMNet/requirements.txt

Lines 8 to 10 in 775e157

https://pytorch-geometric.com/whl/torch-1.4.0/torch_scatter-2.0.4%2Bcu101-cp36-cp36m-linux_x86_64.whl
https://pytorch-geometric.com/whl/torch-1.4.0/torch_sparse-0.6.0%2Bcu101-cp36-cp36m-linux_x86_64.whl
https://pytorch-geometric.com/whl/torch-1.4.0/torch_cluster-1.5.4%2Bcu101-cp36-cp36m-linux_x86_64.whl

As follows, in the above lines, cp36 should be replaced by cp37. After that, all worked. :)

@Joeycho Joeycho closed this as completed Jun 14, 2021
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

2 participants