-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: DataFrame(ndarray, dtype=categoricaldtype) #38857
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
Conversation
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.
lgtm. let's just re-run ci again
# mixed | ||
df = DataFrame([Categorical(list("abc")), list("def")]) | ||
expected = DataFrame( | ||
{0: Series(list("abc"), dtype="category"), 1: list("def")}, columns=[0, 1] | ||
) | ||
tm.assert_frame_equal(df, expected) | ||
|
||
def test_construct_from_listlikes_mismatched_lengths(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.
may pay to split out this test file as getting kind of big
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.
ive been thinking that myself. will address before long
this broke some things locally, am troubleshooting now |
…f-construction-cat
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.
lgtm. can you merge master as i think the tests are now moved
…f-construction-cat
rebased + green |
* BUG: DataFrame(ndarray, dtype=categoricaldtype) * whatsnew * GH ref * elif->if
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff