anode icon indicating copy to clipboard operation
anode copied to clipboard

mark anomalous metrics as such

Open zmarcantel opened this issue 11 years ago • 1 comments

I added a bool field to the data.Datapoint struct that will say "yes, I am an anomaly".

This helps applications parse the output channel without creating a second channel that could potentially block if not properly/diligently read from. It also avoids a less-than-ideal strings.HasSuffix("anomalous") check.

I'm open to discussing other methods, but this seemed the easiest.

zmarcantel avatar Jul 21 '14 15:07 zmarcantel

Thanks for this. At the moment the three sigma analyzer has a separate output channel for anomalous values; is this insufficient for the use case you're thinking of?

I'm not opposed to merging this, just want to understand the use case better.

mattrco avatar Jul 22 '14 11:07 mattrco