[classifier] Refactored recycling; removed boundary=4-state. #1868

pastk · 2022-01-16T12:50:02Z

vng commented

2022-01-16 12:50:02 +00:00

(Migrated from github.com)

@pastk
Closes organicmaps/organicmaps#1839
Closes organicmaps/organicmaps#1869
Closes organicmaps/organicmaps#1889

If somebody can help to update editor's UI (select recycling types like cuisines), we can cover this case https://github.com/mapsme/omim/pull/13450

@pastk Closes https://git.omaps.dev/organicmaps/organicmaps/issues/1839 Closes https://git.omaps.dev/organicmaps/organicmaps/issues/1869 Closes https://git.omaps.dev/organicmaps/organicmaps/issues/1889 If somebody can help to update editor's UI (select recycling types like cuisines), we can cover this case https://github.com/mapsme/omim/pull/13450

👍 1

pastk reviewed 2022-01-16 13:43:08 +00:00

data/mapcss-mapping.csv

pastk commented

2022-01-16 13:43:08 +00:00

можно вкратце объяснить, что какие поля значат?
второе это явно теги ОСМ
первое это тип ОМ, но и последнее тоже, причём последнее редко когда используется
что ещё значит x?
name/int_name - импорт названия, если оно есть в соответствующих тегах?

можно вкратце объяснить, что какие поля значат? второе это явно теги ОСМ первое это тип ОМ, но и последнее тоже, причём последнее редко когда используется что ещё значит `x`? name/int_name - импорт названия, если оно есть в соответствующих тегах?

vng (Migrated from github.com) reviewed 2022-01-16 13:57:43 +00:00

data/mapcss-mapping.csv

vng (Migrated from github.com) commented

2022-01-16 13:57:43 +00:00

name, name_int матчится в стилях отрисовки *.mapcss, но тут это копипаст и смысла не имеет особо, ибо
парсер этого файла ожидает 2, 3, 6 или 7 токенов.

x - значит что тип удаляем (boundary|administrative|4|state) и заменяем его на то что в конце (boundary|administrative|4) при чтении старых данных. name, name_int матчится в стилях отрисовки *.mapcss, но тут это копипаст и смысла не имеет особо, ибо парсер этого файла ожидает 2, 3, 6 или 7 токенов.

biodranik (Migrated from github.com) approved these changes 2022-01-16 23:05:07 +00:00

data/classificator.txt

biodranik (Migrated from github.com) commented

2022-01-16 22:55:12 +00:00

Почему ты выбрал именно organic? Я бы всё матчил на green_waste, во всех странах это "зелёные контейнеры" куда собирают еду и листья.

data/strings/types_strings.txt

biodranik (Migrated from github.com) commented

2022-01-16 22:58:50 +00:00

Может удалить базовый тип вообще отсюда?

data/visibility.txt

					
				@ -105,3 +107,4 @@

				        sanitary_dump_station  00000000000000011111  -

				        school  00000000000000000000  -

				        shelter  00000000000000000000  -

				        shower  00000000000000000000  -

biodranik (Migrated from github.com) commented

2022-01-16 22:59:44 +00:00

Зачем нам файл с нулями? )

editor/xml_feature.cpp

					
				@ -568,3 +568,3 @@

				    uint32_t type = 0;

				    if (k == "amenity" && v == "recycling")

				    if (k == "amenity" && v == "recycling" && xml.HasTag("recycling_type"))

				    {

biodranik (Migrated from github.com) commented

2022-01-16 23:02:29 +00:00

Надо глянуть тагинфо, скорее всего если тип - не центр и не контейнер, то надо матчить его как контейнер.

feature_list/feature_list.cpp

					
				@ -92,2 +89,4 @@

				  {

				    if (ftypes::IsPoiChecker::Instance()(type) || ftypes::IsPlaceChecker::Instance()(type))

				      result = classif().GetReadableObjectName(type);

				  });

