All asserts substituted by ValueError

montefra · montefra · commit 0c1f9f775a2c · 2015-03-04T23:48:30.000+01:00
diff --git a/lib/matplotlib/axis.py b/lib/matplotlib/axis.py
@@ -1728,11 +1728,13 @@ def set_label_position(self, position):
 
         ACCEPTS: [ 'top' | 'bottom' ]
         """
-        assert position == 'top' or position == 'bottom'
         if position == 'top':
             self.label.set_verticalalignment('baseline')
-        else:
+        elif position == 'bottom':
             self.label.set_verticalalignment('top')
+        else:
+            msg = "Position accepts only [ 'top' | 'bottom' ]"
+            raise ValueError(msg)
         self.label_position = position
 
     def _update_label_position(self, bboxes, bboxes2):
@@ -2032,13 +2034,15 @@ def set_label_position(self, position):
 
         ACCEPTS: [ 'left' | 'right' ]
         """
-        assert position == 'left' or position == 'right'
         self.label.set_rotation_mode('anchor')
         self.label.set_horizontalalignment('center')
         if position == 'left':
             self.label.set_verticalalignment('bottom')
-        else:
+        elif position == 'right':
             self.label.set_verticalalignment('top')
+        else:
+            msg = "Position accepts only [ 'left' | 'right' ]"
+            raise ValueError(msg)
         self.label_position = position
 
     def _update_label_position(self, bboxes, bboxes2):
@@ -2083,13 +2087,14 @@ def _update_offset_text_position(self, bboxes, bboxes2):
         )
 
     def set_offset_position(self, position):
-        assert position == 'left' or position == 'right'
-
         x, y = self.offsetText.get_position()
         if position == 'left':
             x = 0
-        else:
+        elif position == 'right':
             x = 1
+        else:
+            msg = "Position accepts only [ 'left' | 'right' ]"
+            raise ValueError(msg)
 
         self.offsetText.set_ha(position)
         self.offsetText.set_position((x, y))
