ustroetz / python-osrm

A Python wrapper around the OSRM API
MIT License
133 stars 55 forks source link

Add annotations to table #23

Closed zifeo closed 6 years ago

ustroetz commented 6 years ago

@mthh could you please have a look?

mthh commented 6 years ago

LGTM as long as the user respects the documentation of our table function which states :

annotations : str, optional
    Either 'duration' (default) or 'distance'

because OSRM also allows you to use both simultaneously by using duration,distance which we currently don't handle correctly (maybe as an improvement for a future PR !?)