Iceberg [Neo/Forge]

Hi there!

Since Embeddium compat has been removed in the latest Iceberg release, could you remove the check for it here...

Iceberg/common/src/main/java/com/anthonyhilyard/iceberg/renderer/VertexCollector.java

Lines 38 to 39 in 52be22f

    
           // If Embeddium 1.0+ is installed on NeoForge, also use the Sodium implementation. 
        
           useSodiumVersion |= Services.getPlatformHelper().getPlatformName().contentEquals("NeoForge") && Services.getPlatformHelper().isModLoaded("embeddium") && Services.getPlatformHelper().modVersionMeets("embeddium", "1.0.0");

...so that your logic will fall back to using vanilla VertexConsumers, rather than crashing with Embeddium installed due to trying to load a Sodium class?

It seems they want to focus on Sodium 0.6 support, which is fine, as Embeddium should be able to work fine with their fallback vanilla logic. Continuing to use the Sodium logic with Embeddium installed seems to be an oversight.

I'm actually curious.. why was Embeddium support removed? I hate to be mean but if this sort of update is going to be made, at the very least make sure that it doesn't result in the game crashing or put a warning on the mod's front page about such breaking change. Spent a good 5 hours trying to figure out why suddenly my private server broke only to discover it was this particular update, not something fun to deal with at all.

@NovaViper

Apologies for the oversight, an update is currently pending that will fix this crash.
@NovaViper Because the Sodium API has diverged from the Embeddium API, and supporting both is not realistic at this time. Perhaps in the future. The changelog of the most recent Iceberg update indicated that Embeddium was no longer supported, that is a good first place to check in the future.

	// If Embeddium 1.0+ is installed on NeoForge, also use the Sodium implementation.
	useSodiumVersion \|= Services.getPlatformHelper().getPlatformName().contentEquals("NeoForge") && Services.getPlatformHelper().isModLoaded("embeddium") && Services.getPlatformHelper().modVersionMeets("embeddium", "1.0.0");

Share to