mapbox / mapbox/mapbox-navigation-android

[Android Auto] Fatal Exception: java.lang.Error: latitude must not be NaN

Open
#7,308 1 comment 1 reaction 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

**Android API:**
**Mapbox Navigation SDK version:**
0.22.0

### Steps to trigger behavior
N/A

### Expected behavior
No crash

### Actual behavior

The following exception is reported a few times in our analytics. Not sure what is provoking this.

```
Fatal Exception: java.lang.Error: latitude must not be NaN
at com.mapbox.maps.Projection.unproject(SourceFile)
at com.mapbox.maps.MapboxMap.unproject(MapboxMap.java:5)
at com.mapbox.maps.plugin.animation.CameraAnimatorsFactory.getFlyTo$lambda-9(CameraAnimatorsFactory.java:95)
at android.animation.KeyframeSet.getValue(KeyframeSet.java:202)
at android.animation.PropertyValuesHolder.calculateValue(PropertyValuesHolder.java:1017)
at android.animation.ValueAnimator.animateValue(ValueAnimator.java:1698)
at android.animation.ValueAnimator.animateValuesInRange(ValueAnimator.java:1512)
at android.animation.AnimatorSet.animateValuesInRange(AnimatorSet.java:1025)
at android.animation.AnimatorSet.animateBasedOnPlayTime(AnimatorSet.java:883)
at android.animation.AnimatorSet.initChildren(AnimatorSet.java:1164)
at android.animation.AnimatorSet.startAnimation(AnimatorSet.java:1384)
at android.animation.AnimatorSet.start(AnimatorSet.java:741)
at android.animation.AnimatorSet.start(AnimatorSet.java:696)
at com.mapbox.navigation.ui.maps.camera.NavigationCamera.startAnimation(NavigationCamera.java:72)
at com.mapbox.navigation.ui.maps.camera.NavigationCamera.requestNavigationCameraToFollowing(NavigationCamera.java:66)
at com.mapbox.navigation.ui.maps.camera.NavigationCamera.requestNavigationCameraToFollowing$default(NavigationCamera.java:17)
at com.mapbox.androidauto.navigation.CarNavigationCamera$locationObserver$1.onNewLocationMatcherResult(CarNavigationCamera.java:136)
at com.mapbox.navigation.core.trip.session.MapboxTripSession.registerLocationObserver(MapboxTripSession.java:25)
at com.mapbox.navigation.core.MapboxNavigation.registerLocationObserver(MapboxNavigation.java:7)
at com.mapbox.androidauto.navigation.CarNavigationCamera$navigationObserver$1.onAttached(CarNavigationCamera.java:11)
at com.mapbox.navigation.core.lifecycle.MapboxNavigationOwner.register(MapboxNavigationOwner.java:18)
at com.mapbox.navigation.core.lifecycle.MapboxNavigationAppDelegate.registerObserver(MapboxNavigationAppDelegate.java:9)
at com.mapbox.navigation.core.lifecycle.MapboxNavigationApp.registerObserver(MapboxNavigationApp.java:11)
at com.mapbox.androidauto.navigation.CarNavigationCamera.onAttached(CarNavigationCamera.java:116)
at com.mapbox.maps.extension.androidauto.CarMapSurfaceOwner.onSurfaceAvailable(CarMapSurfaceOwner.java:128)
at androidx.car.app.utils.RemoteUtils$SurfaceCallbackStub.lambda$onSurfaceAvailable$0(RemoteUtils.java:8)
at androidx.car.app.utils.RemoteUtils.lambda$dispatchCallFromHost$0(RemoteUtils.java)
at androidx.car.app.utils.ThreadUtils.runOnMain(ThreadUtils.java:10)
at androidx.car.app.utils.RemoteUtils.dispatchCallFromHost(RemoteUtils.java:5)
at androidx.car.app.utils.RemoteUtils.lambda$dispatchCallFromHost$1(RemoteUtils.java:15)
at android.os.Handler.handleCallback(Handler.java:958)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:205)
at android.os.Looper.loop(Looper.java:294)
at android.app.ActivityThread.main(ActivityThread.java:8180)
at java.lang.reflect.Method.invoke(Method.java)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:946)
```

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 by reading CarNavigationCamera.kt, especially the locationObserver and navigationObserver paths shown in the stack trace, then trace the request into NavigationCamera and CameraAnimatorsFactory. Investigate how an invalid location reaches MapboxMap.unproject during the Android Auto surface setup. Done means the reported NaN latitude crash no longer occurs, with a regression test if the relevant test entry point is found.

Written by the indexing model from the issue text.

Assessment

Tech stack
android, kotlin
Domain
mobile-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.