FIX: plot Nyquist frequency correctly in Bode plot in Hz by murrayrm · Pull Request #651

FIX: plot Nyquist frequency correctly in Bode plot in Hz by murrayrm · Pull Request #651 · python-control/python-control

Merged

Sep 6, 2021

Merged

FIX: plot Nyquist frequency correctly in Bode plot in Hz#651
bnavigator merged 1 commit intopython-control:masterfrom
murrayrm:fix_bode_nyquist_hz

Copy link Copy Markdown

Member

Quick fix for issue #650, using solution from @dc5048.

Copy link Copy Markdown

Coverage remained the same at 89.923% when pulling 3b90050 on murrayrm:fix_bode_nyquist_hz into 18976c1 on python-control:master.

Copy link Copy Markdown

Contributor

For reference:

import control
tf1 = control.tf(control.drss(3,1,1))
tf1.dt=0.1
control.bode_plot(tf1, Hz=True)

Before	After

(Two different random discrete transfer functions)

bnavigator merged commit 5ab0a1c into python-control:master

bnavigator linked an issue

that may be closed by this pull request

Closed

bnavigator mentioned this pull request

Closed

murrayrm deleted the fix_bode_nyquist_hz branch

murrayrm added this to the 0.9.1 milestone

None yet