SF.net SVN: matplotlib:[8082] trunk/matplotlib

[<as...@us...>]

Revision: 8082
          http://matplotlib.svn.sourceforge.net/matplotlib/?rev=8082&view=rev
Author:   astraw
Date:     2010-01-16 19:19:14 +0000 (Sat, 16 Jan 2010)

Log Message:
-----------
Ensure three minor ticks always drawn (SF# 2924245). Patch by Neil Crighton.

Modified Paths:
--------------
    trunk/matplotlib/CHANGELOG
    trunk/matplotlib/lib/matplotlib/ticker.py

Modified: trunk/matplotlib/CHANGELOG
===================================================================
--- trunk/matplotlib/CHANGELOG	2010-01-16 17:45:32 UTC (rev 8081)
+++ trunk/matplotlib/CHANGELOG	2010-01-16 19:19:14 UTC (rev 8082)
@@ -1,3 +1,6 @@
+2009-01-16 Ensure three minor ticks always drawn (SF# 2924245). Patch
+           by Neil Crighton. -ADS
+
 2010-01-16 Applied patch by Ian Thomas to fix two contouring
            problems: now contourf handles interior masked regions,
            and the boundaries of line and filled contours coincide. - EF

Modified: trunk/matplotlib/lib/matplotlib/ticker.py
===================================================================
--- trunk/matplotlib/lib/matplotlib/ticker.py	2010-01-16 17:45:32 UTC (rev 8081)
+++ trunk/matplotlib/lib/matplotlib/ticker.py	2010-01-16 19:19:14 UTC (rev 8082)
@@ -1320,15 +1320,13 @@
         except IndexError:
             raise ValueError('Need at least two major ticks to find minor '
                              'tick locations')
-        # see whether major step should be divided by 5, 4 or 2. This
+        # see whether major step should be divided by 5, 4. This
         # should cover most cases.
         temp = float(('%e' % majorstep).split('e')[0])
         if temp % 5 < 1e-10:
             minorstep = majorstep / 5.
-        elif temp % 2 < 1e-10:
-            minorstep = majorstep / 4.
         else:
-            minorstep = majorstep / 2.
+            minorstep = majorstep / 4.
 
         tmin = majorlocs[0] - majorstep
         tmax = majorlocs[-1] + majorstep


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.