paleolimbot commented on code in PR #366:
URL: https://github.com/apache/arrow-nanoarrow/pull/366#discussion_r1463943991
##########
python/src/nanoarrow/_lib.pyx:
##########
@@ -220,6 +220,64 @@ cdef class Error:
"""
raise NanoarrowException(what, code, "")
+cdef class CArrowType:
+ """
+ Wrapper around ArrowType to provide implementations in Python access
+ to the values.
+ """
+
+ UNINITIALIZED = NANOARROW_TYPE_UNINITIALIZED
+ NA = NANOARROW_TYPE_NA
+ BOOL = NANOARROW_TYPE_BOOL
+ UINT8 = NANOARROW_TYPE_UINT8
+ INT8 = NANOARROW_TYPE_INT8
+ UINT16 = NANOARROW_TYPE_UINT16
+ INT16 = NANOARROW_TYPE_INT16
+ UINT32 = NANOARROW_TYPE_UINT32
+ INT32 = NANOARROW_TYPE_INT32
+ UINT64 = NANOARROW_TYPE_UINT64
+ INT64 = NANOARROW_TYPE_INT64
+ HALF_FLOAT = NANOARROW_TYPE_HALF_FLOAT
+ FLOAT = NANOARROW_TYPE_FLOAT
+ DOUBLE = NANOARROW_TYPE_DOUBLE
+ STRING = NANOARROW_TYPE_STRING
+ BINARY = NANOARROW_TYPE_BINARY
+ FIXED_SIZE_BINARY = NANOARROW_TYPE_FIXED_SIZE_BINARY
+ DATE32 = NANOARROW_TYPE_DATE32
+ DATE64 = NANOARROW_TYPE_DATE64
+ TIMESTAMP = NANOARROW_TYPE_TIMESTAMP
+ TIME32 = NANOARROW_TYPE_TIME32
+ TIME64 = NANOARROW_TYPE_TIME64
+ INTERVAL_MONTHS = NANOARROW_TYPE_INTERVAL_MONTHS
+ INTERVAL_DAY_TIME = NANOARROW_TYPE_INTERVAL_DAY_TIME
+ DECIMAL128 = NANOARROW_TYPE_DECIMAL128
+ DECIMAL256 = NANOARROW_TYPE_DECIMAL256
+ LIST = NANOARROW_TYPE_LIST
+ STRUCT = NANOARROW_TYPE_STRUCT
+ SPARSE_UNION = NANOARROW_TYPE_SPARSE_UNION
+ DENSE_UNION = NANOARROW_TYPE_DENSE_UNION
+ DICTIONARY = NANOARROW_TYPE_DICTIONARY
+ MAP = NANOARROW_TYPE_MAP
+ EXTENSION = NANOARROW_TYPE_EXTENSION
+ FIXED_SIZE_LIST = NANOARROW_TYPE_FIXED_SIZE_LIST
+ DURATION = NANOARROW_TYPE_DURATION
+ LARGE_STRING = NANOARROW_TYPE_LARGE_STRING
+ LARGE_BINARY = NANOARROW_TYPE_LARGE_BINARY
+ LARGE_LIST = NANOARROW_TYPE_LARGE_LIST
+ INTERVAL_MONTH_DAY_NANO = NANOARROW_TYPE_INTERVAL_MONTH_DAY_NANO
+
+
+cdef class CArrowTimeUnit:
Review Comment:
I gave a few things a try but couldn't get it to work (which is more about
my Cython familiarity than anything). For now I'll leave it as is...there are a
number of things that could be improved about the Cython (including possibly
eliminating it in favour of nanobind, or at least splitting up the giant
_lib.pyx).
--
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]
