Skip to content
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

Batch mode #119

Merged
merged 7 commits into from
Jul 5, 2023
Merged

Batch mode #119

merged 7 commits into from
Jul 5, 2023

Conversation

pshriwise
Copy link
Collaborator

These changes allow the plotter to be used in a batch mode where plot view files (.pltvw) files can be passed to the plotter in the command line to generate images without opening a GUI. This can be handy for figure regeneration of pre-determined model views when the model is still being iterated upon.

$ openmc-plotter -b view1.pltvw view2.pltvw ..

I've added a couple of tests to make sure this feature works as expected and did a little housekeeping there otherwise as well.

Copy link
Contributor

@kkiesling kkiesling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I understand this correctly, the plot view files have to be generated from the GUI (ie saved?) and then they can be used when reopening? They can't be manually made and then passed in right? Maybe add some info to the README about how these files are created as well. Right now it just has info on how to use them assuming they exist.

@kkiesling
Copy link
Contributor

btw this is a fantastic addition and will be very useful!

@pshriwise
Copy link
Collaborator Author

If I understand this correctly, the plot view files have to be generated from the GUI (ie saved?) and then they can be used when reopening? They can't be manually made and then passed in right? Maybe add some info to the README about how these files are created as well. Right now it just has info on how to use them assuming they exist.

That's correct, yep. I'll add a link in the README in the new section on batch mode to the section on saving and exporting.

@kkiesling
Copy link
Contributor

Just wanted to let you know I did not forget about this PR. I just want to test it locally before approving/merging, but my conda/python environments are not happy at the moment. Sorry for the delay!

@shimwell
Copy link
Member

shimwell commented Jul 5, 2023

Is this one ready to merge

@kkiesling
Copy link
Contributor

Is this one ready to merge

Oops despite my last comment I did forget about this [face palm] - I will review today!

@kkiesling
Copy link
Contributor

I tested it out and it indeed does work. Very cool! Merging, thanks @pshriwise!

@kkiesling kkiesling merged commit 14db121 into openmc-dev:develop Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants