-
Notifications
You must be signed in to change notification settings - Fork 283
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
Categorise warnings #5498
Categorise warnings #5498
Conversation
I think I got all of them. Unfortunately the regex test I wrote doesn't cope with any nested brackets in a iris/lib/iris/tests/test_coding_standards.py Lines 137 to 181 in f4e6a35
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #5498 +/- ##
==========================================
+ Coverage 89.37% 89.41% +0.04%
==========================================
Files 89 89
Lines 22446 22539 +93
Branches 5387 5387
==========================================
+ Hits 20061 20154 +93
Misses 1639 1639
Partials 746 746
☔ View full report in Codecov by Sentry. |
fb63268I have confirmed using text searches that the fixed test is finding all |
37df50bThere are still many tests asserting for |
CoverageI have not made a concerted effort to have tests for all the new warning sub-categories. I did have to reference several of them in 37df50b just to get the tests to work. Codecov seems happy with test coverage (#5498 (comment)) already. As noted, there are still many tests asserting for Options
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @trexfeathers, this is going to add lots of flexibility for filtering warnings.
Thanks for wading through, @HGWright! |
* upstream/main: moved latest warning banner logic to conf.py (SciTools#5508) updated layout of top navbar (SciTools#5505) Oblique and Rotated Mercator (SciTools#5548) [pre-commit.ci] pre-commit autoupdate (SciTools#5549) [pre-commit.ci] pre-commit autoupdate (SciTools#5527) Bump scitools/workflows from 2023.09.1 to 2023.10.0 (SciTools#5540) nep29 drop table schedule numpy>1.21 (SciTools#5525) Updated environment lockfiles (SciTools#5545) Replaced `NotImplementedError` with `NotImplemented` (SciTools#5544) Gallery: show colour bar stealing space from multiple axes (SciTools#5537) Updated environment lockfiles (SciTools#5524) Set some memory benchmarks to on-demand to reduce noise. (SciTools#5481) updating docs and stale comment (SciTools#5522) Ensure removal of release candidate from What's New title. (SciTools#5526) Updated environment lockfiles (SciTools#5513) Docs page on filtering warnings (SciTools#5509) Replaced pkg_resources version parser with packager version parser. (SciTools#5511) Categorise warnings (SciTools#5498) Updated all np.product calls to np.prod (SciTools#5493)
🚀 Pull Request
Description
Closes #5472
To do
Consult Iris pull request check list