mcMMO-Dev / mcMMO

The RPG Lover's Mod!
https://mcmmo.org
GNU General Public License v3.0
1.01k stars 862 forks source link

Installing mcMMO dramatically reduces fall damage taken by players #5023

Closed TheBentoBox closed 1 month ago

TheBentoBox commented 4 months ago

I've long felt fall damage on my server was lower than I'd expect, and after some comparisons to vanilla and some plugin searching, I've found mcMMO seems to be the culprit.

To start, see the damage table here. With Feather Falling IV and at least two Protection IV armor pieces, players should die from any fall 103 blocks or greater.

The two tests below were done with full Protection IV armor with Feather Falling IV boots, falling from 102 blocks, on a Purpur 1.20.6 server, with no permissions plugin and on a non-op account.

No plugins: image

With only mcMMO installed: image

Here are my versions (latest mcMMO, recent Purpur): image

In my testing, with the same gear I need to fall 507 blocks or greater for mcMMO to kill me, so the differential seems to to be almost exactly 5x less fall damage with mcMMO installed.

Let me know if I'm missing something. If not, I'd assume the culprit to be some part of the acrobatics roll code. Thanks.

TheBentoBox commented 4 months ago

Also for clarity: in the mcMMO test, I did not proc Roll or Graceful Roll, and I started off as level 0 in Acrobatics.

TheBentoBox commented 4 months ago

I would like to bump this bug as IMO it is fairly serious and needs looking at soon.

nossr50 commented 4 months ago

I would like to bump this bug as IMO it is fairly serious and needs looking at soon.

I'll prioritize looking into it once I do a bugfix (which will most certainly be the next patch I work on)

TheBentoBox commented 4 months ago

Appreciate it bud, thanks for your hard work

nossr50 commented 3 months ago

I've looked into this, and it seems to be intended behaviour, intended behaviour I forgot about. If you are rolling, you are removing 7 damage from the final damage calculated for fall damage, and if you are graceful rolling, this is removing 14.

If you fell from 103 blocks, using the damage table you provided, and assuming you did a graceful roll and had FF IV and Protection II, you would only take 6 damage afterwards because of the amount of damage mcMMO is removing as part of the damage removal from roll.

You can change this behaviour with the config option Skills.Acrobatics.Roll.DamageThreshold which governs how much damage a successful Roll will remove (remember, it is doubled for Graceful Roll).

Now mcMMO could change how it is doing things, for example, it could remove less damage if Feather Fall is involved to make it so players can't just jump off a cliff and survive, and arguably, maybe it should.

I'm going to classify this as a feature request since it's more of a balance change than a bug. Since it's not a bug, it's not as high priority to fix, but I will think about solutions for this problem and see if I can rebalance Roll for enchantments in the near future.

TheBentoBox commented 3 months ago

Sorry, but I believe you are misunderstanding -- I am not rolling or graceful rolling in these test scenarios. See my first reply:

Also for clarity: in the mcMMO test, I did not proc Roll or Graceful Roll, and I started off as level 0 in Acrobatics.

mcMMO is seemingly mutating the base fall damage even when a roll does not proc. I believe this is indeed a bug. Please test it yourself and see if it makes sense to you.

nossr50 commented 3 months ago

Sorry, I thought I replied to this but it looks like I did not. Sorry for misunderstanding your original report, I will look into it mcMMO is indeed mutating the fall damage, I suspect it is not but I will confirm.

TheBentoBox commented 2 months ago

Any update on this?

I want to reiterate my testing scenario. Both pictures are on a freshly installed 1.21.1 Purpur server and represent a 102-block fall; the maximum safe fall with this setup should be 103 blocks:

image

image

I'm trying to be as clear and fair in my testing as I can but I cannot see anything but mcMMO as the culprit and this seems like a serious game balance issue to me.

nossr50 commented 2 months ago

My apologies, this fell by the wayside, I'll reprioritize it and look into it soon.