patchGNU Octave - Patches: patch #8495, signal package: goertzel function


patch #8495: signal package: goertzel function

Submitted by:  None
Submitted on:  Thu 10 Jul 2014 04:36:12 AM UTC  
Category: NonePriority: 5 - Normal
Status: Need InfoPrivacy: Public
Assigned to: Mike Miller <mtmiller>Originator Email: -unavailable-
Open/Closed: Open

Add a New Comment(Rich Markup)

You are not logged in

Please log in, so followups can be emailed to you.


Fri 30 Jan 2015 10:35:30 PM UTC, comment #1:

Thanks for your patch submission. Sorry for the delay in taking a look at it.

Would you mind putting your full name in the copyright line of the source file? It would be preferable to credit the file to someone's real name if possible.

Would you be able to adjust the code to use Octave's coding style? See and

Would you be able to provide some test cases for the algorithm, in Octave's %!test format? This would help ensure that the function works correctly and continues to provide the right answers on all systems going forward.

This function can still be added to the signal package if you are not able to do any of these, but it would help speed things up if the file is made closer to being ready for inclusion.

Mike Miller <mtmiller>
Project AdministratorIn charge of this item.
Thu 10 Jul 2014 04:36:12 AM UTC, original submission:

implementation of görtzel algorithm (equivalent to matlab's one)



(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):

Attached Files
file #31695: added by None (3KiB - text/plain)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by mtmiller (Posted a comment)
  • -unavailable- added by None (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Please enter the title of George Orwell's famous dystopian book (it's a date):



    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 30 Jan 2015 10:35:30 PM UTCmtmillerStatusNone=>Need Info
      Assigned toNone=>mtmiller
      Summarysignal package=>signal package: goertzel function
    Thu 10 Jul 2014 04:36:12 AM UTCNoneAttached File-=>Added, #31695

    Back to the top

    Powered by Savane 3.1-cleanup1