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

Shapefile drop-in should select geometries; use centroid-based selection #620

Open
EricaMaurer opened this issue Sep 28, 2018 · 6 comments
Labels
Luxury Low impact, high level of effort. Used for work prioritization.

Comments

@EricaMaurer
Copy link
Collaborator

EricaMaurer commented Sep 28, 2018

Shapefile drop-in current provides only a boundary for visual comparison. It should also select geometries, using a centroid-based selection.

We still need to decide how this will work with concave geometries — centroids may either be those on the surface of the feature/polygons or based on the bbox. Once we clarify the exact selection strategy, we can move forward.

Shapefile drop-in currently produces erroneous selection-- missing geographies within shapefile drop-in & extra selected areas beyond shapefile boundaries.

User should be able to see selected areas that have a centroid within their shapefile boundary

@allthesignals
Copy link
Collaborator

Can I have some sample Shapefiles for testing purposes?

@hannahkates
Copy link
Contributor

Will most likely use centroid method, but we'll explore other options. We'll also research Carto's methodology.

@hannahkates hannahkates added the Luxury Low impact, high level of effort. Used for work prioritization. label Nov 28, 2018
@hannahkates hannahkates removed the to do label Dec 13, 2018
@andycochran
Copy link
Contributor

Using centroids seems brittle to me, as a centroid often isn't within its polygon. I'd suggest we try programmatically changing the shapefile by putting a negative buffer on its polygons and making the selection using these shrunken polygons. This would prevent a polygon with a boundary that lies on a tract boundary from selecting the adjacent tract.

@allthesignals
Copy link
Collaborator

I think the way this feature works now makes this irrelevant, closing for now unless we should reopen

@EricaMaurer
Copy link
Collaborator Author

This may be something we want to revisit since we have heard from users that they'd like the application to select the geographies within their shapefile. This obviously doesn't work if they upload a shapefile that covers the entire city, but would work if it's a specific study area. Will keep it open for now.

@EricaMaurer EricaMaurer reopened this Mar 5, 2020
@allthesignals
Copy link
Collaborator

Okay thanks @EricaMaurer I'll adjust the issue title and description so that it remains the place where we talk about shapefile-upload-as-selector.

@allthesignals allthesignals changed the title Adjust selection method for shapefile drop-in Shapefile drop-in should select geometries; use centroid-based selection Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Luxury Low impact, high level of effort. Used for work prioritization.
Development

No branches or pull requests

4 participants