eurodatacube / eodash

Software behind the RACE dashboard by ESA and the European Commission (https://race.esa.int), the Green Transition Information Factory - GTIF (https://gtif.esa.int), as well as the Earth Observing Dashboard by NASA, ESA, and JAXA (https://eodashboard.org)
https://race.esa.int
MIT License
89 stars 43 forks source link

Overpass API - new tags #2497

Closed UndeadFairy closed 3 months ago

UndeadFairy commented 4 months ago

During today meeting we discussed the need to add more OSM tags for Overpass API selection for indicator 4 - Flood risk.

Lets's collect here what would be interest. Currently we have school and hospital only.

@sistemagmbh @alexaib please add you ideas to this issue.

UndeadFairy commented 4 months ago

Based on email from Alexandra, we will add only "building"="residential" :heavy_check_mark: and "man_made"="storage_tank" :heavy_check_mark: tags

lubojr commented 4 months ago

Number of "building"="residential" will be absolutely massive for the client to handle to be usable on anything above a size of a small town although I understand the possible importance.

We can limit the total number of features returned and show a warning https://help.openstreetmap.org/questions/20472/how-to-request-a-limited-no-of-features-in-overpass-api but this will be shared for all requested types of features, not just buildings.

alexaib commented 4 months ago

@lubojr for the "building"="residential" is it possible to show a warning to the user that the chosen area is too large for this feature and a smaller area has to be chosen, instead of limiting the total number of features?

lubojr commented 4 months ago

We do not have the framework in place to show custom warnings for a single type of feature and when drawn area is greater than some threshold. I think adding such custom one-off functionality is not desirable in current state of the demonstrator.

I would suggest a counter proposal - to add the "building"="residential" as a possible new tag, but leave it switched "off" as default.

alexaib commented 4 months ago

@lubojr your proposal sounds good

lubojr commented 4 months ago

Implemented and deployed in ideas_i4 branch

alexaib commented 4 months ago

Thank you very much @lubojr