-
Notifications
You must be signed in to change notification settings - Fork 353
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
Improve itinerary description #9
Comments
Any progress on this ? I get not only undetailed but sometimes empty instructions, which look awful in the UI (it result in a empty list item). |
@moimael wow! Do you have an example (waypoint coords or similar) where this happens? |
@moimael the frontend is not really to blame here. The main problems are:
There is some code in OSRM to generate better turn instructions, but currently it is too slow to be used in production: |
@TheMarex good to know. Do you know a relevant issue for OSRM-backend we could l link to here? Project-OSRM/osrm-backend#685 seems like it could be related. |
@TheMarex Thanks for the explaination ! Do you have any pointer to speed up OSRM code ? |
@moimael to be honest its not a really priority at the moment. If you would like to take a step yourself, we are always happy to help new contributors. |
The text description of a route is sometimes hard to follow, and not very detailed. Look into what can be improved.
See for example: #6 (comment)
The text was updated successfully, but these errors were encountered: