The xAbove functions should not include series that match by =. I.e., the filtering should be by > and not by >=.
This is consistent with Graphite Web. See currentAbove (link) and all similar xAbove functions.
I don't know why the isInclusive bool was there originally. I checked GraphiteWeb and it never does that. I also looked at the original PR here introducing this function, and I see no specific references to this.
The
xAbovefunctions should not include series that match by=. I.e., the filtering should be by>and not by>=.This is consistent with Graphite Web. See
currentAbove(link) and all similarxAbovefunctions.I don't know why the
isInclusivebool was there originally. I checked GraphiteWeb and it never does that. I also looked at the original PR here introducing this function, and I see no specific references to this.