Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WFS IIR prefilter #29

Closed
hagenw opened this issue Feb 2, 2015 · 6 comments
Closed

WFS IIR prefilter #29

hagenw opened this issue Feb 2, 2015 · 6 comments
Assignees
Labels

Comments

@hagenw
Copy link
Member

hagenw commented Feb 2, 2015

By executing the test function for the IIR pre-filter from current master (bdd0e4a) in Matlab R2013a

>> test_wfs_iir_prefilter

I got the following error:

Error using fdesign.arbmag/set
There is no enumerated value named 'Nb,Na,F,A'.

Error in /usr/local/MATLAB/R2013a/toolbox/signal/signal/@fdesign/@abstracttypewspecs/schema.p>set_specificationtype (line 65)
@trettberg
Copy link
Contributor

By executing the test function for the IIR pre-filter from current master [...] I got the following error [...]

Cannot reproduce.
Test is passed from current master (da8d4c5) with R2012b, R2013a and R2013b, Linux.

@hagenw hagenw added the bug label Feb 29, 2016
@hagenw
Copy link
Member Author

hagenw commented Feb 29, 2016

I still get an error:

>> test_wfs_iir_prefilter
Error using fdesign.arbmag/set
There is no enumerated value named 'Nb,Na,F,A'.

Error in fdesign.abstracttypewspecs/schema>set_specificationtype (line 64)



Error in fdesign.abstracttypewspecs/abstract_setspecs (line 24)
    set(this, 'SpecificationType', varargin{1});

Error in fdesign.abstracttypewspecs/setspecs (line 7)
abstract_setspecs(this, varargin{:});

Error in fdesign.arbmag (line 119)
this.setspecs(varargin{:});

Error in wfs_iir_prefilter (line 137)
d = fdesign.arbmag('Nb,Na,F,A',Nb,Na,F,abs(H)); %we only consider abs, due to the desired minphase design

Error in test_wfs_iir_prefilter (line 54)
hpre1 = wfs_iir_prefilter(conf)

Linux, Matlab 2015a:

>> ver
----------------------------------------------------------------------------------------------------
MATLAB Version: 8.5.0.197613 (R2015a)
MATLAB License Number: 559163
Operating System: Linux 4.3.0-1-amd64 #1 SMP Debian 4.3.3-7 (2016-01-19) x86_64
Java Version: Java 1.7.0_60-b19 with Oracle Corporation Java HotSpot(TM) 64-Bit Server VM mixed mode
----------------------------------------------------------------------------------------------------
MATLAB                                                Version 8.5        (R2015a)
Optimization Toolbox                                  Version 7.2        (R2015a)
Partial Differential Equation Toolbox                 Version 2.0        (R2015a)
Signal Processing Toolbox                             Version 7.0        (R2015a)
Symbolic Math Toolbox                                 Version 6.2        (R2015a)

@trettberg
Copy link
Contributor

It looks like the DSP System Toolbox is required.

@hagenw
Copy link
Member Author

hagenw commented Feb 29, 2016

This sounds reasonable. I added a check for the DSP System Toolbox and pushed it into the master.
Can you please check, if it finds the toolbox for you and continues.

For me it stops now with the following error under Matlab:

>> test_wfs_iir_prefilter
Error using wfs_iir_prefilter (line 98)
WFS_IIR_PREFILTER: The DSP System Toolbox is required for IIR filter, please use conf.wfs.hpretype="FIR"

Error in test_wfs_iir_prefilter (line 54)
hpre1 = wfs_iir_prefilter(conf)

And this error is returned under Octave:

>> test_wfs_iir_prefilter
error: WFS_IIR_PREFILTER: Not available under Octave, please use conf.wfs.hpretype="FIR"
error: called from
    wfs_iir_prefilter at line 93 column 5
    test_wfs_iir_prefilter at line 54 column 7

@trettberg
Copy link
Contributor

Works fine here.

@hagenw
Copy link
Member Author

hagenw commented Mar 1, 2016

Ok, perfect.

@hagenw hagenw closed this as completed Mar 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants