Change rendering of man_made=embankment and barrier=retaining_wall

[valhentai]

I have some proposition of improvement for the rendering of made_made=embankment and barrier=retaining_wall

• Use the current rendering of made_made=embankment for barrier=retaining_wall. The current rendering of retaining_wall is the same as a normal wall. The use of the embankment would show that there is a difference of level between the two side of the wall
• Use the proposed rendering of natural=earth_bank for the made_made=embankment but with a lighter color. The current rendering is similar to rendering used on other map to show a fortified wall or rampart. On the map it give the impression of an impassable barrier but depending on the slope a embankment can be easily crossed.

The proposed rendering is already used in JOSM.

• If we decide to use the proposed rendering of natural=earth_bank for embankment, maybe render natural=earth_bank too with the same render or different color.

Here an example where i used made_made=embankment. On OSM it look like there is an impassable wall but on the pitcure we can see it is easily climbed.

Maybe I misuse made_made=embankment and should use natural=earth_bank. For me the two are the same with one natural and one artificial.

[imagico]

made_made=embankment and barrier=retaining_wall are features with distinct and different practical meaning so rendering them identically does not seem a very good idea.

natural=earth_bank is covered in #2288.

For better differentiating different kind of barriers see also #3975.

For reference: We currently render:

• natural=cliff
• made_made=embankment
• barrier=retaining_wall as a generic barrier

We don't render

• embankment=yes/cutting=yes - see Render embankment=yes in addition to man_made=embankment #791
• man_made=dyke - see Add rendering for man_made=dyke #823
• natural=earth_bank - see render natural=earth_bank #2288

Any suggestions for changes should take into account the need for an intuitive understanding of the different features and their semantic relationships.

[valhentai]

@imagico What i wanted to say was to use the current rendering of made_made=embankment for barrier=retaining_wall and change the rendering of made_made=embankment. Not use the same rendering for the two. My two first proposition can't be separated.

[imagico]

I got that - and it would be worth trying. What i meant to say is that if you (or someone else) wants to test this idea it is important to see how it relates to the other things that are currently rendered - and possibly also those which are not rendered but we might want to render in the future.

[jeisenbe]

Previously there was an attempt to make a different rendering for barrier=retaining_wall, in #3714 (comment) e.g.:

(barrier=retaining_wall on the west, otherwise man_made=embankment`)

Retaining wall on west, next to green hedge.