search

hafs-community / HAFS

Hurricane Analysis and Forecast System
Other
29 stars 54 forks source link

Allow an easier way to compile with -check all enabled #233

Open BinLiu-NOAA opened 9 months ago

BinLiu-NOAA commented 9 months ago

Description

Provide a clear and concise description of the requested feature/capability. From NCO SPA: In the HAFS v1.0 IT tests, to test the Code Stability with recompiling codes with ‘-check all’ or ‘-check Bounds’, there is no easy way to do but follow the doc, hafs_debug_build, https://docs.google.com/document/d/1QtR76sSEvLn35SRI6Vw9MHmHMmcH9br57_U1Oj7eRog/edit, to update the build scripts or update the CMakeLists.txt.

Please improve the the debug options/configurations in the HAFS build process in the next upgrade.

Proposed solution

How should the new feature/capability be added? If you have thoughts on the implementation strategy, please share them here.

Status (optional)

Do you (or a colleague) plan to work on adding this feature?

Related to (optional)

Directly reference any related issues or PRs in this or other repositories, and describe how they are related. Examples:

BijuThomas-NOAA commented 8 months ago

The buildall.sh together with other build*.sh scripts have been updated by adding the following option:

Supports Debug or Release modes for the build

export BUILD_MODE=${BUILD_MODE:-Release} #|Release|Debug| The default is build in Release mode. However, one can easily change to build in Debug mode.

Details can be found in this commit: f1a6df17f9117f7349dcdcaf2fb1c87d2d86bdc6

BinLiu-NOAA commented 8 months ago

@BijuThomas-NOAA, we probably want to keep this issue open until the changes/fixes go back into HAFS develop branch (or at least go into the production/hafs.v2 branch). Thanks!

BijuThomas-NOAA commented 8 months ago

@BinLiu-NOAA OK, sounds good