mapbox / mapbox/mapbox-navigation-android

Navigation UI offroute stuck fetching new route.

Open
#2,222 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Kotlin
Stars
651
Forks
321
PR merge metrics
No merged PRs in 30d

Description

```
(...)
@Override
public boolean allowRerouteFrom(Point offRoutePoint)
{
return true;
}
(...)

NavigationRoute.builder(getContext()).accessToken(getString(R.string.mapbox_access_token)).origin(origin).destination(destination).enableRefresh(true).build().getRoute(new Callback()
{
@Override
public void onResponse(Call call, Response response)
{
if (response.body() == null || response.body().routes().isEmpty())
return;

startNavigation(response.body().routes().get(0));
}

@Override
public void onFailure(Call call, Throwable t)
{
Timber.e(t);
}
});
```
Navigation UI SDK re routing works on most API and CPU but on the 'Cortex-A55' the re-routing is not completed, instead, it crashes the app by being stuck for too long.

**Android API: 29
**Mapbox Navigation UI SDK version: 0.42.0

### Steps to trigger behavior

1. Go off course.
2. Wait for rerouting

### Expected behavior

After going off course, a new route would be fetched and drawn from the point where the user is currently in.

### Actual behavior

After going off course, the listeners are triggered but a new route is not fetched, crashing the app.

Is this a bug for this kind of CPU or a poor implementation of the SDK?

Thank you
Tiago from Binary Pig

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the allowRerouteFrom(Point offRoutePoint) callback and the NavigationRoute.builder(...).getRoute flow shown in the issue. Reproduce the reroute on Android API 29 using a Cortex-A55 device and compare it with the working devices, including the response and failure callbacks. Done means going off route fetches and draws a new route without the app hanging or crashing.

Written by the indexing model from the issue text.

Assessment

Tech stack
android, kotlin
Domain
mobile
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.