Complete pybind11 wrappings for V2, V3, V4, and add pyBindImathTest (AcademySoftwareFoundation#472)

This does three things:

1. Completes the vector class wrapping, so all previous functionality
   is supported, including interoperability between vector classes
   (i.e. V2i + V2f), and interoperability between vectors and tuples
   and lists (i.e. V2i(1,2) + [1,2]).

2. Adds src/pybind11/PyBindImathTest/pyBindImathTest, which is a
   duplicate of the original test script for the Boost-based bindings,
   with the not-yet-implemented features commented out for now.

3. Adds wrappings for miscellaneous functions in PyImathFun.cpp, which
   are needed by the vector tests.

The vector class tests pass, so the pybind11 functionality is now
consistent with existing Boost-based bindings.

All new wrappings should un-comment out the appropriate tests and
validate that they pass.

Signed-off-by: Cary Phillips <cary@ilm.com>

Author: cary-ilm

.github/workflows/ci_steps.yml

@@ -123,7 +123,7 @@ jobs:
 
       - name: Upload install_manifest.txt
         # Upload the manifest to make it possible to download for inspection and debugging
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@b4b15b8c7c6ac21ea08fcf65892d2ee8f75cf882 # v4.4.3
         with:
           name: ${{ env.INSTALL_MANIFEST }}
           path: _build/${{ env.INSTALL_MANIFEST }}

src/pybind11/CMakeLists.txt

@@ -15,3 +15,8 @@ set(PYBINDIMATH_LIB_SUFFIX "-${IMATH_VERSION_API}" CACHE STRING "String added to
 set(PYBINDIMATH_LIB_PYTHONVER_ROOT "_Python" CACHE STRING "String added as a root to the identifier of the python version in the libraries")
 
 add_subdirectory(PyBindImath)
+if(BUILD_TESTING)
+  enable_testing()
+  add_subdirectory( PyBindImathTest )
+endif()
+

src/pybind11/PyBindImath/CMakeLists.txt

@@ -9,6 +9,7 @@ find_package(pybind11 REQUIRED)
 #
 
 set(PYBINDIMATH_SOURCES
+    PyBindImathFun.cpp
     PyBindImathBox.cpp
     PyBindImathVec.cpp
     PyBindImathPlane.cpp
@@ -120,4 +121,5 @@ if(IMATH_BUILD_APPLE_FRAMEWORKS)
         MACOSX_FRAMEWORK_IDENTIFIER github.com/AcademySoftwareFoundation/Imath.PyBind
         MACOSX_FRAMEWORK_BUNDLE_VERSION PYBINDIMATH_LIB_SUFFIX
         MACOSX_FRAMEWORK_SHORT_VERSION_STRING ${Imath_VERSION})
-endif()
+endif()
+

src/pybind11/PyBindImath/PyBindImath.h

@@ -17,6 +17,8 @@
 
 namespace PyBindImath {
 
+PYBINDIMATH_EXPORT void register_imath_fun(pybind11::module& m);
+
 PYBINDIMATH_EXPORT void register_imath_vec(pybind11::module& m);
 PYBINDIMATH_EXPORT void register_imath_box(pybind11::module& m);
 PYBINDIMATH_EXPORT void register_imath_plane(pybind11::module& m);

src/pybind11/PyBindImath/PyBindImathFun.cpp

@@ -0,0 +1,33 @@
+//
+// SPDX-License-Identifier: BSD-3-Clause
+// Copyright Contributors to the OpenEXR Project.
+//
+
+#include "PyBindImath.h"
+#include <ImathFun.h>
+
+namespace py = pybind11;
+
+namespace PyBindImath {
+
+template <class T>
+void
+register_imath_funT(py::module& m)
+{
+    m.def("cmp", IMATH_NAMESPACE::cmp<T>);
+    m.def("cmpt", IMATH_NAMESPACE::cmpt<T>);
+    m.def("iszero", IMATH_NAMESPACE::iszero<T>);
+    m.def("equal", IMATH_NAMESPACE::equal<T, T, T>);
+
+}
+
+void
+register_imath_fun(py::module& m)
+{
+    register_imath_funT<int>(m);
+    register_imath_funT<float>(m);
+    register_imath_funT<double>(m);
+}
+
+
+} // namespace PyBindImath