Add parameter checks to DayLocator initiator

LindyBalboa · LindyBalboa · commit 7f8c29572dfe · 2016-08-21T11:32:19.000+02:00
Check that interval parameter is an integer greater than zero. Delete unuseful 'optimization' meant to prevent exceeding the MAXTICKS variable. During testing it seemed ineffective. The following Locator.raise_if_exceeds exception was triggered first anyways. resolves #6935
diff --git a/lib/matplotlib/dates.py b/lib/matplotlib/dates.py
@@ -821,19 +821,6 @@ def tick_values(self, vmin, vmax):
 
         self.rule.set(dtstart=start, until=stop)
 
-        # estimate the number of ticks very approximately so we don't
-        # have to do a very expensive (and potentially near infinite)
-        # 'between' calculation, only to find out it will fail.
-        nmax, nmin = date2num((vmax, vmin))
-        estimate = (nmax - nmin) / (self._get_unit() * self._get_interval())
-        # This estimate is only an estimate, so be really conservative
-        # about bailing...
-        if estimate > self.MAXTICKS * 2:
-            raise RuntimeError(
-                'RRuleLocator estimated to generate %d ticks from %s to %s: '
-                'exceeds Locator.MAXTICKS * 2 (%d) ' % (estimate, vmin, vmax,
-                                                        self.MAXTICKS * 2))
-
         dates = self.rule.between(vmin, vmax, True)
         if len(dates) == 0:
             return date2num([vmin, vmax])
@@ -1254,6 +1241,12 @@ def __init__(self, bymonthday=None, interval=1, tz=None):
 
         Default is to tick every day of the month: ``bymonthday=range(1,32)``
         """
+        if interval < 1:
+            raise ValueError("The interval parameter must be an integer "
+                             "greater than or equal to one.")
+        if not isinstance(interval, int):
+            raise ValueError("The interval parameter must be an integer "
+                             "greater than or equal to one.")
         if bymonthday is None:
             bymonthday = range(1, 32)
         elif isinstance(bymonthday, np.ndarray):
