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 Michigan to Land Area #9

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

Conversation

hodgesmr
Copy link

@hodgesmr hodgesmr commented Jan 4, 2018

This pull request serves as an idea for the rest of the state files. I chose to start with Michigan because it illustrates the diff and would be good to solicit feedback.

The current geojson for Michigan renders into the bodies of water around the state. This makes for unexpected shapes when drawn. I noticed that the district geojson provided by @schwanksta in the 2016 files clipped to the land area. To get a state clipped to land, I created a union of all the MI districts, and then dissolved their geometries into fewer polygons.

Before:
screen shot 2018-01-03 at 11 52 09 pm

After:
screen shot 2018-01-03 at 11 51 27 pm

If this change is deemed desirable, I can follow the same process for the other states.

@ogtega
Copy link
Contributor

ogtega commented Jan 5, 2018

Aren't the areas within bodies of water also part of the state's boundaries?

@hodgesmr
Copy link
Author

hodgesmr commented Jan 5, 2018

@ogtega I believe so, and that's part of my question around the general "desirability" of the PR. Depending on your usecase you might want one or the other. Like, if you want to say if a point is "within" a state's jurisdiction, you probably do want to include bodies of water... but if you want to draw a map of the US with state lines, you might not expect the water to be included (this is what I was doing and noticed how weird Michigan and Wisconsin looked).

@konklone
Copy link
Member

konklone commented Jan 6, 2018

I think we'd likely be better off, as a project that intends to have authoritative data, to avoid weakening the authoritativeness of a piece of data in favor of aesthetic impact in some circumstances.

Even on Google Maps, they include water in the visible boundary:

screen shot 2018-01-06 at 6 53 14 pm

Though if there's interest in maintenance, maybe we could maintain both for the several states that have significant water within their boundaries.

Just my thoughts - not trying to shut down discussion.

@schwanksta
Copy link
Collaborator

@konklone I get that argument. Like I said over in #10, I'm not sure I see the utility of a congressional districts repo containing state files either way, but if we want to maintain those (since geojson is nicer, and we can link to them individually), what about starting a new states repo with clipped and unclipped files?

@ogtega
Copy link
Contributor

ogtega commented Jan 7, 2018

@schwanksta it only makes sense

@konklone
Copy link
Member

konklone commented Jan 8, 2018

@schwanksta This repository is/was intended to show the shapes of districts for everyone in the US Congress, and for senators that's their states. We also want the repository to be useful to folks who want to pull down the GeoJSON via permanent URLs -- we have this hooked up to GitHub Pages and CloudFlare so that they are downloadable from https://theunitedstates.io.

I think if another more reliable/maintained source offered consistent perma-URLs to GeoJSON for states or districts, we could refer to that source instead, though we'd at least need to maintain the URLs in place for a while, while we identified clients currently using the data and planned a deprecation schedule.

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.

4 participants