Lfric2lfric multi cpu test#392
Lfric2lfric multi cpu test#392Juan Manuel Castillo Sanchez (ukmo-juan-castillo) wants to merge 8 commits intoMetOffice:mainfrom
Conversation
Lottie Turner (mo-lottieturner)
left a comment
There was a problem hiding this comment.
It looks like checksum_lfric2lfric_oasis_clim_gal9_C12-ral_seuk_C16_lam-lbc_4cpu_azspice_gnu_fast-debug-64bit.txt is the same as checksum_lfric2lfric_oasis_clim_gal9_C12-ral_seuk_C16_lam-lbc_1cpu_azspice_gnu_fast-debug-64bit.txt - have you checked the other new kgos to see if they are the same as their single-cpu equivalents?
|
Hi Juan. Thanks for making those changes - just to note, as per the working practices, the developer should not be resolving comments left by others |
There was a problem hiding this comment.
Hi Juan. I think this is still too many multi-cpu tests in the developer group. Remember that the all group is run nightly, so issues with scaling will still be caught. Can you start by moving all of the new tests into the extra group, and then pick one of them to move back into the ex1a developer group?
PR Summary
Sci/Tech Reviewer:
Code Reviewer: mo-marqh
This PR simply adds rose stem tests to compare running lfric2lfric with multiple CPUs.
Code Quality Checklist
Testing
Test Suite Results - lfric_apps - lfric2lfric_multi_CPU_test/run1
Suite Information
Task Information
✅ succeeded tasks - 106
I just run the lfric2lfric tests as they are the only ones affected by these changes. I compared results with #275 and they do not change. Output generated with 1 CPU and multiple CPUs is the same.
Security Considerations
Performance Impact
AI Assistance and Attribution
Documentation
PSyclone Approval
Sci/Tech Review
(Please alert the code reviewer via a tag when you have approved the SR)
Code Review