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 playground equipment rendering #3161

Closed
wants to merge 2 commits into from
Closed

Add playground equipment rendering #3161

wants to merge 2 commits into from

Conversation

pietervdvn
Copy link

Changes proposed in this pull request:

  • Add icons for playground equipment, such as slides and swings

Test rendering with links to the example places:

See https://www.openstreetmap.org/#map=19/51.20569/3.23290

Before:
screenshot from 2018-04-01 13-52-44

After
screenshot from 2018-04-01 13-38-01

@HolgerJeromin
Copy link
Contributor

HolgerJeromin commented Apr 1, 2018

The svgs should be exported so they dont have all inkscape metadata anymore.

But most important we have to decide if we want to render this high detail in this general purpose style
I am unsure if we should even if i tagged many of them

@kocio-pl
Copy link
Collaborator

kocio-pl commented Apr 1, 2018

Interesting work, especially for z19+. My comments:

  • please be more specific what elements exactly you include in this PR, with links to wiki pages and the numbers of uses
  • save the icons as plain SVG instead of default Inkscape, which adds a lot of unneeded metadata
  • be ready to discuss and tune the shapes 😄

@kocio-pl
Copy link
Collaborator

kocio-pl commented Apr 1, 2018

One more thing - it's good to join paths in the image, see:

https://github.com/gravitystorm/openstreetmap-carto/blob/master/CONTRIBUTING.md#map-icon-guidelines

@pietervdvn
Copy link
Author

  • This renders some playground equipments, as described here in the wiki. It adds support for swing, sandpit, slide, climbing_frame and springy, with respectively 4'255, 3'208, 3099, 1946 and 1840 occurences. Up to date numbers can be found on taginfo.

  • Plain SVG: is done

  • Oh god... Let the bikeshedding begin.

@pietervdvn
Copy link
Author

What do you mean by joining the paths?

Also: yes: I do think we should render this. It shows the richness and possibilities of OSM, and I would love to fill the playgrounds with more details.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Apr 1, 2018

Path > Combine probably.

@Tomasz-W
Copy link

Tomasz-W commented Apr 1, 2018

I'm against - it's creating too much mess and I don't see a need for details of playgrounds.

@pietervdvn
Copy link
Author

pietervdvn commented Apr 1, 2018 via email

@lakedistrictOSM
Copy link

Some feedback on the icons:

  • make the icons smaller (14x14)
  • slide: make the ladder thicker
  • springy: could probably be simplified
  • sandpit: needs simplifying
  • swing: again, could be simplified (perhaps try something like this)

@kocio-pl
Copy link
Collaborator

kocio-pl commented Apr 1, 2018

I don't believe it creates mess. If you go down to the playground and zoom in to z19 it's hard for me to believe that it was by accident and you don't want to see the details other than just the leisure background color all around.

The mess is when some features can be found in different places, like wayside shrine for example - beside being useful outdoor landmark it can be found in the city where once upon the time was the old way and now it's crowded place. That's why we had to tune zoom level to avoid mess while trying to show them relatively early for the more typical cases. Playground equipment is just a detail of a playground and this is where it can be found, so it's natural to expect it here - but of course on later zoom level than the playground as a whole. Z19 is very safe choice in that respect.

@HolgerJeromin
Copy link
Contributor

Could someone render this dense mapped playground?
https://www.openstreetmap.org/way/286032344

@pietervdvn
Copy link
Author

I will have a look at the icons in a few days, when more people had the chance to comment on them. I'll try to fix all issues at once then.

@HolgerJeromin : The difference is small, as play_structure is not rendered. It's difficult to make an icon for that.

screenshot from 2018-04-02 12-17-50

Copy link
Collaborator

@pnorman pnorman left a comment

Choose a reason for hiding this comment

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

I don't believe these icons are worth adding. Their benefit for finding something by looking at the map or orienting yourself when there in person are minimal, nor are there are special considerations which make playground equipment cartographically significant.

@dieterdreist
Copy link

dieterdreist commented Apr 8, 2018 via email

@pnorman
Copy link
Collaborator

pnorman commented Apr 8, 2018

Aside from the issues with the idea in general, the SVGs do meet our map icon style guidelines for canvas size, SVG XML formatting, and likely pixel alignment and reduced complexity.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Apr 8, 2018

Their benefit for finding something by looking at the map or orienting yourself when there in person are minimal, nor are there are special considerations which make playground equipment cartographically significant.

It strongly depends on which scale we're talking about. It's absolutely unsignificant (and I would call it just wrong) on a country or city level. Yet there are many detailed plans for such places on a high zoom level, for example:

https://sherbornplayground.files.wordpress.com/2014/04/playground-design-optional-1558562-preliminary-plan.jpg

Someone took the time to tag these objects, so she thinks it's important enough and I understand very well why micromapping is important for people.

Aside from the issues with the idea in general, the SVGs do meet our map icon style guidelines for canvas size, SVG XML formatting, and likely pixel alignment and reduced complexity.

Complexity is not that big in presented icons and pixel alignment is not top priority for everything other than simple geometrical shapes, but size doesn't look like 14 px in all the cases and it looks like a different style currently - BTW, are these icons yours or taken from some other style (I want to be sure the license is proper for us to use them)?

@pietervdvn
Copy link
Author

Hello everyone,

Yes, this is meant to be a fun gimmick for families, on high zoom level (only 19 ATM). I will look at the icons once more in a few days, when everyone has given his opinion.

I designed them myself (although one piece came from a CC0-image).

@polarbearing
Copy link
Contributor

@pnorman: did you mean "the SVGs do not meet our map icon style guidelines" ? None of the icons works in 14px.

@pietervdvn - sorry but your concept does not work here, for me.

The existing 'playground' icon cites a typical equipment, the see-saw. We would need to find an equipment-agnostic icon for 'playground' if we started rendering specific equipment. These equipment icons would need to cite the general playground icon, so the viewer could recognise the relation between them.

Even if that could be solved, the problem would be that you could not recognise the playground because there are too many equipment icons. Like not finding the wood because there are too many trees.

Oh god... Let the bikeshedding begin.

If you refer to the Bikeshedding in a sense that we are nitpicking on unnecessary details, your criticism is inappropriate. If everybody would throw her own design in the field, we would not recognise anything anymore. Imagine a soccer team with everybody wearing their own dress.

meant to be a fun gimmick for families

Indeed it would be fabulous on a family or kids oriented map style. Don't get me wrong. I contributed a lot to the wiki page and map every swing I find. But this is the general style.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Apr 8, 2018

Even if that could be solved, the problem would be that you could not recognise the playground because there are too many equipment icons. Like not finding the wood because there are too many trees.

This is a vaild problem and thanks for noticing it, but we have it with other types of objects.

Do you recognize park with fountains, artworks, buildings, footways, waterways, ponds, lawns, trees, playgrounds etc? I do, because usually they are rendered later than park. The only bad situation is when objects belonging to something bigger are visible too early. I have no idea how to move cemetery alleys/footpaths to higher zoom levels. Also well mapped hospitals or schools are no longer seen as yellow areas. The same with shops inside marketplace. See #2896.

With playground objects we have no such problems. We see playgrounds from z17 as icon and even earlier as an area. The area is still visible on z19, so you won't forget what are you zooming in.

@matkoniecz
Copy link
Contributor

I admit that I have (again) doubts whatever it is too much or is it still OK to render such details.

I had similar doubts about rubbish bin and benches.

@kocio-pl
Copy link
Collaborator

I would like to hear some arguments about source of these doubts, especially when these objects are rendered only on highest level. Maybe there will be nothing new, but still it's interesting for me to try to understand it.

I have other doubts - IMO it will be hard to just "rescale" these icons to 14 px.

@dieterdreist
Copy link

dieterdreist commented Apr 11, 2018 via email

@dieterdreist
Copy link

dieterdreist commented Apr 11, 2018 via email

@HolgerJeromin
Copy link
Contributor

HolgerJeromin commented Apr 12, 2018

The main problem i see is from polarbearing:
The leisure=playground icon is in the same class of the playground= icons.

first scenario: playground without playground= detail
you zoom in and see the see-saw in every zoom level. Or only till 18 (airport like). This would work

second scenario: playground with only one playground=slide
first option:
you zoom in and see the see-saw in every zoom level. At zoom 19 you see a see-saw and a slide. But in the reality there is no see-saw... Not good

