Skip to content

Code changes to support DEBUG compilation based on appBuilder file#23

Merged
MinsukJi-NOAA merged 1 commit into
NOAA-EMC:developfrom
MinsukJi-NOAA:feature/nems_debug_flag
Jan 21, 2020
Merged

Code changes to support DEBUG compilation based on appBuilder file#23
MinsukJi-NOAA merged 1 commit into
NOAA-EMC:developfrom
MinsukJi-NOAA:feature/nems_debug_flag

Conversation

@MinsukJi-NOAA
Copy link
Copy Markdown
Collaborator

  1. component_MOM6.mk sets the MOM6_MAKEOPT environment variable for compile.sh
  2. component_CICE.mk sets the CICE_MAKEOPT environment variable for comp_ice.backend

Resolves issue #22

…cation

1. component_MOM6.mk sets the MOM6_MAKEOPT environment variable for compile.sh

2. component_CICE.mk sets the CICE_MAKEOPT environment variable for comp_ice.backend
@MinsukJi-NOAA
Copy link
Copy Markdown
Collaborator Author

  • Note that all regression tests for ufs-s2s-model passed.

@RatkoVasic-NOAA
Copy link
Copy Markdown
Collaborator

RatkoVasic-NOAA commented Jan 17, 2020

  • Note that all regression tests for ufs-s2s-model passed.

Regression tests passed on Dell as well.

@MinsukJi-NOAA
Copy link
Copy Markdown
Collaborator Author

NEMSCompset regression tests all passed for ufs-weather-model on Hera.

Copy link
Copy Markdown
Contributor

@DeniseWorthen DeniseWorthen left a comment

Choose a reason for hiding this comment

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

I have confirmed that the correct DEBUG=Y flag is passed to both MOM6 and CICE5 during the build of the coupled app.

@MinsukJi-NOAA MinsukJi-NOAA merged commit daae388 into NOAA-EMC:develop Jan 21, 2020
@MinsukJi-NOAA MinsukJi-NOAA deleted the feature/nems_debug_flag branch January 21, 2020 15:19
hyunchul386 pushed a commit to hyunchul386/NEMS that referenced this pull request Jul 24, 2020
…OAA-EMC#23)

* Added tests for regression test system using rt.sh on Hera.
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.

3 participants