-
Notifications
You must be signed in to change notification settings - Fork 68
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
Reduce number of warnings from tests #1573
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1573 +/- ##
==========================================
+ Coverage 70.57% 70.59% +0.01%
==========================================
Files 325 325
Lines 27061 27066 +5
Branches 3086 3087 +1
==========================================
+ Hits 19098 19106 +8
+ Misses 7380 7377 -3
Partials 583 583 ☔ View full report in Codecov by Sentry. |
/azp run libertem.libertem-data |
Azure Pipelines successfully started running 1 pipeline(s). |
e8df7ef
to
956fadb
Compare
/azp run libertem.libertem-data |
Azure Pipelines successfully started running 1 pipeline(s). |
@matbryan52 there are some deprecation warnings emitted from Other than that, this should be ready to go, and especially for the data tests reduce the number of warnings significantly. |
Fixed in the code itself as well as the testcase. Relates to numpy/numpy#10615 |
👍 yeah - I fixed a few instances of this in the I/O code (and also in |
* `bqplot.Figure` doesn't take a `scales` parameter * Expect and catch `UserWarning`s in `test_empty` for gms/bqp
Can't convert a single-element array directly to a Python `Number`, need to be explicit.
Needs the backport while we still support old-Python.
... so we don't need `importlib_metadata` anymore
/azp run libertem.libertem-data |
Azure Pipelines successfully started running 1 pipeline(s). |
Part of #1414
pkg_resources
→importlib_metadata
bqplot
Contributor Checklist:
Reviewer Checklist:
/azp run libertem.libertem-data
passed