Contour levels must be increasing

ianthomas23 · ianthomas23 · commit e162e884e8c5 · 2015-11-16T12:30:59.000Z
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,3 +1,6 @@
+2015-11-16 Levels passed to contour(f) and tricontour(f) must be in increasing
+           order.
+
 2015-10-21 Added get_ticks_direction()
 
 2015-02-27 Added the rcParam 'image.composite_image' to permit users 
diff --git a/lib/matplotlib/contour.py b/lib/matplotlib/contour.py
@@ -1191,6 +1191,8 @@ def _contour_level_args(self, z, args):
             self.levels = lev
         if self.filled and len(self.levels) < 2:
             raise ValueError("Filled contours require at least 2 levels.")
+        if len(self.levels) > 1 and np.amin(np.diff(self.levels)) <= 0.0:
+            raise ValueError("Contour levels must be increasing")
 
     def _process_levels(self):
         """
@@ -1674,13 +1676,15 @@ def _initialize_x_y(self, z):
           contour(Z,V)
           contour(X,Y,Z,V)
 
-        draw contour lines at the values specified in sequence *V*
+        draw contour lines at the values specified in sequence *V*,
+        which must be in increasing order.
 
         ::
 
           contourf(..., V)
 
-        fill the ``len(V)-1`` regions between the values in *V*
+        fill the ``len(V)-1`` regions between the values in *V*,
+        which must be in increasing order.
 
         ::
 
@@ -1743,8 +1747,8 @@ def _initialize_x_y(self, z):
 
           *levels*: [level0, level1, ..., leveln]
             A list of floating point numbers indicating the level
-            curves to draw; e.g., to draw just the zero contour pass
-            ``levels=[0]``
+            curves to draw, in increasing order; e.g., to draw just
+            the zero contour pass ``levels=[0]``
 
           *origin*: [ *None* | 'upper' | 'lower' | 'image' ]
             If *None*, the first value of *Z* will correspond to the
diff --git a/lib/matplotlib/tests/test_contour.py b/lib/matplotlib/tests/test_contour.py
@@ -9,6 +9,7 @@
 from matplotlib import mlab
 from matplotlib.testing.decorators import cleanup, image_comparison
 from matplotlib import pyplot as plt
+from nose.tools import assert_raises
 
 import re
 
@@ -264,6 +265,14 @@ def test_corner_mask():
         plt.contourf(z, corner_mask=corner_mask)
 
 
+@cleanup
+def test_contourf_decreasing_levels():
+    # github issue 5477.
+    z = [[0.1, 0.3], [0.5, 0.7]]
+    plt.figure()
+    assert_raises(ValueError, plt.contourf, z, [1.0, 0.0])
+
+
 if __name__ == '__main__':
     import nose
     nose.runmodule(argv=['-s', '--with-doctest'], exit=False)
diff --git a/lib/matplotlib/tests/test_triangulation.py b/lib/matplotlib/tests/test_triangulation.py
@@ -10,7 +10,7 @@
 from numpy.testing import assert_array_equal, assert_array_almost_equal,\
     assert_array_less
 import numpy.ma.testutils as matest
-from matplotlib.testing.decorators import image_comparison
+from matplotlib.testing.decorators import cleanup, image_comparison
 import matplotlib.cm as cm
 from matplotlib.path import Path
 
@@ -1021,6 +1021,16 @@ def test_trianalyzer_mismatched_indices():
     triang2 = analyser._get_compressed_triangulation()
 
 
+@cleanup
+def test_tricontourf_decreasing_levels():
+    # github issue 5477.
+    x = [0.0, 1.0, 1.0]
+    y = [0.0, 0.0, 1.0]
+    z = [0.2, 0.4, 0.6]
+    plt.figure()
+    assert_raises(ValueError, plt.tricontourf, x, y, z, [1.0, 0.0])
+
+
 if __name__ == '__main__':
     import nose
     nose.runmodule(argv=['-s', '--with-doctest'], exit=False)
diff --git a/lib/matplotlib/tri/_tri_wrapper.cpp b/lib/matplotlib/tri/_tri_wrapper.cpp
@@ -295,6 +295,12 @@ static PyObject* PyTriContourGenerator_create_filled_contour(PyTriContourGenerat
         return NULL;
     }
 
+    if (lower_level >= upper_level)
+    {
+        PyErr_SetString(PyExc_ValueError,
+            "filled contour levels must be increasing");
+    }
+
     PyObject* result;
     CALL_CPP("create_filled_contour",
              (result = self->ptr->create_filled_contour(lower_level,
diff --git a/lib/matplotlib/tri/tricontour.py b/lib/matplotlib/tri/tricontour.py
@@ -141,13 +141,15 @@ def _contour_args(self, args, kwargs):
 
           tricontour(..., Z, V)
 
-        draw contour lines at the values specified in sequence *V*
+        draw contour lines at the values specified in sequence *V*,
+        which must be in increasing order.
 
         ::
 
           tricontourf(..., Z, V)
 
-        fill the (len(*V*)-1) regions between the values in *V*
+        fill the (len(*V*)-1) regions between the values in *V*,
+        which must be in increasing order.
 
         ::
 
@@ -186,8 +188,8 @@ def _contour_args(self, args, kwargs):
 
           *levels* [level0, level1, ..., leveln]
             A list of floating point numbers indicating the level
-            curves to draw; e.g., to draw just the zero contour pass
-            ``levels=[0]``
+            curves to draw, in increasing order; e.g., to draw just
+            the zero contour pass ``levels=[0]``
 
           *origin*: [ *None* | 'upper' | 'lower' | 'image' ]
             If *None*, the first value of *Z* will correspond to the
diff --git a/src/_contour_wrapper.cpp b/src/_contour_wrapper.cpp
@@ -97,6 +97,12 @@ static PyObject* PyQuadContourGenerator_create_filled_contour(PyQuadContourGener
         return NULL;
     }
 
+    if (lower_level >= upper_level)
+    {
+        PyErr_SetString(PyExc_ValueError,
+            "filled contour levels must be increasing");
+    }
+
     PyObject* result;
     CALL_CPP("create_filled_contour",
              (result = self->ptr->create_filled_contour(lower_level,
