architectury / architectury-api

An intermediary api aimed at easing development of multiplatform mods.
https://discord.architectury.dev/
GNU Lesser General Public License v3.0
315 stars 56 forks source link

[1.20.1 Forge] java.lang.Exception: Mod Loading has failed #459

Open katubug opened 9 months ago

katubug commented 9 months ago

Crash on startup of heavily modded game. It brought me to this screen: https://i.imgur.com/X2Ej8Es.png

The only change I made was deleting a standalone mod from the modlist. I do not believe that's relevant.

Crash report and latest log: https://gist.github.com/katubug/aa879f4aa2d8cff1f007c8fd0484350d

I'm using architectury-9.1.12-forge.jar, Forge is version 47.2.17

Thanks for your time!

katubug commented 9 months ago

Just started up the game without issue. I'm not sure what could have caused it, but I guess it's fine now? I'll leave this open in case you want to look at it, but otherwise feel free to close the issue.

Juuxel commented 9 months ago

at net.minecraft.client.color.item.ItemColors.handler$znn000$embeddium$preRegisterColor

Appears to be an issue with Embeddium, not really sure what. I don't think it's related to Architectury.

katubug commented 9 months ago

I'll chalk this one up to "modded mc be like that sometimes," but I'll keep the embeddium thing in mind in case it keeps happening. Thanks!

ChiefArug commented 9 months ago

FastUtil collections are not thread safe and do not check for thread safety like the built in collections, so this is likely a concurrent modification issue, meaning it will happen sporadically. Definitely something you should report to embeddium.

katubug commented 9 months ago

I've been told by embeddium "This is their bug, they are registering colors on a worker thread instead of queuing the work properly"

embeddedt commented 9 months ago

This appears to be an Architectury bug, AFAIK the vanilla color map is not thread-safe and so event.enqueueWork should be used inside the event handler.

Juuxel commented 9 months ago

Ah, I see. I'll reopen this for further investigation then.