oitsjustjose / Geolosys

A logical, geological system in Minecraft
GNU General Public License v3.0
42 stars 36 forks source link

LAYER type deposits are generating samples but no ore when overriding plutons with data pack #278

Closed Whiskey-Sunshine closed 2 years ago

Whiskey-Sunshine commented 3 years ago

Versions:

What happens:

Ore samples are generated on the surface, but the ore itself is not generated beneath the ground at all when a data pack override is used to change plutons to LAYER type.

What should happen:

The plutons should generate in a layer type as specified by the .json entries

Logs (if necessary):

Strangely enough, logs show no errors on generating plutons.

Additional Comments:

I was admittedly getting very annoyed with sparse plutons and wanted to make things feel a bit more like the older 1.12.2 version, and started getting confused when world stripper was revealing emptiness where ores should have generated. After some time checking repeatedly, I noticed that the issue was coming from only ores that were generated with the LAYER type. DIKE, DENSE, SURFACE_LAYER, and SPARSE all worked correctly, still.

geolosys_overrides.zip geolosys_overrides_no_layer.zip

Using the first datapack, all plutons generated with LAYER chosen (In this case, several coal types, lapis, cinnabar, and mystical world quicksilver) did not generate. With the second, they successfully generated. Some deposit sizes greatly increased for visibility.

Additional screenshots showing missing ores after using worldstripper, with a dense gold pluton showing that that one did generate correctly. https://imgur.com/a/mASaCje

oitsjustjose commented 3 years ago

I believe this should be fixed in the 1.16.Refactor branch but I'm not 100% sure, so I will port your datapack to the new format and verify that it works as expected.

oitsjustjose commented 2 years ago

Stale - closing until someone mentions they can reproduce in Geolosys 6.0.0+