- Notifications
You must be signed in to change notification settings - Fork441
Scaling difference in impulse response compared to scipy.signal#747
-
I had some code that I wanted to port from using importnumpyasnpimportcontrolimportscipy.signal# random state space matricest=np.linspace(0,10,dtype=float)ss=control.drss(3,2,1)A,B,C,D=ss.A,ss.B,ss.C,ss.Ddt=t[1]# compute impulse response with python-controlss=control.ss(A,B,C,D,dt=dt)t_control,h_control=control.impulse_response(ss,T=t)plt.figure()plt.plot(t_control,h_control[0,0])plt.plot(t_control,h_control[1,0])# compute impulse response with scipy.signalss_scipy=scipy.signal.StateSpace(A,B,C,D,dt=dt)t_scipy, (h_scipy,)=scipy.signal.dimpulse(ss_scipy,t=t)plt.figure()plt.plot(t_scipy,h_scipy[:,0])plt.plot(t_scipy,h_scipy[:,1])plt.show() Creates these two here. |
BetaWas this translation helpful?Give feedback.
All reactions
python-control computes the response to a unit area impulse, rather than a unit height impulse:#447
Replies: 1 comment 2 replies
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
python-control computes the response to a unit area impulse, rather than a unit height impulse:#447 |
BetaWas this translation helpful?Give feedback.
All reactions
-
That's a subtle and important detail.Thanks! Would it make sense to document this in impulse_response's docstring or am I just too ignorant of this common concept? |
BetaWas this translation helpful?Give feedback.
All reactions
👍 1
-
I think it would totally make sense to mention it in the docs. |
BetaWas this translation helpful?Give feedback.
All reactions
👍 1