refactor(numpydates): simplify accessors and add inline datetime/timedelta support

- Simplified accessor functions in DateTime64, InlineDateTime64, InlineTimeDelta64, and TimeDelta64 by removing unnecessary function wrappers
- Added support for NumpyDates.InlineDateTime64 and InlineTimeDelta64 in pyjlarray_isarrayabletype and pytypestrdescr
- Refactored pytypestrdescr to use if-elseif structure for better readability
- Updated PyArray.jl to handle unbound units in type conversion

This enhances numpy date type integration while cleaning up code structure.

Author: Christopher Doris

src/JlWrap/JlWrap.jl

@@ -7,6 +7,7 @@ module JlWrap
 
 using ..PythonCall
 using ..Utils
+using ..NumpyDates: NumpyDates
 using ..C
 using ..Core
 using ..Convert

src/JlWrap/array.jl

@@ -223,6 +223,8 @@ pyjlarray_isarrayabletype(::Type{T}) where {T} = T in (
     Complex{Float32},
     Complex{Float64},
 )
+pyjlarray_isarrayabletype(::Type{NumpyDates.InlineDateTime64{U}}) where {U} = true
+pyjlarray_isarrayabletype(::Type{NumpyDates.InlineTimeDelta64{U}}) where {U} = true
 pyjlarray_isarrayabletype(::Type{T}) where {T<:Tuple} =
     isconcretetype(T) &&
     Base.allocatedinline(T) &&
@@ -235,22 +237,44 @@ const PYTYPESTRDESCR = IdDict{Type,Tuple{String,Py}}()
 pytypestrdescr(::Type{T}) where {T} =
     get!(PYTYPESTRDESCR, T) do
         c = Utils.islittleendian() ? '<' : '>'
-        T == Bool ? ("$(c)b$(sizeof(Bool))", PyNULL) :
-        T == Int8 ? ("$(c)i1", PyNULL) :
-        T == UInt8 ? ("$(c)u1", PyNULL) :
-        T == Int16 ? ("$(c)i2", PyNULL) :
-        T == UInt16 ? ("$(c)u2", PyNULL) :
-        T == Int32 ? ("$(c)i4", PyNULL) :
-        T == UInt32 ? ("$(c)u4", PyNULL) :
-        T == Int64 ? ("$(c)i8", PyNULL) :
-        T == UInt64 ? ("$(c)u8", PyNULL) :
-        T == Float16 ? ("$(c)f2", PyNULL) :
-        T == Float32 ? ("$(c)f4", PyNULL) :
-        T == Float64 ? ("$(c)f8", PyNULL) :
-        T == Complex{Float16} ? ("$(c)c4", PyNULL) :
-        T == Complex{Float32} ? ("$(c)c8", PyNULL) :
-        T == Complex{Float64} ? ("$(c)c16", PyNULL) :
-        if isstructtype(T) && isconcretetype(T) && Base.allocatedinline(T)
+        if T == Bool
+            ("$(c)b$(sizeof(Bool))", PyNULL)
+        elseif T == Int8
+            ("$(c)i1", PyNULL)
+        elseif T == UInt8
+            ("$(c)u1", PyNULL)
+        elseif T == Int16
+            ("$(c)i2", PyNULL)
+        elseif T == UInt16
+            ("$(c)u2", PyNULL)
+        elseif T == Int32
+            ("$(c)i4", PyNULL)
+        elseif T == UInt32
+            ("$(c)u4", PyNULL)
+        elseif T == Int64
+            ("$(c)i8", PyNULL)
+        elseif T == UInt64
+            ("$(c)u8", PyNULL)
+        elseif T == Float16
+            ("$(c)f2", PyNULL)
+        elseif T == Float32
+            ("$(c)f4", PyNULL)
+        elseif T == Float64
+            ("$(c)f8", PyNULL)
+        elseif T == Complex{Float16}
+            ("$(c)c4", PyNULL)
+        elseif T == Complex{Float32}
+            ("$(c)c8", PyNULL)
+        elseif T == Complex{Float64}
+            ("$(c)c16", PyNULL)
+        elseif T <: Union{NumpyDates.InlineDateTime64,NumpyDates.InlineTimeDelta64}
+            u, m = NumpyDates.unitpair(T)
+            tc = T <: NumpyDates.InlineDateTime64 ? 'M' : 'm'
+            us =
+                u == NumpyDates.UNBOUND_UNITS ? "" :
+                m == 1 ? "[$(Symbol(u))]" : "[$(m)$(Symbol(u))]"
+            ("$(c)$(tc)8$(us)", PyNULL)
+        elseif isstructtype(T) && isconcretetype(T) && Base.allocatedinline(T)
             n = fieldcount(T)
             flds = []
             for i = 1:n

src/NumpyDates/DateTime64.jl

@@ -27,14 +27,9 @@ end
 
 # accessors
 
-function Dates.value(d::DateTime64)
-    d.value
-end
-
-function unitpair(d::DateTime64)
-    d.unit
-end
+Dates.value(d::DateTime64) = d.value
 
+unitpair(d::DateTime64) = d.unit
 
 # constructors
 

src/NumpyDates/InlineDateTime64.jl

@@ -29,13 +29,10 @@ end
 
 # accessors
 
-function Dates.value(d::InlineDateTime64)
-    d.value
-end
+Dates.value(d::InlineDateTime64) = d.value
 
-function unitpair(::InlineDateTime64{U}) where {U}
-    unitpair(U)
-end
+unitpair(::InlineDateTime64{U}) where {U} = unitpair(U)
+unitpair(::Type{InlineDateTime64{U}}) where {U} = unitpair(U)
 
 # constructors
 

src/NumpyDates/InlineTimeDelta64.jl

@@ -24,13 +24,10 @@ end
 
 # accessors
 
-function Dates.value(d::InlineTimeDelta64)
-    d.value
-end
+Dates.value(d::InlineTimeDelta64) = d.value
 
-function unitpair(::InlineTimeDelta64{U}) where {U}
-    unitpair(U)
-end
+unitpair(::InlineTimeDelta64{U}) where {U} = unitpair(U)
+unitpair(::Type{InlineTimeDelta64{U}}) where {U} = unitpair(U)
 
 # constructors
 

src/NumpyDates/TimeDelta64.jl

@@ -23,13 +23,9 @@ end
 
 # accessors
 
-function Dates.value(d::TimeDelta64)
-    d.value
-end
+Dates.value(d::TimeDelta64) = d.value
 
-function unitpair(d::TimeDelta64)
-    d.unit
-end
+unitpair(d::TimeDelta64) = d.unit
 
 # constructors
 

src/Wrap/PyArray.jl

@@ -278,6 +278,7 @@ pyarray_typestrdescr_to_type(ts::String, descr::Py) = begin
         s = m[3] === nothing ? 1 : parse(Int, m[3])
         us = m[4] === nothing ? "" : m[4]
         u =
+            us == "" ? NumpyDates.UNBOUND_UNITS :
             us == "Y" ? NumpyDates.YEARS :
             us == "M" ? NumpyDates.MONTHS :
             us == "W" ? NumpyDates.WEEKS :