matplotlib-devel Mailing List for matplotlib

It'd be nice to get this into the new release unless it has been
somehow fixed by some other code path in the latest svn... the line
numbers on the diff are no longer accurate, but the idea is still the
same - just change all lines of the form
caplines.extend( self.plot(leftlo, ylo,ls=None,
marker=mlines.CARETLEFT, **plot_kw) )
to
caplines.extend( self.plot(leftlo, ylo, 'k|', marker=mlines.CARETLEFT,
**plot_kw) )
e.g. change the ls=None to 'k|' and the color cycle is not
interrupted. I don't have the 1.1 numpy yet, so I haven't explicitly
tested it, but I don't see why anything would have changed...
On Tue, Feb 12, 2008 at 10:46 PM, Erik Tollerud <eri...@gm...> wrote:
> After a little testing on the subversion repository, the attached diff
> seems to work.
>
> On Feb 10, 2008 12:12 AM, Erik Tollerud <eri...@gm...> wrote:
>> I noticed while making some plots with upper bound error bars that
>> whenever Axes.errorbars is called with any of the errorbars chosen as
>> upper or lower bounds, that the color cycle was off, skipping over 2
>> colors each time another errorbar plot was made (e.g. the first would
>> be green and the second would be cyan instead of blue,green,red,cyan).
>> Looking into the axes class, it appears that if you don't specify a
>> color and want the markers to be drawn over the errorbars, the color
>> cycle has already been skipped over one because of calls to draw the
>> markers into the plot. The solution is to change all the lines that
>> say " ls='None' " in the errorbar method to instead say " ls='k' " -
>> this will prevent those calls from cycling the colors, and hence only
>> the call to actually draw the markers will do so. Can this patch be
>> committed to svn?
>>
>
>
>
> --
> Erik Tollerud
> Graduate Student
> Center For Cosmology
> Department of Physics and Astronomy
> 4155B Frederick Reines Hall
> University of California, Irvine
> Office Phone: (949)824-2996
> Cell: (651)307-9409
> eto...@uc...
>
-- 
Erik Tollerud
Graduate Student
Center For Cosmology
Department of Physics and Astronomy
2142 Frederick Reines Hall
University of California, Irvine
Office Phone: (949)824-2587
Cell: (651)307-9409
eto...@uc...

You're right. Just a typo on my part. I'll commit these changes to SVN 
so they'll make it into the next release.
Cheers,
Mike
Mark Bakker wrote:
> That fixes it on my machine.
> Except that the function GetRealpathAndStat is in cbook.py (I don't 
> even have a mplutil.py file).
> Thanks,
> Mark
>
> On Tue, May 13, 2008 at 7:21 PM, Michael Droettboom <md...@st... 
> <mailto:md...@st...>> wrote:
>
> I suspect on Windows it is safe enough (in most cases) to use the
> realpath as the key, in lieu of anything better. But that would
> require testing on Windows to make sure. Can you try replacing
> the GetRealpathAndStat in mplutil.py with the following?
>
>
> class GetRealpathAndStat:
> def __init__(self):
> self._cache = {}
>
> def __call__(self, path):
> result = self._cache.get(path)
> if result is None:
> realpath = os.path.realpath(path)
> if sys.platform == "win32":
> stat_key = realpath
> else:
>
> stat = os.stat(realpath)
> stat_key = (stat.st_ino, stat.st_dev)
> result = realpath, stat_key
> self._cache[path] = result
> return result
> get_realpath_and_stat = GetRealpathAndStat()
>
>
> Cheers,
> Mike
>
> Mark Bakker wrote:
>
> John, Michael -
>
> Now that we are talking about a new release, did you guys ever
> manage to fix the bug described below. It had to do with greek
> symbols not showing up in postscript files on windows. John
> seemed to have tracked down the source of the problem, but I
> never heard of a solution.
>
> Thanks,
>
> Mark
>
> On Tue, Mar 25, 2008 at 7:50 PM, John Hunter
> <jd...@gm... <mailto:jd...@gm...>
> <mailto:jd...@gm... <mailto:jd...@gm...>>> wrote:
>
> On Tue, Mar 25, 2008 at 12:02 PM, Michael Droettboom
> <md...@st... <mailto:md...@st...>
> <mailto:md...@st... <mailto:md...@st...>>> wrote:
>
> > The *intention* is that the fonts *should* be included
> (with the
> > exception of ps.useafm == True). That was definitely not a
> deliberate
> > change.
> >
> > However, as one of the ones who hasn't been able to
> reproduce this
> > problem, I'm afraid I'm not of much help. From reading the
> code, I'm
> > still completely stumped as to why the font is not embedded.
> Someone
> > will have to step through with a debugger on one of the
> broken
> systems
> > to figure this out, I'm afraid.
>
> I was able to replicate the bug and find the source of the
> problem. I
> am not 100% sure how to fix it, but someone who knows
> os.stat better
> might. The problem is that
> matplotlib.cbook.get_realpath_and_stat
>
> class GetRealpathAndStat:
> def __init__(self):
> self._cache = {}
>
> def __call__(self, path):
> result = self._cache.get(path)
> if result is None:
> realpath = os.path.realpath(path)
> stat = os.stat(realpath)
> stat_key = (stat.st_ino, stat.st_dev)
> result = realpath, stat_key
> self._cache[path] = result
> return result
> get_realpath_and_stat = GetRealpathAndStat()
>
>
> is returning the same stat ino and dev for all the font
> files, and
> thus the renderer.used_characters dictionary is getting
> improper keys
> -- always (0,0). So the first font in the gate, in this
> case Vera, is
> getting a place in the dict and subsequent fonts (the cm*
> ones) are
> not. The basic problem is that the inode and dev appear to
> be unix
> only.
>
> Michael: if you let me know better what this key is
> supposed to be
> doing (can we not simply use the filename for windows?)
> then I can
> attempt or test some fixes.
>
> JDH
>
>
>
> -- 
> Michael Droettboom
> Science Software Branch
> Operations and Engineering Division
> Space Telescope Science Institute
> Operated by AURA for NASA
>
>
-- 
Michael Droettboom
Science Software Branch
Operations and Engineering Division
Space Telescope Science Institute
Operated by AURA for NASA

