Cover more of root_locus: dtime and sisotool by bnavigator · Pull Request #617 · python-control/python-control

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any idea why there are two different sgrid functions? there is this one and there is grid.sgrid. Maybe a different PR, but would be good to unify these and get the best of each of them.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apparently _sgrid_func() was removed in #193 but reintroduced through some merge gone wrong?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.