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

Pin scipy version to one compatible with PAOFLOW #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aj0337
Copy link
Collaborator

@aj0337 aj0337 commented Dec 1, 2024

The version of scipy being installed using setup.py has been pinned to ~=1.9.0.

This was done because

  1. scipy ~=1.14.0 raises AttributeError: module 'scipy.integrate' has no attribute 'simps' since simps was renamed to simpson in these versions. See https://docs.scipy.org/doc/scipy/reference/generated/scipy.integrate.simpson.html

  2. scipy > 1.9 raises TypeError: array type complex256 is unsupported in linalg. Some discussion on this appears here
    BUG: complex256 type created by entering complex128 into linalg.sqrtm causes errors. scipy/scipy#18250

@aj0337 aj0337 requested review from marcobn and Sassafrass6 December 1, 2024 06:03
This was referenced Dec 1, 2024
acd1034 added a commit to acd1034/PAOFLOW that referenced this pull request Jan 30, 2025
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 this pull request may close these issues.

1 participant