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

Get rid of DRIPCG #850

Merged
merged 3 commits into from
Jan 16, 2024
Merged

Get rid of DRIPCG #850

merged 3 commits into from
Jan 16, 2024

Conversation

CoryMartin-NOAA
Copy link
Contributor

A little (tall?) birdie has informed us that we apparently should not be using DRIPCG, and seems we are using DRIPCG in our tests. This PR removes that. Note that the global-workflow will likely need modified in config.atmanl to use the different YAML name.

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 straight forward. Yes, g-w parm/config.gfs/config.atmanl will need to replace

export ATMVARYAML=${HOMEgfs}/sorc/gdas.cd/parm/atm/variational/3dvar_dripcg.yaml

with

export ATMVARYAML=${HOMEgfs}/sorc/gdas.cd/parm/atm/variational/3dvar_pcg.yaml

One question. Why do we switch to PCG instead of DRPCG. I thought the problem was with the I (inexact) part of the PCG. What do JCSDA and GMAO use for the variational DA runs?

@CoryMartin-NOAA
Copy link
Contributor Author

Good question @RussTreadon-NOAA , and I have no good answer! I don't even know why we started with DRIPCG besides that is what a lot of the JCSDA test YAMLs use.

@RussTreadon-NOAA
Copy link
Contributor

@shoyokota may have some insights regarding which minimizer, among various options, we should consider using. Of course, we should test various minimizers in single obs, stand-alone, and cycled modes. Doing so is not a pre-requisite for this PR.

@CoryMartin-NOAA
Copy link
Contributor Author

@DavidNew-NOAA is going to test a few in the next week or so for his AMS poster. But that will be single analysis testing and not cycling. I think GMAO is using DRPCG, or at least Ricardo has a PR to OOPS to fix DRPCG and DRIPCG.

andytangborn
andytangborn previously approved these changes Jan 12, 2024
Copy link
Collaborator

@andytangborn andytangborn left a comment

Choose a reason for hiding this comment

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

Seems simple enough. I can run a single analysis time experiment to see what the impact is going to be for aerosol DA.

@RussTreadon-NOAA
Copy link
Contributor

FYI: Ricardo's OOPS issue is #2475. The corresponding PR is #2476.

DavidNew-NOAA
DavidNew-NOAA previously approved these changes Jan 12, 2024
Copy link
Collaborator

@DavidNew-NOAA DavidNew-NOAA left a comment

Choose a reason for hiding this comment

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

Seems pretty straightforward. I'll get some data on algorithm performance next week with the experiments for my AMS poster.

@CoryMartin-NOAA
Copy link
Contributor Author

@DavidNew-NOAA found that PCG is slow, and NRL is apparently using DRPCG, so I've updated to use DRPCG.

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.

Looks good to me. Approve.

@RussTreadon-NOAA
Copy link
Contributor

g-w PR #2225 was just opened to update the GDASApp hash cloned by g-w. If this PR is merged into develop before #2225 is acted upon, I'll update the GDASApp hash to include changes from this PR.

@CoryMartin-NOAA
Copy link
Contributor Author

Thanks @RussTreadon-NOAA you beat me to it! I think there may be some minor changes that will need to be made to g-w config files when this PR is merged. Like the 3dvar YAML template for example.

I'll merge this now.

@CoryMartin-NOAA CoryMartin-NOAA merged commit 5056169 into develop Jan 16, 2024
5 checks passed
@CoryMartin-NOAA CoryMartin-NOAA deleted the feature/no-more-dripcg branch January 16, 2024 16:38
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.

4 participants