Stoneholm, Underground Villages (Fabric)

Stoneholm, Underground Villages (Fabric)

6M Downloads

Crash on generating village

deadspidervenom opened this issue ยท 14 comments

commented

Describe the bug
As i roam around the world, when it attempts to generate the unground village it instantly crashes.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Underground Village Village

Expected behavior
A village generates

Mod version:
1.2.0 (also did it on 1.1.0)

Fabric API version:
33.1

Minecraft version:
1.16.5

Additional Context:
Crash Log
https://pastebin.com/LJsg3Ur6

commented

Got no idea why this happens. If this is still an issue reopen this.

commented

That is an interesting error. It appears like it is unable to generate the village because it there are no structure pieces in the pool.

commented

Are you using a custom world type?

commented

i have a custom world generator installed, but i am using the default generation, i do have mods that add too the generation.

commented

After a bit of testing (had other issues to work on first) i figured out it seems to be a incompatibility with this mod

https://www.curseforge.com/minecraft/mc-mods/towelette

Which is strange cause i did not think it messed with generation.

commented

That is odd.

I don't think there is much I can do to fix this then since Stoneholm doesn't do anything but generate structures, it doesn't make any changes to how the game handles waterlogging.

commented

After looking at the error log, i looked up part of it and found this post, it might help. It is about a different area entirely, but the error is the same (though plug you mod and fabric in instead.)
https://stackoverflow.com/questions/32101688/illegalargumentexception-bound-must-be-positive

EDIT: a Decent quote for a TLDR

The issue is that you are calling Random.nextInt() with a zero and it doesn't like that. That is happening because the List from getAllItems() is empty. I would prevent this situation by checking that the list has items before performing your logic:

The reason i am posting, is cause i dont understand why out of all the generation mods i am using stoneholm is causing the error.

commented

Hmm maybe yung (from yung's better _____) could help? or maybe this mod dev
https://github.com/frqnny/mostructures

commented

@deadspidervenom maybe try to run your mods with Blame? - https://www.curseforge.com/minecraft/mc-mods/blame-fabric - It should tell where the issue is.
I would ask TelepathicGrunt (https://github.com/TelepathicGrunt/RepurposedStructures-Fabric) because they always know what goes wrong with mods issues.

Btw your FAPI is slightly outdated.

commented

@RDKRACZ Ill give blame a try, and yea i have updated mods since the log was given no changes.

commented

Yeah, I know exactly where the issue is happening. It's when picking the next structure piece to place out in Minecraft code.

Something is making the game not load the structure pools or somehow deleting them.

commented

What i got, the actual crash report had no blame report

****************** Blame Report 1.16.4-2.4.1 ******************

Empty template pool found from : stoneholm:start_pool
The path represented by this is: data/stoneholm/worldgen/template_pool/start_pool.json
Most common cause is that there is a typo in this path to the template pool which then points to a non-existent pool.
Please let the mod author know about this so they can double check the path to make sure it is correct.

commented

****************** Blame Report 1.16.4-2.4.1 ******************
Empty template pool found from : stoneholm:start_pool
The path represented by this is: data/stoneholm/worldgen/template_pool/start_pool.json
Most common cause is that there is a typo in this path to the template pool which then points to a non-existent pool.
Please let the mod author know about this so they can double check the path to make sure it is correct.

Yeah that is super weird. If you actually open the jar in like 7zip and go to data/stoneholm/worldgen/template_pool/ can you find start_pool.json?

commented

I can indeed find it