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

Fix some namelist parameters for tnx2v1 grid #463

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

JorgSchwinger
Copy link
Contributor

There were two BLOM namelist parameter that were not set differently from default for the tnx2v1 grid. This PR fixes this since the defaults did not make sense for the 2-degree grid (although it hasn't been explored if the new setting are optimal). The changes due to the new remain relatively small, but notably there is a small reduction in overall temperature biases. Diagnostics can be found here (comparison valid for physics only):

https://ns2345k.web.sigma2.no/datalake/diagnostics/noresm/jschwing/NOINYOC_T62_tn21_45/BLOM_DIAG/yrs991to1000-NOINYOC_T62_tn21_42/

Copy link
Collaborator

@jmaerz jmaerz left a comment

Choose a reason for hiding this comment

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

Hi Jörg, approved. Nevertheless a stupid question: why not just changing the default value and rather only include the exceptions in the namelist definition? But I guess, this is rather something for the BLOM developers.

@JorgSchwinger
Copy link
Contributor Author

Yes, I agree, but I didn't want to mess around with the BLOM namelist. I don't know where the default value comes from and for which configuration it would be suited.

@jmaerz jmaerz mentioned this pull request Jan 16, 2025
17 tasks
@TomasTorsvik TomasTorsvik self-assigned this Jan 17, 2025
@TomasTorsvik TomasTorsvik added the iHAMOCC Issue mainly concerns the iHAMOCC code base label Jan 17, 2025
@TomasTorsvik TomasTorsvik added this to the release v1.7.0 tag milestone Jan 17, 2025
@TomasTorsvik TomasTorsvik merged commit bfb87be into NorESMhub:master Jan 17, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iHAMOCC Issue mainly concerns the iHAMOCC code base
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants