GNU Octave - Patches: patch #8668, [octave forge] (signal) firls: add...

Group

Main
- Main
- View members
- Search
Homepage
Download
Mailing lists
Source code
Bugs
- Submit new
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search
Tasks
- ~~Submit new~~
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search
Patches
- Submit new
- Browse
- Reset to open
- Digest
- Dependencies
- ~~Export~~
- Get statistics
- Search

patch #8668: [octave forge] (signal) firls: add demos, tests, etc

Submitter:	Rob Sykes <robsykes>
Submitted:	Mon 04 May 2015 01:40:06 PM UTC


Category:	Forge : other	Priority:	5 - Normal
Status:	Postponed	Privacy:	Public
Assigned to:	mtmiller	Open/Closed:	Open

* Mandatory Fields

Add a New Comment Rich Markup

Tue 03 Apr 2018 07:02:40 AM UTC, comment #1:

There has been a lot more work done on an updated firls function with more Matlab compatibility options in bug #51310, along with some test cases. I would rather see the effort combined there.

I will be happy to incorporate more tests and demos into that submission. But I don't think the option you've added here to allow the amplitude vector to be half of the band edge vector is Matlab compatible, so I would rather not include support for that.

I will leave this open as a reminder to adapt the tests and demos into the firls update once it's been added to the package.

Mike Miller <mtmiller>

Mon 04 May 2015 01:40:06 PM UTC, original submission:

Add demos (comparable with those in firpm)
Add initial tests
Support giving 1 amplitude/band
Fix some documentation typos

Rob Sykes <robsykes>

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

Attach Files:


Comment:

Attached Files

file #33896: firls.patch.xz added by robsykes (2KiB - application/x-xz - add demos & initial tests; support giving 1 amplitude/band; fix a couple of typos)

Depends on the following items: None found

Items that depend on this one: None found

Carbon-Copy List

-email is unavailable- added by robsykes (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 logged-in users can vote.

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

Follow 10 latest changes.

Date	Changed by	Updated Field	Previous Value	Replaced by
2019-03-07	mtmiller	Carbon-Copy	Removed 80942	-
2019-03-07	mtmiller	Carbon-Copy	Removed mtmiller	-
2018-04-03	mtmiller	Status	None	Postponed
2018-03-30	mtmiller	Category	Forge : new feature	Forge : other
2018-03-30	mtmiller	Category	None	Forge : new feature
		Summary	signal package: firls: add demos etc.	[octave forge] (signal) firls: add demos, tests, etc
2015-12-22	mtmiller	Assigned to	None	mtmiller
		Summary	[signal] firls: add demos etc.	signal package: firls: add demos etc.
2015-05-04	robsykes	Carbon-Copy	-	Added mtmiller
2015-05-04	robsykes	Attached File	-	Added firls.patch.xz, #33896

GNU Octave - Patches: patch #8668, [octave forge] (signal) firls: add...

patch #8668: [octave forge] (signal) firls: add demos, tests, etc

<a href="#postcomment">Post a Comment</a>

<a href="#discussion">Discussion</a>

<a href="#attached">Attached Files</a>

<a href="#dependencies">Dependencies</a>

<a href="#cc">Mail Notification Carbon-Copy List</a>

<a href="#votes">Votes</a>

<a href="#history">History</a>