mapbox-maps-android icon indicating copy to clipboard operation
mapbox-maps-android copied to clipboard

Compass is queried every 100 millis but filtered by 500 millis

Open yunikkk opened this issue 4 years ago • 2 comments

Seems we're doing a lot of sensor queries that are then ignored, which'll consume more CPU and battery. Could be improved, the interesting thing is how does that impact the battery. Would be nice to compare when battery tests will be available.

References : https://github.com/mapbox/mapbox-maps-android/pull/970/files/bb45db666e9a4d79b6a6927aa592d47fd316096b#r771258957

yunikkk avatar Dec 17 '21 10:12 yunikkk

Do we have a hypothesis about the impact this might have @yunikkk? That would be helpful to understanding the priority to place on the issue.

knov avatar Dec 21 '21 15:12 knov

It's slighly (?) higher consumption of resources, hard to say some exact numbers without making an actual test. Posted it to keep track. Also - the fix should not be hard, couple lines of code, so it's really more matter of comparing & making some benchmarks. IMO aligns well with the SLA effort that we do right now

yunikkk avatar Dec 21 '21 16:12 yunikkk