biodranik (Migrated from github.com) commented

2022-01-16 23:03:38 +00:00

&result нагляднее

vng (Migrated from github.com) reviewed 2022-01-17 07:13:57 +00:00

data/classificator.txt

vng (Migrated from github.com) commented

2022-01-17 07:13:57 +00:00

Взял топ типов из taginfo. В целом, могу "объединить" organic в green_waste. Здесь я хотел сохранить максимальную преемственность из OSM типов, чтобы в будущем завести это в редактор и удовлетворить зеленых ребят по ссылке из описания.

vng (Migrated from github.com) reviewed 2022-01-17 07:14:51 +00:00

data/strings/types_strings.txt

vng (Migrated from github.com) commented

2022-01-17 07:14:51 +00:00

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

vng (Migrated from github.com) reviewed 2022-01-17 07:15:23 +00:00

data/visibility.txt

					
				@ -105,3 +107,4 @@

				        sanitary_dump_station  00000000000000011111  -

				        school  00000000000000000000  -

				        shelter  00000000000000000000  -

				        shower  00000000000000000000  -

vng (Migrated from github.com) commented

2022-01-17 07:15:23 +00:00

Это отдельный вопрос, зачем этот файл вообще. Когда-то отрефакторится, надеюсь.

vng (Migrated from github.com) reviewed 2022-01-17 07:17:07 +00:00

editor/xml_feature.cpp

					
				@ -568,3 +568,3 @@

				    uint32_t type = 0;

				    if (k == "amenity" && v == "recycling")

				    if (k == "amenity" && v == "recycling" && xml.HasTag("recycling_type"))

				    {

vng (Migrated from github.com) commented

2022-01-17 07:17:07 +00:00

Конечно смотрел. Для согласованности базовый тип остается, без него никак в иерархии, но в приложении он ведет себя как контейнер. А наш редактор будет создавать только полные типы или centre или container.

vng commented

2022-01-17 07:21:33 +00:00

(Migrated from github.com)

Еще я хочу убрать type.recycling.toxic и вернуть как в ОСМ batteries. Нет контейнеров для "ядовитых отходов". Это именно батарейки, что сейчас и берется из ОСМ. Нужны будут адекватные переводы.

vng (Migrated from github.com) reviewed 2022-01-17 07:22:56 +00:00

data/classificator.txt

vng (Migrated from github.com) commented

2022-01-17 07:22:56 +00:00

Объединить в одну категорию? Я подумал и склоняюсь что лишнее держать 2 категории по сути одного и того же ..

biodranik (Migrated from github.com) reviewed 2022-01-17 07:23:23 +00:00

data/classificator.txt

biodranik (Migrated from github.com) commented

2022-01-17 07:23:23 +00:00

Просто в редакторе будет путаница у обычных людей, выбирать green waste или organic. Надо спросить у osmеров.

biodranik (Migrated from github.com) reviewed 2022-01-17 07:39:30 +00:00

data/classificator.txt

biodranik (Migrated from github.com) commented

2022-01-17 07:39:30 +00:00

Еще я хочу убрать type.recycling.toxic и вернуть как в ОСМ batteries. Нет контейнеров для "ядовитых отходов". Это именно батарейки, что сейчас и берется из ОСМ. Нужны будут адекватные переводы.

$ tools/unix/translate_categories.sh 'Batteries'
en:Batteries
ar:ﺕﺎﻳﺭﺎﻄﺑ
be:Батарэі
bg:Батерии
cs:Baterie
da:Batterier
de:Batterien
el:Μπαταρίες
es:Baterías
fa:ﺎﻫ ﯼﺮﺗﺎﺑ
fi:Paristot
fr:Batteries
he:תוללוס
hu:Elemek
id:Baterai
it:Batterie
ja:バッテリー
ko:배터리
nb:Batterier
nl:Batterijen
pl:Baterie
pt:Baterias
pt-BR:Baterias
ro:Baterii
ru:Батареи
sk:Batérie
sv:Batterier
sw:Betri
th:แบตเตอรี่
tr:Piller
uk:Батареї
vi:Ắc quy
zh-Hans:电池
zh-Hant:電池

> Еще я хочу убрать type.recycling.toxic и вернуть как в ОСМ batteries. Нет контейнеров для "ядовитых отходов". Это именно батарейки, что сейчас и берется из ОСМ. Нужны будут адекватные переводы. $ tools/unix/translate_categories.sh 'Batteries' en:Batteries ar:ﺕﺎﻳﺭﺎﻄﺑ be:Батарэі bg:Батерии cs:Baterie da:Batterier de:Batterien el:Μπαταρίες es:Baterías fa:ﺎﻫ ﯼﺮﺗﺎﺑ fi:Paristot fr:Batteries he:תוללוס hu:Elemek id:Baterai it:Batterie ja:バッテリー ko:배터리 nb:Batterier nl:Batterijen pl:Baterie pt:Baterias pt-BR:Baterias ro:Baterii ru:Батареи sk:Batérie sv:Batterier sw:Betri th:แบตเตอรี่ tr:Piller uk:Батареї vi:Ắc quy zh-Hans:电池 zh-Hant:電池

biodranik commented

2022-01-17 07:47:23 +00:00

(Migrated from github.com)

Наверное надо сразу пройтись по всей таблице типов и вставить большинство из них, слегка отфильтровав (заматчив) дубликаты. А то получится недофича и недоредактор. Или может лучше просто в метаданные вставить эти теги? Тогда не придётся возиться с классификатором. Хотя в метаданных сразу несколько типов придётся хранить достаточно хитро.

Сейчас нам, получается, придётся различать контейнеры от перерабатывающих центров, и дальше отображать весь список подтипов, что именно там ресайклится.

Просто если мы сделаем удобно и хорошо, то те типы, которых мало, будут более активно добавляться. И можно стать "самой подробной картой ресайклинга в мире". А это тренд сейчас во многих странах.

Наверное надо сразу пройтись по всей таблице типов и вставить большинство из них, слегка отфильтровав (заматчив) дубликаты. А то получится недофича и недоредактор. Или может лучше просто в метаданные вставить эти теги? Тогда не придётся возиться с классификатором. Хотя в метаданных сразу несколько типов придётся хранить достаточно хитро. Сейчас нам, получается, придётся различать контейнеры от перерабатывающих центров, и дальше отображать весь список подтипов, что именно там ресайклится. Просто если мы сделаем удобно и хорошо, то те типы, которых мало, будут более активно добавляться. И можно стать "самой подробной картой ресайклинга в мире". А это тренд сейчас во многих странах.

vng commented

2022-01-17 08:18:17 +00:00

(Migrated from github.com)

Сейчас это лучше вешать на типы, потому что только с ними работает поиск и редактор.
Список в UI при добавлении отображать эквивалентно кухням теперь. И почему "придется отличать"? Тут нет проблем, все так же как и с кафе/ресторан/фастфуд сейчас ..
Я прошелся по топ recycling типам. Посмотрю еще, объединю "однотипные" https://wiki.openstreetmap.org/wiki/Tag:amenity%3Drecycling и пушну. Пусть там останется 10, но самые самые, без заморочек.

- Сейчас это лучше вешать на типы, потому что только с ними работает поиск и редактор. - Список в UI при добавлении отображать эквивалентно кухням теперь. И почему "придется отличать"? Тут нет проблем, все так же как и с кафе/ресторан/фастфуд сейчас .. - Я прошелся по топ recycling типам. Посмотрю еще, объединю "однотипные" https://wiki.openstreetmap.org/wiki/Tag:amenity%3Drecycling и пушну. Пусть там останется 10, но самые самые, без заморочек.

pastk commented

2022-01-17 10:00:50 +00:00

$ tools/unix/translate_categories.sh 'Batteries'
en:Batteries
ru:Батареи
uk:Батареї

По-русски точно будет лучше "Батарейки". Батареи без контекста разве что отопительные :)
В английском без контекста Batteries это зачастую аккумуляторы, а эта категория ресайклинга именно для маленьких батареек для электроники (для car batteries отдельная категория есть). Устоявшегося короткого термина, как в русском, увы, нет, но "Small Batteries" или "Batteries for Electronics" будет всё же лучше и понятнее, по-моему.

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

