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

Update feature/coupled-crow to run the latest ufs-weather-model with prognostic aerosols #338

Closed
Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
b7da671
Checkout and build ufs-weather-model app integrating
rmontuoro Jun 8, 2021
5c8e078
Use proper modulefiles for updated ufs-weather-model
rmontuoro Jun 8, 2021
e2f52cc
Update parsing function for FV3 namelist to work
rmontuoro Jun 8, 2021
ff971a2
Add aerosol scavenging factors (fscav_aero) to FV3 settings.
rmontuoro Jun 8, 2021
d16630f
Properly set NST_SPINUP flag in FV3 namelist.
rmontuoro Jun 8, 2021
c26a0dd
Restore `cplchem` as the flag for coupled aerosols,
rmontuoro Jun 8, 2021
9661d1e
- Set NEMS identifier for coupled aerosol component to 'gocart'.
rmontuoro Jun 8, 2021
52dbd17
Update test cases and input files for coupled aerosols runs.
rmontuoro Jun 8, 2021
bc7c20a
Add scavenging factors to aerosol test cases.
rmontuoro Jun 8, 2021
f81741b
Use updated modulefile names only when building the
rmontuoro Jun 10, 2021
0307896
Load updated modulefiles for coupled aerosols on Hera/Intel.
rmontuoro Jun 10, 2021
6b19a39
Remove GSD-specific functions in forecast postdet.
rmontuoro Jun 10, 2021
8a8194f
Remove no longer needed parsing script used to
rmontuoro Jun 10, 2021
a821a1a
Return proper name for Hera platform and load correct modules.
rmontuoro Jun 10, 2021
c5456b5
Load aerosol-specific modulefiles only for the forecast task.
rmontuoro Jun 13, 2021
d4f246a
Merge branch 'WalterKolczynski-NOAA:feature/aerosol_restart' into fea…
rmontuoro Jun 14, 2021
bfe150a
Replace legacy 'cplgocart' flag with 'cplchem'.
rmontuoro Jun 14, 2021
88f0358
Update tracer list.
rmontuoro Jun 14, 2021
521141d
Merge branch 'WalterKolczynski-NOAA:feature/aerosol_restart' into fea…
rmontuoro Jun 14, 2021
2c40cdc
Restore original path to modulefiles in module_base.hera.
rmontuoro Jun 14, 2021
76f8fa2
Merge 'WalterKolczynski-NOAA:feature/aerosol_restart' into feature/up…
rmontuoro Jun 16, 2021
7e84ea4
Update aerosol_firex_forecast test case.
rmontuoro Jun 22, 2021
165c2b7
update wave ww3_multi.inp file Issue #325
JessicaMeixner-NOAA Jun 23, 2021
6f4a3f2
Update UFS model hash
WalterKolczynski-NOAA Jun 22, 2021
d0de272
Update MOM_input with landmask change option
WalterKolczynski-NOAA Jun 22, 2021
56d5597
Update CICE inputs for new model version
WalterKolczynski-NOAA Jun 22, 2021
434d7c6
Update UFS build script
WalterKolczynski-NOAA Jun 22, 2021
e86db23
Update source of fd_nems.yaml
WalterKolczynski-NOAA Jun 22, 2021
12050b8
Update templates to set ocean mesh file
WalterKolczynski-NOAA Jun 23, 2021
d4910b5
Update Ww3 prepost build
WalterKolczynski-NOAA Jun 23, 2021
351f5aa
Update meditaor name
WalterKolczynski-NOAA Jun 24, 2021
88fb063
Update wave IC
WalterKolczynski-NOAA Jun 24, 2021
1957bb9
Update modules to match UFS
WalterKolczynski-NOAA Jun 24, 2021
2dcf8f2
Update ice configuration to match UFS
WalterKolczynski-NOAA Jun 24, 2021
89c3fa0
Merge branch 'feature/coupled-crow' into feature/model_update
WalterKolczynski-NOAA Jun 28, 2021
6a17f52
Merge 'feature/coupled-crow' into feature/update-aerosols
rmontuoro Jul 1, 2021
4fc8131
Add ammonia and nitrates to the list of cycled GOCART tracers.
rmontuoro Jul 1, 2021
acdc6a6
Update checkout and build script to use the authoritative
rmontuoro Jul 1, 2021
a51d4ef
Add spaces to indent FV3 build section for better readability
KateFriedman-NOAA Jul 8, 2021
5bec2a7
Add FV3_GFS_v16_couplednsst CCPP suite option to UFS coupled build an…
KateFriedman-NOAA Jul 8, 2021
bbcad18
Replace CROW yaml nst_spinup_logic in nsst.yaml with NST_SPINUP variable
KateFriedman-NOAA Jul 8, 2021
cec6188
Add NSST settings to the coupled_free_forecast_wave.yaml case file
KateFriedman-NOAA Jul 9, 2021
d303997
Update nems.configure to inclue eps_imesh
WalterKolczynski-NOAA Jul 14, 2021
771832d
Merge branch 'feature/model_update' into feature/update-aerosols
WalterKolczynski-NOAA Jul 14, 2021
dc970f9
Change forecast module back to module_base
WalterKolczynski-NOAA Jul 14, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions parm/parm_fv3diag/diag_table_aer
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@
# chemical tracers advected by FV3
###
"gfs_dyn", "so2", "so2", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "sulf", "sulf", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "so4", "so4", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "dms", "dms", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "msa", "msa", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "bc1", "bc1", "fv3_history", "all", .false., "none", 2
Expand All @@ -145,8 +145,8 @@
"gfs_dyn", "no3an1", "no3an1", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "no3an2", "no3an2", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "no3an3", "no3an3", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "pp25", "pp25", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "pp10", "pp10", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "pm25", "pm25", "fv3_history", "all", .false., "none", 2
"gfs_dyn", "pm10", "pm10", "fv3_history", "all", .false., "none", 2

"gfs_phys", "ALBDO_ave", "albdo_ave", "fv3_history2d", "all", .false., "none", 2
"gfs_phys", "cnvprcp_ave", "cprat_ave", "fv3_history2d", "all", .false., "none", 2
Expand Down
257 changes: 134 additions & 123 deletions parm/parm_fv3diag/field_table_aer
Original file line number Diff line number Diff line change
@@ -1,158 +1,169 @@
# added by FRE: sphum must be present in atmos
# specific humidity for moist runs
"TRACER", "atmos_mod", "sphum"
"longname", "specific humidity"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"longname", "specific humidity"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
# prognostic cloud water mixing ratio
"TRACER", "atmos_mod", "liq_wat"
"longname", "cloud water mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"longname", "cloud water mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"TRACER", "atmos_mod", "rainwat"
"longname", "rain mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"longname", "rain mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"TRACER", "atmos_mod", "ice_wat"
"longname", "cloud ice mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"longname", "cloud ice mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"TRACER", "atmos_mod", "snowwat"
"longname", "snow mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"longname", "snow mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"TRACER", "atmos_mod", "graupel"
"longname", "graupel mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
"longname", "graupel mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
# prognostic ozone mixing ratio tracer
"TRACER", "atmos_mod", "o3mr"
"longname", "ozone mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
# prognostic so2 mixing ratio tracer
"longname", "ozone mixing ratio"
"units", "kg/kg"
"profile_type", "fixed", "surface_value=1.e30" /
# prognostic subgrid scale turbulent kinetic energy
"TRACER", "atmos_mod", "sgs_tke"
"longname", "subgrid scale turbulent kinetic energy"
"units", "m2/s2"
"profile_type", "fixed", "surface_value=1.e30" /
# prognostic chemistry tracers
"TRACER", "atmos_mod", "so2"
"longname", "so2 mixing ratio"
"units", "ppm"
"profile_type", "fixed", "surface_value=5.e-6" /
# prognostic sulfate mixing ratio tracer
"TRACER", "atmos_mod", "sulf"
"longname", "sulfate mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=3.e-6" /
# prognostic DMS mixing ratio tracer
"longname", "so2 mixing ratio"
"units", "ppm"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=5.e-6" /
"TRACER", "atmos_mod", "so4"
"longname", "sulfate mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=3.e-6" /
"TRACER", "atmos_mod", "dms"
"longname", "DMS mixing ratio"
"units", "ppm"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic msa mixing ratio tracer
"longname", "DMS mixing ratio"
"units", "ppm"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "msa"
"longname", "msa mixing ratio"
"units", "ppm"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic bc1 mixing ratio tracer
"longname", "msa mixing ratio"
"units", "ppm"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "bc1"
"longname", "hydrophobic black carbon mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic bc2 mixing ratio tracer
"longname", "hydrophobic black carbon mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "bc2"
"longname", "hydrophillic black carbon mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic oc1 mixing ratio tracer
"longname", "hydrophillic black carbon mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "oc1"
"longname", "hydrophobic organic carbon mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic oc2 mixing ratio tracer
"longname", "hydrophobic organic carbon mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "oc2"
"longname", "hydrophillic organic carbon mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic dust1 mixing ratio tracer
"longname", "hydrophillic organic carbon mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "dust1"
"longname", "fine dust1 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic dust2 mixing ratio tracer
"longname", "fine dust1 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "dust2"
"longname", "fine dust2 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic dust3 mixing ratio tracer
"longname", "fine dust2 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "dust3"
"longname", "coarse dust3 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic dust4 mixing ratio tracer
"longname", "coarse dust3 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "dust4"
"longname", "coarse dust4 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic dust5 mixing ratio tracer
"longname", "coarse dust4 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "dust5"
"longname", "coarse dust5 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic seas1 mixing ratio tracer
"longname", "coarse dust5 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "seas1"
"longname", "seasalt1 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic seas2 mixing ratio tracer
"longname", "seasalt1 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "seas2"
"longname", "seasalt2 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic seas3 mixing ratio tracer
"longname", "seasalt2 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "seas3"
"longname", "seasalt3 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic seas4 mixing ratio tracer
"longname", "seasalt3 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "seas4"
"longname", "seasalt4 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic seas5 mixing ratio tracer
"longname", "seasalt4 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "seas5"
"longname", "seasalt5 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e-7" /
# prognostic NI mixing ratio tracer
"longname", "seasalt5 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e-7" /
"TRACER", "atmos_mod", "nh3"
"longname", "primary NH3 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e+0" /
"longname", "primary NH3 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e+0" /
"TRACER", "atmos_mod", "nh4a"
"longname", "primary NH4a mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e+0" /
"longname", "primary NH4a mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e+0" /
"TRACER", "atmos_mod", "no3an1"
"longname", "primary NO3an1 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e+0" /
"longname", "primary NO3an1 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e+0" /
"TRACER", "atmos_mod", "no3an2"
"longname", "primary NO3an2 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e+0" /
"longname", "primary NO3an2 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e+0" /
"TRACER", "atmos_mod", "no3an3"
"longname", "primary NO3an3 mixing ratio"
"units", "ug/kg"
"profile_type", "fixed", "surface_value=1.e+0" /
# diagnostic pm2.5 tracer
"TRACER", "atmos_mod", "pp25"
"longname", "primary PM25 mixing ratio"
"units", "ug/m3"
"profile_type", "fixed", "surface_value=1.e+0" /
# diagnostic pm10 tracer
"TRACER", "atmos_mod", "pp10"
"longname", "primary PM10 mixing ratio"
"units", "ug/m3"
"profile_type", "fixed", "surface_value=1.e+0" /
"longname", "primary NO3an3 mixing ratio"
"units", "ug/kg"
"tracer_usage", "chemistry"
"profile_type", "fixed", "surface_value=1.e+0" /
# diagnostic chemistry tracers
"TRACER", "atmos_mod", "pm25"
"longname", "primary PM25 mixing ratio"
"units", "ug/m3"
"tracer_usage", "chemistry", "type=diagnostic"
"profile_type", "fixed", "surface_value=1.e+0" /
"TRACER", "atmos_mod", "pm10"
"longname", "primary PM10 mixing ratio"
"units", "ug/m3"
"tracer_usage", "chemistry", "type=diagnostic"
"profile_type", "fixed", "surface_value=1.e+0" /
# non-prognostic cloud amount
"TRACER", "atmos_mod", "cld_amt"
"longname", "cloud amount"
"units", "1"
"profile_type", "fixed", "surface_value=1.e30" /
"longname", "cloud amount"
"units", "1"
"profile_type", "fixed", "surface_value=1.e30" /
8 changes: 1 addition & 7 deletions scripts/exglobal_forecast.sh
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,6 @@ cpl=${cpl:-.false.}
cplflx=${cplflx:-.false.} # default off,import from outside source
cplwav=${cplwav:-.false.} # ? how to control 1-way/2-way?
cplchem=${cplchem:-.false.} # Chemistry model
cplgocart=${cplgocart:-.false.} # Chemistry model
cplice=${cplice:-.false.} # ICE model

