patchGNU Octave - Patches: patch #9409, [octave forge] (signal) modulate:...


patch #9409: [octave forge] (signal) modulate: new function, partial implementation

Submitted by:  None
Submitted on:  Sun 16 Jul 2017 02:44:11 PM UTC  
Category:  Forge : new function Priority:  5 - Normal
Status:  Postponed Privacy:  Public
Assigned to:  None Originator Email:  -email is 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.


Wed 18 Apr 2018 06:05:16 PM UTC, comment #2:

Here is an independent rewrite with all the modulation schemes and it mostly follows the coding standards. However, I've included some additional comments in the code (with a ###) that may help during the review but may not be necessary otherwise. Also, I am only attaching a .m file as of now and will follow a .patch file after the initial stages of review (if that is fine). Kindly review the patch and inform me of all the changes that are required.

(file #43969)

P Sudeepam <sudeepam>
Fri 30 Mar 2018 02:00:49 AM UTC, comment #1:

Does not include ppm and pwm options as of now.

(Copied text from original submission summary line)

Mike Miller <mtmiller>
Project Administrator
Sun 16 Jul 2017 02:44:11 PM UTC, original submission:


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

Attach Files:

Attached Files
file #43969:  modulate.m added by sudeepam (7KiB - text/x-objcsrc - rewrite with all the modulation schemes. -P Sudeepam)
file #41212:  modulate.m added by None (2KiB - application/octet-stream - Added modulate ..still missing ppm and pwm)


Depends on the following items: None found

Items that depend on this one: None found


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

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

    Only logged-in users can vote.




    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-04-18 sudeepam Attached File- => Added modulate.m, #43969
    2018-03-30 mtmiller StatusNone => Postponed
        SummaryImplemented modulate function for Forge Signal package however it does not include ppm and pwm options as of now. => [octave forge] (signal) modulate: new function, partial implementation
    2017-07-16 None Attached File- => Added modulate.m, #41212

    Back to the top

    Powered by Savane 3.3