Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
WenMeng-NOAA opened this issue Apr 8, 2024 · 2 comments · Fixed by #922
Closed

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

WenMeng-NOAA opened this issue Apr 8, 2024 · 2 comments · Fixed by #922
Assignees
Labels
bug Something isn't working

Comments

@WenMeng-NOAA
Copy link
Collaborator

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

@WenMeng-NOAA
Copy link
Collaborator Author

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:

945cb2c...4770a2f#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
Copy link
Collaborator Author

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

@WenMeng-NOAA WenMeng-NOAA self-assigned this Apr 8, 2024
@WenMeng-NOAA WenMeng-NOAA added the bug Something isn't working label Apr 8, 2024
@WenMeng-NOAA WenMeng-NOAA linked a pull request Apr 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants