Skip to content

Add secondary upload destination#280

Merged
MaxGhenis merged 2 commits intomainfrom
fix/upload-to-public-repo
Feb 23, 2026
Merged

Add secondary upload destination#280
MaxGhenis merged 2 commits intomainfrom
fix/upload-to-public-repo

Conversation

@MaxGhenis
Copy link
Contributor

@MaxGhenis MaxGhenis commented Feb 23, 2026

Reverted in #283 — unnecessary change.

MaxGhenis and others added 2 commits February 23, 2026 07:00
The push workflow uploads to policyengine/policyengine-uk-data-private
but policyengine-uk downloads from policyengine/policyengine-uk-data.
This means new columns (like highest_education) never reach downstream
consumers. Fix by also uploading to the public repo.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
upload_files_to_hf requires a version parameter for tagging.
Without it the upload would fail with TypeError at runtime.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MaxGhenis MaxGhenis merged commit 394117b into main Feb 23, 2026
3 checks passed
MaxGhenis added a commit that referenced this pull request Feb 23, 2026
@MaxGhenis MaxGhenis mentioned this pull request Feb 23, 2026
@MaxGhenis MaxGhenis changed the title Fix: upload datasets to public HuggingFace repo Add secondary upload destination Feb 23, 2026
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