Oh The Biomes You'll Go

Oh The Biomes You'll Go

50M Downloads

[Suggestion] Compatibility with BetterNether

paulevsGitch opened this issue ยท 17 comments

commented

Good day. I am author of BetterNether mod and I want to make a small suggestion about compatibility. As I can see nylium blocks from BYG doesn't have "minecraft:nylium" block tag, could you add it? This will allow plants to grow on them and will correct bonemeal effect. Thank you in advance

commented

Wanted to ask, does my mixin into the nether biome provider work with your mod? And yes I'll get this fixed!

commented

Looks like all other things works fine. Probably because of my own biome provider (something may generate wrong, I don't know what changes you made). If you are interesting I'm trying to cooperate with Cinderscapes developers to integrate my code and their code into unified biome library. And at this time I have a small API for advanced biome generation (like sub-biomes or edges)

2020-07-04_01 12 10

2020-07-04_20 02 11

commented

Is this still oen?

commented

There is an issue with biome generation. It appeared on latest version of BYG.

commented

I think the best way to solve it is to add another condition with "betternether" here

commented

The last one tag is incorrect (must be "byg:nylium_soul_sand", with namespace)

image

P.S.: BetterNether also has a tag "common:soul_ground" for soul-sand-soil like blocks, can you add "Nylium Soul Sand" and "Nylium Soul Soil" into it?

commented

With manually corrected tag it works

2020-07-05_01 40 32

commented

oh my bad didnt realize my misspell

commented

https://github.com/CorgiTaco/BYG/tree/Fabric-1.16.X/src/main/resources/data/common/tags/blocks
does this look correct? Im new to this whole common/tag system

commented

Looks like it is correct

commented

Should I add my nylium blocks that are literally netherrack but are a top block variant too the netherrack tag?

commented

I think yes, this tag was created for all types of netherrack including nylium

commented

Also My nether BP was pulling from fabric api's added nether biomes so thats odd.

commented

Should be it. If i missed something comment on this issue. I'm closing

commented

Now it works, thank you