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
TST: Fix devdeps job #14075
TST: Fix devdeps job #14075
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
astropy/table/tests/test_table.py
Outdated
@@ -2158,7 +2159,15 @@ def test_masking(self, use_nullable_int): | |||
else: | |||
assert t2[name].dtype.kind == "i" | |||
|
|||
assert_array_equal(column.data, t2[name].data.astype(column.dtype)) | |||
with warnings.catch_warnings(): | |||
# This warning pops up when use_nullable_int is False |
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.
table
maintainers should see if ignoring this is ok.
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.
@pllim - looks good, just a small warning on how to avoid getting the warning.
astropy/table/tests/test_table.py
Outdated
@@ -2158,7 +2159,15 @@ def test_masking(self, use_nullable_int): | |||
else: | |||
assert t2[name].dtype.kind == "i" | |||
|
|||
assert_array_equal(column.data, t2[name].data.astype(column.dtype)) | |||
with warnings.catch_warnings(): |
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.
This can be
with np.errstate(invalid='ignore'):
assert_...
(I think that's OK for all numpy versions.)
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.
Good idea. Applied. Thanks!
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.
Looks good! I'd tend to rebase & squash, but rerunning all the CI seems wasteful, so perhaps best to merge as is.
TST: Ignore RuntimeWarning in astropy/table/tests/test_table.py that pops up when use_nullable_int is False and pandas is 1.5.2 Co-authored-by: Marten van Kerkwijk <mhvk@astro.utoronto.ca>
c9bd023
to
b47af9d
Compare
5 commits do look excessive. I squashed them. Thanks for the review! |
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
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.
io.ascii
is good, too, thanks for the fix! Just wondering how this could have slipped under the radar since numpy 1.20...
With numpy 1.23, I see this when I call
|
linkcheck failure is unrelated. |
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.
Looks fine to me.
Thanks, all! |
…075-on-v5.2.x Backport PR #14075 on branch v5.2.x (TST: Fix devdeps job)
…075-on-v5.0.x Backport PR #14075 on branch v5.0.x (TST: Fix devdeps job)
Description
This pull request is fix #14073
I also see some pandas warning, so I'll mark this as draft and see if I can also fix that here.
Checklist for package maintainer(s)
This checklist is meant to remind the package maintainer(s) who will review this pull request of some common things to look for. This list is not exhaustive.
Extra CI
label. Codestyle issues can be fixed by the bot.no-changelog-entry-needed
label. If this is a manual backport, use theskip-changelog-checks
label unless special changelog handling is necessary.astropy-bot
check might be missing; do not let the green checkmark fool you.backport-X.Y.x
label(s) before merge.