-
Notifications
You must be signed in to change notification settings - Fork 47
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
Multiple amenities in a single POI point #633
Comments
This seems to be a good approach. I think we should make sure that the new geometries are still intersecting the building. Currently for the POIs derived from polygons we simply take the centroid. Usually this is fine but the issue comes with buildings that have holes. Maybe we could also make sure here to snap to the closest part of the polygon. |
According to our last meeting, the approach has been changed to handle angles that vary according to the number of repeated entities. This new approach has been implemented and tested and will be added in the next PR. |
Hey @rafleo2008 @EPajares , |
There is a possibility of having more than one amenity in a specific geographic point (like sports complex entry points/shopping mall doors) that groups different amenities. But, It is necessary to avoid to duplicate points. Therefore, a solution is needed.
One first approach is to group different amenities in a new array type column, but there are some issues when the new array type amenity column is implemented. In the score calculations, for example, is possible to match the amenity with a category, but it is not possible to get a double score when we have double matches. (like in thematic_data_sum). And the function will calculate only one value, even though when there are more values in the arrays.
A second approach could be to create an "unnest function" only for accessibility calculations and create "temporal points" only for the calculation process,
I am looking for a suitable approach.
The text was updated successfully, but these errors were encountered: