Bawnorton / BetterTrims

Add new armour trims and give each trim a unique feature.
12 stars 3 forks source link

Copper Trim Buff Completely OP Stacking incorrectly? #36

Closed alexweber closed 6 months ago

alexweber commented 7 months ago

Expected Behaviour The Copper Trim effect is supposed to give a small configurable speed to swim speed

Actual Behaviour Regardless of whether the copper trim buff is enabled or not or what the % is configured to be, when paired with Depth Strider (and also with a redstone trim - probably irrelevant) the boost is completely unreasonable and can't be reduced. Even disabling copper entirely won't make it go away. The only way to not swim at the speed of light is to trim with another material instead.

Now that I think about it maybe the bug has to do with once applied a trim is not respecting updating config values?

Either way, unfortunately it's just too fast and not fair/playable in the current state...

Steps to Reproduce

What version of the mod are you using 2.2.3

Thanks!

Bawnorton commented 7 months ago

Are you playing with any other mods? I can't replicate this behaviour

alexweber commented 7 months ago

Plenty of other mods but none that should affect this, I'll test some more and let you know if I can replicate it in a clean sandbox, thansk for the reply!

NolanHewitt commented 7 months ago

I can replicate this. Movement speed with 4 pieces of copper trim armor makes me move at the speed of light.

https://github.com/Bawnorton/BetterTrims/assets/52480898/e4b55b72-f8b5-4b46-8ee5-badaa7510d5e

NolanHewitt commented 7 months ago

I think I found a fix, in the config set "copper_swim_speed_increase": 0.005, instead of the default of 0.05. Feels much more like a 5% increase per piece with this.

alexweber commented 7 months ago

@NolanHewitt Nice! Doing that definitely worked for me on singleplayer.

Bawnorton commented 6 months ago

Default value adjusted in next release