-
Notifications
You must be signed in to change notification settings - Fork 7.4k
[pcl] PCL VTK9 support. #16700
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[pcl] PCL VTK9 support. #16700
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,15 +1,15 @@ | ||
| diff --git a/CMakeLists.txt b/CMakeLists.txt | ||
| index 6c452ce..2c50b8d 100644 | ||
| index 456009cd3..c1998252f 100644 | ||
| --- a/CMakeLists.txt | ||
| +++ b/CMakeLists.txt | ||
| @@ -303,6 +303,10 @@ set(FLANN_USE_STATIC ON) | ||
| find_package(FLANN 1.7.0 REQUIRED) | ||
| find_package(lz4) | ||
|
|
||
| +if(UNIX AND CMAKE_CXX_COMPILER_VERSION VERSION_LESS 6.0) | ||
| + set (CMAKE_CXX_STANDARD 17) | ||
| + set(CMAKE_CXX_STANDARD 17) | ||
| +endif() | ||
| + | ||
| # libusb-1.0 | ||
| # libusb | ||
| option(WITH_LIBUSB "Build USB RGBD-Camera drivers" TRUE) | ||
| if(WITH_LIBUSB) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,20 +1,26 @@ | ||
| diff --git a/io/CMakeLists.txt b/io/CMakeLists.txt | ||
| index f234971..86ea242 100644 | ||
| index d26e60421..893104b77 100644 | ||
| --- a/io/CMakeLists.txt | ||
| +++ b/io/CMakeLists.txt | ||
| @@ -333,12 +333,13 @@ set(LIB_NAME "pcl_${SUBSYS_NAME}") | ||
| add_definitions(${VTK_DEFINES}) | ||
| PCL_ADD_LIBRARY(${LIB_NAME} COMPONENT ${SUBSYS_NAME} SOURCES ${srcs} ${incs} ${compression_incs} ${impl_incs} ${OPENNI_INCLUDES} ${OPENNI2_INCLUDES}) | ||
| @@ -334,6 +334,10 @@ PCL_ADD_LIBRARY(${LIB_NAME} COMPONENT ${SUBSYS_NAME} SOURCES ${srcs} ${incs} ${c | ||
|
|
||
| target_include_directories(${LIB_NAME} PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include") | ||
| -link_directories(${VTK_LINK_DIRECTORIES}) | ||
| -target_link_libraries("${LIB_NAME}" pcl_common pcl_io_ply ${VTK_LIBRARIES}) | ||
| if(PNG_FOUND) | ||
| target_link_libraries("${LIB_NAME}" ${PNG_LIBRARIES}) | ||
| endif() | ||
|
|
||
| +link_directories(${VTK_LINK_DIRECTORIES}) | ||
| +target_link_libraries("${LIB_NAME}" pcl_common pcl_io_ply ${VTK_LIBRARIES}) | ||
| +if(PNG_FOUND) | ||
| + target_link_libraries("${LIB_NAME}" ${PNG_LIBRARIES}) | ||
| +endif() | ||
| + | ||
| if(LIBUSB_1_FOUND) | ||
| target_link_libraries("${LIB_NAME}" ${LIBUSB_1_LIBRARIES}) | ||
| target_link_libraries("${LIB_NAME}" pcl_common pcl_io_ply) | ||
| if(VTK_FOUND) | ||
| if(${VTK_VERSION} VERSION_LESS 9.0) | ||
| @@ -347,10 +351,6 @@ if(VTK_FOUND) | ||
| endif() | ||
| endif() | ||
|
|
||
| -if(PNG_FOUND) | ||
| - target_link_libraries("${LIB_NAME}" ${PNG_LIBRARIES}) | ||
| -endif() | ||
| - | ||
| if(LIBUSB_FOUND) | ||
| target_link_libraries("${LIB_NAME}" libusb::libusb) | ||
| endif() | ||
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,36 +1,14 @@ | ||
| diff --git a/CMakeLists.txt b/CMakeLists.txt | ||
| index 1cb24ae36..a5414e26e 100644 | ||
| --- a/CMakeLists.txt | ||
| +++ b/CMakeLists.txt | ||
| @@ -320,6 +320,9 @@ IF(WITH_QT) | ||
| ENDIF() | ||
| ADD_DEFINITIONS(-DQT_NO_KEYWORDS) # To avoid conflicts with boost signals/foreach and Qt macros | ||
| ENDIF(QT4_FOUND OR Qt5_FOUND) | ||
| +ELSE() | ||
| + # Unconditionally disable VTK related features since no visualization tools will be built. | ||
| + ADD_DEFINITIONS("-DDISABLE_VTK") | ||
| ENDIF(WITH_QT) | ||
|
|
||
| IF(WITH_SUPERPOINT_TORCH) | ||
| diff --git a/corelib/src/Features2d.cpp b/corelib/src/Features2d.cpp | ||
| index e4048eefb..862cdc68f 100644 | ||
| --- a/corelib/src/Features2d.cpp | ||
| +++ b/corelib/src/Features2d.cpp | ||
| @@ -982,7 +982,7 @@ std::vector<cv::KeyPoint> SIFT::generateKeypointsImpl(const cv::Mat & image, con | ||
| { | ||
| UASSERT(!image.empty() && image.channels() == 1 && image.depth() == CV_8U); | ||
| std::vector<cv::KeyPoint> keypoints; | ||
| -#if defined(RTABMAP_NONFREE) || CV_MAJOR_VERSION > 4 || (CV_MAJOR_VERSION == 4 && CV_MINOR_VERSION >= 3) | ||
| +#if defined(RTABMAP_NONFREE) && ( CV_MAJOR_VERSION > 4 || (CV_MAJOR_VERSION == 4 && CV_MINOR_VERSION >= 3) ) | ||
| cv::Mat imgRoi(image, roi); | ||
| cv::Mat maskRoi; | ||
| if(!mask.empty()) | ||
| @@ -1000,7 +1000,7 @@ cv::Mat SIFT::generateDescriptorsImpl(const cv::Mat & image, std::vector<cv::Key | ||
| { | ||
| UASSERT(!image.empty() && image.channels() == 1 && image.depth() == CV_8U); | ||
| cv::Mat descriptors; | ||
| -#if defined(RTABMAP_NONFREE) || CV_MAJOR_VERSION > 4 || (CV_MAJOR_VERSION == 4 && CV_MINOR_VERSION >= 3) | ||
| +#if defined(RTABMAP_NONFREE) && ( CV_MAJOR_VERSION > 4 || (CV_MAJOR_VERSION == 4 && CV_MINOR_VERSION >= 3) ) | ||
| _sift->compute(image, keypoints, descriptors); | ||
|
|
||
| if( rootSIFT_ && !descriptors.empty()) | ||
| diff --git a/CMakeLists.txt b/CMakeLists.txt | ||
| index c9adfe34..7e397cdb 100644 | ||
| --- a/CMakeLists.txt | ||
| +++ b/CMakeLists.txt | ||
| @@ -325,6 +325,9 @@ IF(WITH_QT) | ||
| ENDIF() | ||
| ADD_DEFINITIONS(-DQT_NO_KEYWORDS) # To avoid conflicts with boost signals/foreach and Qt macros | ||
| ENDIF(QT4_FOUND OR Qt5_FOUND) | ||
| + ELSE() | ||
| + # Unconditionally disable VTK related features since no visualization tools will be built. | ||
| + ADD_DEFINITIONS("-DDISABLE_VTK") | ||
| ENDIF(WITH_QT) | ||
|
|
||
| IF(WITH_TORCH) |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,7 +1,7 @@ | ||||||
| { | ||||||
| "name": "rtabmap", | ||||||
| "version-string": "0.20.3", | ||||||
| "port-version": 1, | ||||||
| "version-string": "0.20.8", | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
This might be |
||||||
| "port-version": 2, | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Since this is a new version update, it should be |
||||||
| "description": "Real-Time Appearance-Based Mapping", | ||||||
| "homepage": "https://introlab.github.io/rtabmap/", | ||||||
| "supports": "windows & !static", | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,5 +1,10 @@ | ||||||||||
| { | ||||||||||
| "versions": [ | ||||||||||
| { | ||||||||||
| "git-tree": "5d777469ff46260991be81cdc4600f5bb8847365", | ||||||||||
| "version-string": "0.20.8", | ||||||||||
| "port-version": 2 | ||||||||||
|
Comment on lines
+5
to
+6
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||
| }, | ||||||||||
| { | ||||||||||
| "git-tree": "07422ec8ed79148e476dbb807db950ed78852ca0", | ||||||||||
| "version-string": "0.20.3", | ||||||||||
|
|
||||||||||
Uh oh!
There was an error while loading. Please reload this page.