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

Error in OSRM routing; how to handle this case? #30

Closed
perrygeo opened this issue Aug 9, 2013 · 5 comments
Closed

Error in OSRM routing; how to handle this case? #30

perrygeo opened this issue Aug 9, 2013 · 5 comments

Comments

@perrygeo
Copy link
Contributor

perrygeo commented Aug 9, 2013

http://router.project-osrm.org/nearest?loc=44.530765,-122.510725
and
http://router.project-osrm.org/nearest?loc=44.530451,-122.517537

both return something like:

'{"version":0.3,"status":0,"mapped_coordinate":[44.530607,],"name":"","transactionId":"OSRM Routing Engine JSON Nearest (v0.3)"}'

Note the incomplete coordinate (no longitude returned). This results in:

line 59, landing.py:  json.loads(binary)

*** ValueError: No JSON object could be decoded

We will need to guard against this case and/or work with OSRM to fix it.

@perrygeo
Copy link
Contributor Author

perrygeo commented Aug 9, 2013

inquiry on the OSRM mailing list: http://lists.openstreetmap.org/pipermail/osrm-talk/2013-August/000239.html

EDIT: Dennis, the lead dev at OSRM, has clearly acknowledged the issue as of Aug 14 but hasn't been very responsive in fixing it.

@perrygeo
Copy link
Contributor Author

perrygeo commented Aug 9, 2013

For now the forestplanner is just ignoring properties where this occurs. Unfortunately, it appears to be a systematic problem with the osrm service so hopefully it gets resolved soon.

@perrygeo
Copy link
Contributor Author

Submitted a github issue and patch to fix it. Project-OSRM/osrm-backend#723

If the patch is not accepted, we may need to fork and run our own server?

@DennisOSRM
Copy link

Already fixed earlier Project-OSRM/osrm-backend@a6c52fd

@perrygeo
Copy link
Contributor Author

@DennisOSRM Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants