-
Notifications
You must be signed in to change notification settings - Fork 167
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
Update rocoto task names #2981
base: develop
Are you sure you want to change the base?
Update rocoto task names #2981
Conversation
Updates the GEFS task names to include the `$RUN` so they are consistent with the task names in the GFS system. This is one step towards unifying the systems where possible. Also adds an underscore between the RUN and task name for *all* tasks to make them more readable.
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.
Changes look good, nice QOL updates, thanks @WalterKolczynski-NOAA !
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.
Changes look good.
One feature request from this would be to have more logical ensemble metatask names. Since the task names include a number, it seems easy enough to just name the metatasks after the root name of the task (see comments). Would this be an easy add to this PR? If not, I'd be happy to open a follow-on issue.
@@ -2680,7 +2680,7 @@ def _get_ecengroups(): | |||
'maxtries': '&MAXTRIES;' | |||
} | |||
|
|||
metatask_dict = {'task_name': f'{self.run}ecmn', | |||
metatask_dict = {'task_name': f'{self.run}_ecmn', |
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.
Not something necessarily for this PR, but it would be nice to have simpler metatask names, like f'{self.run}_ecen'
.
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.
Improving the task names is planned as a next step.
@@ -2851,7 +2851,7 @@ def _get_eposgroups(epos): | |||
'maxtries': '&MAXTRIES;' | |||
} | |||
|
|||
metatask_dict = {'task_name': f'{self.run}epmn', | |||
metatask_dict = {'task_name': f'{self.run}_epmn', |
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.
Ditto, epos
.
@@ -2890,7 +2890,7 @@ def earc(self): | |||
'maxtries': '&MAXTRIES;' | |||
} | |||
|
|||
metatask_dict = {'task_name': f'{self.run}eamn', | |||
metatask_dict = {'task_name': f'{self.run}_eamn', |
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.
earc
.
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.
What @DavidHuber-NOAA said.
An alternative/extension, e.g. eamn -> earcmn
to go with earc#grp#
When I do come to updating the task names, it's going to be to real words like |
Description
Updates the GEFS task names to include the
$RUN
so they are consistent with the task names in the GFS system. This is one step towards abstracting out common elements.Also adds an underscore between the RUN and task name for all tasks to make them more readable.
Type of change
Change characteristics
How has this been tested?
Checklist