Coverage increased (+0.08%) to 90.006% when pullingfaf3145 on roryyorke:rory/check-interconnect-signals into5ab0a1c 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 theinterconnect() function rather than theInterconnectedSystem constructor. The intent ofinterconnect() 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. IfInterconnectedSystem is used directly,check_unused_signals is available to be called after mappings have been set up. I also madeunused_signals public, though I don't know what applications it could have.

While writing this, it occurred to me it wouldn't be allthat hard to output aGraphviz dot file, or, possibly, a data structure that could be fed intoNetworkX, 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.