bugGNU Octave - Bugs: bug #47387, [octave-forge] (signal) fracshift...

 
 

bug #47387: [octave-forge] (signal) fracshift function ignores interpolator parameter

Submitted by:  None
Submitted on:  Fri 11 Mar 2016 12:18:41 PM UTC  
 
Category: Octave Forge PackageSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: Patch SubmittedAssigned to: Mike Miller <mtmiller>
Originator Name: Tim StylesOriginator Email: -unavailable-
Open/Closed: OpenRelease: other
Operating System: GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Fri 11 Mar 2016 04:30:48 PM UTC, comment #1:

Thanks for the report, this looks right and a very simple fix. I'll try to merge this in to the next bug fix release of the signal package.

Mike Miller <mtmiller>
Project AdministratorIn charge of this item.
Fri 11 Mar 2016 12:18:41 PM UTC, original submission:

I'm using the fracshift function in signal-1.2.2/fracshift.m
The help indicates that an optional third parameter can be given to specify the interpolator, but using this option has no effect.

Looking at the code in frachshift.m, line 46 is checking for a fourth parameter instead of a third parameter. Making the following change in fracshift.m causes the third parameter to be used and it works as I would expect:

Anonymous

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jwe (Updated the item)
  • -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 project members can vote.

     

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

     

     

    Follow 14 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 11 Aug 2017 05:43:36 PM UTCjweSummary[ocave-forge] (signal) fracshift function ignores interpolator parameter=>[octave-forge] (signal) fracshift function ignores interpolator parameter
    Fri 11 Aug 2017 05:43:17 PM UTCjweSummarysignal package: fracshift function ignores interpolator parameter=>[ocave-forge] (signal) fracshift function ignores interpolator parameter
    Tue 29 Mar 2016 05:59:24 PM UTCmtmillerAssigned toNone=>mtmiller
    Fri 11 Mar 2016 04:32:24 PM UTCmtmillerStatusNone=>Patch Submitted
    Fri 11 Mar 2016 04:32:13 PM UTCmtmillerRelease3.8.1=>other
      Summaryfracshift function ignores interpolator parameter=>signal package: fracshift function ignores interpolator parameter
    Fri 11 Mar 2016 04:32:01 PM UTCrik5Carbon-CopyRemoved 72865=>-
    Fri 11 Mar 2016 04:30:58 PM UTCrik5StatusPatch Submitted=>None
      Releaseother=>3.8.1
      Summarysignal package: fracshift function ignores interpolator parameter=>fracshift function ignores interpolator parameter
    Fri 11 Mar 2016 04:30:48 PM UTCmtmillerCategoryOctave Function=>Octave Forge Package
      StatusNone=>Patch Submitted
      Release3.8.1=>other
      Summaryfracshift function ignores interpolator parameter=>signal package: fracshift function ignores interpolator parameter

    Back to the top


    Powered by Savane 3.1-cleanup1