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
DM-43925: Add workarounds for pandas bugs when using non-floating-point masked columns. #998
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #998 +/- ##
==========================================
- Coverage 88.97% 88.97% -0.01%
==========================================
Files 344 344
Lines 44199 44226 +27
Branches 9100 9106 +6
==========================================
+ Hits 39327 39351 +24
- Misses 3556 3557 +1
- Partials 1316 1318 +2 ☔ View full report in Codecov by Sentry. |
b6e3cf6
to
d847c64
Compare
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 to me; take or leave the comments as you will.
""" | ||
dataframe = arrow_to_pandas(astropy_to_arrow(astropy_table)) | ||
|
||
if isinstance(index, str): |
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.
Should this raise if it's not a string or None
? If not, what about just if index:
?
@@ -584,6 +588,23 @@ def testWriteMultiIndexDataFrameReadAsAstropyTable(self): | |||
# This test simply checks that it's readable, but definitely not | |||
# recommended. | |||
|
|||
@unittest.skipUnless(atable is not None, "Cannot test writing as astropy without astropy.") | |||
def testWriteAstropyTableWithMaskedColsReadAsSingleIndexDataFrame(self): |
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.
That is quite the mouthful... though I don't have a better suggestion.
d847c64
to
e620aae
Compare
Checklist
doc/changes
configs/old_dimensions