search

NorESMhub / BLOM

Bergen Layered Ocean Model
GNU Lesser General Public License v3.0
16 stars 25 forks source link

Continuous regression testing with NorESM 2.0.6 #306

Closed jmaerz closed 1 month ago

jmaerz commented 7 months ago

Hi @mvertens , I pull over the discussion from #294 on how to set up a regression testing for HAMOCC/NorESM 2.0.6 into a new issue, since it might be relevant also for others (see also https://github.com/NorESMhub/BLOM/wiki/BLOM-iHAMOCC-regression-testing-with-NorESM and #288) and would easily get buried in the former PR.

Since the automatic regression test failed when using NorESM 2.0.6, I followed your suggested step to update the file $SRCROOT/cime/config/cesm/machines/config_machines.xml by replacing the CCSM_CPRNC block under the betzy machine as follows:

 <machine MACH="betzy">
       ......
    <CCSM_CPRNC>UNSET</CCSM_CPRNC>

with

    <CCSM_CPRNC>/cluster/shared/noresm/tools/cprnc/cprnc</CCSM_CPRNC>

I did this and now hitting another error:

/cluster/shared/noresm/tools/cprnc/cprnc: error while loading shared libraries: libnetcdf.so.19: cannot open shared object file: No such file or directory

Any suggestion on this?

mvertens commented 7 months ago

@jmaerz - there is a new PR coming into the noresm2.1 CIME today that should hopefully fix these problems for Noresm2.1. Let me give it a try and make sure it works and then I'll point you to it.

jmaerz commented 7 months ago

All fine - this is not too urgent - I just thought it would be good to get that going at some point (and if I understand correct, this will be soon obsolete anyway with NorESM2.1, since there it then functions as expected after your new PR).

mvertens commented 7 months ago

So I have a bug fix for mo_ocprod.F90 and mo_bgc_params.F90 that is still bfb. Its not too invasive and also involves formatting changes. I'm wondering if it would be okay to just add it to this PR - since its not very big and would fix the github actions tests. Thoughts?

mvertens commented 7 months ago

@jmaerz - this bug fix is implementing what we talked about last night.

jmaerz commented 7 months ago

I guess, I pull this discussion over to #305 and answer there to keep it a bit organized :-)

TomasTorsvik commented 1 month ago

@jmaerz , @mvertens - this issue seems to be resolved and/or outdated. Can it be closed?

jmaerz commented 1 month ago

@TomasTorsvik , from my side yes.