SimpleArrayComplex128[i] = numpy.complex128 fails with cast error

[ThreeMonth03]

How to reproduce

import modmesh as mm
import numpy as np

nc = np.array([1.0 + 0.1j])
mc = mm.SimpleArrayComplex128([1])
mc[0] = nc[0]          # <- fails

print(mc[0].real, mc[0].imag)

Actual

Traceback (most recent call last):
  ...
  mc[0] = nc[0]
RuntimeError: Unable to cast Python instance of type <class 'numpy.complex128'> to C++ type '?' 
(#define PYBIND11_DETAILED_ERROR_MESSAGES or compile in debug mode for details)

Expected

mc[0] should accept a numpy.complex128 (and Python complex) and store it as modmesh::Complex<double> without extra wrapping.

Notes

• If we add additional initializer in WrapComplex.cpp:

           .def(
               py::init(
                   [](const std::complex<value_type> & c)
                   { return std::make_shared<wrapped_type>(wrapped_type{c.real(), c.imag()}); }),
               py::arg("np_complex"))

Constructing mm.complex128 from a Python/numpy complex could work, but assigning an element to SimpleArrayComplex128 doesn't work.

  mc = mm.complex128(np.complex128(1+0.1j))  # ok
  print(mc.real, mc.imag) # 1 0.1j
  ma = mm.SimpleArrayComplex128([1]) #ok  
  mc[0] = np.complex128(1+0.1j) # not ok

[yungyuc]

Agree that SimpleArray constructor should take complex-numbered ndarray.

@j8xixo12 , please comment.

[KHLee529]

Constructing mm.complex128 from a Python/numpy complex could work, but assigning an element to SimpleArrayComplex128 doesn't work.

mc = mm.complex128(np.complex128(1+0.1j)) # ok
print(mc.real, mc.imag) # 1 0.1j
ma = mm.SimpleArrayComplex128([1]) #ok
mc[0] = np.complex128(1+0.1j) # not ok

Could this be fixed by modifying the __setattr__ helper function?

[yungyuc]

Constructing mm.complex128 from a Python/numpy complex could work, but assigning an element to SimpleArrayComplex128 doesn't work.
mc = mm.complex128(np.complex128(1+0.1j)) # ok
print(mc.real, mc.imag) # 1 0.1j
ma = mm.SimpleArrayComplex128([1]) #ok
mc[0] = np.complex128(1+0.1j) # not ok

Could this be fixed by modifying the __setattr__ helper function?

It can be clarified with a PR.