Although the package already provides a good solution, there are a few things, which should be improved/ corrected:
- For FIR architecture it would be useful to add Rectangular Window as baseline.
- In the detailed analysis window the following should be corrected:
- Because the step response represents also a discrete series, it should be a stem plot similar to impulse response.
- When frequency axis is selected to be in Hz, the -3dB crossings should be also in Hz.
- For FIR architecture the order is currently presented to be always zero.
- The pole-zero plot should plot the poles and zeros like zplane. That means (a) when FIR architecture also the poles should be plotted and (b) multiple poles and zeros should be marked by their occurrence number.
- The filter design report should contain the same changes as suggested above (plus the frequency axis according to the setting).
- In the Audio demo it would be useful to make frequency axis in spectrogram also to be adjustable according to the setting in the filter window (Hz vs Normalised). Moreover, for playing with different filter settings it would be nice, if the loaded wave-file remains in the memory.
- In the filter comparison window when using current filter, FIR filters are interpreted as IIR filters. In consequence, IDs greater than four are not loaded. Loading from file is working correct.
- In the save design the version number needs to be updated. Moreover, the file extension .mat could be changed, because there maybe some confusion with .mat files containing data in Matlab format. Octave is correctly handling this, but the change (perhaps in .fil) may improve handling.
Although the package already provides a good solution, there are a few things, which should be improved/ corrected: