Typo in matplotlib/image.py

Line 157 of image.py, there is a typo:

Index: matplotlib/image.py

···

===================================================================
--- matplotlib/image.py (revision 8036)
+++ matplotlib/image.py (working copy)
@@ -144,7 +144,7 @@
         sy = dyintv/viewlim.height
         numrows, numcols = A.shape[:2]
         if sx > 2:
- x0 = (viewim.x0-xmin)/dxintv * numcols
+ x0 = (viewlim.x0-xmin)/dxintv * numcols
             ix0 = max(0, int(x0 - self._filterrad))
             x1 = (viewlim.x1-xmin)/dxintv * numcols
             ix1 = min(numcols, int(x1 + self._filterrad))

Could someone with check in rights fix this in SVN, please.

Cheers,

Gaël

Gael Varoquaux wrote:

Line 157 of image.py, there is a typo:

Index: matplotlib/image.py

--- matplotlib/image.py (revision 8036)
+++ matplotlib/image.py (working copy)
@@ -144,7 +144,7 @@
         sy = dyintv/viewlim.height
         numrows, numcols = A.shape[:2]
         if sx > 2:
- x0 = (viewim.x0-xmin)/dxintv * numcols
+ x0 = (viewlim.x0-xmin)/dxintv * numcols
             ix0 = max(0, int(x0 - self._filterrad))
             x1 = (viewlim.x1-xmin)/dxintv * numcols
             ix1 = min(numcols, int(x1 + self._filterrad))

Could someone with check in rights fix this in SVN, please.

Gael,

It looks like someone already has, as part of additional changes. SVN is now at 8052, and I don't see the problem in either the maintenance branch or the trunk.

Eric

···

Cheers,

Gaël

------------------------------------------------------------------------------
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev _______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Indeed, my mistake. I thought that I had svn uped, but seems like not.

Thanks, and merry Christmas!

Gaël

···

On Thu, Dec 24, 2009 at 03:53:29PM -1000, Eric Firing wrote:

It looks like someone already has, as part of additional changes. SVN
is now at 8052, and I don't see the problem in either the maintenance
branch or the trunk.