vortex icon indicating copy to clipboard operation
vortex copied to clipboard

`sum` statistic should not include `NaN`s

Open joseph-isaacs opened this issue 1 month ago • 1 comments

This can be derived from the NaN count.

joseph-isaacs avatar Nov 03 '25 14:11 joseph-isaacs

I think this is clearly fine for the statistic, since we do the same for Min/Max. It's less clear that it's valid for the compute function, which should arguably be IEEE-compliant (and can short-circuit to NaN if nan_count stat > 0)

lwwmanning avatar Nov 03 '25 14:11 lwwmanning