Fix numpy_num_face_vertices() (#408)

paulmelnikow · Copilot · web-flow · commit fcc90055f8f8 · 2026-03-04T02:52:39.000+09:00
* Test Python + NumPy in CI

* Add comments

* Add more tests

* Add more Python tests

* assert_array_almost_equal

* Try decimal=5

* Try to fix vertex test

* Update a comment

* Upgrade cibuildwheel

* Fix toml

* Match the types and call buf.release()

* Rm buf.release()

* Try to clear license warning on build

* Fix license specifier

* Move python tests out of python so they don’t end up in the wheel

* Add some excludes

* Reformat and exclude some python files

* Update .github/workflows/python.yml

Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;

* Add a note about local numpy install

* Reset a few files

---------

Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/python/bindings.cc b/python/bindings.cc
@@ -149,9 +149,9 @@ PYBIND11_MODULE(tinyobjloader, tobj_module)
     .def(py::init<>())
     .def_readonly("num_face_vertices", &mesh_t::num_face_vertices)
     .def("numpy_num_face_vertices", [] (mesh_t &instance) {
-        auto ret = py::array_t<unsigned char>(instance.num_face_vertices.size());
+        auto ret = py::array_t<unsigned int>(instance.num_face_vertices.size());
         py::buffer_info buf = ret.request();
-        memcpy(buf.ptr, instance.num_face_vertices.data(), instance.num_face_vertices.size() * sizeof(unsigned char));
+        memcpy(buf.ptr, instance.num_face_vertices.data(), instance.num_face_vertices.size() * sizeof(unsigned int));
         return ret;
     })
     .def("vertex_indices", [](mesh_t &self) {
