PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Modifiers to damage of exerted attacks do not modify ailment damage for General's Cry
Nobody has claimed this yet.
- Dominant language
- Lua
- Stars
- 5.4k
- Forks
- 2.4k
- Avg merge
- 1d 12h
- Merged PRs (30d)
- 26
Description
Check version
- I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
- I've checked for duplicate issues by using the search function of the issue tracker
What is the expected behaviour/value?
When allocating passives that increase the damage of exerted attacks, both hit damage and ailment damage of attacks dealt by General's Cry mirages should be increased.
What is the actual behaviour/value?
Ailment damage is not increased for attacks dealt by General's Cry mirages.
This can be worked around by adding another warcry, such as Seismic Cry, and choosing Max Hit calculation mode, however this causes those increases to apply to hit damage twice.
How to reproduce the issue
- Select Tectonic Slam as main skill.
- Link General's Cry
- Equip a weapon that has any chance to crit
- View the tooltip of the Exerted Attack Damage passives, or the Warcry mastery that provides that mod
- Ignite damage is unaffected
- Add Seismic Cry as an additional skill
- Enable Max Hit calculation mode
- View the passive tooltip again
- Ignite damage is increased as much as hit damage was previously
- Hit damage is increased twise as much as before
Character build code
https://pastebin.com/cymZpvNP
Screenshots
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by importing the provided Path of Building character build and reproducing the issue with Tectonic Slam linked to General's Cry. Compare ailment and hit damage with the exerted attack damage passives, then repeat with Seismic Cry and Max Hit calculation mode; done means those modifiers affect ailment damage without applying twice to hit damage.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- lua
- Domain
- game-dev
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100