Repurposed Structures (Neoforge/Forge)

Repurposed Structures (Neoforge/Forge)

58M Downloads

pyramid_ocean causing crashes with TerraForged generation

NielsPilgaard opened this issue ยท 6 comments

commented

Going near an Ocean Pyramid causes a crash in worlds generated with TerraForged

Crash: https://paste.ee/p/FOJJ9

Versions

commented

@NillerMedDild when you are able to, could you please try this jar and see if the height restriction for the processor prevents the crash? It won't solve the issue of someone forcing the pyramid in a weird spot but this should hopefully workaround the issue.
repurposed_structures_forge-3.0.4b+1.16.5.zip

Hmm this might crash if the pyramid spawns in a floating island world and it ends up in the void. I will test that

commented

The crash seems to be occurring when I schedule a tick to a bubble column block so it can automatically create the column of bubble upward when the chunk starts ticking after worldgen is done.

chunk.getBlockTicks().scheduleTick(structureBlockInfoWorld.pos, structureBlockInfoWorld.state.getBlock(), 0);

As for why it is crashing, I do not know. the specific vanilla code seems to do p_205362_1_.getY() >> 4 so for that to be -1, something might be trying to force the ocean pyramid under y = 0 which shouldn't occur in normal terrain and is why I cannot reproduce the crash myself. Some other mod might be messing with structure gen somehow that cause the processor to run below y = 0.
image

I'm gonna try seeing if I can restrict it between 0 and 255 but I would need you to test it

commented

Will do ๐Ÿ‘

commented

I can't reproduce the issue ^^'
Edit: With the old jar, that is

commented

Just tested spawning the ocean pyramid in the end with a slight bump in height so that its bubble columns is right on the edge. Turns out, yeah it crashes as mc will attempt to spawn the entire structure piece and the part of the structure below the world bottom will also run the processor. And negative y values will be fed into the bubble column processor which then crashes the scheduling of fluid ticks. Tested it with the fix and it no more crashing.

v3.0.4 is now released on curseforge for download. Thank you for reporting this!

commented

Damn that was fast, thanks!