-
Notifications
You must be signed in to change notification settings - Fork 28
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
Add chem code and yaml for hybrid. #1206
Conversation
…ues and added alpha_diagb parameter to yaml.
…dd_diagb_climatol to combine these two PRs.
@andytangborn next up, can you make the coding norms tests happy, then I will do my next/final review |
sure. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just naming convention stuff now, otherwise this looks great to me! We can merge this in and then worry about the interpolation, but I think I want the interpolation included before we get the workflow PR ready
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks! This is going to be a fantastic addition!
chem code and yaml for diagb hybrid.