Change allotments color and pattern

[jeisenbe]

Fixes #3411

Changes proposed in this pull request:

• Change allotments color to garden/grass green, i.e. #cdebb0, Lch(90,32,128)
• Change pattern to near-white, light green 2x2 pixel squares on a 4x4 pixel grid
  *Pattern is #f2faeb (Lch97,8,130) approximately 75% white with 25% grass color
• Consolidate allotments on the same lines as garden and grass for text label rendering.
• Add allotments.svg and allotments.md to describe generating this pattern

Explanation:
Allotment gardens are currently a terra-cotta colour, similar to farmyards, with a pattern of white dots every 8 pixels. This looks nice and the relation to farmyards is sensible, but it does not immediately suggest that allotments are a type of garden.

This PR will change allotments to use the same shade of green as landuse=garden (and landuse=grass), while changing the pattern to be distinct from gardens and grass areas.

Test rendering with link:
http://openstreetmap.org/#map=16/52.1940/21.0404
Warsaw z15 master

z16

After z15

z16 after

Also see previous test renderings in issue #3411, eg:
#3411 (comment)
#3411 (comment)
#3411 (comment)

[jeisenbe]

Allotments with gardens and grass

Glauchau, Germany: allotments next to grass

Northern Ireland: allotments by residential gardens

Allotments between an area of grass and an area of garden

[jeisenbe]

Flamengos, Azores Islands
http://www.openstreetmap.org/#map=15/38.55036/-28.65888

z15 - Small patches of allotments with many other green features (orchards, gardens, grass etc)

z16 - Small areas of Allotments, garden, orchard, plant nursery, grass,

z17 - Allotments next to farmland, near botanic garden and orchards

Wales, Great Britain
Trethomas, Wales:
Allotments next to grass/meadow (in center)

Marshfield, Wales. Allotments on mid-right, garden on left by church

[kocio-pl]

Could you provide the code of this pattern but with #ddeec3 as a foreground? All is good with high zoom levels, but on midzoom (when the area is small) it's close to the grass. I'd like to check it, but pattern generation and producing raster image is not that simple.

[kocio-pl]

It looks like on z13 pattern is not used, which is OK for orange, but with the color change it looks exactly like grass (4x zoom to show it):

https://www.openstreetmap.org/way/25345944#map=13/52.1935/21.0956

[jeisenbe]

Right. At z13 there are no patterns shown, so orchards, vineyards and plant nurseries are identical, and gardens are the same as grass. Rock, scree, and are ground are also identical and all types of wetland use one style. I consider it an advantage that allotments use the same color as gardens and lawns, because at z13 almost all allotments are too small to be clearly distinguished by a pattern. It makes sense for allotments to look the same as lawns and gardens at low zoom levels, because allotments are mainly vegetable gardens, flower gardens or lawns depending on the country.

…

On Sun, Dec 9, 2018 at 11:18 AM kocio-pl ***@***.***> wrote: It looks like on z13 pattern is not used, which is OK for orange, but with the color change it looks exactly like grass (4x zoom to show it): https://www.openstreetmap.org/way/25345944#map=13/52.1935/21.0956 [image: pxayfnie] <https://user-images.githubusercontent.com/5439713/49692688-ead41580-fb60-11e8-93e9-d09edcdb1361.png> — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#3548 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AoxshPdnjUoLcqPB7I_GDYzRlSg8OyIHks5u3HL9gaJpZM4Y95Ek> .

[kocio-pl]

You're right, gardens are already grass-looking on z13. Could you produce the lighter green pattern to test it anyway?

[jeisenbe]

I've pushed a commit with the requested code for testing.

15. Allotments - #ddeec3 is used as the background for allotments with a pattern in #f6fbef (approximately 75% white plus 25% #ddeec3, by HSL).

(This commit is not ready to merge; the svg is not included and the file name would need changing, but it will work for testing)

My concern about this option is that it would be introducing yet another shade of green, in between Grass and Farmland, for a pretty specific tag. Also, at high resolution (eg on a retina display), this color can appear to blend with the pattern, with the result looking rather similar to farmland. That's why I've been recommending using the garden / lawn color for the allotments background.

