add selection of recycling types into OSM editor#13450
add selection of recycling types into OSM editor#13450shaforostoff wants to merge 1 commit intomapsme:masterfrom
Conversation
There was a problem hiding this comment.
upd:
Приложение не разрабатывалось как редактор и рантайм приложения очень мало знает про теги и про осм, поэтому с редактором есть куча нюансов, которые сложно учесть.
Текущий реквест делает редактор неработоспособным. Я пока сама не понимаю можно ли без участия ios и android сделать этот функционал. Если можно, то попробую на этой или на следующей неделе сделать. Если нужно участие ios и android превышающее мои познания в мобильной разработке (в аналогичном случае с cuisine там отдельный view controller и т.п. и пока не знаю насколько всё просто/сложно), то не могу дать никакой прогноз -- на платформах много более горящих задач.
| @@ -307,6 +346,15 @@ | |||
| <include field="internet" /> | |||
| </type> | |||
| <type id="amenity-recycling"> | |||
There was a problem hiding this comment.
проблема в том что редактор оперирует внутренними типами, amenity-recycling для него это не amenity-recycling из осма, а то что имеет тип amenity-recycling у нас в рантайме, то есть в данном случае это recycling center
Line 109 in 0d5b275
по идее надо вместо добавления recycling_type внутрь amenity-recycling который является center сделать рядом с amenity-recycling ещё amenity|recycling_container с типами отходов внутри -- это позволит создавать и редактировать и центры и контейнеры и указывать там типы собираемых отходов
There was a problem hiding this comment.
ещё проблема в том что типы в рантайме не знают из чего они когда-то были собраны при генерации карт, надо в FeatureToXml сделать особое сохранение для amenity-recycling и amenity-recycling_container
| <option value="centre" /> | ||
| </value> | ||
| </field> | ||
| <field name="recycling_batteries"> |
There was a problem hiding this comment.
это всё будет крешиться вот тут:
Line 43 in 0d5b275
потому что поля в редакторе существуют не в вакууме, то что хранится в полях должно где-то храниться и в данных.
считается что всё что есть в редакторе кроме "street", "housenumber", "housename", "postcode" и "cuisine", для которых там сделана особая обработка, хранится в metadata
информации о том что recycling* надо поискать в типах нету, поэтому будет поиск в метадате и падение на ассерте
| <option value="centre" /> | ||
| </value> | ||
| </field> | ||
| <field name="recycling_batteries"> |
There was a problem hiding this comment.
с этим полем тоже тяжело -- у нас есть тип recycling-toxic который объединяет батарейки и лампы, recycling_batteries отдельно в данных никак не отражены, переводов для них нет и т.п.
невозможно при сборке данных из двух тегов по принципу или сделать один тип, а потом по данным сказать что там изначально было
|
@shaforostoff see #14157 |
First of all, thank you for completing implementation of recycling types view. I got it on my phone and it is working!
Я показал эту функцию московским активистам, выступающим за раздельный сбор мусора (и его уменьшение в целом), и оказалось что MAPS.ME может служить заменой заброшенному сайту https://recyclemap.ru/
Единственный запрос от них - это возможность добавлять recycling-контейнеры прямо из MAPS.ME
Для этого я добавил самые распространённые типы собираемого вторсырья ( из https://wiki.openstreetmap.org/wiki/Tag:amenity%3Drecycling ). Я не тестировал свои изменения, надеюсь они будут работать из коробки и вам не придётся ничего дорабатывать.