Skip to content

Comment out Nominatim code as it was far exceeding rate limits#278

Open
sean-dunwoody wants to merge 1 commit intomasterfrom
hotfix/15646-mitigate-downtime-issues
Open

Comment out Nominatim code as it was far exceeding rate limits#278
sean-dunwoody wants to merge 1 commit intomasterfrom
hotfix/15646-mitigate-downtime-issues

Conversation

@sean-dunwoody
Copy link
Contributor

The site was far exceeding the Nominatim usage policy, so these requests were being denied. We've disabled this functionality for now as, as far as we can tell, it doesn't seem to be being used anywhere important (and in any case it's not working at present).

See:

https://operations.osmfoundation.org/policies/nominatim/

@portableant
Copy link
Member

Background info: This should be cached as a lookup, like in the API:

The geo lookup was in place to enable linked art data exchange, coordinates in the API and coordinates for maps on the front end of the collection as the axiell system doesn't have them. Geocoding and storing the coords was on the collections roadmap in 2021/2022.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants