Skip to content

Minor Tempo changes#2310

Merged
weiwangncar merged 6 commits into
wrf-model:release-v4.8.0from
AndersJensen-NOAA:wrf_tempo_v3.0.5
May 20, 2026
Merged

Minor Tempo changes#2310
weiwangncar merged 6 commits into
wrf-model:release-v4.8.0from
AndersJensen-NOAA:wrf_tempo_v3.0.5

Conversation

@AndersJensen-NOAA
Copy link
Copy Markdown

@AndersJensen-NOAA AndersJensen-NOAA requested review from a team as code owners April 6, 2026 20:05
dudhia
dudhia previously approved these changes Apr 28, 2026
@dudhia
Copy link
Copy Markdown
Collaborator

dudhia commented Apr 29, 2026

Need to see why test failed.

@dudhia
Copy link
Copy Markdown
Collaborator

dudhia commented May 12, 2026

May need to retest after Shin-Hong update.

@AndersJensen-NOAA
Copy link
Copy Markdown
Author

May need to retest after Shin-Hong update.

@dudhia @weiwangncar What is the status on this retest? Is the failure caused by TEMPO or something else?

@dudhia
Copy link
Copy Markdown
Collaborator

dudhia commented May 18, 2026

May need to retest after Shin-Hong update.

@dudhia @weiwangncar What is the status on this retest? Is the failure caused by TEMPO or something else?

The failed test was not related to TEMPO. We have a Shin-Hong fix we need to test for this PR. Not clear why it showed up here.

@weiwangncar
Copy link
Copy Markdown
Collaborator

@dudhia @AndersJensen-NOAA @islas To retest this PR, we need updated MMM-physics code. Yes, it is strange the failure isn't related to TEMPO, but it could mean some thing is messed up in the memory somewhere.

Trivial change to trigger another test.
@weiwangncar
Copy link
Copy Markdown
Collaborator

@AndersJensen-NOAA @dudhia @islas After making another minor change (lining up lines) in microphysics driver, the regression tests have passed.

@weiwangncar weiwangncar merged commit 06e6998 into wrf-model:release-v4.8.0 May 20, 2026
1 check 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.

3 participants