-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add LocationSource #4331
Comments
Currently the user location marker is internally hardcoded to use the device's GPS positioning to define where it is. Imho this is architecurally a bit at odds with separating data and visualization, but also in a very practical sense it makes the position marker very inflexible. For example let's say you want a virtual walk through of the route, like a play button to preview your journey. Currently we'd have to re-implement the user position marker for this functionality, instead of simply setting its location. Another case would be for using any 3rd party location source, ranging from IP address to airplane schedules to qr codes. |
@tobrun Btw, why the SEMVER-MAJOR tag? If implemented in the same way as in google-play-services, where you provide a default one, but allow overriding it, it wouldn't break API as far as I can tell. |
@mkv123 sorry for only getting back to you now. The initial focus of this ticket was to rename |
Looking a bit more into this it seems that there are two requirements here:
|
Been working on this a bit and need to conclude that this requires a bigger lift as initially thought. Making the system pluggable requires bigger refactors to LocationServices and Telemetry. We ideally pick this up when we introduce a separate telemetry module. I'm going to submit some fixes that I made to telemetry separately. |
We have recently made some progress here:
With mapbox/mapbox-java#269 we now have an abstract
That's achieved with the newly introduced |
Fixed via #7892. |
LocationSource is our equivalent of
LocationServices
.The text was updated successfully, but these errors were encountered: