New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closing all the plots that ARMI opens #1698
Conversation
I think a lot of ARMI PRs are failing right now because the unit tests aren't cleaning up after themselves. In particular, after all the tests are run, a file named This should be remedied. |
Interesting that it looks like it's only happening on windows and not linux |
So, matplotlib released a new version to PyPI.org today. This removed (or changed?) the old color map logic we are using: armi/armi/utils/reportPlotting.py Line 732 in 872e2da
You will see |
@@ -404,13 +404,12 @@ def plotFaceMap( | |||
) | |||
elif referencesToKeep: | |||
# Don't show yet, since it will be updated. | |||
plt.close(fig) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please Delete this one. it cant close with plotBlockDepthMap
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ope! @john-science I guess we didn't test this internally
What is the change?
This PR makes sure that matplotlib plots in ARMI are closed.
Why is the change being made?
Someone recently complained to me that during their recent ARMI Application runs, they were getting
matplotlib
warnings that they had more than 20 plots open at a time.I believe this is the result of an old design idea that ARMI should traffic in figure objects, passing them around freely. Maybe it was a nice idea, we could discuss, but I think it leaves open the real possibility that plots aren't closed when they should be.
Checklist
doc
folder.pyproject.toml
.