valhalla / valhalla/valhalla

Proposal: Report which locations a route error is about

Open
#6,281 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
C++
Stars
6.2k
Forks
981
Avg merge
2d 14h
Merged PRs (30d)
14

Description

While building an app that uses Valhalla I found that a general "failed to route" error left the user experience lacking. On a multi-step route I had to do a human binary search on which location failed.

It would be good to be able to get an index reference to the location or segment that failed the route calculation. That was the UI can match that up with the input location and provide UI affordances (zoom to or visualise the problematic locations/segments).

I have a prototype on forwarding that info into the returned error. But I'm not sure it's an appropriate fix overall. I'll put up a PR for review.

Contributor guide

Open the contributing guide

Research direction

No files or tests are named. Start by tracing the route-calculation error path and reviewing the author's prototype for forwarding the failing location or segment; done means the returned error identifies the relevant input location or segment so a client can match and visualize it.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
api, backend, backend-api-design
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.