Skip to content

Refactor: extract shared parsing helpers - #4

Open
annedranowski wants to merge 2 commits into
chore-uv-gcs-repo-hygienefrom
refactor/shared-src-layout
Open

Refactor: extract shared parsing helpers#4
annedranowski wants to merge 2 commits into
chore-uv-gcs-repo-hygienefrom
refactor/shared-src-layout

Conversation

@annedranowski

Copy link
Copy Markdown
Collaborator

Summary

  • extract duplicated workbook/parsing helpers from active notebooks into src/upperbounds/data/parsing.py
  • add Pydantic experiment configuration models in src/upperbounds/config.py
  • update active notebook variants to import shared parsing helpers instead of redefining them inline
  • add tests covering parsing behavior, config validation, and existing checkpoint helpers

Stack

Validation

  • PYTHONPATH=src .venv/bin/python -m pytest tests/test_parsing.py tests/test_config.py tests/test_checkpoints.py
  • changed notebook code cells parse successfully

Notes

  • notebooks/upper_bound_unknotting_v5_original.ipynb is left untouched as a historical/reference notebook because reserializing it produced noisy diffs.

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