Serene Seasons

Serene Seasons

56M Downloads

Incompatibility with MoreCulling

VoidWalkerAsh-1 opened this issue · 19 comments

commented

What's the issue you encountered?

All I did was have a working modpack and added serene seasons to it with glitchcore as the main dependency, I'm on 1.20.4 with everything up to date, fabric 0.15.6 and the game just crashes when it gets to the mojang screen

Screenshot 2024-02-03 014707
Screenshot 2024-02-03 014720

How can the issue be reproduced?

I really dont know, all I know is when I add the mod it crashes but other mods like Biomes O' Plenty don't

Logs

https://gist.github.com/AshTheNerd/141917a18ccefffa4a03706f0047d79e

Mod Version

SereneSeasons-fabric-1.20.4-9.3.9.18

Additional information

No response

commented

Any updates about this? The issue was fixed by @FxMorin on MoreCulling's side, but needs to be fixed also in SereneSeasons' side.

commented
commented

We could fix this by both changing our redirects to WrapOperation's.
Ill try to find some time to so it.

commented

Just FYI, FxMorin fixed (no partial pun intended) his side of the issue via FxMorin/MoreCulling@eebfcb3.

commented

Serene Seasons doesn't have to do anything. It's already fixed in MoreCulling v0.25.0, and current version of Serene Seasons will work just fine with that.
However MoreCulling v0.25.0 is only available in 1.20.[5-6].
@vico93 I'm asssuming you're playing on 1.20.[1-4], so you may want to open a issue for backport in MoreCulling's side.

commented

Serene Seasons doesn't have to do anything. It's already fixed in MoreCulling v0.25.0, and current version of Serene Seasons will work just fine with that. However MoreCulling v0.25.0 is only available in 1.20.[5-6]. @vico93 I'm asssuming you're playing on 1.20.[1-4], so you may want to open a issue for backport in MoreCulling's side.

I was basically saying what @FxMorin put in the commit:
image
- Fixes crash with Serene Seasons (once they also use WrapOperation)

btw i dont have deep knowledge about how those things work, i only listen the modders 🤷🏻

commented

No worries, I was just letting you know that you can already play with both mods in 1.20.[5-6]. If you want to play on older versions, request a backport of MoreCulling v0.25.0 on their side👍

I think this issue can be closed.

commented

This is most likely a conflict with the mixins from another mod, though from your log it's not clear which one. You'll need to remove mods to figure out which one.

commented

and yes I tried older versions of the 1.20.4 one and the same crash occurs

commented

alright

commented

This is most likely a conflict with the mixins from another mod, though from your log it's not clear which one. You'll need to remove mods to figure out which one.

Its now the only one on in the modpack and its still happening besides cloth config even with cloth config off it still happened

https://gist.github.com/AshTheNerd/2bf174e29531e8945e732aa77f88f5ce

commented

Your log says you still have e.g Iris, Sodium, Wilder Wild and mod menu installed. It is definitely not the only mod

commented

Your log says you still have e.g Iris, Sodium, Wilder Wild and mod menu installed. It is definitely not the only mod

they weren't showing in curse forge, gotta go into the files then and remove them manually

commented

Your log says you still have e.g Iris, Sodium, Wilder Wild and mod menu installed. It is definitely not the only mod

I found the problem. it was apparently the performance mod moreculling-1.20.4-0.22.1

Screenshot 2024-02-03 025259

that's all the mod does so that's weird.

commented

We have a mixin to disable raining based on the season here https://github.com/Glitchfiend/SereneSeasons/blob/1.20.4-multi/common/src/main/java/sereneseasons/mixin/client/MixinLevelRenderer.java#L54. It seems they inject into the exact same place as we do as per https://github.com/FxMorin/MoreCulling/blob/master/src/main/java/ca/fxco/moreculling/mixin/WorldRenderer_rainMixin.java#L60. I'd probably suggest opening an issue on their side as well.

commented

Aight got it, thanks for the help!

commented

So just in case someone else runs into this, I made a modified 24.0 version of More Culling with the changes to the mixin they added to 25.0 so I could run them together on 1.20.4.

If it doesn't work then, sorry. But it works for me.

https://drive.google.com/file/d/1ixFVULTuAigHeYDkGal4dW86iXCnhwn1/view?usp=sharing

commented

Hey, I asked on the More Culling side about the backport, because the incompatibility is still a problem.

Unfortunately, a backport is not planned, so I wanted to ask if the Serene Seasons Devs still plan to fix the incompatibility themselves?

commented

I made a janky compatibility mod for this, if anyone needs it
https://github.com/MUKSC/MoreCulling-SereneSeasons-Compat/releases/tag/v1.0.0