Skip to content

Add -check all and -ftrapuv to all Intel DEBUG builds#2789

Merged
FernandoAndrade-NOAA merged 35 commits intoufs-community:developfrom
BrianCurtis-NOAA:nco_debug_changes
Dec 3, 2025
Merged

Add -check all and -ftrapuv to all Intel DEBUG builds#2789
FernandoAndrade-NOAA merged 35 commits intoufs-community:developfrom
BrianCurtis-NOAA:nco_debug_changes

Conversation

@BrianCurtis-NOAA
Copy link
Collaborator

@BrianCurtis-NOAA BrianCurtis-NOAA commented Jun 26, 2025

Commit Queue Requirements:

  • Fill out all sections of this template.
  • All sub component pull requests have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines) on Ursa
  • Commit 'test_changes.list' from previous step

Description:

Bring in NCO requirements for DEBUG builds.

Commit Message:

* UFSWM - Add -check all and -ftrapuv to all Intel DEBUG builds
  * FV3 - Add -check all and -ftrapuv to all Intel DEBUG builds
    * atmos_cubed_sphere - Add -check all and -ftrapuv to all Intel DEBUG builds
  * stochastic_physics -Add -check all and -ftrapuv to all Intel DEBUG builds

Priority:

  • Normal

Git Tracking

UFSWM:

Sub component Pull Requests:

UFSWM Blocking Dependencies:

Documentation:

  • No documentation update is required for this PR (please explain).

Changes

Regression Test Changes (Please commit test_changes.list):

  • No Baseline Updates.

Input data Changes:

  • None.

Library Changes/Upgrades:

  • No Updates

Testing Log:

  • RDHPCS
    • Hera
    • Ursa
    • Orion
    • Hercules
    • GaeaC6
    • Derecho
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

@BrianCurtis-NOAA BrianCurtis-NOAA changed the title Nco debug changes Add -check all and -ftrapuv to all Intel DEBUG builds Jun 26, 2025
@BrianCurtis-NOAA BrianCurtis-NOAA marked this pull request as ready for review July 29, 2025 13:51
@BrianCurtis-NOAA
Copy link
Collaborator Author

Currently my test_changes.list is similar to #2819 to which I have the same changes. Once that PR goes through, this should not change baselines. That's my only blocker at this point.

@BrianCurtis-NOAA
Copy link
Collaborator Author

Also to note the WW3 changes are going to be merged after this PR, in a PR bringing in a merge from WW3 develop.

@DusanJovic-NOAA
Copy link
Collaborator

I've noticed that some components are being compiled without -check nopointer, which means pointer checks are enabled. However, other components are compiled with -check nopointer, which disables these checks. I think it's important to keep pointer checks enabled across all components.

@BrianCurtis-NOAA
Copy link
Collaborator Author

@DusanJovic-NOAA right, I wanted the scope of this PR to make sure those options were added. I assume removing -check nopointer would bring out numerous build failures or a mass amount of warnings. As such would be good for a different PR.

@JessicaMeixner-NOAA
Copy link
Collaborator

@BrianCurtis-NOAA The WW3 update that includes this is in UFS PR: #2841 NOAA-EMC/WW3#1480

@gspetro-NOAA
Copy link
Collaborator

gspetro-NOAA commented Aug 5, 2025

UPDATE: It looks like adding the flags -shared-intel -diag-disable=10444 will solve the errors we were seeing in UPP PR #1256 on Ursa LLVM. Once @BrianCurtis-NOAA makes that update in UPP PR #1256, we can process that PR quickly on the UPP side. Then the UPP hash in FV3 PR #985 will need to be updated, and possibly the same fix applied to other dependent PRs if the same errors appear in those repos with Ursa LLVM.

EDIT: UPP PR #1256 has been merged!

@BrianCurtis-NOAA
Copy link
Collaborator Author

I added a blocking dep on #2841

@gspetro-NOAA
Copy link
Collaborator

@BrianCurtis-NOAA Do you have a status update/timeline on this PR?

@gspetro-NOAA gspetro-NOAA moved this to Evaluating in PRs to Process Sep 19, 2025
@gspetro-NOAA gspetro-NOAA added the No Baseline Change No Baseline Change label Sep 19, 2025
@BrianCurtis-NOAA
Copy link
Collaborator Author

@BrianCurtis-NOAA Do you have a status update/timeline on this PR?

This one is basically ready to go. There's so many moving part's I'll try to hold off on a mass sync to day before this is ready for top of the queue.

@gspetro-NOAA
Copy link
Collaborator

@BrianCurtis-NOAA Will you hold off on pushing the log till then, too?

@BrianCurtis-NOAA BrianCurtis-NOAA added No Baseline Change No Baseline Change and removed Baseline Updates Current baselines will be updated. GOC There are changes to the GOCART component repository. labels Dec 2, 2025
@FernandoAndrade-NOAA FernandoAndrade-NOAA added the jenkins-ort run ORT testing label Dec 3, 2025
@epic-cicd-jenkins epic-cicd-jenkins removed the jenkins-ort run ORT testing label Dec 3, 2025
@FernandoAndrade-NOAA
Copy link
Collaborator

We should be all set with testing, leaving a note in sub PRs.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Dec 3, 2025

@BrianCurtis-NOAA New stochastic physics hash is NOAA-PSL/stochastic_physics@ae192f1

@gspetro-NOAA
Copy link
Collaborator

@BrianCurtis-NOAA You can update the hash and revert .gitmodules for UFSATM (32870d2.

@FernandoAndrade-NOAA FernandoAndrade-NOAA merged commit 53ffdf0 into ufs-community:develop Dec 3, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A3S There are changes to the atmos_cubed_sphere component repository. No Baseline Change No Baseline Change SP There are changes to the stochastic_physics component repository. UFSATM There are changes to the UFSATM repository.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Add -check all -check noarg_temp_created -ftrapuv to all DEBUG build flags of UFSWM and subcomponents

9 participants