Fix type hint syntax - use ndim=N not numpy.ndim[N]

- Corrected docstring type hints to use proper ndim=1/ndim=2 syntax
- Generated code now has valid type annotations
- All tests pass (10 passed, 1 skipped)

Co-authored-by: jpfeuffer <8102638+jpfeuffer@users.noreply.github.com>

Author: Copilot

autowrap/ConversionProvider.py

@@ -2076,14 +2076,14 @@ def matching_python_type_full(self, cpp_type: CppType) -> str:
         (tt,) = cpp_type.template_args
 
         if self._is_nested_vector(cpp_type):
-            # For 2D arrays, use NDArray type hint
+            # For 2D arrays, use proper NDArray type hint syntax
             (inner_tt,) = tt.template_args
             dtype = self._get_numpy_dtype(inner_tt)
-            return f"numpy.ndarray[numpy.{dtype}_t, numpy.ndim[2]]"
+            return f"numpy.ndarray[numpy.{dtype}_t, ndim=2]"
         else:
-            # For 1D arrays, use NDArray type hint
+            # For 1D arrays, use proper NDArray type hint syntax
             dtype = self._get_numpy_dtype(tt)
-            return f"numpy.ndarray[numpy.{dtype}_t, numpy.ndim[1]]"
+            return f"numpy.ndarray[numpy.{dtype}_t, ndim=1]"
 
     def type_check_expression(self, cpp_type: CppType, argument_var: str) -> str:
         """Check if argument is a numpy array (strict - no lists)."""

tests/test_files/numpy_vector/numpy_vector_wrapper.pyi

@@ -17,49 +17,49 @@ class NumpyVectorTest:
         """
         ...
 
-    def getConstRefVector(self) -> numpy.ndarray[numpy.float64_t, numpy.ndim[1]]:
+    def getConstRefVector(self) -> numpy.ndarray[numpy.float64_t, ndim=1]:
         """
         Cython signature: const libcpp_vector_as_np[double] & getConstRefVector()
         """
         ...
 
-    def getMutableRefVector(self) -> numpy.ndarray[numpy.float64_t, numpy.ndim[1]]:
+    def getMutableRefVector(self) -> numpy.ndarray[numpy.float64_t, ndim=1]:
         """
         Cython signature: libcpp_vector_as_np[double] & getMutableRefVector()
         """
         ...
 
-    def getValueVector(self, size: int ) -> numpy.ndarray[numpy.float64_t, numpy.ndim[1]]:
+    def getValueVector(self, size: int ) -> numpy.ndarray[numpy.float64_t, ndim=1]:
         """
         Cython signature: libcpp_vector_as_np[double] getValueVector(size_t size)
         """
         ...
 
-    def sumVector(self, data: numpy.ndarray[numpy.float64_t, numpy.ndim[1]] ) -> float:
+    def sumVector(self, data: numpy.ndarray[numpy.float64_t, ndim=1] ) -> float:
         """
         Cython signature: double sumVector(libcpp_vector_as_np[double] data)
         """
         ...
 
-    def sumIntVector(self, data: numpy.ndarray[numpy.int32_t, numpy.ndim[1]] ) -> int:
+    def sumIntVector(self, data: numpy.ndarray[numpy.int32_t, ndim=1] ) -> int:
         """
         Cython signature: int sumIntVector(libcpp_vector_as_np[int] data)
         """
         ...
 
-    def createFloatVector(self, size: int ) -> numpy.ndarray[numpy.float32_t, numpy.ndim[1]]:
+    def createFloatVector(self, size: int ) -> numpy.ndarray[numpy.float32_t, ndim=1]:
         """
         Cython signature: libcpp_vector_as_np[float] createFloatVector(size_t size)
         """
         ...
 
-    def create2DVector(self, rows: int , cols: int ) -> numpy.ndarray[numpy.float64_t, numpy.ndim[2]]:
+    def create2DVector(self, rows: int , cols: int ) -> numpy.ndarray[numpy.float64_t, ndim=2]:
         """
         Cython signature: libcpp_vector_as_np[libcpp_vector_as_np[double]] create2DVector(size_t rows, size_t cols)
         """
         ...
 
-    def sum2DVector(self, data: numpy.ndarray[numpy.float64_t, numpy.ndim[2]] ) -> float:
+    def sum2DVector(self, data: numpy.ndarray[numpy.float64_t, ndim=2] ) -> float:
         """
         Cython signature: double sum2DVector(libcpp_vector_as_np[libcpp_vector_as_np[double]] data)
         """

tests/test_files/numpy_vector/numpy_vector_wrapper.pyx

@@ -53,7 +53,7 @@ cdef class NumpyVectorTest:
 
     def getConstRefVector(self):
         """
