pelias / openstreetmap

Import pipeline for OSM in to Pelias
MIT License
112 stars 72 forks source link

Allow configuring whether disused/abandoned venues are removed #537

Closed orangejulius closed 4 years ago

orangejulius commented 4 years ago

Currently, our popularity mapper from #493/ #531 also causes any venues with low popularity (usually those marked as disused or abandoned in OSM) to be removed, and not imported into Pelias:

https://github.com/pelias/openstreetmap/blob/02e4fd7c01fe5644cec010a3f82ddd2bedca58c9/stream/popularity_mapper.js#L202-L203

This might be a reasonable default, but it's also pretty significant. There might be a lot of notable places that fall into those categories. We should look at adding a configuration option to enable or disable removal of those venues.

orangejulius commented 4 years ago

This was fixed in https://github.com/pelias/openstreetmap/pull/539: by default we will now continue to import these venues, but there is a configuration option to remove them if desired.