Skip to content

Tighten managed dataset fallback rules#277

Merged
MaxGhenis merged 4 commits into
mainfrom
codex/managed-runtime-followup
Apr 16, 2026
Merged

Tighten managed dataset fallback rules#277
MaxGhenis merged 4 commits into
mainfrom
codex/managed-runtime-followup

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Summary

  • only fall back to bundled data certification when the data release manifest is unavailable, not on arbitrary fetch or parse failures
  • preserve explicit unmanaged dataset URIs instead of silently rewriting them to local managed mirrors
  • add regression tests for missing data manifests, fallback behavior, and unmanaged dataset source preservation

Testing

  • PATH=/Users/maxghenis/PolicyEngine/policyengine.py/.venv/bin:$PATH /Users/maxghenis/PolicyEngine/policyengine.py/.venv/bin/ruff check src/policyengine/core/release_manifest.py src/policyengine/tax_benefit_models/us/model.py src/policyengine/tax_benefit_models/uk/model.py tests/test_release_manifests.py
  • VIRTUAL_ENV=/Users/maxghenis/PolicyEngine/policyengine.py/.venv PATH=/Users/maxghenis/PolicyEngine/policyengine.py/.venv/bin:$PATH PYTHONPATH=src /Users/maxghenis/PolicyEngine/policyengine.py/.venv/bin/python -m pytest tests/test_release_manifests.py -q

@MaxGhenis MaxGhenis merged commit 159dedb into main Apr 16, 2026
5 checks passed
@MaxGhenis MaxGhenis deleted the codex/managed-runtime-followup branch April 16, 2026 14:23
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