Fix bug in bulk_statistics when providing a DataArray or iris cube with dask array data

[w-k-jones]

Change .data to .values call in get_statistics_from_mask to prevent passing dask array to get_statistics, and added test for dask array input

Fixes #472

• Have you followed our guidelines in CONTRIBUTING.md?
• Have you self-reviewed your code and corrected any misspellings?
• Have you written documentation that is easy to understand?
• Have you written descriptive commit messages?
• Have you added NumPy docstrings for newly added functions?
• Have you formatted your code using black?
• If you have introduced a new functionality, have you added adequate unit tests?
• Have all tests passed in your local clone?
• If you have introduced a new functionality, have you added an example notebook?
• Have you kept your pull request small and limited so that it is easy to review?
• Have the newest changes from this branch been merged?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.67%. Comparing base (706b3ee) to head (0b75b46).
Report is 7 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #474   +/-   ##
=======================================
  Coverage   60.67%   60.67%           
=======================================
  Files          23       23           
  Lines        3537     3537           
=======================================
  Hits         2146     2146           
  Misses       1391     1391           

Flag	Coverage Δ
unittests	60.67% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[w-k-jones]

@freemansw1 @JuliaKukulies are you both happy to review this? Should be quick as it is a very tiny change (1 word) and an added test to check passing dask arrays to bulk statistics works

[JuliaKukulies]

Good catch, thanks for fixing this @w-k-jones!

[JuliaKukulies]

@w-k-jones, I assume that was the reason for the jupyter notebook fail in #417 ?

[w-k-jones]

@w-k-jones, I assume that was the reason for the jupyter notebook fail in #417 ?

Yes, it looks like the same issue, so hopefully it will now be fixed

[freemansw1]

Thanks for the quick fix, @w-k-jones