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

thousands of warnings from UPP code from gfortran #1065

Open
edwardhartnett opened this issue Oct 12, 2024 · 2 comments
Open

thousands of warnings from UPP code from gfortran #1065

edwardhartnett opened this issue Oct 12, 2024 · 2 comments

Comments

@edwardhartnett
Copy link
Collaborator

edwardhartnett commented Oct 12, 2024

When I run UPP through gfortran I get thousands of warnings.

I presented these results in a slideshow to the UPP team some time ago, and there was not much interest. But I am opening an issue to record this.

Build with gfortran -Wall and you will find many defects in UPP code that should be cleaned up. I suggest you clear all warnings and then set up the CI system to prevent careless programmers from introducing more.

Here's the slide about warnings in your code from a presentation about UFS code quality and issues. As the slide says, don't try and fix your warnings without unit testing first, or you will just break more stuff.

image

@gspetro-NOAA
Copy link
Collaborator

Thanks, Ed! Our software integration team is planning to write up a UPP unit testing plan this PI, and I think they will be in touch with you if they haven't been already. I suspect implementation will take quite a while, but it's at least on EPIC's radar. Thanks for creating the issue. :)

@edwardhartnett
Copy link
Collaborator Author

Prototype unit test implementation, with first few tests, can be found here: edwardhartnett#1

Glad to hear this code is getting some testing! Let me know if you need help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

2 participants