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

compilation bugfix for nuopc cap #347

Merged
merged 1 commit into from
May 24, 2024

Conversation

mvertens
Copy link
Contributor

The latest commits to master break the compilation of the nuopc cap. It would be great if regression testing could be carried out on these updates to BLOM since this is the development code moving forwards.

@mvertens mvertens requested review from matsbn and TomasTorsvik May 24, 2024 07:50
@mvertens mvertens added the bug Something isn't working label May 24, 2024
@mvertens mvertens added this to the NorESM2.5 - BLOM/iHAMOCC milestone May 24, 2024
@TomasTorsvik
Copy link
Contributor

@mvertens - is there a branch/tag of NorESM or an Externals_???.cfg file that defines latest development version of NorESM2.5? I agree we should include a test with nuopc cap as this is the direction we will be heading. How will we be able to keep track on what is the latest development version of NorESM?

@JorgSchwinger
Copy link
Contributor

Shouldn't we, just after the release for NorESM2.3 is done, switch over to nuopc?

If 2.5 is going to be based on nuopc, I don't see any particular reason to keep running with mct?

@mvertens
Copy link
Contributor Author

@TomasTorsvik - very good point. I am in the process of putting this external together. I'll have something in place by early next week and send out a note.

@jmaerz
Copy link
Collaborator

jmaerz commented May 24, 2024

@JorgSchwinger , be aware that some -optional- parts of NorESM will still require work to enable the usage of nuopc... - but technically, I agree, would be then the time to do the move towards nuopc.

@TomasTorsvik
Copy link
Contributor

Shouldn't we, just after the release for NorESM2.3 is done, switch over to nuopc?

If 2.5 is going to be based on nuopc, I don't see any particular reason to keep running with mct?

In general, yes, I agree. But there might be some interest in running NorESM2.3 for scientific purposes as this will likely be the last version of NorESM with CMIP6-type model components.

@TomasTorsvik TomasTorsvik merged commit f672e71 into NorESMhub:master May 24, 2024
6 checks passed
@mvertens
Copy link
Contributor Author

Just as an addition note - the regression tests for BLOM will only include DATM, DROF, BLOM and CICE6. They will not include the other NorESM components. Those will be tested in both their 'stand-alone' configurations as well as fully coupled configurations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants