-
Notifications
You must be signed in to change notification settings - Fork 17
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
MAPL develop is non-zero-diff to main - 2024-Sep-11 #3025
Comments
This just getting bizarre. I've reproduced, from what I can tell, the same files are being read, seems be producing the same values. Even more bizarre, the model doesn't to go non-zero diff right away; 4 time steps the model is still zero diff, but then after 30 minutes goes non-zero diff. If I I'm now going to trying running ExtDataDriver.x in the mode where I read in the same emissions as the model, then write them back out immediately with History and see if main and develop produce the same results |
Update: I looked at the nightly tests and it looks like it is AMIP GOCART that has the issue and not Ops:
The Ops emissions test worked. |
I think I figured it out. I think I broke this "flipping" we do in certain cases. It was part of the vertical stuff I said, we don't need cause that's going to be completely redone, well, apparently that was used in some of our current files. And ExtDataDriver.x proves it's worth again |
Good job! |
So I guess what this means is that the ExtData flipping is something we only do in AMIP emissions? Or maybe we got lucky with how the ops is set up? GOCART is still a bit of a black box to me :) |
My nightly tests have informed me that some time late last week, MAPL
develop
became non-zero-diff to MAPLmain
(aka MAPL 2.47). I missed this because of other issues I was having with the tests. Fixed those and new issue exposed.From my testing, the culprit seems to be #3007 . If I build GEOS from commit d888902 from before that PR and compare to MAPL 2.47 it is zero-diff. If I move after that PR (say commit 677ee24), it is non-zero-diff.
The text was updated successfully, but these errors were encountered: