-
Notifications
You must be signed in to change notification settings - Fork 171
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
Archive the experiment directory along with git status/diff output #3105
base: develop
Are you sure you want to change the base?
Archive the experiment directory along with git status/diff output #3105
Conversation
…ories to prevent cleaning other jobs
…lobal-workflow into feature/archive_expdir
Tests were performed on Hercules and the experiment directory tarballs for both cycled and forecast-only modes were found to have been created as expected. Opening for review. |
This .flake8 file was copied from wxflow.
…lobal-workflow into feature/archive_expdir
# Archive the EXPDIR if requested | ||
{% if archive_expdir %} | ||
{% filter indent(width=4) %} | ||
{% include "expdir.yaml.j2" %} | ||
{% endfilter %} | ||
{% endif %} |
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.
Should only need this in one of gdas or gfs, but there is a bit of a coordination problem since either can be run without the other.
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.
Agreed. I added a method to archive.py to determine which RUN
to archive this in and set the archive_expdir
boolean accordingly.
Co-authored-by: Walter Kolczynski - NOAA <[email protected]>
Description
This adds the capability to archive the experiment directory. Additionally, this adds options to run
git status
andgit diff
on theHOMEgfs
global workflow (but not the submodules) and store that information within the experiment directory's archive. These options are specified inconfig.base
with the following defaults:Resolves #2994
Type of change
Change characteristics
How has this been tested?
ARCH_DIFFS=YES
andARCH_EXPDIR_FREQ=6,12
ARCH_EXPDIR=NO
orARCH_HASHES=NO
Checklist