Skip to content

Allow raw CPS source artifact through publication contract#1012

Merged
MaxGhenis merged 1 commit into
mainfrom
codex/raw-cps-publication-contract
May 18, 2026
Merged

Allow raw CPS source artifact through publication contract#1012
MaxGhenis merged 1 commit into
mainfrom
codex/raw-cps-publication-contract

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Summary

  • keep enforcing the leaf-input export contract for final simulation datasets
  • exempt the raw CPS source artifact from the no-computed-policyengine-us-variable check during publication validation
  • add regression coverage showing raw CPS can publish source variables while enhanced CPS still rejects computed exports

Testing

  • uv run pytest tests/unit/test_dataset_validation.py tests/unit/test_upload_completed_datasets.py
  • uv run ruff format policyengine_us_data/utils/dataset_validation.py policyengine_us_data/storage/upload_completed_datasets.py tests/unit/test_dataset_validation.py tests/unit/test_upload_completed_datasets.py
  • uv run ruff check policyengine_us_data/utils/dataset_validation.py policyengine_us_data/storage/upload_completed_datasets.py tests/unit/test_dataset_validation.py tests/unit/test_upload_completed_datasets.py

@MaxGhenis MaxGhenis force-pushed the codex/raw-cps-publication-contract branch from b386006 to e999e89 Compare May 18, 2026 19:26
@MaxGhenis MaxGhenis merged commit 8868ad3 into main May 18, 2026
13 checks passed
@MaxGhenis MaxGhenis deleted the codex/raw-cps-publication-contract branch May 18, 2026 19:56
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