Skip to content

Pin deployed model source revisions#105

Merged
vahid-ahmadi merged 1 commit into
mainfrom
agent/pin-model-source-revisions
Jul 26, 2026
Merged

Pin deployed model source revisions#105
vahid-ahmadi merged 1 commit into
mainfrom
agent/pin-model-source-revisions

Conversation

@vahid-ahmadi

Copy link
Copy Markdown
Contributor

What changed

  • pins OBR, BoE SVAR, FRB/US, and US HANK dependencies to reviewed 40-character Git commits
  • makes Modal fetch those exact detached revisions rather than mutable main branches
  • injects deployed revisions into production adapter provenance
  • corrects BoE SVAR and FRB/US source URLs in provenance
  • updates reproducibility instructions when an exact revision is available
  • adds tests that require full SHAs and keep pyproject pins synchronized with Modal pins

Why

Production provenance previously reported package versions while deployment cloned mutable branches. Several model repositories retain version 0.1.0 across code changes, so a published run could not be reconstructed from its provenance.

Model updates now require an intentional reviewed pin change instead of silently changing the next deployment.

Validation

  • 29 focused provenance, source-pin, and adapter tests passed
  • Ruff passed on changed Python files
  • Python compilation passed
  • git diff --check passed

@vercel

vercel Bot commented Jul 26, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
macromod Ready Ready Preview, Comment Jul 26, 2026 1:52pm

Request Review

@vahid-ahmadi
vahid-ahmadi force-pushed the agent/pin-model-source-revisions branch from 0c56c56 to 10c4fa8 Compare July 26, 2026 13:52
@vahid-ahmadi
vahid-ahmadi marked this pull request as ready for review July 26, 2026 13:53
@vahid-ahmadi
vahid-ahmadi merged commit 7b0ccb3 into main Jul 26, 2026
5 checks passed
@vahid-ahmadi
vahid-ahmadi deleted the agent/pin-model-source-revisions branch July 26, 2026 13:53
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