Carry On

Carry On

112M Downloads

*SlownessMultiplier seems to have little to no effect

Jomik opened this issue ยท 4 comments

commented

Expected Behavior

Slows down considerably.

Actual Behavior

No noticable slow down

Steps to Reproduce

I tried to set the SlownessMultiplier to both 0.1 and 1000, as there is no documentation on it.
But neither seem to have any considerable effect on the player's movespeed.

Version of Minecraft, Carry On, Forge

MC 1.12.2
CarryOn MC1.12 v1.7
1.12.2-forge1.12.2-14.23.0.2545

commented

return (int) (i * CarryOnConfig.settings.blockSlownessMultiplier);
The potionLevel is multiplied with the multiplier, meaning if the multiplier is 0 (as it is with empty chests for example), the multiplier get multiplied with 0, resulting in, of course, 0, so not changing the effect. Try it with a somewhat filled chest!

commented

Oh, you're right. Works!

So I guess my concern is more with

int i = (int)(e.height * e.width);

I'd think you should either call ceil or round on this. As a pig or chicken, etc., is now "free" to move, since the height * width converted to an integer is 0.
And the same for the return statements in both your methods, rather than just discarding the float, why not use round it? :)

commented

Remember, amplifier 0 means Slowness I , so it's not really "free"

commented

Hardly noticeable though. I'd prefer being able to set a higher base in that case. Hmm. But then again, it gets super slow with a full chest ๐Ÿ‘