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

pip install spyrit fails on CREATIS cluster (Fedora) #29

Closed
cmouton opened this issue Jul 21, 2021 · 8 comments
Closed

pip install spyrit fails on CREATIS cluster (Fedora) #29

cmouton opened this issue Jul 21, 2021 · 8 comments

Comments

@cmouton
Copy link
Contributor

cmouton commented Jul 21, 2021

I followed Thomas's recommendation : no more using the cache.
But it is still downloading the .tar.gz and failing.

Is it because there is no wheel for Fedora?

Screen Shot 2021-07-21 at 13 10 06

@cmouton
Copy link
Contributor Author

cmouton commented Jul 21, 2021

I can precise the Python version :
(test_DLMIS) [mouton@linux1 openspyrit]$ python --version
Python 3.9.6

I understand that it is not supported yet (https://pypi.org/project/spyrit/#files).

@cmouton
Copy link
Contributor Author

cmouton commented Jul 22, 2021

@tbaudier : Have you already planned to add the wheels also for Python 3.9?

@tbaudier
Copy link
Collaborator

#32

@cmouton
Copy link
Contributor Author

cmouton commented Jul 22, 2021

Thanks !!!

@tbaudier
Copy link
Collaborator

With python 3.9 and windows, I need to update numpy because v1.19.3 is not available for this version. But for numpy > 1.20.X there is no wheel for python 3.6. I need to fix the version of numpy because fht is an extension of numpy. So I have different possibilities:

  • Do not update numpy and do not create the wheel for windows and python 3.9
  • Update numpy and do not support python 3.6 anymore

@nducros
Copy link
Member

nducros commented Jul 23, 2021

What would be a third option without fht that we no longer need?

@tbaudier
Copy link
Collaborator

We can remove the fixing of the numpy version and it should work for different version of python

@nducros
Copy link
Member

nducros commented Jul 23, 2021

Great. So this is the right time to discard it!

You should only need to replace this line with
H = wh.walsh2_matrix(n)/n

Both Stat_h and optim_had are no longer necessary as I reimplemented the walsh version of both functions (already included)

@tbaudier tbaudier mentioned this issue Jul 23, 2021
@tbaudier tbaudier closed this as completed Sep 8, 2021
jabascal pushed a commit that referenced this issue Nov 27, 2023
New fig and notebook with no answers
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