Remove filter for iris datum FutureWarning
- Dominant language
- Python
- Stars
- 7
- Forks
- 10
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 5
Description
We introduced a filter to ignore the `FutureWarning` being raised by iris about ignoring a datum on load. This warning is being raised even if the source file does not contain a datum, resulting in lots of spurious warnings. https://github.com/SciTools/iris/issues/5749 was opened to fix this behaviour.
For now, we are ignoring this warning on load, and there is an [expected fail unittest](https://github.com/MetOffice/ANTS/blob/abf5e9ebe4b027e54d9a4ffbd94df812b4d7982e/lib/ants/tests/fileformats/test_integration.py#L110) which should flag when the iris issue has been fixed (i.e. when the datum warning is only triggered when the netCDF file actually has a datum that is being ignored).
Contributor guide
Research direction
Start with the expected-fail unittest in lib/ants/tests/fileformats/test_integration.py around line 110, then locate the load-time FutureWarning filter described in the issue. Confirm the Iris behavior has been fixed and remove the workaround when the test shows the warning is emitted only for files containing an ignored datum.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- testing
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100