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 sport POI #55

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

add sport POI #55

wants to merge 4 commits into from

Conversation

progserega
Copy link

Add sport POI with icons:
climbing, athletics, weightlifting, motocross, basketball, bmx, bowling, boxing

@AMDmi3
Copy link
Collaborator

AMDmi3 commented May 12, 2013

Это уже обсуждалось.

  • sport сам по себе - не POI
  • пока мы не умеем несколько POI из одного объекта, это поломает полноценные POI

А вообще, если это и делать, то

  • для всех видов спорта
  • в подкатегории внутри sport_and_entertainment ("по видам спорта")

@AMDmi3 AMDmi3 closed this May 12, 2013
@AMDmi3
Copy link
Collaborator

AMDmi3 commented May 13, 2013

Хотя, @ErshKUS говорит что поддержка нескольких POI из одного объекта таки возможна. Подумаем.

@AMDmi3 AMDmi3 reopened this May 13, 2013
@progserega
Copy link
Author

Это уже обсуждалось.
Где?
sport сам по себе - не POI
Хм.
http://taginfo.openstreetmap.org/search?q=sport#keys
Таки я всегда считал, что если я ставлю точку в josm, тыкаю F3, выбираю "скалолазание", тыкаю OK, вижу теги sport=climbing, name=*, то этого достаточно.
А получается, что обязательно надо выбирать тип - спортивный центр. Иначе - неверно.
Думаю не я один так считал.

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

Мне кажется, если уж выбирать, то полезнее для пользователя убрать вообще "спортивные центры", "стадион" и т.п., а добавить виды спорта: "бег", "велосипед", "плавание", "тяжёлая атлетика", гимнастика, шейпинг, бокс, рукопашный бой и т.п. А уж какие виды этого - пусть выбирает пользователь, тыкая по значкам - открытый стадион или же закрытый спортивный центр. Главное, что там можно поплавать/побегать/полазать.

@progserega progserega closed this May 14, 2013
@progserega
Copy link
Author

А вообще, если это и делать, то
для всех видов спорта
в подкатегории внутри sport_and_entertainment ("по видам спорта")

Согласен. Я добавил пока лишь часть, чтобы "обкатать" на малом. И если такое дополнение будет одобрено, то уж потом "закатывать рукава" по полной :-)

@progserega progserega reopened this May 14, 2013
@AMDmi3
Copy link
Collaborator

AMDmi3 commented May 16, 2013

Где?

В теме на форуме, где-то в районе предложения добавить места для дайвинга.

Ищущий зал скалолазания - ищет именно зал скалолазания, а не "спортивный центр" и "спортивное поле".

Так "зал скалолазания" и будет POI. "Скалолазание" - нет. По-хорошему, sport можно использовать для обозначения, например, для каких видов спорта в спортивном магазине продаётся инвентарь, или какими видам спорта занимаются в спортивной школе, и это совсем не то, чего хочет пользователь, ищущий места для занятия каким-то видом спорта. В общем, sport - это прилагательное, по которому можно сделать фильтр, но не создавать из него POI.

Я ещё забыл самое главное - из sport получатся POI с потерей типа объекта (т.е. для POI "Баскетбол" потеряется что это - площадка, спортивный центр или что-то ещё) и всех его собственных свойств (но останутся общие типа названия и контактной информации), и выглядеть это будет странно, а-ля "Футбол «Локомотив»" для стадиона «Локомотив» со sport=football. Стоит ли вообще такое делать?

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.

None yet

2 participants