Migrate CLI functionality to plugin layer subset 2

[cpelley]

Second/final chunk of plugin changes required by EPP.

Tasks

• improver/cli/extract.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/freezing_rain.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/hail_fraction.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/hail_size.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/interpolate_using_difference.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/lightning_from_cape_and_precip.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/nbhood.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/phase_change_level.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/phase_probability.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/snow_splitter.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/vertical_updraught.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/wet_bulb_freezing_level.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.
• improver/cli/wet_bulb_temperature.py
  • [x] Working acceptance testing.
  • [x] Unittesting coverage.

Issues

• https://github.com/MetOffice/improver_suite/issues/2079
• https://github.com/metoppv/improver/issues/1998
• https://github.com/MetOffice/barebones_workflow/issues/32
• https://github.com/MetOffice/improver_suite/issues/2253

[cpelley]

Finally got this ready for review. This covers the second and final chunk of IMPROVER CLI/plugin changes required specifically for EPP. Cheers

[cpelley]

@bayliffe, directly relevant change in reflection of this work in https://github.com/MetOffice/dagrunner/pull/40 (you need not review that, I mention only out of interest).

[cpelley]

Changes based on feedback from @bayliffe:

• Moved ignore_error into the plugin of from improver/cli/extract.py.
  • Changes to improver_tests/utilities/cube_extraction/test_ExtractSubCube.py to reflect this change in behaviour to the plugin.
• Recommended doc change to improver.utilities.complex_conversion.deg_to_complex.
• Removed static methods deg_to_complex and complex_to_deg from WindDirection.
• Migrated some testing from improver_tests/wind_calculations/wind_direction/test_WindDirection.py to new file improver_tests/utilities/complex_conversion/test_integration.py (roundtrip testing).
  • Changes to improver_tests/wind_calculations/wind_direction/test_WindDirection.py to ensure successful running of tests that remain.

The only thing I haven't done as I'm not quite following is what the issue is, is in regards to https://github.com/metoppv/improver/pull/2003#discussion_r1665628061 However, I responded to what I think is a misunderstanding. *args is being passed to the function. The idea is just 1 parameter could be passed if its a cubelist, or 2 or 3 etc. I have verified that the documentation does highlight what data is actually needed to be provided.

[cpelley]

You happy to approve @bayliffe?

cheers

[cpelley]

I suspect the CI failures with Sphinx are unrelated to this PR (a result of changes in the conda-forge conda environment, prob. needs something pinning or something unpinning). Fix for CI in https://github.com/metoppv/improver/pull/2011

[cpelley]

I have merged in master into this branch and changed the target (since subset 1 was merged I forgot to change it here).

[cpelley]

@mo-robert-purvis, you mind approving once more (it removed your approval after I changed the target to master). cheers

[cpelley]

I just merged master into this branch to resolve the CI failure.

[cpelley]

Thanks both. @bayliffe, I have made the leadtimes optional once more. Thanks for double checking this for me.