> $ tools/unix/translate_categories.sh 'Batteries' > en:Batteries > ru:Батареи > uk:Батареї По-русски точно будет лучше "Батарейки". Батареи без контекста разве что отопительные :) В английском без контекста Batteries это зачастую аккумуляторы, а эта категория ресайклинга именно для маленьких батареек для электроники (для car batteries отдельная категория есть). Устоявшегося короткого термина, как в русском, увы, нет, но "Small Batteries" или "Batteries for Electronics" будет всё же лучше и понятнее, по-моему. По этой причине переводы для других языков лучше бы сделать из русского "батарейки" - больше вероятность, что окажутся корректными. (я бы честно говоря вообще гугл транслейтом массовые переводы не делал, по мне лучше пусть вываливается англоязычный вариант (люди его видят и кто-то идёт дополнить перевод), чем вываливать потенциально кривой перевод, вводящий в заблуждение).

biodranik commented

2022-01-17 10:12:32 +00:00

(Migrated from github.com)

en:Batteries
ar:ﺕﺎﻳﺭﺎﻄﺑ
be:Батарэйкі
bg:Батерии
cs:Baterie
da:Batterier
de:Batterien
el:Μπαταρίες
es:Baterías
fa:ﺎﻫ ﯼﺮﺗﺎﺑ
fi:Paristot
fr:Batteries
he:תוללוס
hu:Elemek
id:Baterai
it:Batterie
ja:バッテリー
ko:배터리
nb:Batterier
nl:Batterijen
pl:Baterie
pt:Baterias
pt-BR:Baterias
ro:Baterii
ru:Батарейки
sk:Batérie
sv:Batterier
sw:Betri
th:แบตเตอรี่
tr:Piller
uk:Батарейки
vi:Ắc quy
zh-Hans:电池
zh-Hant:電池

Надо в английском вписать Small Batteries, и в комменте добавить Household batteries, NOT large car accumulators

``` en:Batteries ar:ﺕﺎﻳﺭﺎﻄﺑ be:Батарэйкі bg:Батерии cs:Baterie da:Batterier de:Batterien el:Μπαταρίες es:Baterías fa:ﺎﻫ ﯼﺮﺗﺎﺑ fi:Paristot fr:Batteries he:תוללוס hu:Elemek id:Baterai it:Batterie ja:バッテリー ko:배터리 nb:Batterier nl:Batterijen pl:Baterie pt:Baterias pt-BR:Baterias ro:Baterii ru:Батарейки sk:Batérie sv:Batterier sw:Betri th:แบตเตอรี่ tr:Piller uk:Батарейки vi:Ắc quy zh-Hans:电池 zh-Hant:電池 ``` Надо в английском вписать Small Batteries, и в комменте добавить Household batteries, NOT large car accumulators

vng commented

2022-01-17 11:21:47 +00:00

(Migrated from github.com)

PTAL. Did not finish with translations, but they are not visible now. Let's finalize "recycling" decomposition.

pastk requested changes 2022-01-17 11:49:18 +00:00

pastk left a comment

как вариант я потом могу по текстовкам в отдельном pr пройтись, мне эта тема довольно близка

data/categories.txt

pastk commented

2022-01-17 11:30:24 +00:00

