Skip to content

fix the MPASSIT build failure and lbc task crashes on Hera; allow setting up RRFSv2x and RRFSv2x_ens directly out-of-the-box; add the SAT_USELIST option to turn on/off radiance DA explicitly#1559

Merged
MatthewPyle-NOAA merged 13 commits into
NOAA-EMC:rrfs-mpas-jedifrom
guoqing-noaa:det_enkf_sat
Jun 22, 2026

Conversation

@guoqing-noaa

@guoqing-noaa guoqing-noaa commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

This PR mostly includes changes needed by RRFSv2x and RRFSv2x_ens as well as @keenaneure and @HaidaoLin-NOAA's TMS data assimilation expriments.

  1. Similar to PR hotfix to enable building MPASBlend and RRFS_UTILS on Hera #1555, this PR fixes the MPASSIT build failure on Hera.

  2. Update the MPAS-Model hash to fix the lbc task crashes on Hera. Check hotfix - safeguard invalid scalar indices to prevent segmentation faults during LBC/IC generation ufs-community/MPAS-Model#263 for details.

  3. @chunhuazhou has requested for a while to be able to set up RRFSv2x and RRFSv2x_ens directly out-of-the-box without manually editing convinfo/satinfo for the deterministic and ensemble runs one by one (lots of times, getkf and jedivar assimilate different set of observations). This PR adds this capability to allow the coexistence of convinfo/satinfo.${NET}[_ens] under the workflow/ directory to facilitate setting up multiple experiments with minimum effort.

  4. Base on the recent conversations with @HaidaoLin-NOAA @keenaneure and our summer interns (Ethan Chang, etc), this PR also streamlines how to turn on/off radiance DA for selected sensors. An option SAT_USELIST is introduced so that now one can explicitly turn on/off given satellite sensors for DA in the exp file. And then satinfo is copied to expdir by default and is only used to update sensor channel information. This provides more convenience that even one does not set up radiance DA at the time of running setup_rocoto.py, one can still easily turn on radiance DA by just modifying SAT_USELIST in therrfs.xml file without rerunning the setup process. This also allows one to do preliminary tests of new satellite sensors without updating satinfo first (although adding channel information to satinfo is still preferred in the long run).

@guoqing-noaa

Copy link
Copy Markdown
Contributor Author

The XML differences are all due to the newly added SAT_USELIST and hence are expected.

+  <envar><name>SAT_USELIST</name><value></value></envar>

@MatthewPyle-NOAA MatthewPyle-NOAA merged commit 993cf4c into NOAA-EMC:rrfs-mpas-jedi Jun 22, 2026
8 checks passed
@guoqing-noaa guoqing-noaa deleted the det_enkf_sat branch June 22, 2026 16:36
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