Plumbing in adjoint lookup table caches to interface#271
Plumbing in adjoint lookup table caches to interface#271DrTVockerodtMO wants to merge 4 commits intoMetOffice:mainfrom
Conversation
|
@cjohnson-pi I am currently running the test-suite and am getting linear KGO failures, none of these changes touch the linear (only the adjoint). The ones that are failing are: 64bit tests are passing, I'll check to see if I have the correct KGOs. |
It looks like I have the right KGOs, not sure what this issue is... |
|
@ss421 I also get some failures in jedi_lfric_tests on ex1a, most of the NWP tests fail in what I presume is the non-linear run on timestep 11. Is this a known problem? They pass on azspice. |
|
Hi Terry, The linear_model kgo failures are an issue on main which'll be fixed this morning. I suspect they weren't updated successfully before being merged yesterday. |
Thanks for that James, that helps narrow it down at least! |
|
Sorry - clicked the wrong button there! |
PR Summary
Sci/Tech Reviewer: @odlomax
Code Reviewer: @jennyhickson
Lengthy but basic changeset which plumbs the adjoint lookup table caches into the interface, i.e.: the second part of #72 . Unfortunately this touches many files because the caches are used at the lowest levels of the transport/solver adjoint trees, but the changeset is very repetitive and is mainly interface changes.
Code Quality Checklist
Testing
trac.log
Test Suite Results - lfric_apps - lfric_apps_adjoint_lookup_cache_part_2_branch_2/run2
Suite Information
Task Information
❌ failed tasks - 17
⌛ waiting tasks - 2
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