spenczar opened a new pull request, #35864:
URL: https://github.com/apache/arrow/pull/35864

   ### What changes are included in this PR?
   
   #35747 asks for additions to all three ListArray types (ListArray, 
FixedSizeListArray, and LargeListArray). This PR only adds for 
FixedSizeListArray. That's just because it's both the easiest to add, and the 
most obviously useful scenario.
   
   This implementation is just in Python and follows the general pattern of 
FixedShapeTensorArrays.
   
   An alternative implementation would be to do this in 
[python/pyarrow/src/arrow/python/numpy_convert.cc](https://github.com/apache/arrow/blob/cbc17a98dfd9fb660523f61ecd361b157d278372/python/pyarrow/src/arrow/python/numpy_convert.cc).
 We could detect 2D Numpy arrays and convert them to FixedShapeTensorArrays. 
That would be a much more significant change; I think it would change the 
behavior of other calls like `pyarrow.array()`. It would also allow for much 
more careful management of copies and memory; it should be possible to 
implement both to_ and from_ with zero copy for primitive numeric types. 
However, it would be a lot more complex, and this is my first Arrow 
contribution, so I figured I'd start a bit smaller.
   
   ### Are these changes tested?
   
   Yes.
   
   ### Are there any user-facing changes?
   
   Yes, and I think I added sufficient documentation.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to