Oh The Biomes You'll Go

Oh The Biomes You'll Go

50M Downloads

Compatibility with Cinderscapes

o-negai opened this issue ยท 11 comments

commented

When i downloaded BYG and started a new world, I used the /locatebiome command and found that none of the biomes from Cinderscapes would come up. They all came up as "Could not find a biome of type within reasonable distance", meaning the biome ID's were valid yet could not be found. I removed BYG and loaded a new world and the biomes from Cinder loaded correctly. I am running Cinderscapes v1.0.4 for 1.16.1 and Biomes You Go 1.0.7 MCV: 1.16.1 on Fabric Loader 0.8.9+build 203-1.16.1.

commented

It looks like the way ahead for this is to implement the same system as smolder (register existing and all future additions of NETHER category biomes).
This would likely be best with a mixin plugin to conditionally load the mixin if Smolder does not exist or to require 0 on that one mixin to allow loading with a failed mixin in the event of smolder's overwrite. Without this, byg (or smolder) will fail to load.

commented

I pull from the fabric api internal biome data for nether biomes, it's their duty to ensure that their biomes are added through the provided fabric api

commented

how do you handle biomes added after your mod loads? if i read it right, you register a new biomelist with the contents that are currently in the list but never address additions after

commented

It shouldn't necessarily matter since the BP is only ever called at world/dimension load?

commented

but the static variables are initialized at class initialization, meaning at class load the list is set, not at world/dimension load

commented

Oh true

commented

Uhhhhhhhhh I'm not sure then

commented

Should i create a custom event for this?

commented

no, check out https://github.com/TerraformersMC/Smolder/blob/master/src/main/java/com/terraformersmc/smolder/Smolder.java

basically they add all nether biomes in the registry, then add a callback for any additional entries that are added later

we do need to test with smolder though to ensure the byg mixin wont cause minecraft to fail to load due to a failed mixin and with smolder doing an overwrite, it should cause conflict (or just add smolder to dependencies and remove the biome supplier completely )

commented

Patched in #31 PR