-
Notifications
You must be signed in to change notification settings - Fork 438
step_info error for MIMO system #574
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
Why line 614? Current master has that line at 807, Your own PR #567 duplicates it in 807 and 817. Can confirm with current master. The problem is the indexing of yout:
|
Why not adjust #567 to handle MIMO systems the way MATLAB's
|
yes my bad, is line 614 for version 8.4 |
Yes, I am trying to fix it, but I commit a partial solution for now. That improve the performance of this function |
step_response have limitations, that function can solve only SIMO systems. My idea is convert the step_info into two functions: step_info4data (t, y, yfinal) for multidimentional, time and output parameters, in the other hand, keep a step_info(sys) as is, but that function can call step_info4data () with the proper input vector for MIMO systems. This can allow use the step_info4data function with time vectors and output vectors imported from externals simulation or system stimulation |
I might be missing something, but the current |
I try with this MIMO system, the function broke at line 614. The step_info function convert the system to SIMO, but cannot calculate the step response characteristic parameters.
The text was updated successfully, but these errors were encountered: