Skip to content

fix: move pypsa dependency to pip#1322

Merged
lkstrp merged 1 commit intoPyPSA:masterfrom
Uzaaft:master
Sep 24, 2024
Merged

fix: move pypsa dependency to pip#1322
lkstrp merged 1 commit intoPyPSA:masterfrom
Uzaaft:master

Conversation

@Uzaaft
Copy link
Copy Markdown
Contributor

@Uzaaft Uzaaft commented Sep 23, 2024

Tried to set this project up, and seems like conda is experiencing some difficulties due to the following:

  • pypsa is installed with conda, and requires highspy
  • highspy is not available on conda for arm64 on osx
  • highspy is being installed in the environment.yml file, but with pip

Seems like this causes conda to error out with highspy not being available to install.

@fneum
Copy link
Copy Markdown
Member

fneum commented Sep 24, 2024

Fine as a temporary measure, but I would like to understand where the problem comes from. Maybe something for @lkstrp?

@lkstrp
Copy link
Copy Markdown
Member

lkstrp commented Sep 24, 2024

Same. It is the same issue as the previously failing ci and only happens on arm macos. And I think related to a new highspy build. Will revert this, once this is solved.

@lkstrp lkstrp merged commit 0d1c36b into PyPSA:master Sep 24, 2024
@lkstrp
Copy link
Copy Markdown
Member

lkstrp commented Sep 24, 2024

@Uzaaft
The issue is fixed now. See #1325. You might need to remove your env and clean the cache of conda/ mamba.

@Uzaaft
Copy link
Copy Markdown
Contributor Author

Uzaaft commented Sep 24, 2024

The issue seems to come from the fact that highspy in conda doesn't have osx-arm64 support, even if the pip variant has.
Previously, pypsa-eur pulled in pypsa from conda, and its dependency graph looked after highspy in conda, and would fail on osx-arm64.

But pypsa on pip was fine since pip has a working version.

At least that's my understanding

@lkstrp
Copy link
Copy Markdown
Member

lkstrp commented Sep 24, 2024

I think that's exactly it. If you install pypsa 30.2 with the build version mentioned in the PR it will work, since the recipe is changed.

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.

3 participants