Ensure NumPy/Scipy are installed from conda-forge if using conda #81
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
A user was getting errors after running the script in a
conda
environment without first installingSciPy
(see below). This was traced to the fact thatSciPy
was getting installed fromPyPI
(as a dependency ofpyoptsparse
) and that package seemed to be incompatible with the other packages (e.g.lapack
) that are installed fromconda-forge
by the script.This change will install
NumPy
andScipy
fromconda-forge
before installingpyoptsparse
if they are not installed, or re-install them fromconda-forge
if they were previously installed fromPyPI
. This only happens whenconda
is enabled.Tests were added to the GitHub workflow to exercise these scenarios.
Also:
SciPy 1.16
, which was released recently.Related Issues
Backwards incompatibilities
None
New Dependencies
None