search

esmf-org / esmf

The Earth System Modeling Framework (ESMF) is a suite of software tools for developing high-performance, multi-component Earth science modeling applications.
https://earthsystemmodeling.org/
Other
156 stars 75 forks source link

Add ESMF development tests for PR and branches #286

Closed danrosen25 closed 4 weeks ago

danrosen25 commented 1 month ago
system compiler comm exhaustive bopt tracing all_tests
ubuntu@22.04 gfortran@11 mpiuni ON O ON ~12m
ubuntu@22.04 gfortran@12 mpich OFF O ON ~22m
macos@14 gfortran@13 mpiuni OFF O OFF ~10m
macos@14 clang+gfortran@14 openmpi ON g ON ~17m
danrosen25 commented 1 month ago

@oehmke @billsacks @theurich I think I'm ready to merge this in today BUT I wanted to hear your thoughts:

Should we keep this as a PR and Manually triggered workflow OR

The issue now is that the cached libraries (openmpi, mpich, hdf5, netcdf, etc) are only retained for 7 days if they aren't touched. Therefore pull request testing will take considerably longer if pull requests aren't submitted weekly.

oehmke commented 1 month ago

I think that for now, let’s keep this as a PR, manually triggered, and weekly (to keep the cache working). After 8.7 and seeing how it goes, we can think about every push. What do you think?

On Sep 5, 2024, at 10:07 AM, Daniel Rosen @.***> wrote:

@oehmke https://github.com/oehmke @billsacks https://github.com/billsacks @theurich https://github.com/theurich I think I'm ready to merge this in today BUT I wanted to hear your thoughts:

Should we keep this as a PR and Manually triggered workflow OR

Add continuous testing (every push) to the develop branch AND/OR Add weekly testing of the develop branch The issue now is that the cached libraries (openmpi, mpich, hdf5, netcdf, etc) are only retained for 7 days if they aren't touched. Therefore pull request testing will take considerably longer if pull requests aren't submitted weekly.

— Reply to this email directly, view it on GitHub https://github.com/esmf-org/esmf/pull/286#issuecomment-2332125529, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE6A7U444V3S6LD7CL7CS7LZVB6TLAVCNFSM6AAAAABNVPMEUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZSGEZDKNJSHE. You are receiving this because you were mentioned.

billsacks commented 4 weeks ago

I don't have strong feelings on the frequency of running these. @oehmke 's suggestion seems reasonable for now.