Skip to content

fix: vendored OpenTPS package markers + dev install hygiene#3

Open
vidyuthdev wants to merge 1 commit into
passlab:masterfrom
vidyuthdev:feature/geometry-service
Open

fix: vendored OpenTPS package markers + dev install hygiene#3
vidyuthdev wants to merge 1 commit into
passlab:masterfrom
vidyuthdev:feature/geometry-service

Conversation

@vidyuthdev

Copy link
Copy Markdown
Contributor
  • Add missing init.py for doseCalculation and protons subpackages
  • tests/conftest.py prepends src/ to sys.path (shields tests from stale strict-mode editable-install finders)
  • scripts/install-dev.sh: purge build artifacts + reinstall cleanly, with a sanity-check import of the MCsquare submodule
  • tests/test_package_layout.py: 17 fast regression tests that catch this class of bug in <2s (use in pre-push if you want)
  • README: document the install script and why it exists

- Add missing __init__.py for doseCalculation and protons subpackages
- tests/conftest.py prepends src/ to sys.path (shields tests from
  stale strict-mode editable-install finders)
- scripts/install-dev.sh: purge build artifacts + reinstall cleanly,
  with a sanity-check import of the MCsquare submodule
- tests/test_package_layout.py: 17 fast regression tests that catch
  this class of bug in <2s (use in pre-push if you want)
- README: document the install script and why it exists
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