Nicer error reporting when geocoder fails about open-buildings HOT 6 OPEN

I tried a simple fix for this

def geocode(data: str):
    try:
        location = ox.geocode_to_gdf(data)
        geom = location.geometry[0]
        geojson = json.loads(json.dumps({"type": "Feature", "geometry": mapping(geom)})) # turn geom tuple into list by (de-)serialising
        return geojson
    except InsufficientResponseError as e: 
         print(f"An error occurred: {e}")

but getting issues with the geojson_to_quadkey function in download_buildings

  File "/usr/local/lib/python3.10/dist-packages/open_buildings-0.10.0-py3.10.egg/open_buildings/download_buildings.py", line 23, in geojson_to_quadkey
    geom = shape(data["geometry"])
TypeError: 'NoneType' object is not subscriptable

@felix-schott can you try reproducing please?

from open-buildings.

Hi @mtravis - not on my laptop at the moment, so can't reproduce but I'm guessing it might be because the programme is not exiting and because the geocoding is unsuccessful, the programme continues and expects a geometry instead of None where it raises the error. try-except is the right approach to suppress the long error message, but we need to exit the programme immediately afterwards. A similar thing happens here - maybe copy that approach? The empty return terminates the programme. (On Linux, it would be better to use sys.exit(1) instead of an empty return but I'm not sure how that behaves on other platforms, so just use the empty return.)

from open-buildings.

I'd also print a more meaningful error message: "No results found for {location} - please try a different search term." Otherwise users don't know what's happening.

from open-buildings.

error

Actually, sorry, you will have to use sys.exit or raise an exception since return will only exit the helper function, not the whole programme. It might be cleaner to return None from the geocoding function if no result could be found, and then use the empty return approach in the calling function if geocoding_result is None. Not sure if that's clear, if I have time, I can give you a code snippet

from open-buildings.

@felix-schott

I've got it working with sys.exit but will try implementing the return None and updating the calling function later.

My error message is sensible looking once it calls the internal message from the event.

from open-buildings.

@felix-schott I'm struggling a bit with to get the script to exit fully without sys.exit(1). I agree it would be cleaner not to use this method but can't get download_buildings to work.

from open-buildings.