tensorflow / lucid

A collection of infrastructure and tools for research in neural network interpretability.
Apache License 2.0
4.67k stars 653 forks source link

skip control inputs when building graphs #186

Closed michaelpetrov closed 5 years ago

michaelpetrov commented 5 years ago

Control inputs are not attached to the graph in a way that makes them visible when iterating using graphdef.node. This causes a crash when trying to attach them as an edge. For now, skipping them seems like a reasonable way to keep the graphs working.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 557


Changes Missing Coverage Covered Lines Changed/Added Lines %
lucid/misc/graph_analysis/overlay_graph.py 0 2 0.0%
<!-- Total: 0 2 0.0% -->
Totals Coverage Status
Change from base Build 555: -0.06%
Covered Lines: 1763
Relevant Lines: 2503

💛 - Coveralls