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

iOS: Please add MGLUserTrackingModeFollowWithCourse #1605

Closed
picciano opened this issue May 20, 2015 · 6 comments · Fixed by #2068
Closed

iOS: Please add MGLUserTrackingModeFollowWithCourse #1605

picciano opened this issue May 20, 2015 · 6 comments · Fixed by #2068
Assignees
Labels
feature iOS Mapbox Maps SDK for iOS

Comments

@picciano
Copy link

It looks like the follow user with heading is off by about 5 to 10 degrees. In this case the user was driving down W Olympic Blvd, but map orientation was quite a bit off. Maybe magnetic heading is being used instead?

@incanus incanus added bug iOS Mapbox Maps SDK for iOS labels May 20, 2015
@picciano
Copy link
Author

Just to clarify this a bit...it seems to be orienting to the orientation of the phone, not the direction of travel. This seems very odd when you use it.

The 5 to 10 degrees was just the effect of holding the phone a slight angle from the direction of travel.

It would seem direction of travel is what is needed here more than the compass direction of the phone.

@picciano
Copy link
Author

Sorry to keep going on and on, but it looks like this is the difference between using the newHeading from didUpdateHeading: and newLocation.course from the didUpdateToLocation: delegate methods of CLLocationManager.

For navigation apps, the course is what is needed. Maybe another user tracking mode is needed?
MGLUserTrackingModeFollowWithCourse

@picciano picciano changed the title Follow user with heading seem off iOS: Please add MGLUserTrackingModeFollowWithCourse May 27, 2015
@picciano
Copy link
Author

We have hacked in a FollowUserWithCourse option. But really look forward to a proper solution with smooth animation and all that jazz! Notice how the user location is not always the center of the map?

ios simulator screen shot jun 23 2015 3 51 25 pm

@1ec5
Copy link
Contributor

1ec5 commented Jun 24, 2015

Smooth animation is covered by #1582.

@picciano
Copy link
Author

Thanks, but I did want to differentiate between "follow user with heading" which is the magnetic direction, and "follow user with course" which is their direction of travel. Navigation apps will need follow with course.

@1ec5
Copy link
Contributor

1ec5 commented Jun 25, 2015

Understood, I left this issue open to track that request.

@1ec5 1ec5 added feature and removed bug labels Jun 26, 2015
@1ec5 1ec5 self-assigned this Aug 13, 2015
1ec5 added a commit that referenced this issue Aug 14, 2015
Added course-tracking. However, rotation has to happen atomically without animation until #1834 is fixed.

Fixes #1605.
@1ec5 1ec5 removed the in progress label Aug 14, 2015
AndwareSsj pushed a commit to AndwareSsj/mapbox-gl-native that referenced this issue Nov 6, 2015
Added course-tracking. However, rotation has to happen atomically without animation until mapbox#1834 is fixed.

Fixes mapbox#1605.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants