Terrestria

Terrestria

2M Downloads

Crash when Waterlogging Sakrua Tree Logs and Leaf Piles

emoace opened this issue ยท 5 comments

commented

This has also happened upon terrain generation, for instance if a Sakura Tree generated in a lake or something along those lines.

When waterlogging Leaf Piles, Not Enough Crashes says that Lithium is the culprit, but it seems the underlying problem is the same as when waterlogging the Sakura Tree Logs.

crash-2021-01-21_22.35.22-server.txt

crash-2021-01-21_22.33.08-client.txt

crash-2021-01-20_23.54.02-server.txt

commented

That looks bad. Does this happen with just Terrestria installed? I noticed that you have a lot of mods installed in those crash reports.

I saw this specific crash a few months back but it was never reported (and I couldn't figure out how to reproduce it), so thanks for reporting this!

commented

You're welcome! Glad I could help!

I created a new instance with only Terrestria, Traverse, Cinderscapes, REI, Fabric API, and Lithium.

I was able to waterlog Sakura Logs no problem even with Lithium on. As for the Leaf Piles, the water was just placed above the pile and disappeared like tall grass shortly after. This is as shown in the pics provided.

2021-01-22_15 19 39
2021-01-22_15 15 44

However, once I added Canvas, Hydrogen, Phosphor, Overworld 2, and Not Enough Crashes, the same crash occurred again:
crash-2021-01-22_15.30.08-client.txt

I couldn't waterlog the Leaf Pile even with more of the mods installed, but I could place a Sakura Log in an existing block of water, but when I placed another log connecting to the Sakura Log, the game crashed:
2021-01-22_15 35 03
2021-01-22_15 35 15
crash-2021-01-22_15.35.08-server.txt

After I removed Hydrogen, the problem did not occur as shown in this photo:
2021-01-22_15 42 06

I even tried it with the bigger pack, removing Hydrogen, and the issue was resolved.

So I would say it is an incompatibility with JellySquid's Hydrogen Mod, at least on my end. I will let you know if this occurs again without Hydrogen.

commented

Good to know. I will try reproducing it too.

commented

Seems to be related to CaffeineMC/hydrogen-fabric#10

commented

Based on my research in CaffeineMC/hydrogen-fabric#19, this is because Terrestria / Terraform construct a BooleanProperty.of("waterlogged") and then use Properties.WATERLOGGED in place of it at times. Terrestria and Terraform should not do this. While this at the very least happens with waterlogged things, I want to go through the code and ensure that there are no other instances of this kind of behavior. Preferably, if a "standard" property exists for something, we should just use that in all cases.