[Scipy-tickets] [SciPy] #1105: Logarithmic chirp frequency sweep incorrect

SciPy Trac scipy-tickets@scipy....
Mon Feb 15 12:11:51 CST 2010


#1105: Logarithmic chirp frequency sweep incorrect
-----------------------------------------+----------------------------------
 Reporter:  johntryan                    |       Owner:  somebody
     Type:  defect                       |      Status:  apply   
 Priority:  normal                       |   Milestone:  0.8.0   
Component:  scipy.signal                 |     Version:  0.7.0   
 Keywords:  chirp logarithmic frequency  |  
-----------------------------------------+----------------------------------

Comment(by warren):

 Replying to [comment:8 stefan]:
 > Thanks, Warren, looks good to me.
 >
 > The only thing I found confusing is the parameter 'f': does it represent
 coefficients, values, etc?  Maybe just clarify that in the documentation.

 Yes, that needs a better explanation.  I'll update the patch later
 today/tonight.

 >
 > I'll wait for your feedback and apply tomorrow morning.


 What do you think about dropping the 'logarithmic' method entirely? I
 agree with John (the creator of the ticket)--the formula is "incorrect" (a
 subjective term, I know).  I suspect the intent for that method was
 actually what is now implemented as the 'geometric' method, but an
 incorrect formula was used.

 Unless it turns out that there is some common use for that particular
 formula, I don't think there is any reason to keep it in chirp().

-- 
Ticket URL: <http://projects.scipy.org/scipy/ticket/1105#comment:9>
SciPy <http://www.scipy.org>
SciPy is open-source software for mathematics, science, and engineering.


More information about the Scipy-tickets mailing list