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

cice6 debug compile #133

Merged

Conversation

MinsukJi-NOAA
Copy link
Collaborator

This PR addresses Issue #132

@junwang-noaa
Copy link
Collaborator

Thanks for making the code changes. Is the cic6 clean up step also working?

@MinsukJi-NOAA
Copy link
Collaborator Author

Yes. compile.sh defaults to 'yes' for clean_before and clean_after, and no errors occurred.

Copy link
Collaborator

@junwang-noaa junwang-noaa left a comment

Choose a reason for hiding this comment

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

May I ask why we remove the tests/compile.sh? I see we still use compile.sh in rt.sh, isn't it?

    ./compile.sh $PATHTR/FV3 $MACHINE_ID "${NEMS_VER}" $COMPILE_NR > ${LOG_DIR}/compile_${COMPILE_NR}.log 2>&1

@MinsukJi-NOAA
Copy link
Collaborator Author

May I ask why we remove the tests/compile.sh? I see we still use compile.sh in rt.sh, isn't it?

This commit removes the CICE6_MAKEOPT flag in compile.sh since CICE6 uses CICE_MAKEOPT.

@DeniseWorthen
Copy link
Collaborator

DeniseWorthen commented Jul 13, 2020

I've tested on both orion and hera; both compiles appear correct for debug=y setting.

I added an issue to CICE to add the the link_mpi=debug flags to the macros.

@DeniseWorthen DeniseWorthen merged commit 600015d into ufs-community:develop Jul 15, 2020
@MinsukJi-NOAA MinsukJi-NOAA deleted the fix/cice6-debug-compile branch July 15, 2020 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants