Skip to content

Conversation

@mnlevy1981
Copy link
Collaborator

  1. with DO_SKEB=True by default, we need to turn it off in the no_stoch_physics test
  2. In order for ERI tests to pass, need stochini=True for hybrid runs

1. with DO_SKEB=True by default, we need to turn it off in the no_stoch_physics
   test
2. In order for ERI tests to pass, need stochini=True for hybrid runs
@mnlevy1981
Copy link
Collaborator Author

The good news is that the no_stoch_physics test passes now. The ERI test is still failing; ref2 is looking for ./INPUT/ocn_stoch.res.nc rather than the reference file. I wonder if the problem is with a branch run rather than hybrid? Switching the logic and testing that, but this might require a MOM6 update to make sure we set stoch_restfile correctly for hybrid and / or branch runs...

@mnlevy1981 mnlevy1981 marked this pull request as ready for review November 12, 2025 03:05
@mnlevy1981
Copy link
Collaborator Author

I have verified that ERI.TL319_t232.G_JRA.derecho_intel.cice-default and SMS.TL319_t232.G_JRA.derecho_intel.mom-no_stoch_physics tests pass; @fischer-ncar reported those (and an ERI test with a B compset that has the same issue as the G compset) as well as an MCC test that is not addressed in this PR... that will need to get fixed in a later tag.

I have not run aux_mom or pr_mom, just the one-off tests.

Copy link
Member

@alperaltuntas alperaltuntas left a comment

Choose a reason for hiding this comment

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

Tests are ongoing. I'll merge this PR for now so that alpha tests can start too.

@alperaltuntas alperaltuntas merged commit 4e912a3 into ESCOMP:main Nov 12, 2025
6 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.

2 participants