Skip to content

chore: carve paper figures/eval out of the package; fix CI collection#4

Merged
nathanchenseanwalter merged 1 commit into
mainfrom
chore/carve-out-paper-code
Jun 22, 2026
Merged

chore: carve paper figures/eval out of the package; fix CI collection#4
nathanchenseanwalter merged 1 commit into
mainfrom
chore/carve-out-paper-code

Conversation

@nathanchenseanwalter

Copy link
Copy Markdown
Contributor

Move all manuscript figure-generation code, paper/benchmark eval scripts, their detection/PR-sweep helpers, and the paper tests into the gitignored dev/paper/ tree so they no longer ship in the wheel or break CI.

  • src/tokeye/extra/eval/{events,sweep,fold_stats}.py -> dev/paper/eval/ (extra/eval now ships only the silbidopy + D3D loaders)
  • scripts/eval/*.py + .ipynb -> dev/paper/eval/; eval_.sh + ablation/eval.sh -> dev/paper/eval/commands/ (paths updated); scripts/test_ablation_smoke.py too
  • output/nature_figures -> dev/paper/figures (was already untracked)
  • 7 paper/ablation tests -> dev/paper/tests/ with a conftest that puts dev/paper/eval on sys.path; tests/ keeps only core + dep-free ablation tests
  • moved scripts/tests now import the helpers locally (from sweep/events/ fold_stats) instead of tokeye.extra.eval.*

CI: the four collection errors (h5py/pybaselines/tifffile/lightning) are gone because the tests importing tokeye.training.big_tf_unet_ablation.step_* moved out of tests/. Also:

  • add [project.optional-dependencies] train so pip install tokeye[train] works
  • align CI python matrix to 3.13/3.14 (matches requires-python >=3.13)

Move all manuscript figure-generation code, paper/benchmark eval scripts,
their detection/PR-sweep helpers, and the paper tests into the gitignored
dev/paper/ tree so they no longer ship in the wheel or break CI.

- src/tokeye/extra/eval/{events,sweep,fold_stats}.py -> dev/paper/eval/
  (extra/eval now ships only the silbidopy + D3D loaders)
- scripts/eval/*.py + *.ipynb -> dev/paper/eval/; eval_*.sh + ablation/eval.sh
  -> dev/paper/eval/commands/ (paths updated); scripts/test_ablation_smoke.py too
- output/nature_figures -> dev/paper/figures (was already untracked)
- 7 paper/ablation tests -> dev/paper/tests/ with a conftest that puts
  dev/paper/eval on sys.path; tests/ keeps only core + dep-free ablation tests
- moved scripts/tests now import the helpers locally (from sweep/events/
  fold_stats) instead of tokeye.extra.eval.*

CI: the four collection errors (h5py/pybaselines/tifffile/lightning) are gone
because the tests importing tokeye.training.big_tf_unet_ablation.step_* moved
out of tests/. Also:
- add [project.optional-dependencies] train so `pip install tokeye[train]` works
- align CI python matrix to 3.13/3.14 (matches requires-python >=3.13)
@nathanchenseanwalter nathanchenseanwalter merged commit 7454c9f into main Jun 22, 2026
3 checks passed
@nathanchenseanwalter nathanchenseanwalter deleted the chore/carve-out-paper-code branch June 22, 2026 04:05
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