That fixes it on my machine.
Except that the function GetRealpathAndStat is in cbook.py (I don't even
have a mplutil.py file).
Thanks,
Mark
On Tue, May 13, 2008 at 7:21 PM, Michael Droettboom <md...@st...> wrote:
> I suspect on Windows it is safe enough (in most cases) to use the realpath
> as the key, in lieu of anything better. But that would require testing on
> Windows to make sure. Can you try replacing the GetRealpathAndStat in
> mplutil.py with the following?
>
> class GetRealpathAndStat:
> def __init__(self):
> self._cache = {}
>
> def __call__(self, path):
> result = self._cache.get(path)
> if result is None:
> realpath = os.path.realpath(path)
> if sys.platform == "win32":
> stat_key = realpath
> else:
> stat = os.stat(realpath)
> stat_key = (stat.st_ino, stat.st_dev)
> result = realpath, stat_key
> self._cache[path] = result
> return result
> get_realpath_and_stat = GetRealpathAndStat()
>
>
> Cheers,
> Mike
>
> Mark Bakker wrote:
>
> > John, Michael -
> >
> > Now that we are talking about a new release, did you guys ever manage to
> > fix the bug described below. It had to do with greek symbols not showing up
> > in postscript files on windows. John seemed to have tracked down the source
> > of the problem, but I never heard of a solution.
> >
> > Thanks,
> >
> > Mark
> >
> > On Tue, Mar 25, 2008 at 7:50 PM, John Hunter <jd...@gm... <mailto:
> > jd...@gm...>> wrote:
> >
> > On Tue, Mar 25, 2008 at 12:02 PM, Michael Droettboom
> > <md...@st... <mailto:md...@st...>> wrote:
> >
> > > The *intention* is that the fonts *should* be included (with the
> > > exception of ps.useafm == True). That was definitely not a
> > deliberate
> > > change.
> > >
> > > However, as one of the ones who hasn't been able to reproduce this
> > > problem, I'm afraid I'm not of much help. From reading the
> > code, I'm
> > > still completely stumped as to why the font is not embedded.
> > Someone
> > > will have to step through with a debugger on one of the broken
> > systems
> > > to figure this out, I'm afraid.
> >
> > I was able to replicate the bug and find the source of the problem.
> > I
> > am not 100% sure how to fix it, but someone who knows os.stat better
> > might. The problem is that matplotlib.cbook.get_realpath_and_stat
> >
> > class GetRealpathAndStat:
> > def __init__(self):
> > self._cache = {}
> >
> > def __call__(self, path):
> > result = self._cache.get(path)
> > if result is None:
> > realpath = os.path.realpath(path)
> > stat = os.stat(realpath)
> > stat_key = (stat.st_ino, stat.st_dev)
> > result = realpath, stat_key
> > self._cache[path] = result
> > return result
> > get_realpath_and_stat = GetRealpathAndStat()
> >
> >
> > is returning the same stat ino and dev for all the font files, and
> > thus the renderer.used_characters dictionary is getting improper keys
> > -- always (0,0). So the first font in the gate, in this case Vera,
> > is
> > getting a place in the dict and subsequent fonts (the cm* ones) are
> > not. The basic problem is that the inode and dev appear to be unix
> > only.
> >
> > Michael: if you let me know better what this key is supposed to be
> > doing (can we not simply use the filename for windows?) then I can
> > attempt or test some fixes.
> >
> > JDH
> >
> >
> >
> --
> Michael Droettboom
> Science Software Branch
> Operations and Engineering Division
> Space Telescope Science Institute
> Operated by AURA for NASA
>
>