Skip to content

Stream reform validation simulations#236

Merged
MaxGhenis merged 1 commit into
mainfrom
codex/stream-reform-validation-memory
Jul 1, 2026
Merged

Stream reform validation simulations#236
MaxGhenis merged 1 commit into
mainfrom
codex/stream-reform-validation-memory

Conversation

@MaxGhenis

Copy link
Copy Markdown
Contributor

Summary

  • avoid retaining every OBBBA stacked reform simulation during reform validation
  • add a regression test that fails if intermediate simulations remain live across stacked states

Validation

  • uv run ruff format packages/populace-build/src/populace/build/us_runtime/reform_validation.py packages/populace-build/tests/test_reform_validation.py
  • uv run ruff check packages/populace-build/src/populace/build/us_runtime/reform_validation.py packages/populace-build/tests/test_reform_validation.py
  • uv run pytest packages/populace-build/tests/test_reform_validation.py

This is needed before rerunning the full sparse 57k release validation: the previous full builder run completed calibration and wrote the H5/diagnostics, then was killed with exit code 137 during reform_validation.json generation after holding multiple large Microsimulation objects.

@MaxGhenis MaxGhenis merged commit a7a77c4 into main Jul 1, 2026
4 checks passed
@MaxGhenis MaxGhenis deleted the codex/stream-reform-validation-memory branch July 1, 2026 12:09
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