[GitHub] [arrow] AlenkaF commented on pull request #35864: GH-35747: Add FixedSizeListArray.to_numpy_ndarray and FixedSizeListArray.from_numpy_ndarray

["AlenkaF (via GitHub)" <gi...@apache.org>]

AlenkaF commented on PR #35864:
URL: https://github.com/apache/arrow/pull/35864#issuecomment-1596733369

   Thank you @spenczar for making a contribution!
   And sorry it took a while to get a review.
   
   The implementation in Python instead of C++ makes sense. It might also make sense to think about the value of having this case (numpy arrays of 1 vs 2 dimensions, how one could still use 2-dim arrays in pyarrow avoiding an error) added to the documentation without adding two new functions to the code. I think that could be enough for a user needing to have an option for 2-dim numpy arrays.
   
   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org