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

Convert v16.3.8 to satingest update instead of obsproc #1804

Merged
merged 5 commits into from
Aug 18, 2023

Conversation

ADCollard
Copy link
Contributor

Description

This is a temporary fix to allow the satingest changes to be implemented on August 29 without causing GSI to crash. Issue with the CRTM means that GSI will crash unless NOAA-21 radiances are switched off.

Obsproc version numbers should remain unchanged unless the obsproc changes are implemented before or at the same time as the satingest changes.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Single cycle test on Hera. Results are unchanged.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes need updates to the documentation. I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing tests pass with my changes
  • Any dependent changes have been merged and published

@WalterKolczynski-NOAA WalterKolczynski-NOAA changed the title Release/gfs.v16.3.8 Convert release/gfs.v16.3.8 to satingest update instead of obsproc update Aug 17, 2023
@WalterKolczynski-NOAA WalterKolczynski-NOAA changed the title Convert release/gfs.v16.3.8 to satingest update instead of obsproc update Convert v16.3.8 to satingest update instead of obsproc update Aug 17, 2023
@WalterKolczynski-NOAA WalterKolczynski-NOAA changed the title Convert v16.3.8 to satingest update instead of obsproc update Convert v16.3.8 to satingest update instead of obsproc Aug 17, 2023
Copy link
Contributor

@WalterKolczynski-NOAA WalterKolczynski-NOAA left a comment

Choose a reason for hiding this comment

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

Looks fine to me.

@WalterKolczynski-NOAA WalterKolczynski-NOAA self-assigned this Aug 17, 2023
@WalterKolczynski-NOAA
Copy link
Contributor

Please also create a new issue to update exglobal_atmos_analysis.sh in develop when you get the chance (all the scripts were moved to global-workflow).

@ADCollard
Copy link
Contributor Author

@WalterKolczynski-NOAA Would you be able to merge this PR and tag with https://github.com/NOAA-EMC/global-workflow/releases/tag/EMC-v16.3.8? This needs to be part of @ilianagenkova 's satingest implementation (scheduled for August 29).

I see this is failing the readthedocs test. Is this expected behavior (I cannot see how my changes would cause this)

I will not update exglobal_atmos_analysis.sh in develop as this should only be a temporary change.

@WalterKolczynski-NOAA
Copy link
Contributor

RTD building was added long after the v16 branch, so just ignore it.

I was waiting for @ilianagenkova to approve before merging. If this is all that is needed for the release, I will then tag it. But it should be tested before I tag it.

@ADCollard
Copy link
Contributor Author

@WalterKolczynski-NOAA Thanks!

I just added @RussTreadon-NOAA as @ilianagenkova just got on a plane to Bulgaria! I spoke with Iliana by phone and she was happy with what I said I had done.

I am running one full cycle test. I will confirm tomorrow with you that it works.

Add information to the release notes, including the addition of PlanetIQ and NOAA-20 OMPS assimilation.
@ADCollard
Copy link
Contributor Author

@WalterKolczynski-NOAA As you may have seen, I got the go ahead to add a couple of sensor updates to this PR. So I really want to test it with those changes before you tag. I am waiting for a new GSI tag to be created right now (need some approvals).

I am going to move ahead with the CDF this afternoon and will assume that tag/EMC-v16.3.8 will be available before NCO take a look.

Thanks.

Copy link
Contributor

@aerorahul aerorahul left a comment

Choose a reason for hiding this comment

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

looks ok to me.

@aerorahul aerorahul merged commit 28a58f0 into NOAA-EMC:release/gfs.v16.3.8 Aug 18, 2023
1 check failed
Copy link
Contributor

@RussTreadon-NOAA RussTreadon-NOAA left a comment

Choose a reason for hiding this comment

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

Changes are consistent with information presented in this PR and GSI PRs #611 and #612.

@WalterKolczynski-NOAA
Copy link
Contributor

@ADCollard If more changes need to be made for this release, please open a new PR with the additional changes. Then once the final version is tested we'll tag it.

@ADCollard
Copy link
Contributor Author

@WalterKolczynski-NOAA @aerorahul Can you tag https://github.com/NOAA-EMC/global-workflow/tree/release/gfs.v16.3.8 as EMC-v16.3.8? I have promised this to NCO by Monday morning!

Should I start a new issue or repurpose #1356 which concerns an upgrade that was originally going to be 16.3.8, but has since slipped?

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.

None yet

5 participants