Update ailment hit calc to account for lucky dmg

[pauloday]

Fixes # .

Description of the problem being solved:

Steps taken to verify a working solution:

Link to a build that showcases this PR:

Before screenshot:

After screenshot:

[deathbeam]

Just quick note, I originally implemented like this but the issue is that the average needs to account for ailment roll average (which is calculated based on stack potential/overstacking), which means the average needs to be calculated in place and not from stored value so it was changed to what it does now (at the cost of not reimplementing the lucky logic), see: https://github.com/PathOfBuildingCommunity/PathOfBuilding-PoE2/pull/143/files#diff-9ea806a499fadd026d5c9d6bcdb3e40ebd379ae78bdf8b997a74dec5ca28aeb0R4116

[pauloday]

In retrospect it seemed a little too easy when I was working on it, I was hoping someone just didn't get around to hooking stuff up lol.

The average roll calculation is already similar to a lucky hit calculation. So you would just need some modification for the lucky case, but I'm not sure what it would be. I did some modeling and I'm pretty confident it'd just be some changes to the consts or maybe new coefficients in the current ailmentRollAverage calculation. I think the main problem is that by time the average roll calc is done all damage types have been combined in calcAilmentSourceDamage. But that fn doesn't have info about whether the hit is lucky either. So I think a solution would be something like:

1. Adding output[damageType .. "LuckyChance"] around line 3312
2. Passing the max and average stacks into calcAilmentSourceDamage
3. Calculating and returning the average roll from there