Skip to content

Conversation

@Tobias-Fischer
Copy link
Contributor

@Tobias-Fischer Tobias-Fischer commented Dec 10, 2022

In file included from $SRC_DIR/ros-humble-slam-toolbox/src/work/lib/karto_sdk/src/Karto.cpp:27: $SRC_DIR/ros-humble-slam-toolbox/src/work/lib/karto_sdk/include/karto_sdk/Karto.h: In member function 'const karto::Pose2& karto::LocalizedRangeScan::GetBarycenterPose() const': $SRC_DIR/ros-humble-slam-toolbox/src/work/lib/karto_sdk/include/karto_sdk/Karto.h:5503:12: error: 'unique_lock' is not a member of 'std'
 5503 |       std::unique_lock<std::shared_mutex> uniqueLock(m_Lock);
      |            ^~~~~~~~~~~

Basic Info

Info Please fill out this column
Ticket(s) this addresses (add tickets here #1)
Primary OS tested on (Ubuntu, MacOS, Windows)
Robotic platform tested on (Steve's Robot, gazebo simulation of Tally, hardware turtlebot)

Description of contribution in a few bullet points

Description of documentation updates required from your changes


Future work that may be required in bullet points

In file included from $SRC_DIR/ros-humble-slam-toolbox/src/work/lib/karto_sdk/src/Karto.cpp:27:
$SRC_DIR/ros-humble-slam-toolbox/src/work/lib/karto_sdk/include/karto_sdk/Karto.h: In member function 'const karto::Pose2& karto::LocalizedRangeScan::GetBarycenterPose() const':
$SRC_DIR/ros-humble-slam-toolbox/src/work/lib/karto_sdk/include/karto_sdk/Karto.h:5503:12: error: 'unique_lock' is not a member of 'std'
 5503 |       std::unique_lock<std::shared_mutex> uniqueLock(m_Lock);
      |            ^~~~~~~~~~~
@SteveMacenski
Copy link
Owner

SteveMacenski commented Dec 10, 2022

@Tobias-Fischer please open against ros2 as well so that all future distributions will have it reflected as well! Then I'll merge the pair

Tobias-Fischer added a commit to Tobias-Fischer/slam_toolbox that referenced this pull request Mar 3, 2023
SteveMacenski pushed a commit that referenced this pull request Mar 3, 2023
@SteveMacenski SteveMacenski merged commit f66f982 into SteveMacenski:humble Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants