Radaer api la side - part two initialisation - #698
Radaer api la side - part two initialisation#698Alan J Hewitt (alanjhewitt) wants to merge 35 commits into
Conversation
|
|
||
| """ | ||
| Copy this template and complete to add your macro | ||
| class vn32_t683(MacroUpgrade): |
There was a problem hiding this comment.
Given the metadata changes, this is the only file where you should add an upgrade macro. When apply_macros.py gets run, it will copy the macro to all relevant places based on the metadata inheritance. Please revert all the other changes to versions.py files.
LFRic macro guidance: https://metoffice.github.io/simulation-systems/Development/macros.html#upgrade-macros-in-lfric
|
Commands used to make test branch and run upgrade macro ! create test branch locally
! if I mess up run this ! push change back to Github
|
PR Summary
Sci/Tech Reviewer:
Code Reviewer:
I developed the original lfric to radaer functionality in 2020. Like a lot of developments back then, it was needed in a hurry and so I developed a fudged solution with technical debt.
We will need user flexibility soon to be able to run radaer with different user settings, hence this part two of two development.
In part two, the previously hard coded lists (from part one) are instead generated from user settings based on the value of i_mode_setup if running with setting "ukca". For settings "glomap_clim" and "dust_and_clim", the user is not allowed to set i_mode_setup as this will always be setting "8" aka SUBCOCSSDU7mode.
These lists are to be determined during the initialisation step and saved for the rest of the run.
I will also include some documentation to describe the RADAER API.
Code Quality Checklist
Testing
trac.log
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