Skip to content

Fix webapp for Shinylive deployment by separating requirements #1245

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

basnijholt
Copy link
Owner

Shinylive/Pyodide was failing because it tried to install packages that are already pre-built in Pyodide (like matplotlib and its C-extension dependencies).

Changes:

  • Split requirements into two files:
    • requirements.txt: Minimal deps for Shinylive (only astral, shiny, shinyswatch)
    • requirements-dev.txt: Full deps for local development (includes matplotlib)
  • Update README to use requirements-dev.txt for local development
  • Update GitHub workflow to use requirements-dev.txt
  • Remove matplotlib version pinning as Pyodide provides it

This fixes the "Can't find a pure Python 3 wheel for 'contourpy'" error when loading the webapp at https://basnijholt.github.io/adaptive-lighting/

basnijholt and others added 2 commits July 21, 2025 15:33
Shinylive/Pyodide was failing because it tried to install packages that
are already pre-built in Pyodide (like matplotlib and its C-extension
dependencies).

Changes:
- Split requirements into two files:
  - requirements.txt: Minimal deps for Shinylive (only astral, shiny, shinyswatch)
  - requirements-dev.txt: Full deps for local development (includes matplotlib)
- Update README to use requirements-dev.txt for local development
- Update GitHub workflow to use requirements-dev.txt
- Remove matplotlib version pinning as Pyodide provides it

This fixes the "Can't find a pure Python 3 wheel for 'contourpy'" error
when loading the webapp at https://basnijholt.github.io/adaptive-lighting/
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