тут путаница со стеклянными банками будет
можно "жестянки", "жестяные банки", "консервные банки"
в английский можно добавить "tin cans", "tins", "tincans"

тут путаница со стеклянными банками будет можно "жестянки", "жестяные банки", "консервные банки" в английский можно добавить "tin cans", "tins", "tincans"

pastk commented

2022-01-17 11:31:09 +00:00

'+ приём обуви

pastk commented

2022-01-17 11:32:04 +00:00

++ food waste, garden waste

data/categories.txt

pastk commented

2022-01-17 11:39:23 +00:00

Carton это как правило композитная упаковка типа тетрапака, соки например в них обычно. С т.з. переработки это не картон!
Картон по-английски cardboard, он либо к бумаге, либо вообще отдельной категорией должен быть.
Устоявшегося русского термина для carton нет, зачастую их все вцелом так и называют тетрапаками, так что можно "Переработка тетрапаков", наверное.

Carton это как правило композитная упаковка типа тетрапака, соки например в них обычно. С т.з. переработки это не картон! Картон по-английски cardboard, он либо к бумаге, либо вообще отдельной категорией должен быть. Устоявшегося русского термина для carton нет, зачастую их все вцелом так и называют тетрапаками, так что можно "Переработка тетрапаков", наверное.

data/mapcss-mapping.csv

pastk commented

2022-01-16 14:33:56 +00:00

Maybe also add recycling:glass? It doesn't intersect with glass_bottles much https://taginfo.openstreetmap.org/keys/recycling%3Aglass#combinations

Maybe also add `recycling:glass`? It doesn't intersect with `glass_bottles` much https://taginfo.openstreetmap.org/keys/recycling%3Aglass#combinations

data/mapcss-mapping.csv

pastk commented

2022-01-16 14:34:11 +00:00

maybe include recycling:electrical_appliances there also? Its ~60% intersection

maybe include `recycling:electrical_appliances` there also? Its ~60% intersection

data/strings/types_strings.txt

					
				@ -2919,0 +2937,4 @@

				  [type.recycling.shoes]

				    en = Shoes

				    be = Абутак

				    ru = Обувь

pastk commented

2022-01-17 11:43:08 +00:00

Консервные банки
или Жестяные

Консервные банки или Жестяные

pastk commented

2022-01-17 11:45:25 +00:00

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

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

pastk commented

2022-01-17 11:46:28 +00:00

Тетрапаки
ну или более длинный вариант - "картонная упаковка для напитков"

Тетрапаки ну или более длинный вариант - "картонная упаковка для напитков"

pastk reviewed 2022-01-17 11:51:43 +00:00

data/strings/types_strings.txt

					
				@ -2919,0 +2937,4 @@

				  [type.recycling.shoes]

				    en = Shoes

				    be = Абутак

				    ru = Обувь

pastk commented

2022-01-17 11:51:43 +00:00

или "Тетрапак и аналоги"

vng (Migrated from github.com) reviewed 2022-01-17 12:17:24 +00:00

data/mapcss-mapping.csv

vng (Migrated from github.com) commented

2022-01-17 12:17:24 +00:00

See replaced_tags.txt. All this "appliances" goes to recycling|small_appliances type.

vng (Migrated from github.com) reviewed 2022-01-17 12:17:56 +00:00

data/mapcss-mapping.csv

vng (Migrated from github.com) commented

2022-01-17 12:17:56 +00:00

Probably, now glass is merged into glass_bottles.

vng (Migrated from github.com) reviewed 2022-01-18 08:31:56 +00:00

data/categories.txt

vng (Migrated from github.com) commented

2022-01-18 08:31:56 +00:00

Понял. cardboard мержится с paper

biodranik (Migrated from github.com) requested changes 2022-01-18 11:21:46 +00:00

biodranik (Migrated from github.com) left a comment

Предлагаю вынести из этого PR и вмержить всё, что не касается recycling:

noexit
borders
форматирование и тесты

