Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

Geocode entities tagged 'location' #11

Closed
alexhebing opened this issue Apr 9, 2019 · 4 comments
Closed

Geocode entities tagged 'location' #11

alexhebing opened this issue Apr 9, 2019 · 4 comments
Assignees
@BeritJanssen
Labels
enhancement New feature or request
Milestone
command line vers...

Comments

@alexhebing
Copy link

alexhebing commented Apr 9, 2019 •
edited
Loading

Show the results of collecting coordinates for each LOCATION in the webinterface (just add them in the table). It should collect coordinates from two systems:

  1. GeoParser
  2. Google API based solution, more info here and here

Implement collecting data from the two Geo systems in a separate application / module in the multiNER app: we might want to separate it later.
@alexhebing alexhebing added the enhancement New feature or request label Apr 9, 2019
@alexhebing alexhebing self-assigned this Apr 9, 2019
@alexhebing alexhebing changed the title Collect coordinates Geocode entities tagged 'location' Apr 9, 2019
@BeritJanssen BeritJanssen self-assigned this Apr 10, 2019
@BeritJanssen
Copy link
Member

It seems that geoparser.io already takes care of the NER part as well. Probably not so suitable then. I came across the geocoder library and it seems to provide options to get lat long data from various sources. I'll request api keys for GeoNames and Google and see how far I get.

@alexhebing
Copy link
Author

That geocoder library looks very promising indeed!

@alexhebing
Copy link
Author

OK, I have it running locally with a personal GeoNames account, and a Google API Key we already had for a previous project. This is just meant for testing, and of course showcasing to the client, whom I will meet with tomorrow.

@alexhebing alexhebing mentioned this issue May 7, 2019
Open
alexhebing pushed a commit that referenced this issue May 24, 2019
Refer #17, #11, #18. First version of command line tool, including ge…
3690eea 
…ocodes, and (bad) HTML output
alexhebing pushed a commit that referenced this issue May 24, 2019
Refer #11 and #18. Change Google method to place, requires language
7af4f15
@alexhebing
Copy link
Author

Continuing this in #18 and #21 : exclude Google, add cachine, and it's done for now.

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

@BeritJanssen BeritJanssen

Labels
enhancement New feature or request
Projects
None yet
Milestone
command line version 1
Development

No branches or pull requests
2 participants
@BeritJanssen @alexhebing