diff --git a/cmake/FindDependencies.cmake b/cmake/FindDependencies.cmake index 75a05875..f35171a9 100644 --- a/cmake/FindDependencies.cmake +++ b/cmake/FindDependencies.cmake @@ -54,6 +54,7 @@ endif() message(STATUS "Configuring PoseLib... done") # COLMAP +set(LSD_ENABLED OFF CACHE BOOL "Disable LSD in COLMAP" FORCE) FetchContent_Declare(COLMAP GIT_REPOSITORY https://github.com/colmap/colmap.git GIT_TAG c9097b031ee00da22609c7ac4b3f6b45b4178de2 @@ -70,7 +71,7 @@ message(STATUS "Configuring COLMAP... done") # JLinkage FetchContent_Declare(JLinkage GIT_REPOSITORY https://github.com/B1ueber2y/JLinkage.git - GIT_TAG 452d67eda005db01a02071a5af8f0eced0a02079 + GIT_TAG 75dadd555f81b1cf1b0f016d8cac76f3b554ba9b EXCLUDE_FROM_ALL ) message(STATUS "Configuring JLinkage...") diff --git a/limap/base/bindings.cc b/limap/base/bindings.cc index 102d6a8f..c1513b8a 100644 --- a/limap/base/bindings.cc +++ b/limap/base/bindings.cc @@ -804,7 +804,7 @@ void bind_camera(py::module &m) { .def(py::init(), R"( Default constructor: identity pose )", - py::arg("initialized") = false) + py::arg("init") = false) .def(py::init(), R"( Copy constructor )", @@ -812,11 +812,11 @@ void bind_camera(py::module &m) { .def(py::init(), R"( Constructor from a quaternion vector and a translation vector )", - py::arg("qvec"), py::arg("tvec"), py::arg("initialized") = true) + py::arg("qvec"), py::arg("tvec"), py::arg("init") = true) .def(py::init(), R"( Constructor from a rotation matrix and a translation vector )", - py::arg("R"), py::arg("tvec"), py::arg("initialized") = true) + py::arg("R"), py::arg("tvec"), py::arg("init") = true) .def(py::init(), R"( Constructor from a Python dict )", diff --git a/limap/base/camera.h b/limap/base/camera.h index ac8a29dd..b60a506e 100644 --- a/limap/base/camera.h +++ b/limap/base/camera.h @@ -87,11 +87,10 @@ class Camera : public colmap::Camera { class CameraPose { public: - CameraPose(bool initialized = false) : initialized(initialized) {} - CameraPose(V4D qvec, V3D tvec, bool initialized = true) - : qvec(qvec.normalized()), tvec(tvec), initialized(initialized) {} - CameraPose(M3D R, V3D T, bool initiallized = true) - : tvec(T), initialized(initialized) { + CameraPose(bool init = false) : initialized(init) {} + CameraPose(V4D qvec, V3D tvec, bool init = true) + : qvec(qvec.normalized()), tvec(tvec), initialized(init) {} + CameraPose(M3D R, V3D t, bool init = true) : tvec(t), initialized(init) { qvec = RotationMatrixToQuaternion(R); } CameraPose(py::dict dict); diff --git a/limap/base/line_dists.cc b/limap/base/line_dists.cc index ee76d014..ab771346 100644 --- a/limap/base/line_dists.cc +++ b/limap/base/line_dists.cc @@ -218,6 +218,8 @@ double compute_distance(const Line2d &l1, const Line2d &l2, case LineDistType::ENDPOINTS_SCALEINV: throw std::runtime_error( "Type error. Scale invariance distance is not supported for Line2d."); + default: + throw std::runtime_error("Distance type not supported."); } return -1.0; } @@ -260,6 +262,8 @@ double compute_distance(const Line3d &l1, const Line3d &l2, return dist_endpoints_scaleinv_oneway(l1, l2); case LineDistType::ENDPOINTS_SCALEINV: return dist_endpoints_scaleinv(l1, l2); + default: + throw std::runtime_error("Distance type not supported."); } return -1.0; } diff --git a/limap/util/simple_logger.h b/limap/util/simple_logger.h index f150b6ec..4051a1ba 100644 --- a/limap/util/simple_logger.h +++ b/limap/util/simple_logger.h @@ -170,7 +170,7 @@ class LogProgressbar : public progressbar { if (update_is_called) progress += inc; std::string out = get_update_str(); - if (out.c_str() != "\r") { + if (out != "\r") { STDLOG(CERR) << out; } } @@ -189,7 +189,7 @@ class SyncLogProgressbar : public progressbar { if (update_is_called) progress += inc; std::string out = get_update_str(); - if (out.c_str() != "\r") { + if (out != "\r") { SYNC_LOG(CERR) << out; } }