Так же выпилить коммит со сгенерированными строками.

А дальше @pastk может взять себе ветку и докинуть на неё нормальные переводы (и заодно потестить, как оно работает в UI и редакторе, если это возможно).
Мержить недоделанное не хочется.

Предлагаю вынести из этого PR и вмержить всё, что не касается recycling: - noexit - borders - форматирование и тесты Так же выпилить коммит со сгенерированными строками. А дальше @pastk может взять себе ветку и докинуть на неё нормальные переводы (и заодно потестить, как оно работает в UI и редакторе, если это возможно). Мержить недоделанное не хочется.

data/categories.txt

biodranik (Migrated from github.com) commented

2022-01-18 11:16:17 +00:00

Давайте сразу беларуский и украинский добавлять. Гуглтранслейтом если что, он хорошо справляется.

data/categories.txt

biodranik (Migrated from github.com) commented

2022-01-18 11:17:38 +00:00

Нельзя мержить paper и cardboard! Везде его отдельно собирают. Cardboard это упаковки от больших коробок, плотный картон, не связанный с едой. Paper это газеты, журналы, писчая бумага.

data/classificator.txt

biodranik (Migrated from github.com) commented

2022-01-18 11:18:01 +00:00

Тут s в конце?

pastk commented

2022-01-18 11:38:11 +00:00

Нельзя мержить paper и cardboard! Везде его отдельно собирают. Cardboard это упаковки от больших коробок, плотный картон, не связанный с едой. Paper это газеты, журналы, писчая бумага.

Тогда glass и glass bottles сразу тоже разделить заодно.

> Нельзя мержить paper и cardboard! Везде его отдельно собирают. Cardboard это упаковки от больших коробок, плотный картон, не связанный с едой. Paper это газеты, журналы, писчая бумага. Тогда glass и glass bottles сразу тоже разделить заодно.

biodranik commented

2022-01-18 12:00:02 +00:00

(Migrated from github.com)

Нельзя мержить paper и cardboard! Везде его отдельно собирают. Cardboard это упаковки от больших коробок, плотный картон, не связанный с едой. Paper это газеты, журналы, писчая бумага.

Тогда glass и glass bottles сразу тоже разделить заодно.

ОСМеры пишут, что сейчас большинство glass это glass bottles. Это близко к правде, т.к. "необычное" стекло принимают обычно только в специальных местах, и очень мало где. Хороший фильтр скорее всего был бы оставить glass, если это не контейнер, и конвертнуть его в glass_bottle, если это контейнер.

Ещё важно поддерживать (хотя бы общим типом) разноцветные контейнеры для стекла. В Швейцарии они всегда отдельные, для каждого цвета свой контейнер: recycling:glass_bottles:colour

