[C++][Python] Conversion of Table to Arrow Tensor #40062
Assignees
Comments
12 tasks
This was referenced May 29, 2024
AlenkaF
added a commit
that referenced
this issue
Jun 5, 2024
…to tensor.cc (#41932) ### Rationale for this change This is a precursor PR to #41870 with the purpose to make the review of #41870 easier (the diff of the code will be visible as it currently isn't because the code was moved to table.cc. I should also live in tensor.cc). ### What changes are included in this PR? The code from `RecordBatch::ToTensor` in record_batch.cc is moved to `RecordBatchToTensor` in tensor.cc. ### Are these changes tested? Existing tests should pass. ### Are there any user-facing changes? No. **This PR does not close the linked issue yet, it is just a precursor!** * GitHub Issue: #40062 Authored-by: AlenkaF <[email protected]> Signed-off-by: AlenkaF <[email protected]>
vibhatha
pushed a commit
to vibhatha/arrow
that referenced
this issue
Jun 5, 2024
…ch.cc to tensor.cc (apache#41932) ### Rationale for this change This is a precursor PR to apache#41870 with the purpose to make the review of apache#41870 easier (the diff of the code will be visible as it currently isn't because the code was moved to table.cc. I should also live in tensor.cc). ### What changes are included in this PR? The code from `RecordBatch::ToTensor` in record_batch.cc is moved to `RecordBatchToTensor` in tensor.cc. ### Are these changes tested? Existing tests should pass. ### Are there any user-facing changes? No. **This PR does not close the linked issue yet, it is just a precursor!** * GitHub Issue: apache#40062 Authored-by: AlenkaF <[email protected]> Signed-off-by: AlenkaF <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
This issue is a part of #40058 and adds
Table→Tensorconversion, using the implementation forRecordBatch.Component(s)
C++, Python
The text was updated successfully, but these errors were encountered: