Cycling over validity time #395
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
jfrost-mo
force-pushed
the
322_cycling_over_validity_time
branch
from
11c5e9f
to
f284625
Compare
This comment was marked as resolved.
This comment was marked as resolved.
jfrost-mo
force-pushed
the
361_include_file_looping
branch
from
3f5fc96
to
d6fcccb
Compare
jfrost-mo
force-pushed
the
322_cycling_over_validity_time
branch
2 times, most recently
from
4dfb37c
to
57a4349
Compare
jfrost-mo
force-pushed
the
361_include_file_looping
branch
from
d6fcccb
to
9a0181d
Compare
This comment was marked as resolved.
This comment was marked as resolved.
jfrost-mo
force-pushed
the
361_include_file_looping
branch
from
9a0181d
to
4dd3dbb
Compare
jfrost-mo
force-pushed
the
322_cycling_over_validity_time
branch
2 times, most recently
from
fe9c469
to
0edfd5d
Compare
This command derives the ID from the title of the recipe _after_ templating has taken place.
jfrost-mo
force-pushed
the
322_cycling_over_validity_time
branch
from
0edfd5d
to
3f2c4ae
Compare
jfrost-mo
force-pushed
the
322_cycling_over_validity_time
branch
3 times, most recently
from
7e04d52
to
ffb4975
Compare
Update tests and docs for cset cookbook
Also adds housekeeping for plot intermediates
This runs the post-steps for a recipe inside the output directory.
jfrost-mo
commented
Mar 19, 2024
jfrost-mo
commented
Mar 19, 2024
cset-workflow/includes/deterministic_plot_surface_air_temp.cylc
Outdated
Show resolved
Hide resolved
This means the include files only need to contain the runtime sections for the recipes. In future we might even be able to avoid that.
Centralise scheduling into flow.cylc
It requires quite a lot of extra per-filename setup, which is only needed for large runs.
This means that adding a new recipe does not need to touch flow.cylc Fixes #460 This required adding a glob jinja global, and fixing some workflow loops caused by certain recipes.
Break out workflow running commands into code blocks Fixes #236
On the front page of the docs and the README. Code dates can be done when those files are next changed.
Improve workflow documentation
Strings must be quoted in Jinja
JorgeBornemann
approved these changes
Apr 8, 2024
jfrost-mo
commented
Apr 8, 2024
Co-authored-by: JorgeBornemann <32248422+JorgeBornemann@users.noreply.github.com>
|
With all the comments now being address, or moved out into seperate issues, I'll go ahead and merge this. I'm aiming to get a release out next week, which should give us time to clean up any last bit from this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Based on #387
Fixes #322
To do