Add debug compilation flag specification at top level#17
Merged
DeniseWorthen merged 8 commits intoJan 21, 2020
Merged
Conversation
1. Debug flags are specified in coupledFV3_MOM6_CICE_debug.appBuilder 2. Flags are used by NEMS MOM6 and CICE submodules to initiate debug makes
Collaborator
Author
|
Attach build log with and without debug flags. Also attach regression test report. |
Collaborator
|
We also need to update two files in conf for adding the -link_mpi=dbg to the FFLAGS_DEBUG settings for the configure.fv3_coupled.cheyenne.intel and configure.fv3_coupled.hera.intel. These have not yet been added to the develop branch but should be. |
DeniseWorthen
requested review from
DeniseWorthen,
JessicaMeixner-NOAA and
binli2337
add -link_mpi=dbg to fflags_debug for cheyenne
add -link_mpi=dbg to fflags_debug for hera
added to correct conf files configure.fv3_coupled.MACHINE
Collaborator
|
I've confirmed the top-level app build passes the flags down to MOM6 and CICE5. Once we merge the related PRs to the develop branches of CICE, MOM6-interface and NEMS, you'll need to amend you PR to point to NOAA_EMC repos (instead of your forks) and we can approve. Thanks! |
binli2337
approved these changes
Jan 21, 2020
JessicaMeixner-NOAA
approved these changes
Jan 21, 2020
DeniseWorthen
referenced
this pull request
in DeniseWorthen/ufs-s2s-model
Jan 21, 2020
Add debug compilation flag specification at top level (#17)
JessicaMeixner-NOAA
pushed a commit
to JessicaMeixner-NOAA/ufs-s2s-model
that referenced
this pull request
Feb 3, 2020
* Add debug compilation flag specification at top level 1. Debug flags are specified in coupledFV3_MOM6_CICE_debug.appBuilder 2. Flags are used by NEMS MOM6 and CICE submodules to initiate debug makes
JessicaMeixner-NOAA
pushed a commit
to JessicaMeixner-NOAA/ufs-s2s-model
that referenced
this pull request
Apr 23, 2020
Update NEMS,FV3,CICE (ufs-community#28)
ShanSunNOAA
pushed a commit
to ShanSunNOAA/ufs-s2s-model
that referenced
this pull request
Oct 28, 2020
…l_update_and_aux_output_dom Combine "Update to MYNN-EDMF" and "Output of auxiliary 2d/3d arrays from CCPP ufs-community#17"
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resolves issue #16