Revamped Tall Mountains - #3010
Conversation
|
Hey, I know that this biome is really cool, but can you please move it into a cave? It would look much better there. |
careeoki
left a comment
careeoki
left a comment
There was a problem hiding this comment.
This is undeniably very cool, and is a lot more detailed than what we have now. But
- Can you please split the cold mountain into a seperate PR? Yeah technically this is only 2 biomes, but there's a lot of individual subbiomes to look at here. (i also really don't like how snow biomes with trees look without #1364 , they just look weird with snow on the ground and nothing on the trees.)
- I think it's a little too tall. I would aim for 1000 blocks, for the sanity of anyone wanting to climb this. I like the amount of slopes there are, so see if you are able to just squish them closer vertically.
- There are a lot of big flat walls like this. Ideally there should be a smoother slope upwards. Again, since you added more slopes, you might be able to make it nicer by lowering the base biome.
|
It would be cool if we could force what biomes are next to it or by some other way control how steep the climb is, as it gets pretty ridiculous... in some cases. Here same mountain from two different sides: 
|
|
I updated the mountains base to be at 128 for this and the cold mountain PR but forgot to put a comment in to notify people about it. |
careeoki
left a comment
careeoki
left a comment
There was a problem hiding this comment.
Biome-wise it looks good, except one minor thing.
But something is happening here and creating these completely flat mountain sides.
@IntegratedQuantum you're better at identifying these technical problems than I am
| "cubyz:grass", | ||
| "4 to 6 cubyz:soil", | ||
| }, | ||
| .structures = .{ |
There was a problem hiding this comment.
I think you could reduce the amount of trees here. All of these big trees make it hard to navigate up the mountain (not in a fun way)
|
|
||
| .maxSubBiomeCount = 1, | ||
| .stoneBlock = "cubyz:slate/base", | ||
| .music = "cubyz:out_of_breath", |
There was a problem hiding this comment.
You should update this to the new music format (see #3044)
careeoki
moved this from Waiting for artistic review
to In artistic review
in PRs to review
|
I think you could take advantage of the young oak SBBs here, especially since the mountains parent biome uses them. |
8afdb32 to
ebed6d5
Compare
|
ok I have no idea what I'm doing with github and just messed a bunch of stuff up because I don't know what I'm doing |
|
#3098 is the new PR since I accidentally closed this one |
It's a bug in the code, only happens at the border of the map fragments, I'll make an issue, no need to worry about it here. |
|
thanks quantum |
5 participants
I've been thinking about current cubyz mountain generation, and realized that we can do better. With better generation settings I've created an even better tall mountain, with more trees to make it more realistic. I'm not completely happy or settled on the design yet, but I can say for sure it is an improvement. There are also some generation bugs with the mountains I'll need to fix.