Closed mark-petersen closed 6 years ago
@jonbob, @vanroekel and @maltrud, I'm out this week. I ran all my standard tests in MPAS-Ocean on this and it passed. If it works in ACME, any of you can go ahead and merge it.
@jonbob the changes look good to me visually. Could you run the domain decomposition and threading tests for salinity restoring to verify no unforeseen issues arise in E3SM?
@mark-petersen, @vanroekel, @jonbob i tested Mark's branch using GMPAS-OIECO-OIDMS-IAF_oEC60to30v3 with both eco forcing and SSS restoring and it appears to work fine. the restart files have both of the correct forcing group names, and it also restarted without a problem. i didn't check exact restart but hope to do that fairly soon. but i'd say don't wait for that.
Testing within E3SM:
Continued testing:
I'll try testing with just BGC and then also with just salinity restoring
Continued testing:
Continued testing:
Obviously a problem with threading and BGC. Do we want to continue with this PR and open an issue for this? Or put this PR on hold and fix here? @maltrud ? @mark-petersen ?
I'd like to merge this as-is because it adds back some previously removed lines that are needed and pass the ERS test with both salinity restoring and BGC. Then let's look at the BGC threading problem separately. @jonbob or @maltrud if you agree, go ahead and approve this PR.
@mark-petersen - as soon as you want to get this into mpas-dev, we can make an ACME PR as well
i agree. i thought i had caught all of the threading issues with BGC, but looks like not.
This change is required to run with multiple forcing groups in ACME, like both salinity restoring and BGC. They were put in by Adrian in PR #1333, but then removed inadvertently in PR #1427.