Bug 89924 - make MM_LOCATION_GPS_REFRESH_TIME_SECS configurable
Summary: make MM_LOCATION_GPS_REFRESH_TIME_SECS configurable
Status: RESOLVED FIXED
Alias: None
Product: ModemManager
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
Hardware: Other All
: medium enhancement
Assignee: Aleksander Morgado
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-06 19:48 UTC by Fabrice Bellet
Modified: 2017-07-31 06:31 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Added new API method to request GPS refresh rate update (23.89 KB, patch)
2015-12-07 13:33 UTC, Aleksander Morgado
Details | Splinter Review
Description of the option in the man page (1.10 KB, patch)
2017-07-30 22:46 UTC, Valentin Blot
Details | Splinter Review

Description Fabrice Bellet 2015-04-06 19:48:40 UTC
Some applications like gnome-maps, may have an interest in a location update from GPS more frequent than every 30 seconds. Most devices can update their location every second. Ideally this interval should be configurable.

For example geoclue has the possibility to upload wifi neighbouring ESSID and 3GPP data to mozilla geolocation service, when a GPS fix is available (with the hope that GPS coordinates can improve the location accuracy of these wifi/3GPP data). Such case would benefit from a more frequent interaction with ModemManager, so the Wifi/3GPP/NMEA GPS data stay *in sync* as much as possible, when the user is moving around.
Comment 1 Aleksander Morgado 2015-12-07 13:33:58 UTC
Created attachment 120388 [details] [review]
Added new API method to request GPS refresh rate update

Hey Fabrice, could you test the attached patch on top of ModemManager git master?
Comment 2 Aleksander Morgado 2015-12-07 13:36:16 UTC
With the patch on, you could do for example:
$ mmcli -m 0 --location-set-gps-refresh-rate=5
Comment 3 Fabrice Bellet 2015-12-10 21:26:04 UTC
Hi Aleksander, I just tested the patch, and it works for me. Thanks!
Comment 4 Aleksander Morgado 2015-12-21 16:49:18 UTC
This was merged to git master and will be available in the next major release.
Comment 5 Valentin Blot 2017-07-30 22:46:11 UTC
The new option for mmcli should be documented in the man page.
Comment 6 Valentin Blot 2017-07-30 22:46:48 UTC
Created attachment 133138 [details] [review]
Description of the option in the man page
Comment 7 Aleksander Morgado 2017-07-31 06:31:08 UTC
(In reply to Valentin Blot from comment #6)
> Created attachment 133138 [details] [review] [review]
> Description of the option in the man page

Pushed to git master and mm-1-6, thanks.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.