If the user is running AnomalyDetectionTs() we can assume that they are looking for outliers. Therefore, could a warning be thrown if the user sets a percentage (max_anoms) that results in max_outliers being 0?
On a similar note, I think this degenerate case demonstrates that it is safer to iterate over seq_len(max_outliers) rather than 1:max_outliers.
If the user is running
AnomalyDetectionTs()we can assume that they are looking for outliers. Therefore, could a warning be thrown if the user sets a percentage (max_anoms) that results inmax_outliersbeing 0?On a similar note, I think this degenerate case demonstrates that it is safer to iterate over
seq_len(max_outliers)rather than1:max_outliers.