search

NOAA-EMC / UPP

Other
35 stars 95 forks source link

Correct failing to build UPP on Gaea-C5 #993

Closed DavidBurrows-NCO closed 1 month ago

DavidBurrows-NCO commented 1 month ago

I heard from Rahul that ufs-community/ufs-weather-model #2326 may be merging soon. I am hoping to get this UPP update in before 2326 merges. Thanks for any help.

Refs Merged PR #945 Refs Issue #992 Refs ufs-community/ufs-weather-model #2326

FernandoAndrade-NOAA commented 1 month ago

@WenMeng-NOAA this should be fine to remove for now, RTs utilize a python helper script to summarize differences in results for the RT logs. Given that the RTs aren't fully configured on Gaea yet, this shouldn't change anything a the moment.

WenMeng-NOAA commented 1 month ago

@WenMeng-NOAA this should be fine to remove for now, RTs utilize a python helper script to summarize differences in results for the RT logs. Given that the RTs aren't fully configured on Gaea yet, this shouldn't change anything a the moment.

@FernandoAndrade-NOAA Thanks for clarifying. This PR is for updating the modulefiles for Gaea-C5 only. There is no need for running the UPP RTs.

WenMeng-NOAA commented 1 month ago

This PR is ready for merging.