Skip to content

Point jcb-gdas to Argo and ADT QC updates#2111

Merged
guillaumevernieres merged 4 commits intorelease/gfs.v17from
feature/argo_adt_qc
Apr 21, 2026
Merged

Point jcb-gdas to Argo and ADT QC updates#2111
guillaumevernieres merged 4 commits intorelease/gfs.v17from
feature/argo_adt_qc

Conversation

@JohnSteffen-NOAA
Copy link
Copy Markdown
Collaborator

@JohnSteffen-NOAA JohnSteffen-NOAA commented Apr 7, 2026

Description

This PR brings in jcb-gdas configuration changes to marine QC for Argo and ADT (altimeters) observations to remove bad or biased obs from assimilation.

Update
Bugfixes added to the recentering time offset and ice thickness rescaling
Additional files generated for marine ensemble mean to be archived
IODA lat/lon check included

Companion PRs

https://github.com/NOAA-EMC/jcb-gdas/pull/233

Issues

Resolves #2100
Refs #2096

Automated CI tests to run in Global Workflow

  • atm_jjob
  • C96C48_ufs_hybatmDA
  • C96C48_hybatmsnowDA
  • C96_gcafs_cycled
  • C48mx500_3DVarAOWCDA
  • C48mx500_hybAOWCDA
  • C96C48_ufsgsi_hybatmDA
  • C48_ufsenkf_atmDA
  • C96C48_hybatmDA

@JohnSteffen-NOAA JohnSteffen-NOAA self-assigned this Apr 7, 2026
@JohnSteffen-NOAA JohnSteffen-NOAA added draft draft PR often used for discussion DO NOT MERGE PR is not ready to be merged yet gfsv17 labels Apr 7, 2026
@JohnSteffen-NOAA JohnSteffen-NOAA marked this pull request as draft April 7, 2026 17:37
…es the jcb-gdas hash and includes marine ensemble mean for archiving
@JohnSteffen-NOAA JohnSteffen-NOAA force-pushed the feature/argo_adt_qc branch 2 times, most recently from 0588228 to f8e9d89 Compare April 21, 2026 14:23
@guillaumevernieres guillaumevernieres marked this pull request as ready for review April 21, 2026 18:21
Copy link
Copy Markdown
Collaborator

@shlyaeva shlyaeva left a comment

Choose a reason for hiding this comment

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

See a comment below, I think we need a different commit for ioda.

Comment thread build.sh
Comment on lines +144 to +148
# Hack OOPS one line change to harden code for operations,
# remove once the change is merged into the JCSDA develop branch and we can update the submodule
sed -i 's| ASSERT(spaces_.size() >0);| // ASSERT(spaces_.size() >0);|' $dir_root/sorc/oops/src/oops/base/ObsSpaces.h


Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

My eyes are closed for this part.

Comment thread sorc/ioda
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think we need a different commit: https://github.com/JCSDA/ioda/commits/release/2.11/, commit JCSDA/ioda@724af19.

@guillaumevernieres guillaumevernieres merged commit 32d2cce into release/gfs.v17 Apr 21, 2026
5 checks passed
@guillaumevernieres guillaumevernieres deleted the feature/argo_adt_qc branch April 21, 2026 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO NOT MERGE PR is not ready to be merged yet draft draft PR often used for discussion gfsv17

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants