Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add selection of recycling types into OSM editor #13450

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shaforostoff
Copy link
Contributor

@shaforostoff shaforostoff commented Jul 21, 2020

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 ). Я не тестировал свои изменения, надеюсь они будут работать из коробки и вам не придётся ничего дорабатывать.

Copy link
Contributor

@tatiana-yan tatiana-yan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upd:

Приложение не разрабатывалось как редактор и рантайм приложения очень мало знает про теги и про осм, поэтому с редактором есть куча нюансов, которые сложно учесть.

Текущий реквест делает редактор неработоспособным. Я пока сама не понимаю можно ли без участия ios и android сделать этот функционал. Если можно, то попробую на этой или на следующей неделе сделать. Если нужно участие ios и android превышающее мои познания в мобильной разработке (в аналогичном случае с cuisine там отдельный view controller и т.п. и пока не знаю насколько всё просто/сложно), то не могу дать никакой прогноз -- на платформах много более горящих задач.

@@ -307,6 +346,15 @@
<include field="internet" />
</type>
<type id="amenity-recycling">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

проблема в том что редактор оперирует внутренними типами, amenity-recycling для него это не amenity-recycling из осма, а то что имеет тип amenity-recycling у нас в рантайме, то есть в данном случае это recycling center

amenity|recycling;[amenity=recycling][recycling_type=centre];;name;int_name;109;

по идее надо вместо добавления recycling_type внутрь amenity-recycling который является center сделать рядом с amenity-recycling ещё amenity|recycling_container с типами отходов внутри -- это позволит создавать и редактировать и центры и контейнеры и указывать там типы собираемых отходов

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ещё проблема в том что типы в рантайме не знают из чего они когда-то были собраны при генерации карт, надо в FeatureToXml сделать особое сохранение для amenity-recycling и amenity-recycling_container

<option value="centre" />
</value>
</field>
<field name="recycling_batteries">
Copy link
Contributor

@tatiana-yan tatiana-yan Jul 27, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

это всё будет крешиться вот тут:

bool TypeDescriptionFromXml(pugi::xml_node const & root, pugi::xml_node const & node,

потому что поля в редакторе существуют не в вакууме, то что хранится в полях должно где-то храниться и в данных.
считается что всё что есть в редакторе кроме "street", "housenumber", "housename", "postcode" и "cuisine", для которых там сделана особая обработка, хранится в metadata

информации о том что recycling* надо поискать в типах нету, поэтому будет поиск в метадате и падение на ассерте

<option value="centre" />
</value>
</field>
<field name="recycling_batteries">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

с этим полем тоже тяжело -- у нас есть тип recycling-toxic который объединяет батарейки и лампы, recycling_batteries отдельно в данных никак не отражены, переводов для них нет и т.п.

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

@pataquets
Copy link

@shaforostoff see #14157

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants