Skip to content

[pob1-port] Fix skill Area of Effect rounding and minion skill AoE values #2479

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
LocalIdentity merged 2 commits into from
Aug 30, 2026
+50 −26
Merged

[pob1-port] Fix skill Area of Effect rounding and minion skill AoE values #2479

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0d02fdb
Apply changes from https://github.com/PathOfBuildingCommunity/PathOfB…
LocalIdentity Aug 30, 2026
076bccb
Fix port
Aug 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/Data/SkillStatMap.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -654,7 +654,7 @@ return function(mod, flag, skill)
mod("AreaOfEffect", "MORE", nil, 0, 0, { type = "Condition", var = "CastOnFrostbolt" }),
},
["active_skill_area_of_effect_radius_+%_final"] = {
mod("AreaOfEffect", "MORE", nil),
mod("AreaOfEffectRadius", "MORE", nil),
},
["active_skill_area_of_effect_+%_final"] = {
mod("AreaOfEffect", "MORE", nil),
Expand Down
5 changes: 4 additions & 1 deletion src/Modules/CalcBreakdown.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,12 +95,15 @@ return function(modDB, output, actor)
})
end

function breakdown.area(base, areaMod, total, incBreakpoint, moreBreakpoint, redBreakpoint, lessBreakpoint, label)
function breakdown.area(base, areaMod, total, incBreakpoint, moreBreakpoint, redBreakpoint, lessBreakpoint, label, radiusMod)
local out = {}
t_insert(out, label)
if base ~= total then
t_insert(out, s_format("%.1fm ^8(base radius)", base / 10))
t_insert(out, s_format("x %.2f ^8(square root of area of effect modifier)", m_floor(100 * m_sqrt(areaMod)) / 100))
if radiusMod and radiusMod ~= 1 then
t_insert(out, s_format("x %.2f ^8(final radius modifier)", radiusMod))
end
t_insert(out, s_format("= %.1fm", total / 10))
end
if incBreakpoint and moreBreakpoint and redBreakpoint and lessBreakpoint then
Expand Down
69 changes: 45 additions & 24 deletions src/Modules/CalcOffence.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -156,12 +156,28 @@ local function calcDamage(activeSkill, output, cfg, breakdown, damageType, typeF
round(summedMax * inc * more * moreMaxDamage + addMax)
end

---Calculates the area percentage after the client integer rounding stages.
---@param incArea number @Additive modifier
---@param moreArea number @Multiplicative modifier
---@return number
local function calcAreaMod(incArea, moreArea)
local baseAreaPercent = roundSymmetric(incArea * 100)
local finalAreaPercent = roundSymmetric(m_max(0, moreArea) * 100)
local areaPercent = baseAreaPercent * finalAreaPercent / 100
areaPercent = areaPercent >= 0 and m_floor(areaPercent) or m_ceil(areaPercent)
return m_max(0, areaPercent) / 100
end

---Calculates skill radius
---@param baseRadius number
---@param areaMod number
---@param radiusMod number|nil
---@return number
local function calcRadius(baseRadius, areaMod)
return m_floor(baseRadius * m_floor(100 * m_sqrt(areaMod)) / 100)
local function calcRadius(baseRadius, areaMod, radiusMod)
local radiusPercent = m_floor(100 * m_sqrt(areaMod))
-- Final radius modifiers apply after Area of Effect has been converted to radius.
radiusPercent = m_floor(radiusPercent * (radiusMod or 1))
return m_floor(baseRadius * radiusPercent / 100)
end

---Calculates the tertiary radius for Molten Strike, correctly handling the deadzone.
Expand All @@ -180,30 +196,31 @@ end
---@param baseRadius number
---@param incArea number @Additive modifier
---@param moreArea number @Multiplicative modifier
---@param radiusMod number|nil @Final radius modifier
---@return number, number, number, number @Next breakpoint: increased, more; Previous breakpoint: reduced, less
local function calcRadiusBreakpoints(baseRadius, incArea, moreArea)
local radius = calcRadius(baseRadius, round(round(incArea * moreArea, 10), 2))
local function calcRadiusBreakpoints(baseRadius, incArea, moreArea, radiusMod)
local radius = calcRadius(baseRadius, calcAreaMod(incArea, moreArea), radiusMod)
local incAreaBreakpoint, redAreaBreakpoint, moreAreaBreakpoint, lessAreaBreakpoint
if radius > 0 then
incAreaBreakpoint = 0
repeat
incAreaBreakpoint = incAreaBreakpoint + 1
local newRadius = calcRadius(baseRadius, round(round((incArea + incAreaBreakpoint / 100) * moreArea, 10), 2))
local newRadius = calcRadius(baseRadius, calcAreaMod(incArea + incAreaBreakpoint / 100, moreArea), radiusMod)
until (newRadius > radius)
redAreaBreakpoint = 0
repeat
redAreaBreakpoint = redAreaBreakpoint + 1
local newRadius = calcRadius(baseRadius, round(round((incArea - redAreaBreakpoint / 100) * moreArea, 10), 2))
local newRadius = calcRadius(baseRadius, calcAreaMod(incArea - redAreaBreakpoint / 100, moreArea), radiusMod)
until (newRadius < radius)
moreAreaBreakpoint = 0
repeat
moreAreaBreakpoint = moreAreaBreakpoint + 1
local newRadius = calcRadius(baseRadius, round(round(incArea * moreArea * (1 + moreAreaBreakpoint / 100), 10), 2))
local newRadius = calcRadius(baseRadius, calcAreaMod(incArea, moreArea * (1 + moreAreaBreakpoint / 100)), radiusMod)
until (newRadius > radius)
lessAreaBreakpoint = 0
repeat
lessAreaBreakpoint = lessAreaBreakpoint + 1
local newRadius = calcRadius(baseRadius, round(round(incArea * moreArea * (1 - lessAreaBreakpoint / 100), 10), 2))
local newRadius = calcRadius(baseRadius, calcAreaMod(incArea, moreArea * (1 - lessAreaBreakpoint / 100)), radiusMod)
until (newRadius < radius)
end
return incAreaBreakpoint, moreAreaBreakpoint, redAreaBreakpoint, lessAreaBreakpoint
Expand Down Expand Up @@ -425,7 +442,8 @@ function calcs.offence(env, actor, activeSkill)

local function calcAreaOfEffect(skillModList, skillCfg, skillData, skillFlags, output, breakdown)
local incArea, moreArea = calcLib.mods(skillModList, skillCfg, "AreaOfEffect", "AreaOfEffectPrimary")
output.AreaOfEffectMod = round(round(incArea * moreArea, 10), 2)
output.AreaOfEffectMod = calcAreaMod(incArea, moreArea)
local radiusMod = calcLib.mod(skillModList, skillCfg, "AreaOfEffectRadius")
if skillData.radiusIsWeaponRange then
local range = 0
if skillFlags.weapon1Attack then
Expand All @@ -439,29 +457,29 @@ function calcs.offence(env, actor, activeSkill)
if skillData.radius then
skillFlags.area = true
local baseRadius = skillData.radius + (skillData.radiusExtra or 0) + skillModList:Sum("BASE", skillCfg, "AreaOfEffect")
output.AreaOfEffectRadius = calcRadius(baseRadius, output.AreaOfEffectMod)
output.AreaOfEffectRadius = calcRadius(baseRadius, output.AreaOfEffectMod, radiusMod)
output.AreaOfEffectRadiusMetres = output.AreaOfEffectRadius / 10
if breakdown then
local incAreaBreakpoint, moreAreaBreakpoint, redAreaBreakpoint, lessAreaBreakpoint = calcRadiusBreakpoints(baseRadius, incArea, moreArea)
breakdown.AreaOfEffectRadius = breakdown.area(baseRadius, output.AreaOfEffectMod, output.AreaOfEffectRadius, incAreaBreakpoint, moreAreaBreakpoint, redAreaBreakpoint, lessAreaBreakpoint, skillData.radiusLabel)
local incAreaBreakpoint, moreAreaBreakpoint, redAreaBreakpoint, lessAreaBreakpoint = calcRadiusBreakpoints(baseRadius, incArea, moreArea, radiusMod)
breakdown.AreaOfEffectRadius = breakdown.area(baseRadius, output.AreaOfEffectMod, output.AreaOfEffectRadius, incAreaBreakpoint, moreAreaBreakpoint, redAreaBreakpoint, lessAreaBreakpoint, skillData.radiusLabel, radiusMod)
end
if skillData.radiusSecondary then
local incAreaSecondary, moreAreaSecondary = calcLib.mods(skillModList, skillCfg, "AreaOfEffect", "AreaOfEffectSecondary")
output.AreaOfEffectModSecondary = round(round(incAreaSecondary * moreAreaSecondary, 10), 2)
output.AreaOfEffectModSecondary = calcAreaMod(incAreaSecondary, moreAreaSecondary)
baseRadius = skillData.radiusSecondary + (skillData.radiusExtra or 0)
output.AreaOfEffectRadiusSecondary = calcRadius(baseRadius, output.AreaOfEffectModSecondary)
output.AreaOfEffectRadiusSecondary = calcRadius(baseRadius, output.AreaOfEffectModSecondary, radiusMod)
output.AreaOfEffectRadiusSecondaryMetres = output.AreaOfEffectRadiusSecondary / 10
if breakdown then
local incAreaBreakpointSecondary, moreAreaBreakpointSecondary, redAreaBreakpointSecondary, lessAreaBreakpointSecondary
if not skillData.projectileSpeedAppliesToMSAreaOfEffect then
incAreaBreakpointSecondary, moreAreaBreakpointSecondary, redAreaBreakpointSecondary, lessAreaBreakpointSecondary = calcRadiusBreakpoints(baseRadius, incAreaSecondary, moreAreaSecondary)
incAreaBreakpointSecondary, moreAreaBreakpointSecondary, redAreaBreakpointSecondary, lessAreaBreakpointSecondary = calcRadiusBreakpoints(baseRadius, incAreaSecondary, moreAreaSecondary, radiusMod)
end
breakdown.AreaOfEffectRadiusSecondary = breakdown.area(baseRadius, output.AreaOfEffectModSecondary, output.AreaOfEffectRadiusSecondary, incAreaBreakpointSecondary, moreAreaBreakpointSecondary, redAreaBreakpointSecondary, lessAreaBreakpointSecondary, skillData.radiusSecondaryLabel)
breakdown.AreaOfEffectRadiusSecondary = breakdown.area(baseRadius, output.AreaOfEffectModSecondary, output.AreaOfEffectRadiusSecondary, incAreaBreakpointSecondary, moreAreaBreakpointSecondary, redAreaBreakpointSecondary, lessAreaBreakpointSecondary, skillData.radiusSecondaryLabel, radiusMod)
end
end
if skillData.radiusTertiary then
local incAreaTertiary, moreAreaTertiary = calcLib.mods(skillModList, skillCfg, "AreaOfEffect", "AreaOfEffectTertiary")
output.AreaOfEffectModTertiary = round(round(incAreaTertiary * moreAreaTertiary, 10), 2)
output.AreaOfEffectModTertiary = calcAreaMod(incAreaTertiary, moreAreaTertiary)
baseRadius = skillData.radiusTertiary + (skillData.radiusExtra or 0)
if skillData.projectileSpeedAppliesToMSAreaOfEffect then
local incSpeedTertiary, moreSpeedTertiary = calcLib.mods(skillModList, skillCfg, "ProjectileSpeed")
Expand All @@ -487,7 +505,7 @@ function calcs.offence(env, actor, activeSkill)
local radiusForBaseRadius = {}
local radiiOccurrences = {}
for adjustedBaseRadius, occurrenceCount in pairs(baseRadiiOccurrences) do
local radiusForDeviation = calcRadius(adjustedBaseRadius, output.AreaOfEffectModTertiary)
local radiusForDeviation = calcRadius(adjustedBaseRadius, output.AreaOfEffectModTertiary, radiusMod)
radiusForBaseRadius[adjustedBaseRadius] = radiusForDeviation
sumOfRandomRadii = sumOfRandomRadii + radiusForDeviation * occurrenceCount
radiiOccurrences[radiusForDeviation] = (radiiOccurrences[radiusForDeviation] or 0) + occurrenceCount
Expand All @@ -499,14 +517,17 @@ function calcs.offence(env, actor, activeSkill)
local incAreaBreakpointTertiary, moreAreaBreakpointTertiary, redAreaBreakpointTertiary, lessAreaBreakpointTertiary = m_huge, m_huge, m_huge, m_huge
t_insert(out, skillData.radiusTertiaryLabel)
t_insert(out, s_format("R ^8(base radius)^7 x %.2f ^8(square root of area of effect modifier)", m_floor(100 * m_sqrt(output.AreaOfEffectModTertiary)) / 100))
if radiusMod ~= 1 then
t_insert(out, s_format("x %.2f ^8(final radius modifier)", radiusMod))
end
local baseRadii = {}
for adjustedBaseRadius in pairs(baseRadiiOccurrences) do
t_insert(baseRadii, adjustedBaseRadius)
end
table.sort(baseRadii, function(a,b) return a < b end)
for _, adjustedBaseRadius in ipairs(baseRadii) do
t_insert(out, s_format("%.1f%% ^8chance of^7 %.1fm ^8base radius resulting in^7 %.1fm ^8final radius", baseRadiiOccurrences[adjustedBaseRadius] / marginWidth * 100, adjustedBaseRadius / 10, radiusForBaseRadius[adjustedBaseRadius] / 10))
local incAreaBreakpointTertiaryIntermediate, moreAreaBreakpointTertiaryIntermediate, redAreaBreakpointTertiaryIntermediate, lessAreaBreakpointTertiaryIntermediate = calcRadiusBreakpoints(adjustedBaseRadius, incAreaTertiary, moreAreaTertiary)
local incAreaBreakpointTertiaryIntermediate, moreAreaBreakpointTertiaryIntermediate, redAreaBreakpointTertiaryIntermediate, lessAreaBreakpointTertiaryIntermediate = calcRadiusBreakpoints(adjustedBaseRadius, incAreaTertiary, moreAreaTertiary, radiusMod)
incAreaBreakpointTertiary = m_min(incAreaBreakpointTertiary, incAreaBreakpointTertiaryIntermediate)
moreAreaBreakpointTertiary = m_min(moreAreaBreakpointTertiary, moreAreaBreakpointTertiaryIntermediate)
redAreaBreakpointTertiary = m_min(redAreaBreakpointTertiary, redAreaBreakpointTertiaryIntermediate)
Expand All @@ -518,10 +539,10 @@ function calcs.offence(env, actor, activeSkill)
breakdown.AreaOfEffectRadiusTertiary = out
end
else
output.AreaOfEffectRadiusTertiary = calcRadius(baseRadius, output.AreaOfEffectModTertiary)
output.AreaOfEffectRadiusTertiary = calcRadius(baseRadius, output.AreaOfEffectModTertiary, radiusMod)
if breakdown then
local incAreaBreakpointTertiary, moreAreaBreakpointTertiary, redAreaBreakpointTertiary, lessAreaBreakpointTertiary = calcRadiusBreakpoints(baseRadius, incAreaTertiary, moreAreaTertiary)
breakdown.AreaOfEffectRadiusTertiary = breakdown.area(baseRadius, output.AreaOfEffectModTertiary, output.AreaOfEffectRadiusTertiary, incAreaBreakpointTertiary, moreAreaBreakpointTertiary, redAreaBreakpointTertiary, lessAreaBreakpointTertiary, skillData.radiusTertiaryLabel)
local incAreaBreakpointTertiary, moreAreaBreakpointTertiary, redAreaBreakpointTertiary, lessAreaBreakpointTertiary = calcRadiusBreakpoints(baseRadius, incAreaTertiary, moreAreaTertiary, radiusMod)
breakdown.AreaOfEffectRadiusTertiary = breakdown.area(baseRadius, output.AreaOfEffectModTertiary, output.AreaOfEffectRadiusTertiary, incAreaBreakpointTertiary, moreAreaBreakpointTertiary, redAreaBreakpointTertiary, lessAreaBreakpointTertiary, skillData.radiusTertiaryLabel, radiusMod)
end
end
output.AreaOfEffectRadiusTertiaryMetres = output.AreaOfEffectRadiusTertiary / 10
Expand Down Expand Up @@ -1652,7 +1673,7 @@ function calcs.offence(env, actor, activeSkill)
end
end
local incArea, moreArea = calcLib.mods(skillModList, skillCfg, "TrapTriggerAreaOfEffect")
local areaMod = round(round(incArea * moreArea, 10), 2)
local areaMod = calcAreaMod(incArea, moreArea)
output.TrapTriggerRadius = calcRadius(data.misc.TrapTriggerRadiusBase, areaMod)
output.TrapTriggerRadiusMetre = output.TrapTriggerRadius / 10
if breakdown then
Expand Down Expand Up @@ -1753,7 +1774,7 @@ function calcs.offence(env, actor, activeSkill)
end

local incArea, moreArea = calcLib.mods(skillModList, skillCfg, "MineDetonationAreaOfEffect")
local areaMod = round(round(incArea * moreArea, 10), 2)
local areaMod = calcAreaMod(incArea, moreArea)
output.MineDetonationRadius = calcRadius(data.misc.MineDetonationRadiusBase, areaMod)
output.MineDetonationRadiusMetre = output.MineDetonationRadius / 10
if breakdown then
Expand Down
Loading