factorio/flib
1
0
Fork 0
mirror of https://github.com/factoriolib/flib.git synced 2025-09-04 17:00:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix flib_math.sum ignoring the first element (#69)

Browse Source
This commit is contained in:
David Luposchainsky
2024-07-28 05:07:34 +02:00
committed by GitHub
parent b089999856
commit 798436ceed
2 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
math.lua
View File

@@ -131,7 +131,7 @@ end
--- @param set number[]
--- @return number
function flib_math.sum(set)
local sum = set[2] or 0
local sum = set[1] or 0
for i = 2, #set do
sum = sum + set[i]
end

12
tests/test_math.lua
View File

@@ -93,16 +93,16 @@ end
function Test_sum()
Test.assertEquals(math.sum(values1), 100)
Test.assertEquals(math.sum(values2), 185)
Test.assertEquals(math.sum(values3), -5)
Test.assertEquals(math.sum(values4), -117)
Test.assertEquals(math.sum(values2), 170)
Test.assertEquals(math.sum(values3), -20)
Test.assertEquals(math.sum(values4), -128)
end
function Test_mean()
Test.assertEquals(math.mean(values1), 25)
Test.assertEquals(math.mean(values2), 37)
Test.assertEquals(math.mean(values3), -1)
Test.assertEquals(math.mean(values4), -23.4)
Test.assertEquals(math.mean(values2), 34)
Test.assertEquals(math.mean(values3), -4)
Test.assertEquals(math.mean(values4), -25.6)
end
function Test_midrange()