> > Нельзя мержить paper и cardboard! Везде его отдельно собирают. Cardboard это упаковки от больших коробок, плотный картон, не связанный с едой. Paper это газеты, журналы, писчая бумага. > > Тогда glass и glass bottles сразу тоже разделить заодно. ОСМеры [пишут](https://t.me/OpenStreetMapOrg/87387), что сейчас большинство glass это glass bottles. Это близко к правде, т.к. "необычное" стекло принимают обычно только в специальных местах, и очень мало где. Хороший фильтр скорее всего был бы оставить glass, если это не контейнер, и конвертнуть его в glass_bottle, если это контейнер. Ещё важно поддерживать (хотя бы общим типом) разноцветные контейнеры для стекла. В Швейцарии они всегда отдельные, для каждого цвета свой контейнер: `recycling:glass_bottles:colour`

pastk commented

2022-01-18 13:13:27 +00:00

Could you please fix organicmaps/organicmaps#1839 (comment) also while you are at it.
Also adding amenity=waste_transfer_station could be useful for environmental activism.

And dump stations amenity=sanitary_dump_station / waterway=sanitary_dump_station are not related to recycling, but are very important to campervan/motorhome/boat users! (upd: this one is better done separately)

Could you please fix https://git.omaps.dev/organicmaps/organicmaps/issues/1839#issuecomment-1015390153 also while you are at it. Also adding `amenity=waste_transfer_station` could be useful for environmental activism. And dump stations `amenity=sanitary_dump_station` / `waterway=sanitary_dump_station` are not related to recycling, but are very important to campervan/motorhome/boat users! (upd: this one is better done separately)

vng commented

2022-01-18 18:28:08 +00:00

(Migrated from github.com)

Предлагаю:

Разделить картон и бумагу
Оставить мерж glass и glass_bottles.
Судя по этому цветов мало но есть, задаются разными способами и все равно самый частый "white;green;brown". Мы это типами красиво не поддержим, я предлагаю не заморачиваться тут. Ну и это совсем оверкилл отмечать каждый контейнер, они же стоят группами и на карте ставится одна точка для всей группы.

Предлагаю: - Разделить картон и бумагу - Оставить мерж glass и glass_bottles. - Судя по [этому](https://taginfo.openstreetmap.org/search?q=glass) цветов мало но есть, задаются разными способами и все равно самый частый "white;green;brown". Мы это типами красиво не поддержим, я предлагаю не заморачиваться тут. Ну и это совсем оверкилл отмечать каждый контейнер, они же стоят группами и на карте ставится одна точка для всей группы.

vng (Migrated from github.com) reviewed 2022-01-19 07:17:50 +00:00

data/classificator.txt

vng (Migrated from github.com) commented

2022-01-19 07:17:50 +00:00

сохраняем написание как в ОСМ

biodranik commented

2022-01-19 09:17:51 +00:00

(Migrated from github.com)

Ок, подчисти бранч, @pastk допилит переводы.

vng commented

2022-01-19 13:34:54 +00:00

(Migrated from github.com)

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

pastk commented

2022-01-19 16:43:33 +00:00

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

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

Иконку у landuse=landfill поменяете заодно? organicmaps/organicmaps#1839 (comment)

> Можно переводы и категории скинуть мне сюда файлом, я все вставлю. Да, можно попробовать так, после того, как функциональные правки будут закончены. Иконку у `landuse=landfill` поменяете заодно? https://git.omaps.dev/organicmaps/organicmaps/issues/1839#issuecomment-1015390153

vng commented

2022-01-19 16:58:08 +00:00

(Migrated from github.com)

Да, и добавлю waste_transfer_station, sanitary_dump_station, mast, flagpole

🎉 1

vng commented

2022-01-20 12:19:30 +00:00

(Migrated from github.com)

PTAL

pastk reviewed 2022-01-20 13:27:59 +00:00

data/styles/clear/include/Icons.mapcss

pastk commented

2022-01-20 13:26:18 +00:00

flagpole is a more significant landmark compared to power towers and masts - let's make it more visible, e.g. z15- like a lighthouse

`flagpole` is a more significant landmark compared to power towers and masts - let's make it more visible, e.g. z15- like a lighthouse

data/styles/clear/include/Icons.mapcss

pastk commented

2022-01-20 13:19:18 +00:00

Let's make it visible at lower zoom levels.
E.g. amenity=fuel is visible from z14 - dump stations are also used by motorists and its not a lot of them and usually they're located off dense areas like city centers so they won't clog up the map - I suggest making it z15- at least.

Let's make it visible at lower zoom levels. E.g. `amenity=fuel` is visible from z14 - dump stations are also used by motorists and its not a lot of them and usually they're located off dense areas like city centers so they won't clog up the map - I suggest making it z15- at least.

pastk commented

2022-01-20 13:32:46 +00:00

LGTM
(please consider my minor comments)

I'll do the strings.

LGTM (please consider my minor comments) I'll do the strings.

biodranik (Migrated from github.com) approved these changes 2022-01-20 13:53:08 +00:00

data/categories.txt

biodranik (Migrated from github.com) commented

2022-01-20 13:39:33 +00:00

Просто Батарейки будет тоже нормально, если иконка ресайклинга.

data/editor.config

					
				@ -313,3 +313,3 @@

				      <type id="amenity-recycling_container">

				      <type id="amenity-recycling-container">

				        <include field="name" />

				        <include field="website" />

biodranik (Migrated from github.com) commented

2022-01-20 13:40:34 +00:00

Всякие новые типы не вставляем в редактор? Батарейки там например.

biodranik (Migrated from github.com) commented

2022-01-20 13:41:17 +00:00

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

data/strings/types_strings.txt

biodranik (Migrated from github.com) commented

2022-01-20 13:51:24 +00:00

    en = Holding Tank Dump Station
    be = Зліўны рэзервуар для нечыстот
    de = VE-Station
    fr = Station de vidange
    it = Camper service
    ru = Сливной резервуар для нечистот
    uk = Зливний резервуар для нечистот

```suggestion en = Holding Tank Dump Station be = Зліўны рэзервуар для нечыстот de = VE-Station fr = Station de vidange it = Camper service ru = Сливной резервуар для нечистот uk = Зливний резервуар для нечистот ```

vng (Migrated from github.com) reviewed 2022-01-20 14:02:34 +00:00

data/editor.config

					
				@ -313,3 +313,3 @@

				      <type id="amenity-recycling_container">

				      <type id="amenity-recycling-container">

				        <include field="name" />

				        <include field="website" />

vng (Migrated from github.com) commented

2022-01-20 14:02:33 +00:00

Не вставляются, но это все надо будет делать по аналогии (прям точь-в-точь) с cuisine.

vng (Migrated from github.com) reviewed 2022-01-20 14:03:52 +00:00

data/categories.txt

vng (Migrated from github.com) commented

2022-01-20 14:03:52 +00:00

Наименование "Батарейки" в types.strings
Если не против что поиск "батарейки" тоже будет вести в урну, я исправлю.

Наименование "Батарейки" в types.strings Если не против что поиск "батарейки" тоже будет вести в урну, я исправлю.

vng commented

2022-01-20 14:57:03 +00:00

(Migrated from github.com)

PTAL

pastk approved these changes 2022-01-20 15:29:21 +00:00

pastk commented

2022-01-20 15:51:44 +00:00

Please add waterway=sanitary_dump_station (merge into amenity=sanitary_dump_station).
Upd: sanitary_dump_station=* also

Please add `waterway=sanitary_dump_station` (merge into `amenity=sanitary_dump_station`). Upd: `sanitary_dump_station=*` also

pastk commented

2022-01-21 06:48:24 +00:00

https://www.openstreetmap.org/way/884317426
Recycling centre is displayed as landuse=commercial

Could it be because maps need to be regenerated with new type changes?
Or type display priorities just need to be adjusted

https://www.openstreetmap.org/way/884317426 Recycling centre is displayed as landuse=commercial ![Screenshot_1642747363](https://user-images.githubusercontent.com/18434508/150479649-0c93c9a2-eda6-4bd2-8ae2-e87cf3d4137a.png) Could it be because maps need to be regenerated with new type changes? Or type display priorities just need to be adjusted

vng commented

2022-01-21 08:01:32 +00:00

(Migrated from github.com)

"Could it be because maps need to be regenerated with new type changes?"
Exactly! Now you can't test this changes in full.

But in this particular case, the object also will have 2 separate types. And we should adjust "display priority" amenities over landuses ..

"Could it be because maps need to be regenerated with new type changes?" Exactly! Now you can't test this changes in full. But in this particular case, the object also will have 2 separate types. And we should adjust "display priority" amenities over landuses ..

This repo is archived. You cannot comment on pull requests.

No reviewers

biodranik

pastk