Skip to content

Conversation

@JanStreffing
Copy link
Contributor

No description provided.

@JanStreffing JanStreffing self-assigned this Nov 17, 2025
@JanStreffing JanStreffing added the enhancement New feature or request label Nov 17, 2025
@mandresm
Copy link
Contributor

Looks good to me but for some reason, CI tests are failing. I'm looking into that. Also including fesom2.7 into the esm_tests

@mandresm
Copy link
Contributor

FESOM is failing to compile in the first test I ran. See /work/ab0246/a270152/testing/comp/fesom/fesom-2.7/comp.out

@JanStreffing
Copy link
Contributor Author

I thought it was a good idea to turn OMP on by default. Seems thats not the case.

Two solutions:
a) turn omp off in the fesom 2.7 in the comp_command.
b) make a change in the cmakelist.txt in fesom to prevent this error:

if(${ENABLE_OPENMP})
   target_link_libraries(${PROJECT_NAME}.x PRIVATE OpenMP::OpenMP_Fortran)
endif()

I suggest we do both. a) for a quick fix now, b) for 2.7.1.

@mandresm
Copy link
Contributor

The errors during compilation still persist after having set -DOPENMP=OFF. See agian /work/ab0246/a270152/testing/comp/fesom/fesom-2.7/comp.out

@JanStreffing
Copy link
Contributor Author

Sorry for the long wait. Can you do the test again? It seems to have been cleaned up.

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.

3 participants