-
Notifications
You must be signed in to change notification settings - Fork 5
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
Adjust shapes at edges of zipcode domain to match zips instead of all pumas #8
Comments
Does adjusting these shapes affect the model computation, or is the goal solely to make display better? |
Display only. |
For now anyway. If/when doing more with census, we just need to define things clearly. |
@tsibley I uploaded shapes with better rural boundaries. Copying slack comment here too: I think they'll be good enough to display, but I defer to you on UI/UX if you still aren't happy with the viz. We can further refine these if we want to, but no time before launch. I did the clipping based on zipcodes, and that left a few edge cases that I don't like (such as including Tiger Mountain where no one lives but excluding Snoqualmie and North Bend). But these cover 95%+ of the SFS samples from year 1 |
Thanks! These definitely look better. I've switched the map over to them. :-) I would still like to clip these (at least for display) to not go over water as well. |
Will remove water before Monday morning. |
Done. Cropped out Lakes Union, Washington, and Sammamish. |
Compare zipcode boundary https://github.com/seattleflu/seattle-geojson/blob/master/sfs_domain_geojsons/sfs_domain_zipcodes.geojson
to puma-matched boundary https://github.com/seattleflu/seattle-geojson/blob/master/sfs_domain_geojsons/sfs_domain_neighborhood%2Bpuma.geojson
The puma matching puts a lot of visual mass on low density populations we aren't really serving. We should consider restricting the boundaries to the zip code edge.
The potential downside is making it harder to match to census data, but if we do that at the tract level, I don't think it's a problem.
The text was updated successfully, but these errors were encountered: