paleolimbot commented on issue #39933: URL: https://github.com/apache/arrow/issues/39933#issuecomment-1928657729
I believe reticulate 1.35.0 changed the "name" of the capsule. I am not sure what it was before and I am not sure what it is now...we started using external pointers on the R side in Arrow 7.0.0, which converted to Python capsules automatically, because it felt better than converting the pointer to an R `double()`, sending that to Python, then static casting that to a `uintptr_t` and calling it a pointer again. One of the errors is along the lines of `PyCapsule_GetPointer() called with incorrect name`, so perhaps the name check was made stricter on the Python side when capsules started being used there. On the R side, we could go back to calling `_import_from_c` with a floating-point value (although I would rather not). We also have the ability to call it with a string representation of the address, which is also lossless, but this is rejected by pyarrow (I just checked). I also just checked passing a floating point value and that results in the test passing for me. The code on the R side that could change is here: https://github.com/apache/arrow/blob/672238ff6352fa388b54182d8ae1667f9e99c327/r/R/python.R#L341-L353 -- 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]
