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

Backend: New new geocoding library #178

Open
laredotornado opened this issue Mar 17, 2022 · 7 comments · Fixed by #228
Open

Backend: New new geocoding library #178

laredotornado opened this issue Mar 17, 2022 · 7 comments · Fixed by #228

Comments

@laredotornado
Copy link
Collaborator

We are using the following GeoCoding library

geopy==1.21.0

for resolving addresses to latitude and lonitudes. However, this library now has a dependence on Google, which we would prefer not to use. We need to use a new library, which would fit in to the file directory/services/location_service.py in the "get_coords" method.

The only requirement is that this library be Python based and free to use, even if it is rate-limited.

@stepsbystep
Copy link
Collaborator

Sign me up @laredotornado!

@laredotornado
Copy link
Collaborator Author

Thanks @stepsbystep ! assigned

@stepsbystep
Copy link
Collaborator

New geocoding library is tested and in use with the current parse_coop_csv @laredotornado!

We are now using the OpenStreet Maps nominatim library
Check the following for more information
https://www.natasshaselvaraj.com/a-step-by-step-guide-on-geocoding-in-python/
https://nominatim.org/release-docs/latest/api/Overview/

There are two pushes. The first has the new code but is not properly described. The second push adds a line of documentation.

@stepsbystep
Copy link
Collaborator

I've created the getGeoLoc object class. It functions correctly in parse_coop_csv.

@stepsbystep
Copy link
Collaborator

LocationServices updated but not working yet

@SteveEdiger SteveEdiger moved this from To do to Moved to production in Version 3.1 (CLASSIC - DONT USE) Sep 22, 2022
@SteveEdiger
Copy link
Collaborator

SteveEdiger commented Oct 11, 2022 via email

@SteveEdiger SteveEdiger moved this from Moved to production to Test-QA in Version 3.1 (CLASSIC - DONT USE) Dec 2, 2022
@maxgraziano maxgraziano self-assigned this May 8, 2024
@maxgraziano
Copy link
Collaborator

Fixed in '3.1-model-refactor'. Continues to use GeoPy but fetches data from Nominatim as opposed to Google.

@maxgraziano maxgraziano linked a pull request May 29, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Test-QA
Development

Successfully merging a pull request may close this issue.

4 participants