-
Notifications
You must be signed in to change notification settings - Fork 823
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
Render natural=shrubbery #4530
Render natural=shrubbery #4530
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The general thing first: As indicated in #4473 (comment) the situation from last September that has led to the consensus to close that issue has not substantially changed. I don't want to prejudice the possibility that this assessment could change but so far no argument is in sight that is likely to result in that.
Independent of that there are a few other issues with your PR:
- your patterns are PNG only - we usually want patterns to be designed as SVGs so they can be rendered at different resolutions in good quality.
- the pattern design does not allow for an intuitive distinction towards the other features we render with the same base color (natural=scrub and natural=wood). Since most of the features with this tag are small a coarse pattern is not a good idea, the only pattern that you could really consider for this would be a relatively fine structure pattern similar to for example leisure=garden and natural=beach.
- you overload the progression of wood/forest and scrub colors which indicate a difference in height of vegetation in this style with a completely different use to indicate some kind of density of vegetation. That is not a good idea. You will have heath-height shrubbery in the same color as a rainforest with >50m high trees.
- Interpretation of
shrubbery:density
is highly questionable - it is ostentatiously non-verifiable and there is no indication that its very limited application in the database follows any consistent verifiable principles beyond the domains of the individual mappers who use it (which is just 40 by the way). The only hint at what the different density values mean is the wiki page and that is full of weasel words ('generally', 'might', 'effectively') and is available in English language only - hence no one who does not speak English has a chance to apply these tags in any consistent way.
Different color for dense shrubbery sounds to me like interesting approach. My questions:
|
I was following the literal example of
Thanks. I did it mainly to stay near to
Any colours or hues you wish to suggest?
It does that — two, three, or four lobes on the symbol getting 'denser' — I'll see if I can create a screenshot more clearly showcasing it.
Currently I leave out the symbols but keep the same fill colour. That seems to work OK visually and rewards mappers for using |
No, the number of vegetation related green colors in this style is already at the limit of what can realistically be intuitively understood. And in its current generic urban greenery use the tag does not in any way fit into the schema we have for vegetation related green colors. If you'd want to support the generic urban greenery use the most suitable option would be a design somewhat similar to |
Render `natural=shrubbery`. The three `shrubbery:density` values (`sparse`, `medium`, and `dense`) are reflected in the pattern. Fixes gravitystorm#4473
f372f04
to
6731bdb
Compare
I've changed the image patterns to SVG as requested. @kocio-pl Here is a sample of the three SamplesThe example below includes some patches of shrubbery that were already tagged as I'm quite satisfied with how the rendering turns out here. NowWith this PR |
I have seen the request for review but as indicated i continue to stand behind the decision to close #4473. It was @kocio-pl who re-opened the issue and suggested to work on a PR so i think it would probably be more productive if he provides feedback how he thinks this can be developed in a way that can achieve consensus approval from the maintainers. It seems unlikely from my perspective but as said i don't want to prejudice the possibility. |
As a mere contributor I cannot select who the reviewer is, nor can I request specific reviewers via this UI. I have not specifically requested you to reassess your review. A maintainer will probably have to click some buttons to change who the reviewer is. |
Sorry, my mistake then. I always thought you can request a review from a specific person. |
@kocio-pl do you want to review or comment on this? |
See my comments in #4473 (comment) - I think it is probably still too soon to render this new tag, until it clearly overtakes |
Closing - see #4473 (comment) and reasons above. |
@pnorman What is the relevance of that code snippet for this PR? If it's 'too soon' to render this tag, then why close the PR? Is something wrong with the PR itself? |
Code snippet was a mispaste, corrected it above. The reason for closing this PR is we're declining rendering this tag. |
@pnorman Could you give an indication of the number of uses needed to consider rendering? |
@jeisenbe How can we measure its use in urban gardens in comparison to |
Can this please be rendered. The map now looks broken/imcomplete and is not reflecting the map data for people to recognise their surroundings. By minimum at least render it the same as 'scrub'; as it is a form of greenery. To be honest, I found it a bit silly that a very specific tag as 'flowerbed' is being rendered; but has far less pratical implecation than shrubbery. That tag is far older (2015), but only started to become popular by 2020. It took until 2022 before it had 25k tags and today has 45k tags. In contrast, Shrubbery was implemented in 2021 and today has 24k tags in use. I do believe that these both are 'micromapping' tags and people start using them when the more rudimentary mapping is already done. But then again; so might lampposts, bollards, or manholes. Or a swing in a playground. |
I don't see a reason to not at least render this the same way as |
Render
natural=shrubbery
. The threeshrubbery:density
values(
sparse
,medium
, anddense
) are reflected in the pattern.The pattern is modelled on the pattern used for
natural=scrub
(same distribution of symbols). Colours used matchnatural=scrub
too, except when theshrubbery:density=dense
, then the same colour asbarrier=hedge
is used to reflect that in both cases the vegetation is generally speaking impassable.Fixes #4473
Before
(Here
barrier=hedge
witharea=yes
is tagged. In the 'after' screenshotsnatural=shrubbery
withshrubbery:density=dense
replaces it.)After
Showing both
shrubbery:density=dense
andshrubbery:density=medium
:In the following screenshot some of the
natural=shrubbery
is tagged withshrubbery:density=sparse
and noshrubbery:density
tag for comparison: