GmsCore icon indicating copy to clipboard operation
GmsCore copied to clipboard

Merge LocalNLPBackend into MicroG

Open Iey4iej3 opened this issue 1 year ago • 6 comments

Due to the latest change, it seems better to merge LocalNLPBackend into MicroG, cf. https://github.com/Helium314/Local-NLP-Backend/issues/18

LocalNLPBackend seems to be the best NLP backend to maintain a local database for network locations.

Iey4iej3 avatar May 30 '23 11:05 Iey4iej3

Local location learning added via https://github.com/microg/GmsCore/commit/16e37cbad87749c853394547c02a7dd7df3516e3

mar-v-in avatar Jul 04 '23 11:07 mar-v-in

Pleased to see this development - however, one question (acknowledging, that this may still be 'work in progress'): In the above mentioned commit, the "Mozilla Location Service" functionality is made invisible (and I assume also deactivated?), if no "API Key" has been defined in the build properties. What is the intention/necessity of this? Will there be follow-on commits?

MSe1969 avatar Jul 08 '23 21:07 MSe1969

Custom builds without API key never could use Mozilla Location Service (the service doesn't allow access without such API key). The change just hides the setting in those cases, as users otherwise might be confused.

mar-v-in avatar Jul 08 '23 21:07 mar-v-in

Local location learning added via 16e37cb

It might be nice to add some other LocalNLP features.

  • Import and Export: for easy migration between devices and from the original LocalNLPBackend
  • Kalman filter: for bad GPS
  • Bad emitter discard: to drop some Wifi AP which is movable, such as Phone Hotspot, Wifi on car...

CoelacanthusHex avatar Jul 29 '23 07:07 CoelacanthusHex

  • Kalman filter: for bad GPS
  • Bad emitter discard: to drop some Wifi AP which is movable, such as Phone Hotspot, Wifi on car...

The code already:

  • Filters out GPS data with lower reported accuracy than 25m
  • Ignores Wifi AP that seemingly are reachable in an area of more than 200m within 14 days
  • Ignores cells that seemingly are reachable in an area of more than 10km within 14 days
  • Ignores Wifi AP which by SSID are assumed to be movable (based on the lists at MovingWifiHelper.kt:25-66)

Not perfect for now, but I'd say it's good enough.

  • Import and Export: for easy migration between devices and from the original LocalNLPBackend

Indeed a good idea

mar-v-in avatar Jul 29 '23 07:07 mar-v-in

It seems to be a good start to support csv imports.

Iey4iej3 avatar Dec 26 '23 18:12 Iey4iej3