-
Notifications
You must be signed in to change notification settings - Fork 5
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
Re-instate hofx scaling in observation operators. #328
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These definitely fix the issue w/ JoJc getting to be negative - this no longer happens; 3DFGAT runs to the end and produces a healthy increment.
Now, I seem to be seeing more difference w/ GSI results than I was seeing before ... I will make some plots and illustrate this elsewhere. I think for the purposes here - the changes should be taken and move to develop. The difference in increments is another issue.
This is good by me ... and btw: did pull the latest JEDI to be in sync w/ the yaml adjustments here.
Thanks for the hard work JJ - and I know Fabio was heavily involved too. Thanks to both of you.
I took a quick look of the definition of the |
@Dooruk I don't see any failed tests in my JEDI built yesterday or today. I suggest you have another run after checking out the latest JEDI. |
Ok, triggering another with the last evening build. |
@gmao-jjin3 it failed again, there is just minor difference in |
Okay. Updated some benchmark numbers. |
Thanks @gmao-jjin3 that worked. I can't tag your github handle but Fabio thank you for your help! |
Add back hofx scaling in linear and non-linear observation operators for convectional data.
Dependencies
Solves the issue #304