Andrew, One of my worry is that this can results in inconsistent ouputs between backends. Your patch only affects backends with compositing capabilities. And backends such as ps backend will still render images at the bottom of all other artists.
I think it is often sufficient if we draw images at the bottom but respect zorders among images, and I guess this actually solves the issue of the original post. So, my inclination is to respect zorders between images by default (images still drawn at the bottom), and optionally with all other artists. Anyhow, I guess the cases affected by this issue is quite rare, and I'm actually fine if you submit the patch as is. Regards, -JJ On Mon, Nov 9, 2009 at 11:21 AM, Andrew Straw <straw...@astraw.com> wrote: > Hi All, > > I have addressed what I think is a long-standing wart: zorder is mostly > ignored for imshow(). (See e.g. > http://old.nabble.com/Re%3A--Matplotlib-users--imshow-zorder-tt19047314.html#a19047314 > ) The question is whether I should apply the attached patch. > > The worry is that someone is relying on the old behavior of images being > underneath everything else. To a degree, this could be achieved by having > the default image zorder be lower than everything else. Currently, imshow() > creates images with zorder 1, which is pretty low on the hierarchy, but > there may be something lower. (The artist base class default is 0, but the > Line2D class bumps it to 2.) All tests pass with the change, and manual > inspection of several of the pylab_examples don't show any difference before > and after the patch. > > -Andrew > > ------------------------------------------------------------------------------ > Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day > trial. Simplify your report design, integration and deployment - and focus > on > what you do best, core application coding. Discover what's new with > Crystal Reports now. http://p.sf.net/sfu/bobj-july > _______________________________________________ > Matplotlib-devel mailing list > Matplotlib-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/matplotlib-devel > > ------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ Matplotlib-devel mailing list Matplotlib-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
