Adding Enspell Damage Percent Multiplier Effect#9503
Adding Enspell Damage Percent Multiplier Effect#9503lefneer311 wants to merge 5 commits intoLandSandBoat:basefrom
Conversation
Adds the math for enspell damage percent multipliers and adds Composure calculation
|
✨ Hello and thanks for the PR! ✨ 🤖: This is a friendly automated reminder that the maintainers won't look at your PR until you've properly completed all of the checkboxes in the pre-filled template. |
Line 814 had a double space after the comma before the comment.
| mult += (float)weaponPct / 100.0f; | ||
|
|
||
| // 4) Apply multiplier exactly once | ||
| damage = (int32)std::floor(damage * mult); |
There was a problem hiding this comment.
Nit, but we don't like c-style casts on new code
| mult += (float)weaponPct / 100.0f; | |
| // 4) Apply multiplier exactly once | |
| damage = (int32)std::floor(damage * mult); | |
| mult += static_cast<float>(weaponPct) / 100.0f; | |
| // 4) Apply multiplier exactly once | |
| damage = std::floor<int32>(static_cast<float>(damage) * mult); |
There was a problem hiding this comment.
Understood! I believe I understand the intent behind the suggested change.
What about this? Is this more closely aligned with the standard?
| mult += (float)weaponPct / 100.0f; | |
| // 4) Apply multiplier exactly once | |
| damage = (int32)std::floor(damage * mult); | |
| mult += static_cast<float>(weaponPct) / 100.0f; | |
| // 4) Apply multiplier exactly once | |
| damage = static_cast<int32>(std::floor(static_cast<float>(damage) * mult)); |
Thank you again for your help.
|
Other than that, looks fine but... what in retail uses that augment? Mostly curious (but I would like to verify the mod anyway) |
Thank you for your feedback! It's a rarely utilized modifier that's only present on a handful of items. Enspell damage builds aren't exactly meta, but they can be fun. http://ffxidb.com/items/20614 - augmented rank 15 has 150% With the current configuration, it's difficult to replicate the high percentage effects. The Mod (899) without a multiplier would allow for +~32%. |
I affirm:
What does this pull request do?
Resolves bug #4518
This adds enspell damage % multiplier calculations, including the base effect from RDM JA Composure to battleutils.cpp, wires up the augment for enspell damage %+, and adds the modifier entries to mod.lua and modifier.h.
Steps to test these changes
Switch to RDM main (level 50+), use an enspell, engage a target and note the additional effect damage
Activate composure, engage a target and note the additional effect damage is 3x (+200%)
Use !additem with the augment ID 899 for +enspell dmg % on items and equip them
Note the additional effect damage multiplier scale