MetOffice / MetOffice/ANTS

Remove filter for iris datum FutureWarning

Open
#91 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.