Skip to content

Fix macOS builds by switching pyscipopt from pip to conda-forge#1688

Merged
lkstrp merged 1 commit intoPyPSA:masterfrom
stefan-oet:bugfix/pyscipopt-on-macos
May 14, 2025
Merged

Fix macOS builds by switching pyscipopt from pip to conda-forge#1688
lkstrp merged 1 commit intoPyPSA:masterfrom
stefan-oet:bugfix/pyscipopt-on-macos

Conversation

@stefan-oet
Copy link
Copy Markdown
Contributor

Closes # (if applicable).

Changes proposed in this Pull Request

The new lock files approach introduced in #1660 caused macOS builds to fail because installing pyscipopt via pip does not include scip.

The proposed solution is to fetch pyscipopt from conda-forge instead of pip. This package properly bundles scip providing a complete and working environment (see updated locked files included in this PR)

The downside is that we have to drop support for ARM linux as pyscipopt does not currently provide builds via conda-forge or pip for this platform. Users will either need to install it from source or get if via a package manager when available (e.g. Arch Linux ARM package)

@lkstrp

Checklist

  • I tested my contribution locally and it works as intended.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to envs/environment.yaml.
  • Changes in configuration options are added in config/config.default.yaml.
  • Changes in configuration options are documented in doc/configtables/*.csv.
  • Sources of newly added data are documented in doc/data_sources.rst.
  • A release note doc/release_notes.rst is added.

Copy link
Copy Markdown
Member

@lkstrp lkstrp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As already discussed, that's fine. It's unlikely that anyone needs to run Eur on Linux ARM
Thanks for the fixes! @stefan-oet

@lkstrp lkstrp enabled auto-merge (squash) May 14, 2025 08:55
@lkstrp lkstrp merged commit 7567ae6 into PyPSA:master May 14, 2025
12 checks passed
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.

2 participants