cartodb4/dataservices-api
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #610 from CartoDB/ch102254

Browse Source 
HERE Bulk geocoder: Read all responses
This commit is contained in:
Raúl Marín 2020-11-19 10:03:42 +01:00 committed by GitHub
commit f72dabd0a9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
NEWS.md
View File

@ -1,9 +1,10 @@
Nov 11th, 2020
Nov 18th, 2020
==============
* Version `0.39.2` of the server extension
* Version `0.23.4` of the Python library
* Better messages on remote errors.
* Fix error on unknown status codes.
* Here bulk geocoder: Return also errors and no-matches.
Sep 23th, 2020
==============

7
server/lib/python/cartodb_services/cartodb_services/here/bulk_geocoder.py
View File

@ -126,7 +126,7 @@ class HereMapsBulkGeocoder(HereMapsGeocoder, StreetPointBulkGeocoder):
status=polling_root.find('./Response/Status').text)
def _download_results(self, job_id):
result_r = self.session.get("{}/{}/result".format(self.BATCH_URL, job_id),
result_r = self.session.get("{}/{}/all".format(self.BATCH_URL, job_id),
params=self.credentials_params,
timeout=(self.connect_timeout, self.read_timeout))
root_zip = zipfile.ZipFile(io.BytesIO(result_r.content))
@ -147,6 +147,9 @@ class HereMapsBulkGeocoder(HereMapsGeocoder, StreetPointBulkGeocoder):
precision,
[match_type] if match_type else []
)))
elif row['matchLevel'] == 'NOMATCH':
results.append((row['recId'], [], {}))
elif row['matchLevel'] == 'FAILED':
results.append((row['recId'], [], {'error': 'Bulk geocoder failed'}))
return results