Add multiple markers and rotations to AnimatedMarkerActivity

[zugaldia]

As mentioned in #5242 we currently have an experimental AnimatedMarkerActivity in the TestApp that animates a marker position using a ValueAnimator. We want to expand this activity to reflect common scenarios (e.g. a car marker), and create a concrete example that can help us profile the performance of different marker animations approaches.

In particular, we want to add the following elements:

• Several markers animating concurrently in the same visible region.
• One marker acting as a selected marker that remains on top of the others [android] #5285 - bring selected MarkerView to the front #5294 [android] #5296 - Add a container ViewGroup for MarkerViews #5298
• The markers could rotate as well as change their location.

Related sample in the same category: #5282.

/cc: @ivovandongen @tobrun @bleege @cammace

[zugaldia]

Follow progress in https://github.com/mapbox/mapbox-gl-native/tree/5299-sample-cars-driving branched out from release-android-v4.1.0.

[ivovandongen]

I've been trying some different combinations. Using both Markers and MarkerViews. All using ValueAnimators to move around the Markers. As expected, the results are not as smooth as we'd like. (This is all on a Nexus 5X)

First off the MarkerViews

One car:

Same thing, with gpu/cpu overlays

10 cars

10 cars, with gpu/cpu overlays

[ivovandongen]

Next up: the markers (GL).

1 car

1 car, debug

10 cars

10 cars, debug

Results are very similar with to Android based View Markers.

[ivovandongen]

Every step in the animation results in the following log statements btw:

06-10 16:26:05.732 19469-19469/com.mapbox.mapboxsdk.testapp D/mbgl: [JNI]: nativeUpdateMarker
06-10 16:26:05.780 19469-19469/com.mapbox.mapboxsdk.testapp D/mbgl: [Android]: NativeMapView::invalidate()
06-10 16:26:05.784 19469-19469/com.mapbox.mapboxsdk.testapp D/mbgl: [Android]: NativeMapView::invalidate()
06-10 16:26:05.785 19469-19469/com.mapbox.mapboxsdk.testapp D/mbgl: [Android]: NativeMapView::invalidate()

Although not much needs to be done on the core side (mainly, calculating screen position from the new coordinates), it seems that a lot happens anyways.

[zugaldia]

Great job @ivovandongen , this is great progress.

Did you notice if any of the following had any lower/higher noticeable performance impact?

• The "speed" of the car.
• The "distance" traveled.
• Enabling [android] #5285 - bring selected MarkerView to the front #5294.
• The type of animation, e.g. AccelerateDecelerateInterpolator vs LinearInterpolator.
• Rotations.

[tobrun]

I'm going to run with this branch and try out proposed solution for this from #5242.

[tobrun]

Merged with #5300 on release-android-v4.1.0

[AlejandroP27]

Hello, I am very glad that I am a super rookie on the subject of Android, serious layers of telling me or explaining how you did to give movements to multiple markes if it is not much trouble. thank you very much

[zugaldia]

Hi @AlejandroP27, thanks for using our Maps SDK. However, this is not a support forum — if you have more implementation questions, please check out our excellent help docs or post your question on Stack Overflow. Thanks!