Skip to content

Bring in NCO Intel DEBUG build changes.#985

Merged
DusanJovic-NOAA merged 15 commits into
NOAA-EMC:developfrom
BrianCurtis-NOAA:nco_debug_changes
Dec 3, 2025
Merged

Bring in NCO Intel DEBUG build changes.#985
DusanJovic-NOAA merged 15 commits into
NOAA-EMC:developfrom
BrianCurtis-NOAA:nco_debug_changes

Conversation

@BrianCurtis-NOAA
Copy link
Copy Markdown
Collaborator

Description

Bring in NCO DEBUG flag change for ops

NCO requires that all DEBUG builds have at least the following: -check all -ftrapuv

What bug does it fix, or what feature does it add? Meets implementation standards for ops debug builds
Is a change of answers expected from this PR? No

Issue(s) addressed

Testing

How were these changes tested? UFSWM Regression Testing Suite
What compilers / HPCs was it tested with? Gaea C6, Intel
Are the changes covered by regression tests? Yes
Have the ufs-weather-model regression test been run? Yes
On what platform? GaeaC6
Will the code updates change regression test baseline? No

Dependencies

@BrianCurtis-NOAA
Copy link
Copy Markdown
Collaborator Author

Could a CM look over and give preliminary approval for the code changes. This is a requirement for UFSWM.

@DusanJovic-NOAA DusanJovic-NOAA self-requested a review September 23, 2025 14:22
@BrianCurtis-NOAA
Copy link
Copy Markdown
Collaborator Author

@DusanJovic-NOAA not sure who your # 2 reviewer will be, please assign

@BrianCurtis-NOAA
Copy link
Copy Markdown
Collaborator Author

@BrianCurtis-NOAA I think your atmos_cubed_sphere hash needs an update to bd2b399?

https://github.com/BrianCurtis-NOAA/ufsatm/tree/nco_debug_changes/fv3 it is?

@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

gspetro-NOAA commented Dec 3, 2025

@BrianCurtis-NOAA I think your atmos_cubed_sphere hash needs an update to bd2b399?

https://github.com/BrianCurtis-NOAA/ufsatm/tree/nco_debug_changes/fv3 it is?

No, I confused myself, but it looks fine (which is why I deleted the comment), but I'm perplexed because the WM repo_check is failing for reasons related to atmos_cubed_sphere. I suppose it's just because this hasn't been merged. Sorry for the false alarm.

@BrianCurtis-NOAA
Copy link
Copy Markdown
Collaborator Author

@BrianCurtis-NOAA I think your atmos_cubed_sphere hash needs an update to bd2b399?

https://github.com/BrianCurtis-NOAA/ufsatm/tree/nco_debug_changes/fv3 it is?

No, I confused myself, but it looks fine (which is why I deleted the comment), but I'm perplexed because the WM repo_check is failing for reasons related to atmos_cubed_sphere. I suppose it's just because this hasn't been merged. Sorry for the false alarm.

No problem, always appreciated!

@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

Could a CM please merge this PR?

@DusanJovic-NOAA DusanJovic-NOAA merged commit 32870d2 into NOAA-EMC:develop Dec 3, 2025
2 of 5 checks passed
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