-        getConstRefVector(self) -> numpy.ndarray[numpy.float64_t, numpy.ndim[1]]
+        getConstRefVector(self) -> numpy.ndarray[numpy.float64_t, ndim=1]
         """
         _r = self.inst.get().getConstRefVector()
         # Convert C++ vector to numpy array COPY (Python owns data)
@@ -65,7 +65,7 @@ cdef class NumpyVectorTest:
 
     def getMutableRefVector(self):
         """
-        getMutableRefVector(self) -> numpy.ndarray[numpy.float64_t, numpy.ndim[1]]
+        getMutableRefVector(self) -> numpy.ndarray[numpy.float64_t, ndim=1]
         """
         _r = self.inst.get().getMutableRefVector()
         # Convert C++ vector to numpy array COPY (Python owns data)
@@ -77,7 +77,7 @@ cdef class NumpyVectorTest:
 
     def getValueVector(self,  size ):
         """
-        getValueVector(self, size: int ) -> numpy.ndarray[numpy.float64_t, numpy.ndim[1]]
+        getValueVector(self, size: int ) -> numpy.ndarray[numpy.float64_t, ndim=1]
         """
         assert isinstance(size, int) and size >= 0, 'arg size wrong type'
 
@@ -91,7 +91,7 @@ cdef class NumpyVectorTest:
 
     def sumVector(self, numpy.ndarray[numpy.float64_t, ndim=1] data ):
         """
-        sumVector(self, data: numpy.ndarray[numpy.float64_t, numpy.ndim[1]] ) -> float
+        sumVector(self, data: numpy.ndarray[numpy.float64_t, ndim=1] ) -> float
         """
         assert isinstance(data, numpy.ndarray), 'arg data wrong type'
         # Convert 1D numpy array to C++ vector (fast memcpy)
@@ -107,7 +107,7 @@ cdef class NumpyVectorTest:
 
     def sumIntVector(self, numpy.ndarray[numpy.int32_t, ndim=1] data ):
         """
-        sumIntVector(self, data: numpy.ndarray[numpy.int32_t, numpy.ndim[1]] ) -> int
+        sumIntVector(self, data: numpy.ndarray[numpy.int32_t, ndim=1] ) -> int
         """
         assert isinstance(data, numpy.ndarray), 'arg data wrong type'
         # Convert 1D numpy array to C++ vector (fast memcpy)
@@ -123,7 +123,7 @@ cdef class NumpyVectorTest:
 
     def createFloatVector(self,  size ):
         """
-        createFloatVector(self, size: int ) -> numpy.ndarray[numpy.float32_t, numpy.ndim[1]]
+        createFloatVector(self, size: int ) -> numpy.ndarray[numpy.float32_t, ndim=1]
         """
         assert isinstance(size, int) and size >= 0, 'arg size wrong type'
 
@@ -137,7 +137,7 @@ cdef class NumpyVectorTest:
 
     def create2DVector(self,  rows ,  cols ):
         """
-        create2DVector(self, rows: int , cols: int ) -> numpy.ndarray[numpy.float64_t, numpy.ndim[2]]
+        create2DVector(self, rows: int , cols: int ) -> numpy.ndarray[numpy.float64_t, ndim=2]
         """
         assert isinstance(rows, int) and rows >= 0, 'arg rows wrong type'
         assert isinstance(cols, int) and cols >= 0, 'arg cols wrong type'
@@ -158,7 +158,7 @@ cdef class NumpyVectorTest:
 
     def sum2DVector(self, numpy.ndarray[numpy.float64_t, ndim=2] data ):
         """
-        sum2DVector(self, data: numpy.ndarray[numpy.float64_t, numpy.ndim[2]] ) -> float
+        sum2DVector(self, data: numpy.ndarray[numpy.float64_t, ndim=2] ) -> float
         """
         assert isinstance(data, numpy.ndarray), 'arg data wrong type'
         # Convert 2D numpy array to nested C++ vector