Chailotl / particular

Enhances Minecraft's ambience with hand-crafted visual effects like fireflies, falling leaves, and waterfall cascades
GNU Lesser General Public License v3.0
38 stars 21 forks source link

Modded entities hitting water cause client to crash. #35

Open mutantfrogs opened 4 months ago

mutantfrogs commented 4 months ago

I am playing with Supplementaries (1.20-2.8.17) on a multiplayer server (Fabric 1.20.1), and whenever a bomb hits the water, it causes everyone's client to crash if they have Particular (1.1.1) installed. When particular is disabled, entity hitting doesn't cause a crash.

YnwLNE commented 4 months ago

Can confirm, it's usually larger mobs/animals that trigger the crash.

Topodic commented 4 months ago

Experiencing this with Supplementaries bombs as well. Disabling waterSplash in Particular's config seems to fix it in the meantime without having to disable the mod entirely, thankfully.

Fyoncle commented 4 months ago

any newsss?

ThexXTURBOXx commented 2 months ago

I've got a bug report about this over at https://github.com/ThexXTURBOXx/Balkons-WeaponMod-Legacy/issues/67 and it can be reproduced very easily and consistently. In our case, the culprit is clear: https://github.com/Chailotl/particular/blob/9536d7300fca14290afb88b4b0799527bb7b7291/src/main/java/com/chailotl/particular/mixin/InjectEntity.java#L90

The Collections#max call must have some safeguard against velocities being empty. Otherwise, such crashes can occur pretty easily. So, the real problem is that mods tend to override the tick function - this causes onSetVelocity to never get called and hence velocities remains empty (and thus, a max call crashes).

lihaan commented 1 month ago

Also facing this issue when firing cannonballs into the water (Small Ships mod)

ThexXTURBOXx commented 1 month ago

@lihaan The bugfix is already there - we just need to wait for #41 to get merged. Alternatively, build the mod from source yourself

ThexXTURBOXx commented 1 month ago

I noticed that this project is GPL licensed. So, I can upload my builds without any problems by stating the following:

There is a 1.21 and a 1.20.x build in this zip file: hotfixes.zip Choose whatever one you need!