-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feature/plotting tools #13
Conversation
Clean up
Daint slurm
…gration into feature/plotting_tools
launch jenkins |
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.
Thanks for polishing the plotting tools!
|
||
# Reuse infrastructure to read in grid variables | ||
# add path to integration test to reuse existing grid logic | ||
sys.path.append( |
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.
[OPTIONAL] This is the daint path you had, I realized that I also had it hard coded to the path in the docker container. Should we have an environment flag IS_DOCKER to switch between the two? Or maybe relative to this path?
I realize that this will be eliminated fairly soon so probably not worth it.
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.
I am going to pass on refactoring this, because it'll be gone soon and because I think this script will be use on daint
mostly to get the data you want quickly (and on all arch)
@@ -0,0 +1,12 @@ | |||
from PIL import Image |
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.
Since we said in Readme that we don't include plotting requirement, this is probably okay. But is this package in the fv3viz
environment?
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.
No it's not, it's more a small utility to make gif from the command line
Insure model output directory is created
Feature/logger
Standing
cli
scriptExample to run
Pace
on daint.Plotting tools