search

NOAA-EMC / UPP

Other
36 stars 98 forks source link

The inline post failure at ufs-weather-model for debug mode #923

Closed WenMeng-NOAA closed 5 months ago

WenMeng-NOAA commented 6 months ago

The inline post failed at ufs-weather-model debug mode (here).

WenMeng-NOAA commented 6 months ago

Here are suggestions from @DusanJovic-NOAA 👍

I think the message you see is just a warning not an error. The warning just says that temporary arrays are created before passing them to the subroutine. Creating a lot of temporary arrays can be expensive. The real question is why we see all these warnings now and not before. And I think that's because the compiler flags are changed here:

https://github.com/NOAA-EMC/UPP/compare/945cb2cef5e8bd5949afd4f0fc35c4fb6e95a1bf...4770a2f509b7122e76c4f004210031a58ae9502c#diff-f2d185634247633fd9fbcaee7a0bfea46c22f49b874999ab5ab69af2eb2c7e30

"-check all" is more strict than the previous warning flags. I'm sure you will also see an increased number of runtime warnings in standalone upp runs. Check the stderr/stdout file in the regression test for example.

One way to avoid all these warnings is to either revert to old (current) compiler flags. or add "-check noarg_temp_created" after "-check all".

WenMeng-NOAA commented 6 months ago

Wen ran the single test "rap_clm_lake_debug" in ufs-weather-model with Dusan's fix. The test was successfully completed.