Height Map Landscapes have Two Edges marked as Cliffs about openloco

[LeftofZen]

hey Glen, can you send me the exact PNG you used for this, just so I can test it as you have it

edit: nevermind, I've managed to repro it with your land gen settings

[LeftofZen]

Whatever texture is listed last will be applied to these two edges

Actually, it is the 'first' terrain type here, Brown Rock, but the Around Cliffs generator that is the real issue - this is the very last generator that is run in the code, and seems to be overwriting the terrain with whatever land type you've set for it. Unfortunately this generator is not yet reverse-engineered so I cannot examine exactly what and why this is happening, and why the game things these areas are "around cliffs"

[glenjimen]

I've attached the png in case you'd still like it.

Actually, it is the 'first' terrain type here, ...

I didn't explain that part quite right, if you have multiple terrain types set to 'around cliffs' then only the last alphabetically will be applied. I think this is the expected behavior, actually.

You can see multiple terrain types set to around cliffs, but only sand is applied to the edges and to actual cliffs throughout the map.
Anyway, that's not really a problem in need of a solution.

Also, with the new paint terrain tools you've added, it's easy enough to go thru and fix up those two edges :-)

[glenjimen]

FYI, more details. The terrain type stops getting applied to an edge if the land reaches a height of zero.

[LeftofZen]

We've fixed this in an experimental branch; Aaron reverse-engineered the generateTerrainAroundCliffs method just for this, and in that method the cliff detection code runs on all tiles of the map. Bringing in that range 1 tile from the edge of the map means the edge-of-map 'cliff' will no longer be detected as a cliff, fixing the problem. You can also solve this already by just not using the "around cliffs" terrain colouring.

[glenjimen]

As of release v24.04, the cliff coloring is halfway solved :-)

Now only one edge is being colored as a cliff. Same settings as before. Here is the heightmap I used, in case it's helpful. I was able to reproduce this reliably with multiple different heightmaps.

It's bizarre that your applied solution only reduced the number of edges effected. Still, pretty easy to go in and manually clean up that edge, so not a huge high priority thing.

[LeftofZen]

Ah, I didn't actually put the fix into this release! I totally forgot about it, and it required #2383 which was only merged 12 hours ago, just before I went to bed and as the other guys tagged the release. I will get this in for next release for sure, though that may be while away.

Interesting that the behaviour still changed though!