-
Notifications
You must be signed in to change notification settings - Fork 46
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
clean up "mpas15-3" grid entries #153
Comments
Wait @adamrher I'm not clear what you are suggesting. Sam's suggestion was to change mpas15-3conus to mpas15-3. I think the way to go is to change mpas15-3 to mpas15-3conus. Either way should work, but it has impact on how models will refer to the grid. I'm thinking we need to do the later so only mpas15-3conus is used. Do you agree with that? |
Yes, I agree. I guess I wasn't clear on that point. |
The other thing I should have asked is who is going to do this? I'm assuming you will. And once it's done you'll let us know we'll integrate into both CAM and CTSM. Does that sound right? |
Yeah I can do a PR that fixes this and #152. What is to "integrate into both CAM and CTSM" ? We are not supporting this grid in CESM3.0, and neither mpasa15-3 nor mpasa15-3conus are referenced anywhere in CAM. Are they referenced in CTSM anywhere? |
Oh, I was going to say it's in the Grids file. But, now I notice it has a question mark... mpasa15-3conus? So this is just something that needs to happen post CESM3. That helps to know. Thanks... |
Yep. There is no X in the "Install in CESM3" column for 15-3conus grid. But there is a X? for uniform 15 km ... that's my next task to figure out. |
@slevis-lmwg notes:
I responded
This
mpasa15-3conus
grid is not installed in any other externals. I checked with Bill Skamarock and he said the origins of this partial grid install occurred during a time when we were more aspirational in getting these big grids running in CESM. This grid is not runnable at present, but maybe it might be in the future so I'd just recommend fixing the inconsistencies and not removingmpasa15-3conus
altogether.@PeterHjortLauritzen @ekluzek
The text was updated successfully, but these errors were encountered: