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

consistent namelist cires_ugwp_nml for ugwpv0 and ugwpv1 #128

Open
junwang-noaa opened this issue Nov 14, 2023 · 3 comments
Open

consistent namelist cires_ugwp_nml for ugwpv0 and ugwpv1 #128

junwang-noaa opened this issue Nov 14, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@junwang-noaa
Copy link

junwang-noaa commented Nov 14, 2023

Description

Both ugwpv0 and ugwpv1 are using the same namelist cires_ugwp_nml, but they define the namelist in their own subroutines cires_ugwp_module.F90 and cires_ugwpv1_module.F90. The same namelist contains different variables, which causes confusions and issues in up level configurations/templates/script that need to test the two ugwp schemes. E.g. lauch_level is a namelist variable for ugwpv0, but is a hardcoded variable for ugwpv1. The possible solutions are:

  1. only define one namelist cires_upwp_nml, and this namelist will be used by both ugwp schemes.
  2. ugwpv0 and ugwpv1 define their own namelist with different namelist names.

The impact to top level namelist template can be seen in UFS weather model PR#1965:
ufs-community/ufs-weather-model#1941 (comment)

which is confusing and error-prone.

Steps to Reproduce

Please provide detailed steps for reproducing the issue.

  1. namleist in ugwpv0:
    namelist /cires_ugwp_nml/ knob_ugwp_solver, knob_ugwp_source,knob_ugwp_wvspec, knob_ugwp_azdir, &
  2. namelist in ugwpv1:
    namelist /cires_ugwp_nml/ knob_ugwp_solver, knob_ugwp_source,knob_ugwp_wvspec, knob_ugwp_azdir, &
  3. seeing differences between 1) and 2).

Additional Context

Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

  • Machine
  • Compiler
  • Suite Definition File or Scheme
  • Reference other issues or PRs in other repositories that this is related to, and how they are related.

Output

Please include any relevant log files, screenshots or other output here.

@junwang-noaa junwang-noaa added the bug Something isn't working label Nov 14, 2023
@SamuelTrahanNOAA
Copy link
Collaborator

I prefer option 1. Make a new cires_ugwp_namelist.F90 and have it read the namelist. Any ugwp schemes use that module to read namelists.

@junwang-noaa
Copy link
Author

@mdtoyNOAA @yangfanglin @Qingfu-Liu FYI.

@lisa-bengtsson
Copy link
Collaborator

I'm preparing a PR to the global workflow HR3 tag with the convection namelist changes, will commit to Jessica Mexiner's HR3 working branch once this is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants