pastvu icon indicating copy to clipboard operation
pastvu copied to clipboard

Check region by point - replace Google calling to OSM/Yandex request

Open paul-k-pastvu opened this issue 6 years ago • 22 comments

Popup menu in the https://pastvu.com/admin/region/check :

image

Some errors from Google.

Maybe remove this or replace it to OSM / Yandex request instead?

paul-k-pastvu avatar Feb 06 '20 18:02 paul-k-pastvu

Google geocoding requires API key these days: https://maps.googleapis.com/maps/api/geocode/json?latlng=40.714224,-73.961452

kabalin avatar Nov 04 '20 15:11 kabalin

There is OSM Overpass mechanism https://www.openstreetmap.org/query?lat=60&lon=30&xhr=1 изображение or there is OSM Nominatim for ex. https://nominatim.openstreetmap.org/ui/reverse.html?format=html&lat=59.6942440819683&lon=30.443973541259766&zoom=18

mkgrgis avatar Nov 04 '20 16:11 mkgrgis

Я бы предложил и саму карту, и эту функцию заменить на OSM, недолго думая. Вообще поставиь в админский интерфейс ту же карту с тем же селектором, что и основная. Мне так было бы сильно удобнее.

paul-k-pastvu avatar Nov 04 '20 16:11 paul-k-pastvu

Я бы предложил и саму карту, и эту функцию заменить на OSM, недолго думая.

Карта там и так OSM, патч для использования Nominatim здесь #228

Вообще поставиь в админский интерфейс ту же карту с тем же селектором, что и основная. Мне так было бы сильно удобнее.

Ну это в отдельный тикет лучше вынести.

kabalin avatar Nov 04 '20 20:11 kabalin

Ну это в отдельный тикет лучше вынести.

Ну это так, к слову. 10-й приоритет.

paul-k-pastvu avatar Nov 04 '20 20:11 paul-k-pastvu

Реализованный механизм расчищает дорогу к https://github.com/PastVu/pastvu/issues/142. Там становится возможна автопривязка большинства регионов.

mkgrgis avatar Nov 05 '20 11:11 mkgrgis

Реализованный механизм расчищает дорогу к #142. Там становится возможна автопривязка большинства регионов.

Да, это кстати. Но тут вот какое дело: зачача #142 сформулирована очень рамочно и очень неконкретно. И, по сути, пока чтт нужна ровно одному человкеку - тому админу, который занимается регионами, то есть пока что в основном мне. Так что я понизил ей приоритет пока что. Остальное напишу там.

paul-k-pastvu avatar Nov 05 '20 22:11 paul-k-pastvu

The best:

image

paul-k-pastvu avatar Nov 07 '20 23:11 paul-k-pastvu

Ещё вопрос по этой задаче - а нельзя ли делать ссылками уровни из Nominatim? Чтобы они вели на соответствующие объекты nominatim.openstreetmap.org ?

paul-k-pastvu avatar Nov 07 '20 23:11 paul-k-pastvu

Там скорре можно приеделать ссылки на отношения ОСМ, которые можно качать в geoJSON через https://github.com/tyrasd/osmtogeojson и отрисовывать на этой же карте или просматривать вживую на ОСМ. Кстати, к координатам сверху лучше, для избежания путаницы, приписать φ= для широты и λ= для долготы, т.к. указывают в разном порядке в разных странах. А так будет международная геодезическая нотация, которую не нужно переводить.

mkgrgis avatar Nov 08 '20 06:11 mkgrgis

Ещё вопрос по этой задаче - а нельзя ли делать ссылками уровни из Nominatim? Чтобы они вели на соответствующие объекты nominatim.openstreetmap.org ?

#242

kabalin avatar Nov 08 '20 14:11 kabalin

Багрепорт от @klimashkin - В английской версии image

У меня в английской версии - иначе: image

То есть это двойной баг:

  1. Не подтягиваются английские версии названий регионов PastVu
  2. Названия регионов Nominatim отображаются не на языке интерфейса сайта, а на языке интерфейса пользовательского окружения (браузера или ОС)
  • по п.1. - может, это давно так было, просто никто туда не заглядывал.

paul-k-pastvu avatar Nov 08 '20 16:11 paul-k-pastvu

Хм, может быть локальный кэш? У меня работает.

Названия регионов Nominatim отображаются не на языке интерфейса сайта, а на языке интерфейса пользовательского окружения (браузера или ОС)

Это я не менял, нужно чтобы по-русски всегда?

kabalin avatar Nov 08 '20 17:11 kabalin

Хм, может быть локальный кэш? У меня работает.

Очистка кеша не помогла. Я там выше обновил багрепорт - я вижу две связанные проблемы.

paul-k-pastvu avatar Nov 08 '20 17:11 paul-k-pastvu

Там скорре можно приеделать ссылки на отношения ОСМ, которые можно качать в geoJSON через https://github.com/tyrasd/osmtogeojson и отрисовывать на этой же карте или просматривать вживую на ОСМ.

Отрисовывать - это приятно, но роскошь. Кроме того, напоминаю, что там бардак, и это могут быть вообще не отношения, а хоть Node, хоть Way, причем в любом месте иерархии.

У меня есть кустик более полезных задач, где хотелось бы отрисовывать границы регионов на карте. Но это не горит совсем, есть дела поважнее.

Кстати, к координатам сверху лучше, для избежания путаницы, приписать φ= для широты и λ= для долготы, т.к. указывают в разном порядке в разных странах. А так будет международная геодезическая нотация, которую не нужно переводить.

Не возражаю.

paul-k-pastvu avatar Nov 08 '20 17:11 paul-k-pastvu

Названия регионов Nominatim отображаются не на языке интерфейса сайта, а на языке интерфейса пользовательского окружения (браузера или ОС) Это я не менял, нужно чтобы по-русски всегда?

Если включен русский интерфейс сайта - запрашивать русские (если нету - то что он там возвращает в в виде дефолтных)

Если включен английский интерфнес сайта - запрашивать английские (если нету в явном виде - дефолтные)

paul-k-pastvu avatar Nov 08 '20 17:11 paul-k-pastvu

Если включен русский интерфейс сайта - запрашивать русские (если нету - то что он там возвращает в в виде дефолтных)

Если включен английский интерфнес сайта - запрашивать английские (если нету в явном виде - дефолтные)

Добавил язык в запрос (см #242 ). Пробуйте еще раз, если опять не работает я уберу шаблоны для формирования ссылки (у меня по прежнему упорно работает в Хроме).

Screenshot

kabalin avatar Nov 08 '20 17:11 kabalin

Могу сделать ссылку не на последнюю запись, а на все записи (но ссылка будет вести на одну и ту-же страницу) или вообще вынести ее отдельной строкой ниже последнего элемента.

kabalin avatar Nov 08 '20 17:11 kabalin

  1. Не подтягиваются английские версии названий регионов PastVu ....
  • по п.1. - может, это давно так было, просто никто туда не заглядывал.

Да, только русский вариант был. Добавил коммит в #242 чтобы учитывался язык.

kabalin avatar Nov 08 '20 22:11 kabalin

Упустили мелочь https://github.com/PastVu/pastvu/issues/131#issuecomment-723535463

mkgrgis avatar Nov 09 '20 14:11 mkgrgis

Язык победили, но ссылка на OSM указывает куда-то не туда:

image

Сейчас: https://nominatim.openstreetmap.org/ui/details.html?place_id=115595528

Должно быть: https://nominatim.openstreetmap.org/ui/details.html?osmtype=R&osmid=173763

paul-k-pastvu avatar Nov 15 '20 21:11 paul-k-pastvu

Добавил, но кажется не совсем то что вы хотели. Там в ответе API нет информации по всей цепочке, только данные по точке (см https://nominatim.org/release-docs/develop/api/Output/#json). Для формирования ссылки я сначала использовал place_id (как сейчас работает), но можно ссылку сделать как у вас в примере с osm_id (см патч #269), а вот прямо чтобы район показывал так нельзя (я не нашел), но вы там можете в открывшемся окне нажать на "детали" в последней колонке напротив интересующего вас элемента и попасть куда надо.

kabalin avatar Nov 16 '20 21:11 kabalin

Вроде бы все сделано по этой задаче, закрываю.

kabalin avatar Jul 09 '23 09:07 kabalin