OCNTIM=${OCNTIM:-3600}
Expand Down Expand Up @@ -134,7 +133,6 @@ esac
[[ $cplflx = .true. ]] && MOM6_predet
#[[ $cplwav = .true. ]] && WW3_predet #no WW3_predet at this time
[[ $cplice = .true. ]] && CICE_predet
[[ $cplchem = .true. ]] && GSD_predet

case $RUN in
'gfs') FV3_GFS_det;;
Expand All @@ -144,7 +142,6 @@ esac #no run type determination for data atmosphere
[[ $cplflx = .true. ]] && MOM6_det
[[ $cplwav = .true. ]] && WW3_det
[[ $cplice = .true. ]] && CICE_det
[[ $cplchem = .true. ]] && GSD_det

echo "MAIN: RUN Type Determined"

Expand All @@ -159,7 +156,6 @@ esac #no post determination set up for data atmosphere
[[ $cplflx = .true. ]] && MOM6_postdet
[[ $cplwav = .true. ]] && WW3_postdet
[[ $cplice = .true. ]] && CICE_postdet
[[ $cplchem = .true. ]] && GSD_postdet
echo "MAIN: Post-determination set up of run type finished"

echo "MAIN: Writing name lists and model configuration"
Expand All @@ -172,8 +168,7 @@ esac #no namelist for data atmosphere
[[ $cplflx = .true. ]] && MOM6_nml
[[ $cplwav = .true. ]] && WW3_nml
[[ $cplice = .true. ]] && CICE_nml
[[ $cplchem = .true. ]] && GSD_nml
[[ $cplgocart = .true. ]] && GOCART_rc
[[ $cplchem = .true. ]] && GOCART_rc

