-
Notifications
You must be signed in to change notification settings - Fork 304
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
COMPASS: rename e3sm_coupling and move it to be its own test #585
Comments
As part of this work, I would take out the steps that involve other E3SM components (e.g. generating mapping files to atmosphere and land grids). This work will need to move to a different set of scripts, a different repo, and a different set of tools that is still in the planning stage. |
@jonbob, @vanroekel, @maltrud and @mark-petersen, I'd appreciate your thoughts before I start on this. |
Yes, I agree that moving I don't think we should remove mapping file capability until it is available and tested in another process. I think it still makes sense to create the sea ice initial condition here in COMPASS, since the sea ice mesh is the same MPAS mesh and it is a convenient place to do it. We will also be adding special graph files for load balancing and potentially remove low latitude cells, and I think those steps make sense in this COMPASS system. So rather than name it |
Could you elaborate on why you think these make more sense in COMPASS than in a repo under E3SM-Project? It seems to me that this would be equally convenient to do in the "other" system. And given that MPAS-Seaice has opted out of COMPASS, it seems like its initialization shouldn't be tied to COMPASS if there is an alternative. But I'm open to understanding why it's easier to keep it here.
I'm definitely not good with this. It implies basically the same thing as the old name, namely that it provides everything you need for an E3SM initial condition. It should be clearer that this is just ocean (and maybe sea ice). |
@xylar - I agree with your proposed changes. I think the renaming is consistent with ongoing work in other parts of the system, as is removing the steps that involve other E3SM components. Please let me know if I can help in any way. |
As part of this work, I will add I will remove the runoff metadata from COMPASS because we have decided that will be handled in the new repo in E3SM-Project, so we won't necessarily even know the approach at the COMPASS stage. |
@xylar after some thought I can see the advantages of |
@xylar I think these are good changes and in line with what we are slowly working toward elsewhere. I also agree we go with e3sm_ocean_init |
Moved to MPAS-Dev/compass#11 |
I would like to move the
e3sm_coupling
step to be its own "test" (at the same level asinit
andspin_up
) in eachglobal_ocean
test case.My reasoning for renaming is that we want to make clear that, once this code is more robust, it will generate an E3SM ocean (and possibly sea-ice, though that can be part of this discussion) initial condition. I don't have a great suggestion for the new name but something like
e3sm_ocean_init
.My reasoning for moving it to its own "test" is:
init
and typically also afterspin_up
. If it is its own test, we can use aprerequisite
tag to indicate this, see Add support for a prerequisite tag in COMPASS #559 and Add prerequisite tags to all necessary test cases #560.The text was updated successfully, but these errors were encountered: