matplotlib-devel

[matplotlib-devel] small patch for axes.specgram

[Glen W. M. <Gle...@sw...>]

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

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

[Michael D. <md...@st...>]

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 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
> 
> 
> 
> ------------------------------------------------------------------------
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc.
> Still grepping through log files to find problems?  Stop.
> Now Search log events and configuration files using AJAX and a browser.
> Download your FREE copy of Splunk now >> http://get.splunk.com/
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Matplotlib-devel mailing list
> Mat...@li...
> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

-- 
Michael Droettboom
Science Software Branch
Operations and Engineering Division
Space Telescope Science Institute
Operated by AURA for NASA

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

[John H. <jd...@gm...>]

On Nov 14, 2007 8:00 AM, Michael Droettboom <md...@st...> 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 the
other spectral methods as well (eg psd, csd and cohere) for the sake
of consistency in the API?

JDH

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

[Glen W. M. <Gle...@sw...>]

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?

Relax, patch attached.

Also, I am getting an error in numerix/__init__.py when it gets imported
by a script that is run by apache.  Apparently, sys.argv has length 0,
so "del a" on line 38 fails.

Thanks,
Glen