|
From: | Mike Miller |
Subject: | [Octave-patch-tracker] [patch #8668] [octave forge] (signal) firls: add demos, tests, etc |
Date: | Tue, 3 Apr 2018 03:02:40 -0400 (EDT) |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:59.0) Gecko/20100101 Firefox/59.0 |
Update of patch #8668 (project octave): Status: None => Postponed _______________________________________________________ Follow-up 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. _______________________________________________________ Reply to this item at: <http://savannah.gnu.org/patch/?8668> _______________________________________________ Message sent via/by Savannah http://savannah.gnu.org/
[Prev in Thread] | Current Thread | [Next in Thread] |