TerraFirmaCraft

TerraFirmaCraft

2M Downloads

Pyrite not generating

Allmoz opened this issue ยท 3 comments

commented

Hi,

It seems pyrite is not being generated with gold veins.

After some reading the code, in constants.py, the pyrite generation is defined as part of native gold generation for all the igneous rocks using rock categories "['igneous_extrusive', 'igneous_intrusive']" for the spoiler_rocks:

'normal_native_gold': preset_vein('native_gold', 'cluster', ['igneous_extrusive', 'igneous_intrusive'], 'pyrite', 20, ['igneous_extrusive', 'igneous_intrusive'], preset=NORMAL_S_METAL_ORE),

But at the moment of checking those rocks in vein_ore_blocks in world_gen.py, the rocks categories are not being expanded to all the rock names, so it never generates that spoiler_ore:

if vein.spoiler_ore is not None and rock in vein.spoiler_rocks:

Pyrite is the only being affected right now, cose is the only one using a rock category in "spoiler_rocks". Maybe expanding spoiler_rocks with expand_rocks may be enough to solve it:

def expand_rocks(rocks_list: List[str], path: Optional[str] = None) -> List[str]:

Cheers!

commented

Nice, i reader a little and saw the todo in the spoiler_rock, i will wait.

Thanks!

commented

Yes, this is a known issue, it's not getting fixed in 1.18 due to affecting existing worlds, and is fixed in 1.20 by #2516