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

Make job names consistent. #586

Merged
merged 3 commits into from
Jan 18, 2022

Conversation

aerorahul
Copy link
Contributor

@aerorahul aerorahul commented Jan 16, 2022

Currently, there is a inconsistency in the line #PBS -N, where sometimes one of the following is used to attribute the jobname.

  • #PBS -N gdas_ - if this job belongs to the gdas family
  • #PBS -N gfs_ - if this job belongs to the gfs family
  • #PBS -N %RUN%_ - Lets the job inherit from the variable defined in the suite (and this variable is ( and has to be) defined in the suite.

This PR:

  • removes the hard-wired nature of gdas or gfs and uses %RUN%. This will also open the door to unify several of the ecf scripts from the gdas and gfs families. For e.g. several of the gdas and gfs families can share the same ecf script, with differences sometimes in resources (if at all). And those differences can be moved up to the suite definition (if desired).

This work is part of #398

Including @WeiWei-NCO for feedback

…om %RUN%. This commit uses %RUN% to make it consistent and possibly will open the door for further consolidation between gdas and gfs families
Copy link
Contributor

@WalterKolczynski-NOAA WalterKolczynski-NOAA left a comment

Choose a reason for hiding this comment

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

Looks fine except the noted question about keeping gdas after %RUN% has been added.

ecf/scripts/enkfgdas/analysis/create/jenkfgdas_diag.ecf Outdated Show resolved Hide resolved
ecf/scripts/enkfgdas/analysis/create/jenkfgdas_update.ecf Outdated Show resolved Hide resolved
ecf/scripts/enkfgdas/analysis/recenter/jenkfgdas_sfc.ecf Outdated Show resolved Hide resolved
ecf/scripts/enkfgdas/forecast/jenkfgdas_fcst.ecf Outdated Show resolved Hide resolved
ecf/scripts/enkfgdas/post/jenkfgdas_post_master.ecf Outdated Show resolved Hide resolved
Copy link
Member

@KateFriedman-NOAA KateFriedman-NOAA left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for making this update @aerorahul !

@KateFriedman-NOAA KateFriedman-NOAA merged commit 3a34176 into feature/ops-wcoss2 Jan 18, 2022
@KateFriedman-NOAA KateFriedman-NOAA deleted the feature/ecf_jobnames branch January 18, 2022 18:33
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