Skip to content

Commit

Permalink
Merge pull request #810 from pybamm-team/issue-803-graphviz
Browse files Browse the repository at this point in the history
#803 make visualise fail only raise logger error
  • Loading branch information
valentinsulzer committed Feb 12, 2020
2 parents b3d90d7 + d6caf83 commit f68b303
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions pybamm/expression_tree/symbol.py
Original file line number Diff line number Diff line change
Expand Up @@ -285,9 +285,13 @@ def visualise(self, filename):

new_node, counter = self.relabel_tree(self, 0)

DotExporter(
new_node, nodeattrfunc=lambda node: 'label="{}"'.format(node.label)
).to_picture(filename)
try:
DotExporter(
new_node, nodeattrfunc=lambda node: 'label="{}"'.format(node.label)
).to_picture(filename)
except FileNotFoundError:
# raise error but only through logger so that test passes
pybamm.logger.error("Please install graphviz>=2.42.2 to use dot exporter")

def relabel_tree(self, symbol, counter):
""" Finds all children of a symbol and assigns them a new id so that they can be
Expand Down

0 comments on commit f68b303

Please sign in to comment.