Skip to content

Preflight checkpoint recalibration output root#205

Merged
MaxGhenis merged 1 commit into
mainfrom
codex/fix-183-recalibrate-output-preflight
Jun 3, 2026
Merged

Preflight checkpoint recalibration output root#205
MaxGhenis merged 1 commit into
mainfrom
codex/fix-183-recalibrate-output-preflight

Conversation

@anth-volk
Copy link
Copy Markdown
Contributor

Summary

Fixes #183.

  • Preflight pe_us_recalibrate_from_checkpoint --output-root before starting checkpoint recalibration.
  • Require --output-root to already exist, be a directory, and pass best-effort writable/searchable permission checks.
  • Avoid writing a sentinel/probe file or creating the output directory during preflight.
  • Keep later artifact writes using the preflighted output directory.

Tests

  • ruff check src/microplex_us/pipelines/pe_us_recalibrate_from_checkpoint.py tests/pipelines/test_recalibrate_from_checkpoint.py
  • git diff --check
  • PYTHONPATH=/Users/administrator/Documents/PolicyEngine/microplex-us/src PYTHONDONTWRITEBYTECODE=1 /Users/administrator/Documents/PolicyEngine/worktrees/microplex-us/fix-pe-rebuild-smoke-issues/.venv/bin/python -m pytest -q -p no:cacheprovider tests/pipelines/test_recalibrate_from_checkpoint.py
  • PYTHONPATH=/Users/administrator/Documents/PolicyEngine/microplex-us/src PYTHONDONTWRITEBYTECODE=1 /Users/administrator/Documents/PolicyEngine/worktrees/microplex-us/fix-pe-rebuild-smoke-issues/.venv/bin/python -m microplex_us.pipelines.pe_us_recalibrate_from_checkpoint --help

Note: --help exited 0, with sandbox Arrow CPU-info warnings.

@anth-volk anth-volk marked this pull request as ready for review June 3, 2026 20:43
@anth-volk anth-volk requested a review from MaxGhenis June 3, 2026 20:43
@MaxGhenis MaxGhenis merged commit 036fa06 into main Jun 3, 2026
5 checks passed
@MaxGhenis MaxGhenis deleted the codex/fix-183-recalibrate-output-preflight branch June 3, 2026 23:26
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.

PE-US checkpoint resume validates output directory only after expensive recalibration

2 participants