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

Consider dropping GeocodeFarm support #445

Closed
KostyaEsmukov opened this issue Sep 13, 2020 · 2 comments
Closed

Consider dropping GeocodeFarm support #445

KostyaEsmukov opened this issue Sep 13, 2020 · 2 comments
Milestone

Comments

@KostyaEsmukov
Copy link
Member

GeocodeFarm is the only geocoder which tests constantly fail due to the service availability issues. I have observed:

  • A week-long expired TLS certificate
  • Sporadic "No results" errors (8ce18b1)
  • Intermittent connection refused errors

In fact, this is the only service for which I had to add an environment variable to allow quickly disabling tests without a commit: a6af73a. And I occasionally have to switch on that env to make the test suite pass.

The current issue is that along with "connection refused" errors the service returns an inadequate address value: #437

I believe that level of service is inappropriate for a public API, so I'm considering removing GeocodeFarm geocoder in geopy 2.2, to avoid anyone wasting their time on a service which is that unstable.

@KostyaEsmukov
Copy link
Member Author

Removed in 5ac2473 and will be released as geopy 2.2.

@APIVersa
Copy link

APIVersa commented Apr 18, 2024

May I suggest adding it back, as it is now extremely stable and reliable. It was a downtime issue back then, but was resolved after the time it was removed from here. There are people still asking to use your solution, but cannot as it isn't in the newer packages, forcing them to use other soultions.

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

No branches or pull requests

2 participants