Skip to content

Conversation

dustinswales
Copy link
Member

@dustinswales dustinswales commented Jul 14, 2025

SOURCE: Dustin Swales NOAA-GSL

DESCRIPTION OF CHANGES:
SCM changes identical to FV3UFS NOAA-EMC/fv3atm #950.

ISSUE:
N/A

ASSOCIATED PRs:

Copy link
Member

@scrasmussen scrasmussen left a comment

Choose a reason for hiding this comment

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

Approving because you said there are fewer changes here but pointing them out just to double check, in FV3UFS NOAA-EMC/fv3atm #950 we have the following class of changes that aren't in this PR

  • get_tracer_index -> get_physics_tracer_index
  • integer, parameter :: physics_no_tracer = -99

@grantfirl
Copy link
Collaborator

Approving because you said there are fewer changes here but pointing them out just to double check, in FV3UFS NOAA-EMC/fv3atm #950 we have the following class of changes that aren't in this PR

  • get_tracer_index -> get_physics_tracer_index
  • integer, parameter :: physics_no_tracer = -99

Yes, I think that everything is good. FV3 and SCM now use different solutions for the get_tracer_index thing. The SCM has it defined in GFS_typedefs, but the FV3 uses the one from FMS (which the SCM doesn't use).

@grantfirl
Copy link
Collaborator

The debug mode failure should be debugged at some point, but not here. It doesn't fail on any other platforms.

@grantfirl grantfirl merged commit 1ea577d into NCAR:main Aug 6, 2025
12 of 13 checks passed
@scrasmussen
Copy link
Member

The debug mode failure should be debugged at some point, but not here. It doesn't fail on any other platforms.

Agreed, I was able to reproduce this error locally which will make it easier to debug. I'll track it down.

@grantfirl
Copy link
Collaborator

The debug mode failure should be debugged at some point, but not here. It doesn't fail on any other platforms.

Agreed, I was able to reproduce this error locally which will make it easier to debug. I'll track it down.

Oh, good. I was not able to reproduce a failure locally. Which version of GNU were you able to reproduce with? I have 12.4 on my Mac.

@scrasmussen
Copy link
Member

The debug mode failure should be debugged at some point, but not here. It doesn't fail on any other platforms.

Agreed, I was able to reproduce this error locally which will make it easier to debug. I'll track it down.

Oh, good. I was not able to reproduce a failure locally. Which version of GNU were you able to reproduce with? I have 12.4 on my Mac.

I'm using 12.2 on Derecho (by locally I meant off of Github lol).

Using a debugger I found the specific place where the code is breaking due to NaNs, still working on why though. PR #607 has that branch.

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