EnhancedVisuals

EnhancedVisuals

35M Downloads

Crash with Carpet: NeoForged

Closed this issue · 7 comments

commented

Minecraft 1.21.1
NeoForge 21.1.208
CreativeCore 2.13.11
EnhancedVisuals 1.8.23
Carpet: NeoForged 1.0.4

This is similar to #174 but for Minecraft 1.21.1. Here is a gist containing the log.

Line 73 states Caused by: org.spongepowered.asm.mixin.throwables.MixinApplyError: Mixin [enhancedvisuals.mixins.json:ExplosionAccessor from mod enhancedvisuals] from phase [DEFAULT] in config [enhancedvisuals.mixins.json] FAILED during APPLY

Line 117 states Caused by: org.spongepowered.asm.mixin.transformer.throwables.InvalidMixinException: Incompatible @Accessor getX (for x:D) in enhancedvisuals.mixins.json:ExplosionAccessor from mod enhancedvisuals previously written by carpet.mixins.ExplosionAccessor (for X:D) [ -> ACCESSOR Applicator Phase -> enhancedvisuals.mixins.json:ExplosionAccessor from mod enhancedvisuals -> Apply Accessors]

Edited for clarity.

commented

What is Carpet? This crash seems to be very very weird.

commented

What is Carpet? This crash seems to be very very weird.

Carpet: NeoForged is a NeoForge Port of the Fabric mod Carpet

Carpet allows for a lot of detailed control over various aspects of Minecraft in general and adds many useful utility commands.

commented

Sorry, but I'm not sure what is going wrong there. I do not see a way to fix it from my end. Maybe report the issue to them. You can link it here and I will join the conversation.

commented

I guess the basic usage of @Accessor can conflict with other mods. I was able to fix it on my end by adding the carpet name to the accessor method names (here), which you can do as well to prevent future conflicts if you want.

commented

I added @Unique which do the same thing. Thanks for pointing this one out. New version has been released.

commented

Correction: Unique does not work, I have renamed the method names instead as @chililisoup suggested.

commented

Dang, it would be really nice if it did work for that