terrapower / armi / 11535245133

91%

master: 87%

Build Type

push

github

Committed by web-flow

Commit Message

Avoiding closing plots that are meant to be interactive (#1978)

The plt.show() options are intended for:

1. Interactive use on ipython or notebooks
2. Automated use during sphinx-gallery generation

Running plt.show() followed by plt.close() prevents at
least the 2nd use case from working, as seen in #1977.

This change puts the close() only in the code branches
that are expected to be run non-interactively, e.g.
compute code. That allows the system to keep memory
clear during big runs while still allowing interactive
and gallery usage.

Fixes #1977

Run Details

6 of 8 new or added lines in 3 files covered. (75.0%)

1 existing line in 1 file now uncovered.

27067 of 30053 relevant lines covered (90.06%)

0.9 hits per line