bugGNU Octave - Bugs: bug #66766, [octave-forge] signal package...

 
 

bug #66766: [octave-forge] signal package -> Missing function sos2cell ( patch )

Submitter:  Abinash Singh <abinashsingh>
Submitted:  Sat 08 Feb 2025 07:27:30 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Feature Request
Status:  Patch Submitted Assigned to:  None
Originator Name:  Abinash Singh Open/Closed:  * Open
Release:  * 9.4.0 Release: 
Operating System:  * Any Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 22 Feb 2025 02:53:16 PM UTC, comment #1: 

Thanks for the patch!

I took a quick look at the patch and have a few comments:

1. The textinfo section has empty non comment lines between parts of the documentation, so the texinfo section will stop at those blank lines.  You should keep the lines in the text info section as lines sarrting with ##

2. You should include some tests as well at the end of the file

3. The INDEX file should include a reference to the new function - in the Filter Conversion section ?

John Donoghue <lostbard>
Group Member
Sat 08 Feb 2025 07:27:30 PM UTC, original submission:  

This patch implements sos2cell function which was under the "missing function" category.

source: https://wiki.octave.org/Signal_package

Please review it and provide me feedback. I'm also working on its inverse i.e cell2sos

Thank You

Abinash Singh <abinashsingh>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #56863:  sos2cell.patch added by abinashsingh (3KiB - text/x-patch - Please provide me feedback for improvements)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by lostbard (Posted a comment)
  • -email is unavailable- added by nrjank (Updated the item)
  • -email is unavailable- added by abinashsingh (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2025-02-08 nrjank StatusNone Patch Submitted
    2025-02-08 abinashsingh Attached File- Added sos2cell.patch, #56863

    Back to the top

    Powered by Savane 3.14-7003.
    Corresponding source code