Oh The Biomes You'll Go Refabricated

Oh The Biomes You'll Go Refabricated

11M Downloads

[Crash]: Setting `forceBYGEndBiomeSource` to `false` crashes game

saltyseadoggo opened this issue ยท 11 comments

commented

Minecraft Version

1.18.2

BYG Version

1.4 RC 15

Terrablender Version

1.1.0.99

Mod Loader

Fabric

Mod Loader Version

0.14.5

What happened?

Changing "forceBYGEndBiomeSource": true, to "forceBYGEndBiomeSource": false, in end-biomes.json5 causes the game to crash on startup.

crash-log

crash-2022-05-20_16.05.08-client.txt

latest.log

latest.log

commented

@Cosmic-Goat some file editors save JSON5 as corrupted files for whatever reason, try changing it

commented

I'm also getting this exact same crash, when changing any setting. I've tried editing the file in both vim and emacs with no special modes or anything fancy enabled. I'm using a Linux system, if that makes a difference.

This is likely a bug with BYG and not vim or emacs, because no other mod has any issues when changing configs, and I haven't run into any file corruption issues before using these editors.

commented

@Cosmic-Goat send your crash then

commented

latest.log
Crash Report

I've also cloned the repo and I'll try debugging it to see if I can find what's causing the issue on my end.

commented

As far as I can tell, Jankson freaks out about something when trying to read the config here (when editing nether-biomes.json5, for example). I can also confirm that editing the file in IntelliJ works fine for some reason. This is a really strange bug, sorry to bring it up!

Maybe file an issue with Jankson?

commented

send your config file, seems to have gotten corrupted @saltyseadoggo

commented

send your config file, seems to have gotten corrupted @saltyseadoggo

https://gist.github.com/saltyseadoggo/1b7ddba0bef9cf7b2a5db248b8439a94
This config file comes from deleting config/byg folder entirely, running the game to let it regenerate, exiting the game, opening the file in gedit, changing the one setting, and saving it.

commented

I also just had this happen when setting the weight of the subzero hypogeal biome on a fresh config from 0 to 2. It looked like the same crash text.

commented

@saltyseadoggo im under the impression the program you're saving the file with is causing issues, try a different one maybe?

commented

I opened the nether biome config file in IntelliJ and set the hypogeal's rarity there, and no crash. I guess gedit does something funky with .json5 files. It might be related to how it seemed to think they were written in C?

So that's issue resolved, I guess.
While I have you here, why are some of the Nether biomes disabled? Are they going to have breaking changes or something? Asking before I enable them in case doing so will cause issues down the line

commented

@saltyseadoggo disabled? they were probably not added back, and a good place to ask in the future is discord, please keep them issues related to issues