OreCruncher/DynamicSurroundings2

Biome sounds/ambience do not work correctly in The Twilight Forest.

Closed this issue · 5 comments

Dynamic Surroundings Version: 4.0.4.1
Twilight Forest Version: 4.0.412
Forge Version: 36.1.4
Minecraft Version: 1.16.5
Latest Log: https://pastebin.com/V2zPkBwy

I've noticed that when testing the twilight forest with dynamic surroundings that the ambience that would normally play when in forest-type biomes doesn't play and instead it plays Cave sounds even when I'm exposed to the sky and there are no blocks above me.

I'm assuming this is because the twilight forest's surface level is lower than the Overworld's and thus dynamic surroundings thinks that I'm in a cave, although that's just my guess as biome sounds don't seem to be working right across the whole dimension.

(Also, I'm unsure if this is the right github for this issue so just let me know and I'd be happy to move this issue to the other repository)

Apparently they have yet to fix that issue. The world is reporting sea level at 64 (same as overworld) though it generates at a lower Y value. Had this issue back in 1.12.2 as well.

I see, I'll report it on their github as well then.

Update: It is done: TeamTwilight/twilightforest#1056

Vanilla issue actually, World::getSeaLevel hardcodes the value

public int getSeaLevel() {
   return 63;
}

Sea level is properly set for our noise settings as well as our chunk provider:
https://github.com/TeamTwilight/twilightforest/blob/1.16.x/src/generated/resources/data/twilightforest/worldgen/noise_settings/forest_noise_config.json#L4
and
https://github.com/TeamTwilight/twilightforest/blob/1.16.x/src/main/java/twilightforest/world/ChunkGeneratorTwilightBase.java#L43

Not to mention.. the World's method is outright deprecated
image

Another note: I've no idea how you expect the client to even have any of this information to begin with. Servers with custom datapacks can change sea level purely server side.

I opened a general "support Twilight Forest" issue in the regular Dynamic Surroundings issue tracker. Next update will contain config changes for Twlight Forest which will define sea level for the dimension.

Awesome, Much appreciated!