If a system is passed in, both the root locus and bode functions do both computations and produce a plot. In my opinion, it would be better to separate the computation from the plotting. The plotting commands should then return matplotlib axes and maybe figure instances and the computation methods would return the results of the computations, e.g. roots, mag/phase, etc.
Right now there aren't clean ways to get hold of the figure and axes (I don't consider .gca() and .gcf() that "clean"). And if you are using a plotting library other than matplotlib you may not want to call functions that create matplotlib plots.
There maybe other plotting functions in the library that I haven't play with yet too that this is applicable too.
If a system is passed in, both the root locus and bode functions do both computations and produce a plot. In my opinion, it would be better to separate the computation from the plotting. The plotting commands should then return matplotlib axes and maybe figure instances and the computation methods would return the results of the computations, e.g. roots, mag/phase, etc.
Right now there aren't clean ways to get hold of the figure and axes (I don't consider
.gca()and.gcf()that "clean"). And if you are using a plotting library other than matplotlib you may not want to call functions that create matplotlib plots.There maybe other plotting functions in the library that I haven't play with yet too that this is applicable too.