Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clip states closer to land area based on 2016 districts #10

Open
wants to merge 4 commits into
base: gh-pages
Choose a base branch
from

Conversation

hodgesmr
Copy link

@hodgesmr hodgesmr commented Jan 4, 2018

This is an extension of #9.

I went ahead and followed the same process but for all 50 states: using @schwanksta's 2016 district geojson to rebuild the state lines much closer to land masses.

If this is not a desirable change, then we can just close both PRs.

@schwanksta
Copy link
Collaborator

I think this is a very desirable change in general, though two things: a) the Census actually publishes state shapefiles that are clipped to land (https://www.census.gov/geo/maps-data/data/cbf/cbf_state.html), but b) do we even need states in the repo? I'm not sure of the utility of maintaining our own, but I admit I've never asked anyone.

@ogtega
Copy link
Contributor

ogtega commented Jan 5, 2018

Like I said here, additionally, once I resolve a coordinate's district it is really easy to get the coordinate's state. Maybe move the state topology files into their own repo?

@hodgesmr
Copy link
Author

hodgesmr commented Jan 5, 2018

@schwanksta a) awesome, thanks for the links! Was digging around for that before I decided to try to generate them, this repo was much easier to find than the census page.

From @ogtega:

Maybe move the state topology files into their own repo?

This seems like a fair idea, given this repo's name. But I also get @schwanksta's question of utility given availability from the Census.

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.

3 participants