[python-control-discuss] consistency of documentation and default args for root_locus()

[Scott C. L. <sli...@ca...>]

The documentation for root_locus() (in rlocus.py) claims two return values, whereas there is actually only one: the root list, "rlist".

rlocus() in matlab.py has default argument klist=None.  Inside the function body, if klist is found to be None, then it is set to logspace(-3, 3).  Is there a reason to do that rather than use

  def rlocus(sys, klist = logspace(-3, 3), **keywords)

Should I start sending patches instead of complaining?  :-)

Thanks,
Scott