westonpace commented on code in PR #35356:
URL: https://github.com/apache/arrow/pull/35356#discussion_r1267391740
##########
python/pyarrow/array.pxi:
##########
@@ -2542,8 +2542,14 @@ cdef class DictionaryArray(Array):
def dictionary_decode(self):
"""
Decodes the DictionaryArray to an Array.
+
+ See :func:`pyarrow.compute.dictionary_decode` for usage.
+
+ Returns
+ -------
+ dictionary_decode : Array
"""
- return self.dictionary.take(self.indices)
+ return _pc().dictionary_decode(self)
Review Comment:
> I think it's good to unify the dictionay_decode logic in Python
This is a good point too. I'll let @jorisvandenbossche or @AlenkaF weigh in
as well. We can move forward as-is for now and add this back if they want.
--
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]
