Example:

import pycolmap
p = pycolmap.Point2D()
p.point3D_id = pycolmap.INVALID_POINT3D_ID

On the other hand, I have made several attempts to improve the doc rendering of Image class (e.g., B1ueber2y@e205e62). But no luck so far.

Thanks. Nit: For consistency across Python, I would suggest calling it INVALID_POINT3D_ID etc. It could also make sense to adjust the str __repr__ to print out kInvalidPoint3DId or INVALID_POINT3D_ID instead of -1?

Thanks. Nit: For consistency across Python, I would suggest calling it INVALID_POINT3D_ID etc. It could also make sense to adjust the str __repr__ to print out kInvalidPoint3DId or INVALID_POINT3D_ID instead of -1?

Thanks. I have updated the names to upper cases. Now the __repr__ is controlled in the C++ side after the recent update: https://github.com/colmap/colmap/blob/main/src/colmap/scene/image.cc#L107 (while the one for point2D is missing still), so the pybind part does not help.

On the other hand, I have made several attempts to improve the doc rendering of Image class (e.g., B1ueber2y@e205e62). But no luck so far.

You need to replace

"camera_id"_a = kInvalidCameraId,

by

py::arg_v("camera_id", kInvalidCameraId, "pycolmap.INVALID_CAMERA_ID"),

and so on.

On the other hand, I have made several attempts to improve the doc rendering of Image class (e.g., B1ueber2y@e205e62). But no luck so far.

You need to replace

"camera_id"_a = kInvalidCameraId,

by

py::arg_v("camera_id", kInvalidCameraId, "pycolmap.INVALID_CAMERA_ID"),

and so on.

Thanks. Updated.

Thanks, can you also update this one?

colmap/src/pycolmap/scene/point2d.cc

Line 28 in cd97b65

"point3D_id"_a = kInvalidPoint3DId)

I think this was already updated?

I missed it, sorry.