Review comments

Author: robjmcgibbon

velociraptor/autoplotter/__init__.py

@@ -70,10 +70,10 @@
          units: end value units
        # Restriction of binning vertically
        lower:
-         value: lower value for median/mean line
+         value: floor value when computing median/mean line
          units: lower value units
        upper:
-         value: upper value for median/mean line
+         value: ceiling value when computing median/mean line
          units: upper value units
      # Generic metadata for plot
      metadata:

velociraptor/autoplotter/lines.py

@@ -252,6 +252,9 @@ def create_line(
             mask = isnan(x)
             masked_x = masked_x[~mask]
 
+        if (self.lower is not None) and (self.upper is not None):
+            assert self.upper > self.lower
+
         if self.lower is not None:
             self.lower.convert_to_units(y.units)
             mask = masked_y < self.lower