Hi John,
I'm a bit confused. When you say 1st instance, you mean
if self.axison and self._frameon:
self.patch.draw(renderer)
at line 1452?
And you're saying that you tend to keep only the second instance,
instead of both?
(I'm sorry if I misunderstood you. I often have problems with my english)
I guess we need both. What I see is that self.patch is for axes
background (a "filled" patch), and self.frame is for axes border (a
patch with facecolor="none"). Aren't they?
(We may have self.patch.set_edgecolor("none") explicitly though).
Regards,
-JJ
On Mon, Jul 28, 2008 at 10:45 AM, John Hunter <[EMAIL PROTECTED]> wrote:
> On Mon, Jul 28, 2008 at 9:26 AM, Jae-Joon Lee <[EMAIL PROTECTED]> wrote:
>> Hello,
>>
>> I just noticed that,if I use imshow(), part of axes border is not
>> clearly visible (the image hides the border).
>> And this seems to be due to the following changes in axes.draw()
>> method made in r5882.
>>
>> --- lib/matplotlib/axes.py (revision 5881)
>> +++ lib/matplotlib/axes.py (revision 5882)
>> @@ -1503,8 +1503,6 @@
>> artists.extend(self.tables)
>> if self.legend_ is not None:
>> artists.append(self.legend_)
>> - if self.axison and self._frameon:
>> - artists.append(self.frame)
>>
>> dsu = [ (a.zorder, i, a) for i, a in enumerate(artists)
>> if not a.get_animated() ]
>>
>>
>> Don't we need axes.frame to be drawn? Recovering those two lines
>> solved my problem.
>
> apparently the frame drawing code was in Axes.draw and Paul removed
> the 2nd instance of it. The first instance (which is the one that
> remains) draws the frame under the artists, and the 2nd instance (the
> one you want restored) draws the frame above the artists. I tend to
> agree that it is the 2nd one we want. Does anyone have a different
> opinion before I fix this?
>
> JDH
>
> Apparently someone moved the frame drawing code to be earlier so it
> would be under the artists.
>
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