case $RUN in
'data') DATM_model_configure;;
Expand Down Expand Up @@ -216,7 +211,6 @@ if [ $machine != 'sandbox' ]; then
[[ $cplflx = .true. ]] && MOM6_out
[[ $cplwav = .true. ]] && WW3_out
[[ $cplice = .true. ]] && CICE_out
[[ $cplchem = .true. ]] && GSD_out
[[ $esmf_profile = .true. ]] && CPL_out
else
echo "MAIN: Running on sandbox mode, no output linking"
Expand Down
7 changes: 4 additions & 3 deletions sorc/build_ufs_coupled.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,16 @@ if [ $target = hera ]; then target=hera.intel ; fi
if [ $target = orion ]; then target=orion.intel ; fi
if [ $target = stampede ]; then target=stampede.intel ; fi

MOD_PATH=$cwd/ufs_coupled.fd/modulefiles/$target
MOD_PATH=$cwd/ufs_coupled.fd/modulefiles

module purge
module use $MOD_PATH
module load fv3
module load ufs_${target}
cd ufs_coupled.fd/
if [[ -d build ]]; then rm -Rf build; fi
if [[ -d GOCART ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

So you have switched to using your fork, but as soon as this PR goes into ufs-weather-model this build check will not work anymore. Do we address that now or later?

Copy link
Author

Choose a reason for hiding this comment

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

In the most recent versions of ufs-weather-modelall modulefiles have been renamed as: ufs_<machine>.<compiler_settings>. This change introduces the update and should work from now on.

Copy link
Contributor

Choose a reason for hiding this comment

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

But you didn't update the hash of what is checked out for the ufs-weather-model for the coupled case. Therefore this will break the coupled case as is, unless I missed that.

Copy link
Contributor

Choose a reason for hiding this comment

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

This was my issue as well. The currently non-aerosol coupled-crow ufs model version is still using the old naming convention. So this PR would break that until #324 is also merged.

Copy link
Contributor

Choose a reason for hiding this comment

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

(And vice versa)

Copy link
Author

Choose a reason for hiding this comment

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

You are right. My assumption was that we were going to update to the latest ufs-weather-model for the coupled case, too, before or at the same time as this PR. I will address this issue by adding a conditional block.

Copy link
Contributor

Choose a reason for hiding this comment

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

I hope we will update ufs-weather-model for the other case very soon, but as of now #324 is remains an unassigned task.

Copy link
Author

Choose a reason for hiding this comment

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

Commit f81741b addresses this issue. Updated modulefiles are only loaded to build the ufs-weather-model with prognostic aerosols. This model is identified though a hidden file, created when using option -a during checkout.

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like I'm too late and the complications have already been added, but I'm going to try and tackle #324 after we clear a couple of the other PRs out. So let's cool it with adding more if-block exceptions for aerosols when we want to be removing them.

Copy link
Contributor

Choose a reason for hiding this comment

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

At least until we can discuss it in the afternoon.

./build.sh
module load ufs_aerosols_${target}
CMAKE_FLAGS="-DAPP=ATMAERO" CCPP_SUITES="FV3_GFS_v16" ./build.sh
else
CMAKE_FLAGS="-DS2S=ON -DWW3=ON" CCPP_SUITES="FV3_GFS_v15p2_coupled,FV3_GFS_v16_coupled,FV3_GFS_v16" ./build.sh
fi
4 changes: 2 additions & 2 deletions sorc/checkout.sh
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,9 @@ if [ ${COUPLED:-"NO"} = "NO" ]; then
else
if [[ ! -d ufs_coupled.fd ]] ; then
if [ "${AEROSOL}" = "YES" ] ; then
git clone https://github.com/NOAA-EMC/FV3-GOCART ufs_coupled.fd >> ${topdir}/checkout-ufs_coupled.log 2>&1
git clone https://github.com/rmontuoro/ufs-weather-model.git ufs_coupled.fd >> ${topdir}/checkout-ufs_coupled.log 2>&1
cd ufs_coupled.fd
git checkout develop
git checkout feature/gocart
else
git clone https://github.com/ufs-community/ufs-weather-model ufs_coupled.fd >> ${topdir}/checkout-ufs_coupled.log 2>&1
cd ufs_coupled.fd
Expand Down
Loading