Repository navigation
story 10 3 division boundary crosswalk - #132
Merged
Merged
Conversation
cbwinslow
commented
Oct 10, 2026
cbwinslow
commented
Owner
- docs: add state campaign finance to the source catalog
- Add division_boundary and geography_crosswalk tables (Story 10.3)
- Link divisions to CD119 and SLD 2024 boundaries by identifier (Story 10.3)
- Add Census relationship and block equivalency crosswalk loaders (Story 10.3)
- docs: record Story 10.3 status and open items
- Link letter-suffixed state districts by rule; record 10.3 relationship load
- Link the remaining state districts through a reviewed OCD crosswalk (Story 10.3)
Expand-only migration a1d4c8e6f372 per ADR-0006: link political divisions to dated boundary vintages and record cross-geography relationships with explicit weight types, so a relationship overlap never masquerades as a population weight. Downgrade refuses to drop populated tables.
…10.3) State FIPS from the OCD state code plus the district code equals the TIGER GEOID; no name comparison. Numeric-coded state legislative districts are seeded as OCD divisions from the loaded boundaries; letter-coded districts are reported unlinked for a reviewed mapping. A link to a wrong vintage or boundary family fails the run.
…y 10.3) Plan, size preview, approve, resumable download, stage, load and validate for the 119th-Congress and 2024 state legislative relationship files (weight_type none, areas kept as evidence) and block equivalency files (whole-block assignment, weight 1). Registers both datasets and their field checklists; stage tables ship in the same expand-only revision.
…p load Census codes Alaska senate and MD/MN/ND/SD house seats with letters (01A); OpenStates writes the same seat as sldl:1a. The rule is code-to-code and verified against the now-readable opencivicdata_division. Vermont and Massachusetts senate stay unlinked pending a reviewed mapping.
…Story 10.3) Massachusetts, New Hampshire House, Vermont, DC wards and Puerto Rico share no code with OpenStates. scripts/build_ocd_sld_crosswalk.py derives each OCD id from the official Census district name and refuses to write unless every id is in the pinned official OCD list, is unique, and exists in OpenStates. The linker reads the 545 reviewed pairs by identifier and retires the 380 numeric-seeded divisions that matched nothing in OpenStates.
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
⛔ Files ignored due to path filters (1)
📒 Files selected for processing (35)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Four downgrade-refusal tests hard-coded the previous head revision, so adding a migration broke them. They now ask Alembic for the head.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.