Updates for cycling w/o IAU#445
Merged
KateFriedman-NOAA merged 3 commits intoSep 23, 2021
Merged
Conversation
- if-block that changed LATB_CASE to "190" when LATB_CASE is "192" is removed - if-block was needed previously to handle missing fix file for this case - new fix set includes symlinks to redirect fix files for certain resolutions - LATB_CASE change created forecast output on the wrong grid dimensions; removing block resolves that Refs: NOAA-EMC#427
- add WRITE_NSFLIP variable to parsing_model_configure_FV3.sh with default set to .false. - add WRITE_NSFLIP variable to config.base and set to .true. - running with WRITE_NSFLIP=.true. writes latitudes in current GFSv16 order (positive to negative) - running with WRITE_NSFLIP=.false. writes latitudes negative to positive and causes issues with the GSI Refs: NOAA-EMC#427
…ture/coupled_sprint_cycling Refs: NOAA-EMC#427
KateFriedman-NOAA
requested review from
WalterKolczynski-NOAA and
aerorahul
WalterKolczynski-NOAA
approved these changes
Sep 23, 2021
kayeekayee
pushed a commit
to kayeekayee/global-workflow
that referenced
this pull request
May 30, 2024
…-EMC#445) Clean up the exponential cloud overlapping method in RRTMG, which reflects the practice of the pre-2018 operational HWRF model.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Tested feature/coupled_sprint on WCOSS-Dell atmos-only with IAU turned off. Successfully cycled 5.5 cycles at C192C96L127 with IAU off:
Mars: /gpfs/dell2/emc/modeling/save/Kate.Friedman/expdir/fcscycnoiau
Issue with IAU has been resolved in ufs-community/ufs-weather-model#782. Will test with IAU on when feature/coupled_sprint reaches the associated ufs-weather-model hash (ufs-community/ufs-weather-model@29fa453).
Thankfully only minimal updates were needed so far to cycle feature/coupled_sprint with recent forecast updates:
WRITE_NSFLIPvariable and set it to.true.(default is.false.)to get latitudes written out in GFSv16 orderRef: #427