-
Notifications
You must be signed in to change notification settings - Fork 523
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
Port robot_state to ROS 2 #59
Closed
Closed
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
159f45b
Port robot_state to ROS2
vmayoral 1b79abb
robot_state, adapt to ROS2
vmayoral 346b932
robot_state, remove log.h
vmayoral a226084
Restore broken comment
vmayoral c736767
robot_state, apply clang format
vmayoral 4caa947
Change logger to include separator
vmayoral 9c5e111
Restoring invertVelocity method in robot_state
ahcorde c5cf4a5
Remove some comments related con printfs and ROS logging
ahcorde a17c5d9
robot_state fix Time related issues
ahcorde 813681f
Fixing robot_state test_aabb pointers
ahcorde d76dd5c
robot_state restore harmonizePositions methods
ahcorde File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,41 +1,64 @@ | ||
set(MOVEIT_LIB_NAME moveit_robot_state) | ||
|
||
if(CMAKE_CXX_COMPILER_ID STREQUAL "Clang") | ||
# clang is picky about typeid(*solver) | ||
add_compile_options(-Wno-potentially-evaluated-expression) | ||
endif() | ||
|
||
add_library(${MOVEIT_LIB_NAME} | ||
src/attached_body.cpp | ||
src/conversions.cpp | ||
src/robot_state.cpp | ||
) | ||
set_target_properties(${MOVEIT_LIB_NAME} PROPERTIES VERSION "${${PROJECT_NAME}_VERSION}") | ||
|
||
target_link_libraries(${MOVEIT_LIB_NAME} moveit_robot_model moveit_kinematics_base moveit_transforms ${catkin_LIBRARIES} ${urdfdom_LIBRARIES} ${urdfdom_headers_LIBRARIES} ${Boost_LIBRARIES}) | ||
|
||
add_dependencies(${MOVEIT_LIB_NAME} ${catkin_EXPORTED_TARGETS}) | ||
set_target_properties(${MOVEIT_LIB_NAME} PROPERTIES VERSION ${${PROJECT_NAME}_VERSION}) | ||
target_link_libraries(${MOVEIT_LIB_NAME} moveit_robot_model moveit_kinematics_base moveit_transforms ${rclcpp_LIBRARIES} ${rmw_implementation_LIBRARIES} ${urdfdom_LIBRARIES} ${urdfdom_headers_LIBRARIES} ${Boost_LIBRARIES}) | ||
ament_target_dependencies(${MOVEIT_LIB_NAME} | ||
moveit_robot_model | ||
moveit_kinematics_base | ||
moveit_transforms | ||
urdf | ||
tf2_geometry_msgs | ||
) | ||
|
||
install(TARGETS ${MOVEIT_LIB_NAME} | ||
LIBRARY DESTINATION ${CATKIN_PACKAGE_LIB_DESTINATION} | ||
ARCHIVE DESTINATION ${CATKIN_PACKAGE_LIB_DESTINATION}) | ||
install(DIRECTORY include/ DESTINATION ${CATKIN_GLOBAL_INCLUDE_DESTINATION}) | ||
|
||
# Unit tests | ||
if(CATKIN_ENABLE_TESTING) | ||
find_package(moveit_resources REQUIRED) | ||
include_directories(${moveit_resources_INCLUDE_DIRS}) | ||
|
||
catkin_add_gtest(test_robot_state test/robot_state_test.cpp) | ||
target_link_libraries(test_robot_state moveit_test_utils ${catkin_LIBRARIES} ${urdfdom_LIBRARIES} ${urdfdom_headers_LIBRARIES} ${MOVEIT_LIB_NAME}) | ||
|
||
# As an executable, this benchmark is not run as a test by default | ||
add_executable(robot_state_benchmark test/robot_state_benchmark.cpp) | ||
target_link_libraries(robot_state_benchmark moveit_test_utils ${catkin_LIBRARIES} ${urdfdom_LIBRARIES} ${urdfdom_headers_LIBRARIES} ${MOVEIT_LIB_NAME} ${GTEST_LIBRARIES}) | ||
|
||
catkin_add_gtest(test_robot_state_complex test/test_kinematic_complex.cpp) | ||
target_link_libraries(test_robot_state_complex moveit_test_utils ${catkin_LIBRARIES} ${urdfdom_LIBRARIES} ${urdfdom_headers_LIBRARIES} ${MOVEIT_LIB_NAME}) | ||
LIBRARY DESTINATION lib | ||
ARCHIVE DESTINATION lib) | ||
install(DIRECTORY include/ DESTINATION include) | ||
|
||
catkin_add_gtest(test_aabb test/test_aabb.cpp) | ||
target_link_libraries(test_aabb moveit_test_utils ${catkin_LIBRARIES} ${urdfdom_LIBRARIES} ${urdfdom_headers_LIBRARIES} ${MOVEIT_LIB_NAME}) | ||
endif() | ||
# # Unit tests | ||
# if(BUILD_TESTING) | ||
# find_package(ament_cmake_gtest REQUIRED) | ||
# find_package(moveit_resources REQUIRED) | ||
# | ||
# include_directories(${moveit_resources_INCLUDE_DIRS}) | ||
# | ||
# ament_add_gtest(test_robot_state test/robot_state_test.cpp) | ||
# ament_target_dependencies(test_robot_state | ||
# tf2_geometry_msgs | ||
# ${MOVEIT_LIB_NAME} | ||
# urdf | ||
# visualization_msgs | ||
# ) | ||
# | ||
# # As an executable, this benchmark is not run as a test by default | ||
# add_executable(robot_state_benchmark test/robot_state_benchmark.cpp) | ||
# ament_target_dependencies(robot_state_benchmark | ||
# moveit_test_utils | ||
# tf2_geometry_msgs | ||
# ${MOVEIT_LIB_NAME} | ||
# urdf | ||
# visualization_msgs | ||
# ) | ||
# | ||
# ament_add_gtest(test_robot_state_complex test/test_kinematic_complex.cpp) | ||
# ament_target_dependencies(test_robot_state_complex | ||
# moveit_test_utils | ||
# ${MOVEIT_LIB_NAME} | ||
# urdf | ||
# visualization_msgs | ||
# ) | ||
# | ||
# ament_add_gtest(test_aabb test/test_aabb.cpp) | ||
# ament_target_dependencies(test_aabb | ||
# moveit_test_utils | ||
# ${MOVEIT_LIB_NAME} | ||
# urdf | ||
# visualization_msgs | ||
# tf2_geometry_msgs | ||
# ) | ||
# endif() |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert these kind of changes