Skip to content
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

MizuRoute lake grids #75

Merged
merged 13 commits into from
Dec 17, 2022
Merged

MizuRoute lake grids #75

merged 13 commits into from
Dec 17, 2022

Conversation

ekluzek
Copy link
Contributor

@ekluzek ekluzek commented Dec 13, 2022

This adds some new grids for mizuRoute lake HDMA-lake option, and also changes the mizuRoute grid aliases to the convention discussed previously.

Fixes #67
Uses the convention as addressed in #70

@ekluzek ekluzek added the enhancement New feature or request label Dec 13, 2022
Copy link
Collaborator

@fischer-ncar fischer-ncar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you need to do any additional testing.

Copy link
Collaborator

@jedwards4b jedwards4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I DO think that aliases are intended to be short and these are getting to be too long for my comfort, but we've already discussed this.

@ekluzek
Copy link
Contributor Author

ekluzek commented Dec 16, 2022

I ran the test SMS_Ld1_D.f09_f09_rHDMAlk_mg17.I2000Clm50SpMizGs.cheyenne_intel.mizuroute-default and it PASSes.

@ekluzek
Copy link
Contributor Author

ekluzek commented Dec 16, 2022

I just looked at the results of query_config --grids, and I realize that I didn't fix all of the mizuRoute grids. I thought I had fixed them, but these still need to be changed:

hcru_hcru_mt13_rMERIT hcru_hcru_mt13_rHDMA f09_f09_mg17_rHDMA f09_f09_mg17_rMERIT f19_f19_mg17_rHDMA f19_f19_mg17_rMERIT

@ekluzek
Copy link
Contributor Author

ekluzek commented Dec 16, 2022

Oh, the problem is that there are mizuRoute grids defined for MCT -- but I should just remove those.

@ekluzek
Copy link
Contributor Author

ekluzek commented Dec 17, 2022

OK I removed the mizuRoute grids from MCT, and did a namelist check for the new aliases, and they all work. So this is ready from my perspective. @fischer-ncar can you merge it now?

@fischer-ncar fischer-ncar merged commit e0cc73e into ESMCI:main Dec 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add some new grids for mizuRoute with Lakes
3 participants