Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update wet bulb freezing level cube name #1956

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

mspelman07
Copy link
Contributor

@mspelman07 mspelman07 commented Oct 12, 2023

Currently the wet bulb freezing level cube is called "wet_bulb_freezing_level". However the hail size cli requires the name the be "wet_bulb_freezing_level_altitude" to extract it from a cubelist.

This PR updates the cube name so it can be used with the hail size plugin.

Updated acceptance test data: metoppv/improver_test_data#35

Testing:

  • Ran tests and they passed OK
  • Added new tests for the new feature(s)

@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (7e2ed68) 98.39% compared to head (635f9bd) 98.39%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1956   +/-   ##
=======================================
  Coverage   98.39%   98.39%           
=======================================
  Files         123      123           
  Lines       11808    11808           
=======================================
  Hits        11618    11618           
  Misses        190      190           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MoseleyS MoseleyS merged commit fdc96a4 into metoppv:master Oct 13, 2023
10 checks passed
@MoseleyS MoseleyS mentioned this pull request Oct 13, 2023
2 tasks
MoseleyS pushed a commit to MoseleyS/improver that referenced this pull request Aug 22, 2024
…ed by hail_size cli. (metoppv#1956)

Co-authored-by: Marcus Spelman <[email protected]>

This doesn't need a second review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants