Re: [matplotlib-devel] small patch for axes.specgram

2007-11-14 Thread Glen W. Mabey
On Wed, Nov 14, 2007 at 08:59:15AM -0600, John Hunter wrote: > Glen, this looks fairly benign (the center frequency bit) so I am > happy to add it, but could I trouble you to make similar patch the > other spectral methods as well (eg psd, csd and cohere) for the sake > of consistency in the API?

Re: [matplotlib-devel] small patch for axes.specgram

2007-11-14 Thread John Hunter
On Nov 14, 2007 8:00 AM, Michael Droettboom <[EMAIL PROTECTED]> wrote: > I'll defer to others with more experience with specgram to assess the > rest of your patch. Glen, this looks fairly benign (the center frequency bit) so I am happy to add it, but could I trouble you to make similar patch th

Re: [matplotlib-devel] small patch for axes.specgram

2007-11-14 Thread Michael Droettboom
The fix for mlab.py is obviously necessary, so I've committed it to SVN. I'll defer to others with more experience with specgram to assess the rest of your patch. Cheers, Mike Glen W. Mabey wrote: > Hello, > > This is fairly minor, but I'd wonder if you'd consider including it. > > It adds an

[matplotlib-devel] small patch for axes.specgram

2007-11-13 Thread Glen W. Mabey
Hello, This is fairly minor, but I'd wonder if you'd consider including it. It adds an optional Fc parameter to the specgram method of the Axes class, and makes the calculation of the extent a bit more efficient. Also, a quick fix for mlab.py. Thank you, Glen Mabey *** zipsrc/matplotlib-200711