-
Notifications
You must be signed in to change notification settings - Fork 438
control.matlab.freqresp is inconsistent with MATLAB #573
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
Comments
Looking through the code, it looks like the |
Agh. I must have been looking at some different version. I think the problem is not that the function is missing, but that it is inconsistent with MATLAB. Re-editing title and description. Sorry for the confusion. |
Hi, I update the control to version 0.9.0. LTI system can be callable. But I want to caculate the freqresp of a FRD system to get the complex frequency response. If there any solution? |
Use |
As pointed out in issue #550, the freqresp function returns magnitude, phase, and frequency instead of the complex frequency response. This is OK for the main package (since you can just evaluate the system itself to get the complex frequency response), but it is inconsistent with MATLAB's freqresp function. At the least, control.matlab.freqresp should be consistent with MATLAB.
See discussion in PR #449 and issue #434.
The text was updated successfully, but these errors were encountered: