Skip to content

Fix Assimulo pip extra metadata#110

Merged
bernalde merged 1 commit into
masterfrom
fix/issue-109-assimulo-extra
Jul 14, 2026
Merged

Fix Assimulo pip extra metadata#110
bernalde merged 1 commit into
masterfrom
fix/issue-109-assimulo-extra

Conversation

@parkyr

@parkyr parkyr commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Closes #109.

Summary

  • Remove the unresolvable assimulo==3.4.3 PyPI requirement from the assimulo optional extra and requirements-assimulo.txt.
  • Keep cython>=0.29,<3 as the pip-installable helper for source-build workflows.
  • Document that Assimulo itself is supported through the conda-forge environment or a local source build, not through PyPI.

Related: #8 is packaging-modernization context only; this PR does not depend on it.

Validation

  • uv pip install -e '.[assimulo]' --dry-run --python .venv/bin/python --cache-dir /tmp/pharmapy-uv-cache-issue109
  • uv pip install -r requirements-assimulo.txt --dry-run --python .venv/bin/python --cache-dir /tmp/pharmapy-uv-cache-issue109
  • .venv/bin/python -m pytest --collect-only
  • .venv/bin/python -m pytest tests/ -m 'not assimulo'
  • git diff --check

Branch Hygiene

@parkyr
parkyr marked this pull request as ready for review July 10, 2026 16:23
@bernalde
bernalde merged commit 64d1e4b into master Jul 14, 2026
2 checks passed
@bernalde
bernalde deleted the fix/issue-109-assimulo-extra branch July 14, 2026 13:37
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.

[packaging] assimulo optional extra cannot be installed by pip (not on PyPI)

2 participants