Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Animated Marker Example App Not Working #5107

Closed
bleege opened this issue May 23, 2016 · 4 comments
Closed

Animated Marker Example App Not Working #5107

bleege opened this issue May 23, 2016 · 4 comments
Labels
Android Mapbox Maps SDK for Android bug

Comments

@bleege
Copy link
Contributor

bleege commented May 23, 2016

The AnimatedMarkerActivity test example is currently not working. It should show a default marker moving on a straight line between Brussels and Washington, DC. Currently however the marker is not displayed on the map. This is likely due to changes from the build out of the View Based Annotations.

We also should remove the phrase Experimental from the Activity name as this should be a supported feature as part of the View Base Annotations API.

device-2016-05-23-145256

@bleege bleege added bug Android Mapbox Maps SDK for Android labels May 23, 2016
@bleege bleege added this to the android-v4.1.0 milestone May 23, 2016
@tobrun
Copy link
Member

tobrun commented May 23, 2016

I noticed this also, when zooming in a bit you can see the marker appearing again. My first guess is that this isn't related to changes made by view annotations and this is an issue with gl based annotations.

@tobrun
Copy link
Member

tobrun commented May 23, 2016

Confirming that his seems to be related to zoomlevel. I did a test animating at higher value:

ezgif com-video-to-gif 19

@tobrun
Copy link
Member

tobrun commented May 23, 2016

We also should remove the phrase Experimental from the Activity name as this should be a supported feature as part of the View Base Annotations API.

This feature is both part of the GL based Annotations as the View based Annotations API. Both Marker as MarkerView expose `setPosition(LatLng position) and both will need to call the same JNI methods. The reason why I marked it experimental is that animations aren't smooth yet. This is something I'm trying to profile/measure and hoping that perf best practices as #5103 will resolve it.

@tobrun
Copy link
Member

tobrun commented May 23, 2016

With help from @1ec5, we were able to find this is core bug and created a follow up issue in #5109. Going to close this in favour of #5109.

@tobrun tobrun closed this as completed May 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android bug
Projects
None yet
Development

No branches or pull requests

2 participants