GeezerVison

Coverage increased (+0.08%) to 90.006% when pulling faf3145 on roryyorke:rory/check-interconnect-signals into 5ab0a1c on python-control:master.

Quick responses on a few things, based on a quick read:

• I really like this functionality and completely agree we should include it!

• I think it would be OK to put this in the interconnect() function rather than the InterconnectedSystem constructor. The intent of interconnect() was to provide some additional processing and a more user-friendly interface.

• It looks like there are some glitches in the documentation (ignore_inputs appears twice).

Thanks for the feedback. I've fixed the docstring.

think it would be OK to put this in the interconnect()

OK, good. If InterconnectedSystem is used directly, check_unused_signals is available to be called after mappings have been set up. I also made unused_signals public, though I don't know what applications it could have.

While writing this, it occurred to me it wouldn't be all that hard to output a Graphviz dot file, or, possibly, a data structure that could be fed into NetworkX, to allow respectively visualization and graph-theoretic analysis of the signal flow, but I don't have time for any of that at the moment.

Connecting @petercorke into this discussion because he is working on a nice-looking graphical renderer for this kind of interconnection. Not sure it is released yet though.