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.

Sign up for GitHub

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
Open

add selection of recycling types into OSM editor #13450

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
48 changes: 48 additions & 0 deletions data/editor.config
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,45 @@
<!-- A list of values is hard-coded, so string in this case. -->
<value type="string" many="yes" />
</field>
<field name="recycling_type">
<tag k="recycling_type" />
<value type="list"/>
<option value="container" />
<option value="centre" />
</value>
</field>
<field name="recycling_batteries">
Copy link
Contributor

@tatiana-yan tatiana-yan Jul 27, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

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

omim/editor/editor_config.cpp

Line 43 in 0d5b275

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

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

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

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 отдельно в данных никак не отражены, переводов для них нет и т.п.

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

<tag k="recycling:batteries" />
<value type="boolean" />
</field>
<field name="recycling_cans">
<tag k="recycling:cans" />
<value type="boolean" />
</field>
<field name="recycling_clothes">
<tag k="recycling:clothes" />
<value type="boolean" />
</field>
<field name="recycling_glass_bottles">
<tag k="recycling:glass_bottles" />
<value type="boolean" />
</field>
<field name="recycling_paper">
<tag k="recycling:paper" />
<value type="boolean" />
</field>
<field name="recycling_plastic_bottles">
<tag k="recycling:plastic_bottles" />
<value type="boolean" />
</field>
<field name="recycling_scrap_metal">
<tag k="recycling:scrap_metal" />
<value type="boolean" />
</field>
<field name="recycling_small_appliances">
<tag k="recycling:small_appliances" />
<value type="boolean" />
</field>
<field name="opening_hours">
<tag k="opening_hours" />
</field>
Expand Down Expand Up @@ -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

omim/data/mapcss-mapping.csv

Line 109 in 0d5b275

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

<include field="recycling_type" />
<include field="recycling_batteries" />
<include field="recycling_cans" />
<include field="recycling_clothes" />
<include field="recycling_glass_bottles" />
<include field="recycling_paper" />
<include field="recycling_plastic_bottles" />
<include field="recycling_scrap_metal" />
<include field="recycling_small_appliances" />
<include field="name" />
<include field="operator" />
<include field="website" />
Expand Down