Adding initial Met Office configuration for VerPy database and plotti…#2134
Adding initial Met Office configuration for VerPy database and plotti…#2134RachelNorth wants to merge 1 commit into
Conversation
|
Hi, having pulled this PR for testing - I've had an issue with the change in verpy_plot/opt/rose-app-metoffice.conf. |
Hi mo-tomosevans , This will be resolved in #2103, where |
JorgeBornemann (JorgeBornemann)
left a comment
There was a problem hiding this comment.
All good.
Reflecting here what we discussed in person that the optional configurations in the metplus_ascii2nc app go in the main CSET repository rather than the restricted files, because they are unlikely to have any site-sensitive information.
Also noting that some of the changes will benefit other sites rather than Met Office.
| MET_INSTALL_DIR=${MET_DIR} | ||
| CONDA_VENV_LOCATION = ${CONDA_METPLUS_VENV_LOCATION} | ||
| METPLUS_OBS_DIR=${METPLUS_OBS_DIR} | ||
| METPLUS_OPT_CONFIG_KEYS = metoffice |
There was a problem hiding this comment.
The METPLUS_OPT_CONFIG_KEYS environment configuration is likely to be redundant, as it should not be using this file if it was no already metoffice, but is also harmless.
…ng capability
Adds configuration and modifications needed to run the VerPy database creation and plotting (of METplus verification results) at the Met Office. As part of #2133.
Contribution checklist
Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.
rose-suite.conf.examplehas been updated if new diagnostic added.