Warsaw, z16

z15

z14

[kocio-pl]

Thanks. It is too light to see the pattern, so maybe 75% grass + 25% farmland would be better. I would like to have something that looks different than grass even when the area is too small to see the pattern anyway, but the pattern is important when zooming in.

[jeisenbe]

Here are a few more options. I checked for what could count as 75% grass plus farmland; the option with the most contrast from grass is #d8ecba, Lch(91,27,124)

16. Allotments in #d8ecba with 75 percent white for pattern
    z15a
    
    z13
    
    z14
    
    z15b
    
    z15c
    

I feel that the tests above were not much of an improvement; the color is still too close to grass. I also tried the current campsite / caravan site color, but this is not much of an improvement (it's also similar to grass).

Here is one more option. I've shifted the color to be more blue, similar to orchards, vineyards and plant nurseries, but lighter and less saturated. This makes the allotments more similar to parks, leisure and orchard, though it is still close to the grass/garden color. #d7f3c9 is Lch(93,24,134)

17. Allotments in #d7f3c9 with 75 percent white for pattern
    z15a
    
    z15b
    
    z13
    
    z14
    
    z15c
    
    z15d
    

(I also tried both of these background colors with 100% white squares for the pattern, but this is too much contrast, so I haven't bothered to upload these).

[Adamant36]

Hhhmm, I don't know about the squares. It makes the area look out of place since nothing else uses them and they are to uniform. Maybe a pattern would be better. Either with or without them though, what about a slightly darker tint to farmland. I'm sick of green colors. So I liked that allotments weren't green. It doesn't seem like the right color for it. I don't think of gardens as green. Let alone allotments. Its more of a civic amenity (like university grounds) or a social facility then a surface.

[kocio-pl]

They are similar to both garden and farmland, so they should be green also. Park has social meaning too, but green is for vegetation.

I will look at the examples closer later.

[kocio-pl]

Thanks - I tested it and it's too light, I don't see the pattern. So let's go back to the original idea (just keep "converted to PNG" in documentation).

[kocio-pl]

I have one more idea - what about using green shade of former scrub? It's quite dark (so the pattern would be visible) and distinguishable from other greens.

[jeisenbe]

The previous scrub color (#b5e3b5) is identical to the current golf course color, so I believe we would need to change golf courses to use this.

It was test number 5 back in the issue discussion, see #3411 (comment)

The golf color is also fairly similar to the current orchard/vineyard/plant_nursery background color, #aedfa3.

Golf left (Lch 86,29,143), orchard/plant_nursery right (Lch 84,36,138)

I previously tested the orchard/plant nursery color, #aedfa3. It could work, and I suppose allotments are similar to plant nurseries (although I would think thye are closer to grass/garden normally). I tested this with the larger square pattern before, but I'll need to try it with the smaller squares. Here's a couple of examples with the larger pattern:

I can push commits with both of this options, for testing, if interested.

[kocio-pl]

Yes, please test this color, but with current pattern.

[jeisenbe]

Test 18: Allotments in golf-color #b5e3b5 with 75% opacity white pattern:
Lch(86,29,143)
z15

z16

Test 19: Allotments in plant_nursery/orchard/vineyard color #aedfa3.
Lch(84,36,138)

z15

z16

[kocio-pl]

It looks too dark, especially on z13, when we have no pattern, so I believe original change proposition is the best at this moment.

[jeisenbe]

OK, so we are going with #cdebb0, same as gardens and grass. I've pushed the changes. This should be identical to the initial commit, as seen here: #3548 (comment)

[kocio-pl]

Hm, do we have to drop the pattern at z13? Could you try how would it be without dropping it at all?

[jeisenbe]

@kocio-pl, the other green landuse patterns generally begin at z14, eg vineyards, orchards, plant_nursery, garden, cemetery. These areas are generally fairly small and the pattern is unlikely to be visible at low zoom levels for most allotments (though your huge ones in Warsaw may be an exception)

Some of the natural patterns begin much earlier, because they can cover large areas. Specific wetland patterns begin at z14, along with beach and scrub. Screen/shingle and forests begin at z13, quarries and reef at z10. Bare_rock, sand, (generic) wetland and glacier patterns begin at z5.

The other issue is that z12 is where the landcover colors start to be faded, so it becomes harder to distinguish subtle differences.

I've tested this on the two larger towns/cities in southern Wales. Cardiff has several fairly larges areas of allotments; Newport only has small allotments (as is common in Britain).

Cardiff z14, with pattern:

z13, No pattern:

With pattern:

z12, No pattern:

With pattern:

Newport z14

z13 No pattern:

With pattern:

z12 No pattern:

With pattern:

It looks like the pattern could work on z13, though it's only clearly visible on the larger allotments and can be confusing on small allotments that show only a few pixels. But z12 won't work.

[kocio-pl]

Since I see nothing bad with rendering pattern on small allotments and there are more places with the big ones, like here:

https://www.openstreetmap.org/#map=13/50.3657/30.6137
https://www.openstreetmap.org/#map=13/52.4557/13.4252
https://www.openstreetmap.org/#map=13/54.1680/19.4217

let's render allotments pattern from z13+. I see no problems with other patterns on low zoom levels so far, so it might be tweaked in the future in general (also in gardens etc.), but it's not worth digging even deeper for now.

[kocio-pl]

@jeisenbe It would be nice to have this merged in the new release on Friday, do you think you can add z13 rendering soon?

[jeisenbe]

I’ve pushed the change. The allotments pattern should now render from z13.

…

On Wed, Dec 19, 2018 at 8:56 PM kocio-pl ***@***.***> wrote: @jeisenbe <https://github.com/jeisenbe> It would be nice to have this merged in the new release on Friday, do you think you can add z13 rendering soon? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3548 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AoxshFtqKcXDw0dTgIDUg-DNHDVk5vfVks5u6il6gaJpZM4Y95Ek> .

[kocio-pl]

Thanks, that was quite hard work to test, now let's see how it works full time on the map...

[dktue]

I love it on the map! :-)

[polarbearing]

Thanks for all the fine-tuning, and good to see the little-squares idea works so well. Here is a nice before-after effect on adjacent tiles ;-)

[kocio-pl]

I think some boundary line is missing in many cases.

[polarbearing]

Indeed, which surprised me a bit since the top post shows boundaries both for before and after.
However in the code I see no line drawn, neither before nor after. Maybe they are barriers?

Should we focus on this one alone, or open a more general issue 'review boundaries for filled area'? At least school needs it darkened, there is one open.

[kocio-pl]

Please make as simple PRs as possible, because I feel overwhelmed currently. It does not hurt to make more merges, while reviewing complex proposals with many days of discussions and interdependencies is very hard for me and still we have no other active reviewers with merging rights.

This is general remark, not only related to this proposition. Sorry for this inconvenience.

[jeisenbe]

The similar features leisure=garden and landuse=plant_nursery / =orchard / =vineyard also lack outlines, so perhaps there should be a new issue to discuss this in general before adding more outlines.

…

On Thu, Dec 27, 2018 at 12:22 AM kocio-pl ***@***.***> wrote: Please make as simple PRs as possible, because I feel overwhelmed currently. It does not hurt to make more merges, while reviewing complex proposals with many days of discussions and interdependencies is very hard for me and still we have no other active reviewers with merging rights. This is general remark, not only related to this proposition. Sorry for this inconvenience. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3548 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AoxshAdJ47-EEy_jQG2sMz2u37_C_vRuks5u85RPgaJpZM4Y95Ek> .

[Adamant36]

Its questionable to me if things like orchards or vineyards, which dont usually represent the actual landuse boundary and only the bounds of the crop being grown are easy to seperate from other things on their own, need outlines in the first place (maybe not with plant nurserys though). So I suggest it goes through rigorous disscussion in their own seperate issues if they are going to be considered.