-
Notifications
You must be signed in to change notification settings - Fork 79
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
Change CESM's default aoflux_grid to xgrid #502
Comments
Yes, I agree - can we do this as part of the beta04 tag? |
That sounds good as a goal. I'm thinking that we should wait to change the CESM default until we have an updated ESMF tag, which is still going to be a little while before it's ready, so the timing will depend on that. My thinking is that, even though the standard CESM configurations don't seem to need this ESMF fix, there are probably some less-common configurations that will trigger the error you reported last year, so it will be best to have a fix in place in ESMF before we switch over to using xgrid by default... but I don't feel strongly about that plan if you or others would like to move ahead sooner with the switch to xgrid by default. |
When we make this change, I feel we should introduce at least one test – probably in aux_cmeps, but maybe in CESM's prealpha / prebeta test suite – that tests with aoflux_grid = ogrid, to make sure that option continues to work. |
We should also resolve #510 . (I have added this to the initial comment above.) |
We want to use the exchange grid by default for the aoflux_grid.
There have been some issues getting this to work in all cases, but we seem to be getting closer to having this working. Specifically:
After those three pieces are ready and we have updated CESM to point to an ESMF development snapshot with the fix noted above, I think we should switch the default to aoflux_grid to "xgrid" so that we are testing with that configuration. @jedwards4b do you agree with this timing?
The text was updated successfully, but these errors were encountered: