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

meadow/grassland and orchard/vineyard color unification #1655

Merged
merged 2 commits into from
Jul 27, 2015

Conversation

imagico
Copy link
Collaborator

@imagico imagico commented Jul 14, 2015

Attempting to reduce clutter and variety of green tones:

  • unify landuse=meadow and natural=grassland (already hinted in Use a random symbology for forests #1242)
  • unify base color of landuse=orchard and landuse=vineyard (slightly stronger than current vineyard to improve contrast to scrub)
  • new fine grained pattern for landuse=vineyard, this is based on the standard pattern in German and Swiss maps - not sure how recognizable it is in other cultures. But it seems a huge improvement compared to the very coarse previous pattern which works particularly badly on fine grained vineyard parcels.
  • patterns for orchard/vineyard start at z=14 (vineyard was starting at z=13, orchard was always rendered with pattern).
  • move landuse=conservation to nature reserve boundaries (long overdue IMO but not really tested on real world objects).

Examples (from around here):

z=17
z=16
z=15
z=14
z=13
z=12

* unify landuse=meadow and natural=grassland
* unify base color of landuse=orchard and landuse=vineyard
* new fine grained pattern for landuse=vineyard
* patterns for orchard/vineyard start at z=14 instead of z=13
* move landuse=conservation to nature reserve boundaries
@kocio-pl
Copy link
Collaborator

Very good idea.

@matkoniecz
Copy link
Contributor

I think it would be better as separate PRs, at least "move landuse=conservation to nature reserve boundaries" should be separate. Maybe also "meadow/grassland" and "orchard/vineyard color unification" would be better as separate.

It simplifies testing, discussion and code review.

@matkoniecz
Copy link
Contributor

new fine grained pattern for landuse=vineyard, this is based on the standard pattern in German and Swiss maps - not sure how recognizable it is in other cultures

Completely unrecognisable for me. There is no standard pattern for vineyards for maps made in Poland as this feature is extremely rare here.

@matkoniecz
Copy link
Contributor

@imagico - is this test place is maybe an area close to you where you may perform local survey? In that case - can you look at https://www.openstreetmap.org/note/396185 https://www.openstreetmap.org/note/396187 https://www.openstreetmap.org/note/396188 https://www.openstreetmap.org/note/396189

@kocio-pl
Copy link
Collaborator

@matkoniecz BTW: there is a lengthy discussion going on tagging list about rural highways - a proposition was given to tag them as highway=service+service=rural when they are not really highway=track. I don't know however if this is the case.

@imagico
Copy link
Collaborator Author

imagico commented Jul 14, 2015

Actually i did this together to be able to test how these colors fit together. Changing colors one at a time is difficult since there are always interdependencies. You'd start changing the orchard color and then notice you'd better change it again to better accommodate the change in grass color.

I can take the change of landuse=conservation out here if there are doubts about it. Just wanted to make sure that the color change is not meant to apply to landuse=conservation.

Regarding the pattern - this type is also used on french maps, on Italian maps i have seen a somewhat similar 'L' shaped symbology - see page 6 here, the pattern also appears to be at least somewhat intuitive considering the appearance of current industrialized vineyards - at least not worse than the orchard pattern. Spanish maps use a different symbol which however seems much less intuitive. And that together covers more than two thirds of the world wine production...

@matkoniecz
Copy link
Contributor

Actually i did this together to be able to test how these colors fit together.

OK, I hoped that at least landuse=conservation may be tested independently.

@matkoniecz
Copy link
Contributor

And that together covers more than two thirds of the world wine production...

To make it clear - I am not thinking that lack of recognition for this symbol in Poland as a valid reason for rejecting this change (Poland has about 50 mapped vineyards).

@vincentdephily
Copy link

Consider the opinion of a French man if you will : I don't recognize those patterns either. I think the original pattern (3 raisins on the vine) is much more recognizable for the vineyards. The orchard pattern isn't great, but I'm not sure what would be better. Perhaps an apple is the most universally-recognizable fruit ?

As for the color changes, they look good to me.

@imagico
Copy link
Collaborator Author

imagico commented Jul 14, 2015

I realize this is a question on how much abstraction is desirable and how abstraction is made. The grape symbol uses fairly little abstraction in design but is quite abstract in the way it connects the landuse to the product produced (a bit like symbolizing forest with piles of logs). Someone unfamiliar with the product, visiting the vineyard only during winter, will not be able to make this connection. This makes the pattern unique in this style at the moment, all other patterns use symbols that are abstractions of the appearance of the area in question itself.

One problem of this is it cannot be made any smaller than it is without being unrecognizable. This means this needs to be a very coarse pattern which essentially makes it unsuitable for the lower zoom levels.

I would be open to a less abstract visualization of the appearance of a vineyard but this seems difficult without being misleading. My current suggestion might be unclear but i don't think it is misleading in the way it positively creates a wrong association.

Does anyone have examples for use of the grapes symbology in patterns in non-OSM maps?

@pnorman
Copy link
Collaborator

pnorman commented Jul 14, 2015

Actually i did this together to be able to test how these colors fit together. Changing colors one at a time is difficult since there are always interdependencies. You'd start changing the orchard color and then notice you'd better change it again to better accommodate the change in grass color.

One solution is to first do the colour changes in a spreadsheet, calculating colour differences. This can cut back the number of tests you have to do on the map, and let you be more precise in what to check on each iteration.

@imagico
Copy link
Collaborator Author

imagico commented Jul 21, 2015

It seems there is agreement about the colors but some objections concerning the vineyard pattern. I can change this back to the old one but since all objections so far have been based on a subjective difficulty to recognize the meaning of the pattern i am not really sure this is sufficient basis to decide this is really overall the preferable solution.

So again my primary reasons for using a different pattern summarized:

  • the old pattern due to its coarse structure is difficult to read with fine grained vineyard mapping - see here for example. While there are many wine growing areas where mapping currently is fairly coarse and not dividing the individual plots it would be preferable if the style did actually encourage fine grained mapping. Areas like here and here look all right with the current styling at the moment but would look much worse when mapped in minute detail at z=13-15.
  • the proposed pattern is a fairly common way to depict vineyards in maps, it is used in many non-OSM maps and also in several OSM maps (like here and here).
  • the pattern works well together with the orchard pattern underlining the idea that vineyards are more or less a subtype of orchard.

Further input (arguing either for or against the pattern change or suggesting a third option) would be good.

@pnorman
Copy link
Collaborator

pnorman commented Jul 21, 2015

The new pattern looks okay to me, but i haven't had a chance to load it up in Kosmtik.

@yohanboniface
Copy link

i haven't had a chance to load it up in Kosmtik.

Kosmtik bug?

@pnorman
Copy link
Collaborator

pnorman commented Jul 21, 2015

Kosmtik bug?

E_NOTIME

@yohanboniface
Copy link

E_NOTIME

Huhu ok ;)

