Skip to content

refactor: extract shared rl runtime - #6

Draft
annedranowski wants to merge 1 commit into
refactor/shared-invariantsfrom
refactor/shared-rl-runtime
Draft

refactor: extract shared rl runtime#6
annedranowski wants to merge 1 commit into
refactor/shared-invariantsfrom
refactor/shared-rl-runtime

Conversation

@annedranowski

Copy link
Copy Markdown
Collaborator

Summary

  • Extracts the shared Spherogram RL environment and PPO load/train runtime from the two target notebooks.
  • Updates unknotting_curr_ver and unknotting_connect_sum_experiments to use the shared runtime module.
  • Keeps this PR stacked on refactor/shared-invariants so the RL runtime diff can be reviewed separately.

Validation

  • Parsed edited notebooks for Python syntax.
  • Ran PYTHONPATH=src .venv/bin/python -m compileall -q src tests.
  • Ran PYTHONPATH=src .venv/bin/python -m pytest tests/test_rl_runtime.py tests/test_invariants.py tests/test_parsing.py tests/test_config.py tests/test_checkpoints.py tests/test_paths_workbook_targets.py.

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