[android] Update A-GPS data automatically once per day

biodranik (Migrated from github.com) reviewed 2022-02-06 11:57:03 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-06 11:57:02 +00:00

Надо учесть несколько вещей.

Настройку андроида:

boolean assistedGpsEnabled = 0 != Settings.Global.getInt(context.getContentResolver(Settings.Global.ASSISTED_GPS_ENABLED));

Наши настройки доступа к сети в настройках. Нужно обязательно показать пользователю диалог, в идеале поясняющий, что сейчас через сеть подгрузятся данные для скорейшего определения позиции. Я не знаю точно, но по косвенным источникам AGPS качает от 50 до 500 килобайт в зависимости от area. При включённом роуминге, и путешествии на неделю, это будет неприятным сюрпризом.
Вообще весь этот код стоит запускать ТОЛЬКО тогда, когда обычный GPS подход зафейлился и не работает. И когда есть интернет подключение + наш пользователь разрешил (п.2 выше), само собой. Система должна сама прекрасно обрабатывать agps данные в фоне и сама должна их обновлять.

Наша основная проблема сейчас — когда никакой сети нет и не будет. А позиция не определяется. Т.е. не в agps.

Надо учесть несколько вещей. 1. [Настройку](https://android.googlesource.com/platform/frameworks/base/+/84e2756c0f3794c6efe5568a9d09101ba689fb39/core/java/android/provider/Settings.java#2611) андроида: ``` boolean assistedGpsEnabled = 0 != Settings.Global.getInt(context.getContentResolver(Settings.Global.ASSISTED_GPS_ENABLED)); ``` 2. Наши настройки доступа к сети в настройках. Нужно обязательно показать пользователю диалог, в идеале поясняющий, что сейчас через сеть подгрузятся данные для скорейшего определения позиции. Я не знаю точно, но по косвенным источникам AGPS качает от 50 до 500 килобайт в зависимости от area. При включённом роуминге, и путешествии на неделю, это будет неприятным сюрпризом. 3. Вообще весь этот код стоит запускать ТОЛЬКО тогда, когда обычный GPS подход зафейлился и не работает. И когда есть интернет подключение + наш пользователь разрешил (п.2 выше), само собой. Система должна сама прекрасно обрабатывать agps данные в фоне и сама должна их обновлять. Наша основная проблема сейчас — когда никакой сети нет и не будет. А позиция не определяется. Т.е. не в agps.

rtsisyk reviewed 2022-02-06 16:07:44 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

rtsisyk commented

2022-02-06 16:07:44 +00:00

Author

Owner

Settings.Global.ASSISTED_GPS_ENABLED

Я не вижу такую опцию в текущем SDK и документации. Её просто нет.

Настройки доступа к сети проверяются:

    if (!NetworkPolicy.getCurrentNetworkUsageStatus())
      return;

Ввиду того, что в Андроиде данную опцию из настроек уже давно выпилили, а диалог в данном коде не вызывается, то скачивание будет либо только по wifi либо если разрешили где-то в другом месте.

Моё понимание, что наличие актуальных данных agps снижает time to first fix в том числе при полностью оффлайновом использовании. Во всяком случае Pixel 4a после обновления A-GPS стал находить локацию при полном оффлайне за 1-2 минуту, а до этого не смог найти за 10.

1. Settings.Global.ASSISTED_GPS_ENABLED Я не вижу такую опцию в текущем SDK и документации. Её просто нет. 2. Настройки доступа к сети проверяются: ``` if (!NetworkPolicy.getCurrentNetworkUsageStatus()) return; ``` Ввиду того, что в Андроиде данную опцию из настроек уже давно выпилили, а диалог в данном коде не вызывается, то скачивание будет либо только по wifi либо если разрешили где-то в другом месте. 3. Моё понимание, что наличие актуальных данных agps снижает time to first fix в том числе при полностью оффлайновом использовании. Во всяком случае Pixel 4a после обновления A-GPS стал находить локацию при полном оффлайне за 1-2 минуту, а до этого не смог найти за 10.

rtsisyk reviewed 2022-02-06 16:09:54 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

rtsisyk commented

2022-02-06 16:09:53 +00:00

Author

Owner

Также хотел бы добавить, что OSMAnd запускает обновление данных A-GPS раз в 16 часов без каких либо дополнительных условий вроде "GPS подход зафейлился".

biodranik (Migrated from github.com) reviewed 2022-02-08 15:20:16 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-08 15:20:15 +00:00

Это же неправильная проверка. Она не будет работать, если пользователь разрешил мобильный интернет. Надо учитывать текущую настройку "Всегда спрашивать", или "Всегда".

biodranik (Migrated from github.com) reviewed 2022-02-08 15:21:05 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-08 15:21:05 +00:00

Это в дополнение к проверке на наличие активного соединения.

rtsisyk reviewed 2022-02-09 11:17:07 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

rtsisyk commented

2022-02-09 11:17:07 +00:00

Author

Owner

на не будет работать, если пользователь разрешил мобильный интернет.

Не понял, если честно. А вообще в ведре эта NetworkPolicy вообще нигде не вызывает диалог. То есть юзеру чисто физически негде разрешить или запретить.

> на не будет работать, если пользователь разрешил мобильный интернет. Не понял, если честно. А вообще в ведре эта NetworkPolicy вообще нигде не вызывает диалог. То есть юзеру чисто физически негде разрешить или запретить.

biodranik (Migrated from github.com) reviewed 2022-02-09 12:04:05 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-09 12:04:05 +00:00

На iOS выскакивает диалог (потому что вызывается соответствующий код в нужных местах). Полагаю, этот кусок был почему-то выпилен. Посмотри, пожалуйста, как было сделано раньше в апстриме. Надо вернуть этот функционал, он будет нужен и для проверок обновлений карт тоже, и для синхронизации меток.

rtsisyk reviewed 2022-02-10 06:15:40 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

rtsisyk commented

2022-02-10 06:15:40 +00:00

Author

Owner

Я поменял на явную проверку wifi. Юзкейс простой. Пришел вечером в отель. Врубил wifi. Обновился. НА следущий день gps работает быстрее.

biodranik (Migrated from github.com) reviewed 2022-02-10 10:24:59 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-10 10:24:59 +00:00

Это плохой подход. Многие пользуются только мобильной связью, для них этот фикс вообще пройдёт незамеченным. У нас есть в настройках приложения галочка для использования интернета, зачем городить костыли? Просто используй эту галочку, как положено.

rtsisyk reviewed 2022-02-11 06:46:30 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

rtsisyk commented

2022-02-11 06:46:30 +00:00

Author

Owner

Во-первых, галочки у нас в настройках в Андроиде уже давно нет. Она была выпилена вместе с рекламой, так как другого юзера этой галочки не существовало. Во-вторых, оригинальная версия патча как раз таки использовала NetworkPolicy а не напрямую проверяла wifi. Что в итоге здесь предлагается сделать?

biodranik (Migrated from github.com) reviewed 2022-02-11 07:57:37 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-11 07:57:37 +00:00

Вот же настройка. Есть и должна работать. С чем ты споришь?

Вот же настройка. Есть и должна работать. С чем ты споришь? ![image](https://user-images.githubusercontent.com/170263/153555711-8d5108b0-3b30-4e2b-8510-9ca6fdbb8397.jpeg)

rtsisyk reviewed 2022-02-11 08:09:58 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

rtsisyk commented

2022-02-11 08:09:57 +00:00

Author

Owner

Видимо вернули все же. Хорошо. В оригинальной версии патча эта настройка учитывается. В чем была претензия тогда?

biodranik (Migrated from github.com) reviewed 2022-02-11 08:52:54 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-11 08:52:54 +00:00

Надо показывать диалог, если стоит всегда спрашивать, а не просто учитывать настройку.

biodranik (Migrated from github.com) reviewed 2022-02-11 08:54:11 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-11 08:54:10 +00:00

Или передизайнить вообще подход, и сделать отдельный диалог, где можно будет выставлять доступ для каждого случая:

Обновление AGPS
Проверка обновлений карт
Загрузка KML картинок в Place Page
Загрузка wiki

И т.п.

Или передизайнить вообще подход, и сделать отдельный диалог, где можно будет выставлять доступ для каждого случая: - Обновление AGPS - Проверка обновлений карт - Загрузка KML картинок в Place Page - Загрузка wiki И т.п.

rtsisyk reviewed 2022-02-11 09:16:32 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

rtsisyk commented

2022-02-11 09:16:32 +00:00

Author

Owner

Если мы будем для загрузки AGPS показывать диалог, это вынесет мозг юзеру окончательно. Пусть оно обнорвляется только на wifi. Это вполне достойное решение, которое будет работать. Показ диалога для загрузки карт и загрузки кратинок не вход в scope этого патча и делать это я не буду.

biodranik (Migrated from github.com) reviewed 2022-02-11 09:25:38 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-11 09:25:38 +00:00

Я же объяснял, почему решение только для wifi — плохое. Есть большое количество пользователей с анлимом, которые никогда не подключаются к вайфаю, но тем не менее хотят быстро определять позицию, когда мобильная связь недоступна (в горах, хайках, путешествиях по своим родным странам, особенно во времена ковида).

vng (Migrated from github.com) reviewed 2022-02-11 10:15:41 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

vng (Migrated from github.com) commented

2022-02-11 10:15:41 +00:00

Надо показывать диалог, если стоит все

Мы обновляем AGPS незаметно в фоне (по крайней мере сейчас). Как это будет выглядеть? Пользователь просто открыл карты и тут вдруг вылетает диалог про Мобильный интернет?

> Надо показывать диалог, если стоит все Мы обновляем AGPS незаметно в фоне (по крайней мере сейчас). Как это будет выглядеть? Пользователь просто открыл карты и тут вдруг вылетает диалог про Мобильный интернет?

biodranik (Migrated from github.com) reviewed 2022-02-11 10:30:20 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-11 10:30:20 +00:00

Проблема именно в "незаметно в фоне". Должно быть заметно и под контролем. Т.е. с учётом настроек пользователя на моём скрине выше.

Я писал, как это может выглядеть: если в настройке стоит "спрашивать", то появляется диалог с описанием действия, для подтверждения пользователем.

Проблема именно в "незаметно в фоне". Должно быть заметно и под контролем. Т.е. с учётом настроек пользователя на моём скрине выше. Я писал, как это может выглядеть: если в настройке стоит "спрашивать", то появляется диалог с описанием действия, для подтверждения пользователем.

rtsisyk reviewed 2022-02-12 04:45:29 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

rtsisyk commented

2022-02-12 04:45:29 +00:00

Author

Owner

Вернул патч в оригинальное состояние на использование NetworkPolicy.getCurrentNetworkUsageStatus(). На всякий случай ниже привожу ссылку на реализацию этого метода:

45fa4c32fd/android/src/com/mapswithme/util/NetworkPolicy.java (L114-L129)

Кажется, что соответствует тому, что здесь просят:

if (connected && (wifi || setting(always)) updateAGPS();

CC: @biodranik @vng

Вернул патч в оригинальное состояние на использование `NetworkPolicy.getCurrentNetworkUsageStatus()`. На всякий случай ниже привожу ссылку на реализацию этого метода: https://github.com/organicmaps/organicmaps/blob/45fa4c32fd3825961e91bc298205ab0463356478/android/src/com/mapswithme/util/NetworkPolicy.java#L114-L129 Кажется, что соответствует тому, что здесь просят: ``` if (connected && (wifi || setting(always)) updateAGPS(); ``` CC: @biodranik @vng

biodranik (Migrated from github.com) reviewed 2022-02-12 08:25:47 +00:00

android/src/com/mapswithme/maps/location/LocationHelper.java

biodranik (Migrated from github.com) commented

2022-02-12 08:25:47 +00:00

А откуда берётся boolean acceptedInRoaming = Config.getMobileDataRoaming();? И когда она true?

А откуда берётся ` boolean acceptedInRoaming = Config.getMobileDataRoaming();`? И когда она true?

biodranik (Migrated from github.com) approved these changes 2022-02-12 08:26:32 +00:00

vng (Migrated from github.com) approved these changes 2022-02-12 09:44:05 +00:00

Rows
Columns

[android] Update A-GPS data automatically once per day #2032