@matkoniecz
Copy link
Contributor

I can change this back to the old one but since all objections so far have been based on a subjective difficulty to recognize the meaning of the pattern

And I am not considering my doubts as good reason to reject/change PR - but I will not merge it.

Maybe @math1985 or @gravitystorm may try looking at this PR?

@matthijsmelissen
Copy link
Collaborator

I haven't had time to look at it yet.

@matthijsmelissen
Copy link
Collaborator

I think both having a more pictographic and a more simplified representation have their merits. I find it hard to pick between the two.

@gravitystorm What do you think?

pnorman added a commit that referenced this pull request Jul 27, 2015
meadow/grassland and orchard/vineyard color unification
@pnorman pnorman merged commit 5439572 into gravitystorm:master Jul 27, 2015
@gravitystorm
Copy link
Owner

I'm happy with the abstract symbols for pattern fills. I'm slightly biased since (as @imagico links above) I use similar symbols in my own maps. But more importantly, pattern fills can repeat the symbol many hundreds of times, so they should be as simple as possible to reduce visual noise - it's more likely that an abstract symbol will be visually simpler than a (often complex) icon.

@matkoniecz
Copy link
Contributor

@imagico - are you planning to make & test new PR for

move landuse=conservation to nature reserve boundaries

?

@imagico
Copy link
Collaborator Author

imagico commented Sep 11, 2015

Opened a new issue for that: #1826

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.

8 participants