Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incompatibility with Emotecraft #2371

Open
Lolip-p opened this issue Mar 9, 2024 · 5 comments
Open

Incompatibility with Emotecraft #2371

Lolip-p opened this issue Mar 9, 2024 · 5 comments
Labels
A-mods Area: Mod compatibility F-help-wanted Flag: Help wanted

Comments

@Lolip-p
Copy link

Lolip-p commented Mar 9, 2024

Bug Description

Using the Emotecraft mod 2.2.5 and Sodium 0.5.6+ or 0.5.8 the following happens: the bones in the elbows and knees disappear. Animations stop looking beautiful. The animation used is the same. This error occurs starting from version 0.5.6. It is also present in the newest version at the moment, 0.5.8.

Emotecraft : https://github.com/KosmX/emotes/releases/tag/2.2.5

It will be more clear to look at the screenshots:

0.5.3

mc_sod0 5 3

0.5.6+ or 0.5.8

mc_sod0 5 8

Reproduction Steps

  1. Minecraft version 1.20.1 Quilt or Fabric.
  2. Launch Sodium 0.5.3 and Emotecraft 2.2.5. Select an animation where there are bends in the joints. We see that everything is fine.
  3. Launch Sodium 0.5.6+ or 0.5.8 and Emotecraft 2.2.5. We select the same animation, we see that it is broken.

Log File

latest.log

Crash Report

crash-2024-03-09_13.27.56-client.txt

@Lolip-p Lolip-p added the S-needs-triage Status: Needs triage label Mar 9, 2024
@jellysquid3
Copy link
Member

The mod author can set a mixin rule to disable features.render.entity, but this won't work with Iris, and it will reduce performance in the general case.

Not really sure why this mod is broken. Sodium should just be rendering what it's asked to. Needs investigation.

@jellysquid3 jellysquid3 added A-mods Area: Mod compatibility and removed S-needs-triage Status: Needs triage labels Mar 11, 2024
@jellysquid3 jellysquid3 changed the title Poor compatibility with Emotecraft Incompatibility with Emotecraft Mar 11, 2024
@dima-dencep
Copy link

Not really sure why this mod is broken

EmoteCraft replaces the quads in cuboids with its own when playing the bending animation, which is not possible while sodium completely replaces cuboids

But they have a second way where they just replace the Cuboid render with their own, which also doesn't work with sodium because sodium overrides the vanilla renderer calls.

This all works by redirecting the render in ModelPart, which is exactly what was overridden by sodium

@jellysquid3
Copy link
Member

Hmmm... we'll have to figure out some kind of interface they can use then. I don't see a way to avoid a conflict here otherwise. Suggestions welcome.

@jellysquid3 jellysquid3 added the F-help-wanted Flag: Help wanted label Mar 18, 2024
@dima-dencep
Copy link

I think @KosmX will comment here or fix this incompatibility herself
Most likely this incompatibility is similar to #1627

@KosmX
Copy link

KosmX commented Mar 19, 2024

I'm very busy (and will be for some time), but I can make a compatibility mode (make my mod use sodium if sodium is loaded, or behave normal otherwise)
I don't (yet) know how can I change sodium to render differently..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-mods Area: Mod compatibility F-help-wanted Flag: Help wanted
Development

No branches or pull requests

4 participants