Skip to content

ARROW-1309: [Python] Handle nested lists with all None values in Array.from_pandas#947

Closed
wesm wants to merge 3 commits intoapache:masterfrom
wesm:ARROW-1309
Closed

ARROW-1309: [Python] Handle nested lists with all None values in Array.from_pandas#947
wesm wants to merge 3 commits intoapache:masterfrom
wesm:ARROW-1309

Conversation

@wesm
Copy link
Member

@wesm wesm commented Aug 7, 2017

No description provided.

wesm added 3 commits August 7, 2017 19:16
Change-Id: Iffa7511f223dcfa0e81099f84e20e73fa0bbad9a
Change-Id: I222934709b134d08ccdfb8181c0c6fd1b5574718
Change-Id: I00d47a8decf872a7688a036528b3f24565917700
@wesm
Copy link
Member Author

wesm commented Aug 8, 2017

+1. This is failing due to ARROW-1338, unrelated to this

@asfgit asfgit closed this in 66ab6b2 Aug 8, 2017
@wesm wesm deleted the ARROW-1309 branch August 8, 2017 02:08
return Status::NotImplemented("mask not supported in object conversions yet");
}

if (is_strided()) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have a failing test that checks support for strided arrays?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like not. I'll add one, since we should be using

class Ndarray1DIndexer {
for this iteration rather than going directly to PyArray_DATA

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants