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

Pylearn-parsimony is not compatible with numpy version >=1.20 #38

Open
jchiquet opened this issue Apr 6, 2023 · 5 comments · Fixed by #39
Open

Pylearn-parsimony is not compatible with numpy version >=1.20 #38

jchiquet opened this issue Apr 6, 2023 · 5 comments · Fixed by #39

Comments

@jchiquet
Copy link

jchiquet commented Apr 6, 2023

While trying to use this module with a recent version of Python and numpy, scipy, and understand the error underlying desanou/multiscale_glasso#1, I got the following error:

Module 'numpy' has no attribute 'float'.np.floatwas a deprecated alias for the builtinfloat. To avoid this error in existing code, use floatby itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, usenp.float64` here. The aliases was originally deprecated in NumPy 1.20; for more details and guidance see the original release note at:

https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations

It seems that parsimony use at least once np.float :

array = np.asarray(array, dtype=np.float)

Do you plan to publish a release correcting this?

Thanks

@DimitriPapadopoulos
Copy link
Member

Fixed in master but need a release.

@DimitriPapadopoulos
Copy link
Member

And by the way, are you using this package with Python 2.7 or Python 3?

@jchiquet
Copy link
Author

jchiquet commented Apr 6, 2023

Hey, thanks for quick fix !

I am trying to use it with Python 3. Seems to work up to 3.8, but would like to go 3.10...

@DimitriPapadopoulos
Copy link
Member

Issues with 3.10 or 3.11?

@jchiquet
Copy link
Author

jchiquet commented Apr 6, 2023

Issues with 3.10 or 3.11?

Issue noted with 3.10, not tested with 3.11...

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

Successfully merging a pull request may close this issue.

2 participants