second option:
you zoom in and see the see-saw in every zoom level till 18. At zoom 19 you see a slide.
We swap the wrong see-saw with the correct slide. "where is see-saw gone?" Not good

@pietervdvn
Copy link
Author

@polarbearing @HolgerJeromin
Yes, the 'polarbearing' is a problem, but I think it is only small one.

The only 'perfect' option I currently see is a new icon for playgrounds which does not feature a playground equipment icon (e.g. a kid running as seen with fitnesses icons or something similar), although I do not have a very good idea for that and I'm not willing to pick up that battle.

IMHO, showing the seesaw at high zoom levels is not perfect, but not a big issue. I'd rather have a map which shows icons and one to much, then having no icons as well.

However, hiding the seesaw-icon at high zoom levels is problematic. A playground without playground equipment would suddenly lose all icons. For now, I would let the seesaw in place, even if no seesaw is present.

On the other criticisms: as they icons render only on Z19, I don't think the cluttering of the map will pose any problem.

I'm having a look to the icons again to make them 14px.

And yes, rendering playground outlines would be great, but I'll only be trying that if this icon set goes somewhere.

@meased
Copy link
Contributor

meased commented Apr 12, 2018

Assuming that there was a nice 14x14 icon for each type of equipment, and each of these icons could be interpreted as a playground at lower zoom levels (may be a big assumption), then perhaps we could use the seesaw icon if no other equipment is tagged, or if equipment is tagged, just pick one and use that as the playground icon for that playground on all zoom levels.

@HolgerJeromin
Copy link
Contributor

However, hiding the seesaw-icon at high zoom levels is problematic. A playground without playground equipment would suddenly lose all icons.

Same with airport or cities :-)

We had an generic icon proposal (for kindergarten) in #120 (comment) which could be used in leisure color for playgrounds.

Copy link
Contributor

@nebulon42 nebulon42 left a comment

Choose a reason for hiding this comment

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

Thanks for putting a lot of effort into this. I'm with @pnorman here in thinking that this is too much detail even for z19. It is always hard having to be against a change especially from a first-time contributor who took up the hassle and get into contributing here.

@kocio-pl
Copy link
Collaborator

Since there are few voices against having this feature at all and I think it would be hard even for a skilled graphic designer to make the icons 14 px, I will close this ticket without merging. Sorry, I hope next time you will have more luck.

However i was talking lately with @dotevo and he said he was trying to create special app exactly for showing playground equipment, so you could talk with him about that.

@kocio-pl kocio-pl closed this Apr 27, 2018
@dotevo
Copy link

dotevo commented Apr 27, 2018

Ohh... I like this idea :-/ It is very useful for parents. I think the benches look bolder than this.

@SomeoneElseOSM
Copy link
Contributor

@pietervdvn I'd be happy to take a pull request at https://github.com/SomeoneElseOSM/openstreetmap-carto-AJT/ for symbols to show at zoom 20 and above (that style is designed for around up to zoom 24), but the graphics would still need to look characteristic even when rendered in 14x14 pixels - something that would be tricky with e.g. https://github.com/pietervdvn/openstreetmap-carto/blob/60cdab62b8696ce8d550897a16d078df9a8c2e7e/symbols/playground/swing.svg as it stands.

@pietervdvn
Copy link
Author

pietervdvn commented Apr 28, 2018 via email

@carstenhag
Copy link

I don't see how this would be bad for the highest possible zoom level. The user just sees a green area if we don't render playground equips on playgrounds.

@TheAdventurer64
Copy link

Could someone start working on this? It would be much appricaited.

@polarbearing
Copy link
Contributor

There are specialised maps now that render these features. I doubt it is useful on the general style.

@JesseWeinstein
Copy link

JesseWeinstein commented Oct 16, 2020

What are the specialized maps that render this? Got links?

(edited to add: sadly, https://taginfo.openstreetmap.org/tags/playground=swing#projects -- does not list any specialized maps. @polarbearing -- which ones were you referring to?)

@niedakh

This comment was marked as off-topic.

@SomeoneElseOSM

This comment was marked as off-topic.

@BertMule
Copy link

I was not aware of distinction between playground-items.
I thought it was just either playground or some 'playhouse'.

The latter is what I missed in a park, in issue #4816.

I would be happy if it was just those 2 possibilities, not going into further (too much) detail.

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