vibhatha commented on PR #13687:
URL: https://github.com/apache/arrow/pull/13687#issuecomment-1193009915

   > @vibhatha add docstrings to the Python/Cython code: 
https://numpydoc.readthedocs.io/en/latest/format.html (see the example on the 
bottom, or look through the Arrow source)
   
   @lidavidm I have already included the docs in the Cython in the original PR, 
but that function is in compute.pyx and only referred as an import in the 
compute.py. I was not sure whether to mark the method in Cython underscore 
function and call it in the compute.py and add the docs there. That was the 
confusing part. 


-- 
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