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

Should masterShapeDB point to seattleflu/geo-json? #4

Closed
famulare opened this issue Apr 3, 2019 · 5 comments
Closed

Should masterShapeDB point to seattleflu/geo-json? #4

famulare opened this issue Apr 3, 2019 · 5 comments
Assignees

Comments

@famulare
Copy link
Member

famulare commented Apr 3, 2019

We need to agree on shapes, but that requires choosing a valid domain. See seattleflu/seattle-geojson#1 for discussion.

@tsibley
Copy link
Member

tsibley commented Apr 3, 2019

My 2¢: I think I need our geospatial shapes in the database anyway, so that location could become the authority for shape data if we wanted.

@grhuynh
Copy link
Collaborator

grhuynh commented Apr 15, 2019

I'll try to connect the masterSpatialDB to seattleflu/geo-json, assuming those files are not being further modified?

@famulare
Copy link
Member Author

Great! They are fixed for now. They might change in the future, but the format should remain geojson. The CRA_NAME and NEIGHBORHOOD_... are the least stable because they are only defined for Seattle city limits right now. In principle, we can expand those, but as far as I can tell we have to get neighborhood-level shapes from each town, as I can't find a central repository.

@famulare
Copy link
Member Author

@grhuynh If/when you do that, please branch off of master, and when you think it all works, put in a pull request.

@famulare
Copy link
Member Author

Another comment. I just noticed that the seattle-geojsons are literally seattle only. So, keep an option to point to the original shapefiles too, as those are all king county. In May, we collectively should expand the curated scope and how we selectively access it: (see seattleflu/seattle-geojson#1 (comment))

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

No branches or pull requests

3 participants