Skip to content

Publish TRACE TRO declarations for US data releases#746

Merged
MaxGhenis merged 5 commits intomainfrom
codex/trace-tro-export-release-manifest
Apr 21, 2026
Merged

Publish TRACE TRO declarations for US data releases#746
MaxGhenis merged 5 commits intomainfrom
codex/trace-tro-export-release-manifest

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Summary

  • add a TRACE TRO builder for US data release manifests
  • upload trace.tro.jsonld alongside the floating and versioned release manifests
  • add focused regression coverage for TRO generation and HF commit operations

Testing

  • PYTHONPATH=/Users/maxghenis/PolicyEngine/_codex_prs/policyengine-us-data-trace-tro-export-release-manifest /Users/maxghenis/PolicyEngine/policyengine-us-data/.venv/bin/pytest /Users/maxghenis/PolicyEngine/_codex_prs/policyengine-us-data-trace-tro-export-release-manifest/policyengine_us_data/tests/test_release_manifest.py /Users/maxghenis/PolicyEngine/_codex_prs/policyengine-us-data-trace-tro-export-release-manifest/policyengine_us_data/tests/test_trace_tro.py
  • /Users/maxghenis/PolicyEngine/policyengine-us-data/.venv/bin/python -m ruff check policyengine_us_data/utils/data_upload.py policyengine_us_data/utils/release_manifest.py policyengine_us_data/utils/trace_tro.py policyengine_us_data/tests/test_release_manifest.py policyengine_us_data/tests/test_trace_tro.py

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 13, 2026

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

Project Deployment Actions Updated (UTC)
pipeline-diagrams Error Error Apr 13, 2026 6:28pm

Request Review

MaxGhenis and others added 5 commits April 20, 2026 18:58
- Drop the non-canonical ArrangementBinding wrapper; trov:contributedToArrangement
  links directly to the arrangement (matching trov.ttl)
- Add pe:* extension namespace and surface model version, git sha, data-build
  fingerprint, and data build id as structured fields on the performance node
  so a verifier can cross-check against policyengine.py's bundle TRO without
  parsing prose
- Add pe:emittedIn / pe:ciRunUrl / pe:ciGitSha / pe:ciGitRef attestation
  derived from GITHUB_ACTIONS env vars
- Switch composition fingerprint to "\n".join(sorted(hashes)), matching
  policyengine.py's algorithm so fingerprints are recomputable across repos
- Make composition/arrangement/performance index numbering consistent (all 1)
- Ship policyengine_us_data/schemas/trace_tro.schema.json and validate
  emitted TROs against it in the test suite
- Replace the tautological serialize-twice determinism test with a
  build-twice round-trip
- Cover local vs github-actions emission context in tests

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
ruff format has been failing on every open PR because cps.py drifted
out of canonical form. Apply `ruff format` so this PR's lint passes.
Tracked upstream in #804.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@MaxGhenis MaxGhenis force-pushed the codex/trace-tro-export-release-manifest branch from 6883d29 to 1c94821 Compare April 20, 2026 22:58
@MaxGhenis MaxGhenis marked this pull request as ready for review April 21, 2026 02:47
@MaxGhenis MaxGhenis merged commit 37271be into main Apr 21, 2026
8 of 9 checks passed
@MaxGhenis MaxGhenis deleted the codex/trace-tro-export-release-manifest branch April 21, 2026 02:47
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