[BUG] Intervals.ofSeq does not work if nan is in sequence about fsharp.stats

[bvenn]

This is the problem:

min nan 1  //results in nan
max nan 1  //results in nan

1. < nan //false
1. > nan //false

I think the best option is to return a empty Interval if the collection contains a nan. This leads to an additional check for each element if it is nan, but I think this is a necessary step.

[bvenn]

After reevaluation we came to the conclusion that a nan in a collection should throw an exception, so the user has full control and notices that something is odd in the data.

The function should be updated to allow overloads with arguments that define what happens to nan values.

[bvenn]

Intervals.ofSeqBy snd (List.indexed [5;5;5;5])

results in ClosedInterval ((0, 5), (0, 5)). While not wrong by default, it would be beneficial if the Intergal maximum is the last occurrence within the collection. It can by fixed by modification of

FSharp.Stats/src/FSharp.Stats/Intervals.fs

Line 40 in 262f1ac

let mmax,mmaxV = if current > maximum then current,e.Current else maximum,maximumV

to

let mmax,mmaxV = if current >= maximum then current,e.Current else maximum,maximumV