PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Mana Cost per Second with Max Unleash Seals is incorrect
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 open and closed issues by using the search function of the issue tracker
Check for support
- I've checked that the calculation is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).
What platform are you running Path of Building on?
Windows
What is the value from the calculation in-game?
In-game using Unleash Support, I believe it should be calculated as
Mana Cost * Cost Multiplier * Hit Rate = Mana Cost * Cost Multiplier * (1 / Time to Gain Max Seals)
So the value would be 15.13 Mana/s (with placeholder stats):
- Mana Cost = 23
- Cost Multiplier = 1.4
- Hit Rate = 0.47
- Time to Gain Max Seals = 2.13s
- Cast Rate = 1.5s
23* 1.4 * (1 / 2.13)= 15.13 Mana/s`
What is the value from the calculation in Path of Building?
In PoB using Unleash support, when Hit Rate is based on Time to Gain Max Seals (https://github.com/PathOfBuildingCommunity/PathOfBuilding/issues/9601), PoB calculates it as Mana Cost * Cost Multiplier * Time to Gain Max Seals (calling the time "Full Unleash Speed"). It appears to be using max seal time as the rate instead.
So PoB produces the incorrect value 68.57 Mana/s (with placeholder stats):
- Mana Cost = 23
- Cost Multiplier = 1.4
- Hit Rate = 0.47
- Time to Gain Max Seals = 2.13s
- Cast Rate = 1.5/s
23* 1.4 * 2.13 = 68.57 Mana/s
When instead, Hit Rate is based on Cast Rate (currently when Cast Rate * 1.1 > Time to Gain Max Seals), PoB calculates it as Mana Cost * Cost Multiplier * Cast Rate * 1.1
- Mana Cost = 23
- Cost Multiplier = 1.4
- Hit Rate = 0.47
- Time to Gain Max Seals = 2.13s
- Cast Rate = 3/s
23* 1.4 * 3.3 = 106.26 Mana/s
How to reproduce the issue
- Select Arc as the main skill
- Add Unleash Support to Arc
- Tick on Do you wait for Max Unleash Seals?
- See incorrect first Mana Cost per Second
- Enter into custom modifiers
100% increased cast speed - See incorrect second Mana Cost per Second
PoB for PoE1 build code
Screenshots
When Time to Gain Max Seals is greater than 110% of Cast Rate:
When 110% of Cast Rate is greater than Time to Gain Max Seals:
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
Reproduce the issue with Arc and Unleash Support using the listed steps and the linked PoE1 build code. Start from the Unleash mana-cost-per-second calculation, checking both the max-seal-time and cast-rate branches. Done means both branches use the reported hit-rate behavior and match the expected Mana/s values.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- lua
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100