Add 1 day reg test that matches a benchmark case#57
Add 1 day reg test that matches a benchmark case#57JessicaMeixner-NOAA wants to merge 8 commits into
Conversation
Update datesuite tests to include 2013 tests update baseline area on hera
|
Jun has asked that we update to the latest post libraries so they can remove the older ones. I see you have an update for conf/fv3 on dell, but can we update fv3_coupled conf files with this commit also? We want post/8.0.6 |
|
@DeniseWorthen yes, we can update all the post libraries on this commit. In addition we will be updating to use the latest fv3atm from PR NOAA-EMC/ufsatm#81 |
|
Jessica,
I thought we will have a clean s2s commit with Moorthi's changes which is
in current fv3atm (PR#77 and PR#44) with wave test added, then we
willcommit Shan's fraction grid change to s2s, is the plan changed?
…On Thu, Mar 19, 2020 at 11:51 AM Jessica Meixner ***@***.***> wrote:
@DeniseWorthen <https://github.com/DeniseWorthen> yes, we can update all
the post libraries on this commit.
In addition we will be updating to use the latest fv3atm from PR
NOAA-EMC/ufsatm#81 <NOAA-EMC/ufsatm#81>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#57 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI7D6TKC5K4W2FYRSUTJ7LTRII5ONANCNFSM4LOSIGPQ>
.
|
|
@junwang-noaa If we do that, then this PR needs to go in without updating to fv3atm to NOAA-EMC/ufsatm#81 and then I need to debug the WW3 build so that the WW3 test can be added here with this current fv3atm. |
|
That is my understanding. So we have:
1) fv3atm PR#44 and #77 with atm<-> wave coupling, in s2s we will have
with atm<->wav coupling, and fv3-mom6-cice5-wav enabled
2) fv3atm PR #81 with fraction grid changes and CCPP updates, in s2s we
will have no baseline results change but have ccpp tests added
I understand that we need a 1 day benchmark test in 2). But I do hope we
get 1) done first. If you'd like to change the plan, please let Bin/Denise
know. Bin supposed to work on merge #81 to s2s, do you want that to be in
your PR? Then we have to resolve this 1 day test difference issue.
…On Thu, Mar 19, 2020 at 12:11 PM Jessica Meixner ***@***.***> wrote:
@junwang-noaa <https://github.com/junwang-noaa> If we do that, then this
PR needs to go in without updating to fv3atm to NOAA-EMC/ufsatm#81
<NOAA-EMC/ufsatm#81> and then I need to debug the
WW3 build so that the WW3 test can be added here with this current fv3atm.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#57 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI7D6TIPFCMZFUFNCPDPX6TRII72TANCNFSM4LOSIGPQ>
.
|
|
I'm closing this commit since PR #60 will break this... I'll make a new PR with an updated baseline. @DeniseWorthen since I have to make an updated baseline (it's made, just need to move it to a "develop-YYYYMMDD" from it's current "update-20200321" spot. Do you want me to add any of the rearrangements that will be needed in upcoming PRs or just go with how it is now? |
|
Don't make extra work for yourself. I'll do it when I commit the cmeps and restart tests. |
This PR closes issue #46