Skip to content

Pin the graph extra to microcosm main instead of #873's head - #541

Merged
MaxGhenis merged 1 commit into
masterfrom
repin-microcosm-main-20261015
Oct 15, 2026
Merged

MaxGhenis merged 1 commit into
masterfrom
repin-microcosm-main-20261015

Conversation

@MaxGhenis

Copy link
Copy Markdown
Contributor

The optional graph extra pinned microcosm-frame and microcosm-graph to 3ff92b0a, the head of PolicyEngine/microcosm#873, which is not on microcosm main. Under ARCH-1 (Max, 2026-10-15), #873 retires once dynamics pins a main commit (microcosm#1285, the release contract). This PR repins both packages to microcosm main c28b4537.

Validation

By a GPT-6.1 Sol build lane, 20261015-061437-md-repin-microcosm:

  • Install. Fresh venvs on Python 3.13.15 and 3.14.7, each with uv pip install -e '.[dev,graph]'. Both packages' direct_url.json confirms the exact commit, and require_graph() passes.
  • Tests. The CI graph-integration tests, which cover every test importing microcosm.*, give 155 passed with no skips on each Python version.
  • Tier counts. Unchanged, and the tier-policy test passes against full collection.
  • Reproduction.
    • canonical.py is byte-identical between the two commits.
    • keyed_uniform draws are byte-identical between 3ff92b0a and main on 315 cases. Main changes only temporal NumPy coordinates and signed float zero; dynamics passes integer person/year/draw coordinates and string process names.
  • Hashes. Implementation and node hashes do change: runtime.py:642 and trajectory.py:573 hash keyed_uniform's whole defining module via source_hash. No committed graph reproduction pin depends on them, and the frozen candidate spec hashes in tests/test_m6_engine_refit.py are unchanged. No runs/*.json, gates.yaml, Python or test file changes.

Docs

docs/population-graph.md now:

  • names the main pin;
  • drops the "core change must be reviewed first" caveat, since the code is on main;
  • rewords the receipt note for main's executor;
  • lists tests/test_graph_trajectory_accounting.py in the test command CI already runs.

Dynamics needs no #873-only feature, so microcosm#873 can close once this merges.

🤖 Generated with Claude Code

microcosm-frame and microcosm-graph were pinned to 3ff92b0a, the head of
microcosm#873, which is not on main. Under ARCH-1 (Max, 2026-10-15) #873
retires once dynamics pins a main commit. Repin both to main c28b4537.

Validated by a GPT-6.1 Sol build lane (20261015-061437-md-repin-microcosm):
the CI graph-integration tests pass on Python 3.13 and 3.14 (155 each, no
skips); keyed_uniform draws are byte-identical to 3ff92b0a on 315 integer
coordinate/stream cases; canonical.py is byte-identical. Implementation
hashes change because runtime.py and trajectory.py hash keyed_uniform's
defining module, but no committed reproduction pin depends on them. No
Python, test, runs/ or gates.yaml change; tier counts unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
social-security-model Ready Ready Preview Oct 15, 2026 12:24pm UTC

Request Review

@MaxGhenis
MaxGhenis merged commit 2c0eb63 into master Oct 15, 2026
12 checks passed
@MaxGhenis

Copy link
Copy Markdown
Contributor Author

Merged at reviewed head f34888d (merge commit 2c0eb63) by the dynamics coordinator. Gates: gh pr checks exit 0 (graph-integration 3.13 and 3.14, all four pytest shards), MERGEABLE/CLEAN, not a draft, no change requests.

Independent review: Subfleet 20261015-082342-md541-review (hard tier, served by Opus 5.5 with a shell), APPROVE, with no blocker and no major finding. It checked:

  • the installed microcosm-frame and microcosm-graph byte-match c28b4537;
  • 155 passed with no skips on 3.13 and 3.14;
  • keyed_uniform gives byte-identical draws on 169,164 dynamics-shaped keys;
  • mortality, 5-year and accounted trajectories match end to end across the two pins on 3 seeds;
  • no committed hash pins the moved node keys;
  • the API surface used at 3ff92b0a is all still present on main.

One minor finding is left for a follow-up: the comment at src/populace_dynamics/graph/trajectory.py:140-141 still describes the old pin's gate behaviour. Behaviour is unaffected.

Master moved from 22e3f82 to e9ad042 after CI ran. None of those 63 commits touch pyproject.toml, docs/population-graph.md, src/populace_dynamics/graph, tests/test_graph_* or the workflows. This PR adds no tests, so tier counts are unaffected.

Next: closing microcosm#873, which microcosm main (#1285) supersedes.

@MaxGhenis
MaxGhenis deleted the repin-microcosm-main-20261015 branch October 16, 2026 02:53
MaxGhenis added a commit that referenced this pull request Oct 16, 2026
…iers

Master gained #525, #528, #534, #535, #538, #539, #541, #543, #545,  since
this branch's base (9af5392). Conflicts were the two tier files only;
tests/track_u2/test_u2_loader.py, changed on both sides, merged cleanly:
the merged file is master's plus exactly this branch's one added test,
and this branch's plus exactly master's changes. Every other file equals
its one changing side.

Tier files rewritten from `pytest --collect-only -q -m <tier>` on the
merged tree: unit 9,902, artifact 4,244, integration_psid 1,423,
reproduction_legacy 520, oracle_policyengine 308 (16,397), which is
master's counts plus this branch's +1 unit.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
MaxGhenis added a commit that referenced this pull request Oct 16, 2026
…iers

Master gained #525, #528, #534, #535, #538, #539, #541, #543 and #545 since
this branch's base (9af5392). Conflicts were the two tier files only;
tests/track_u2/test_u2_loader.py, changed on both sides, merged cleanly:
the merged file is master's plus exactly this branch's one added test,
and this branch's plus exactly master's changes. Every other file equals
its one changing side.

Tier files rewritten from `pytest --collect-only -q -m <tier>` on the
merged tree: unit 9,902, artifact 4,244, integration_psid 1,423,
reproduction_legacy 520, oracle_policyengine 308 (16,397), which is
master's counts plus this branch's +1 unit.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch was successfully deployed

1 active deployment
Preview — f34888d3 Deployed Oct 15, 2026 by vercel[bot]
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