This repository was archived by the owner on Jan 26, 2026. It is now read-only.
Open
Conversation
…econfigure .cfg file with the first params from the orb_slam
… the r200; adjust the readme
… again This reverts commit 3c6c96e.
Fix the coordinate transformation from the orb_slam frame to the ros frame.
…ns a point must have to get into the ros point cloud
…eye_s_camera Add support for MyntEye S camera
…_rgbd added TUM2 launch and config file
…e different from the camera frame
…anch Add method for getting map transform and pose estimate in a user-specified frame
…ranch Add ZED2 launch file
Removed references to ipubot_slam and replaced with orb_slam2
Update orb_slam2_zed2_stereo.launch
…ublish_gba_status Publish GBA status
…/master Fix typo on README
Fixed bug (missing tf2-geometry) in Dockerfile and added setup.bash sourcing
…missing-deps Add dependency definition for 'tf2_geometry_msgs' and 'tf2_ros'
…om Node.h and from cv.
…out fixes are in the source files. This gets rid of the issue of 'orb_slam2/lib/liborb_slam2_ros.so: undefined reference to TaskQueue::TaskQueue<boost::shared_ptr<pcl::PointCloud<pcl::PointXYZRGBA> >, boost::shared_ptr<pcl::PointCloud<pcl::PointXYZRGBA> >, boost::shared_ptr<pcl::PointCloud<pcl::PointXYZRGBA> >, cv::Mat>::~TaskQueue()'
This file contains hidden or 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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Hi,
I think I fixed your dense_map branch. More specifically I fixed the annoying bug while linking against mt_task_queue:
orb_slam2/lib/liborb_slam2_ros.so: undefined reference to TaskQueue::TaskQueue<boost::shared_ptr<pcl::PointCloud<pcl::PointXYZRGBA> >, boost::shared_ptr<pcl::PointCloud<pcl::PointXYZRGBA> >, boost::shared_ptr<pcl::PointCloud<pcl::PointXYZRGBA> >, cv::Mat>::~TaskQueue()Please see my comments in the code where I made the changes- it should be all clearly explained as to what was wrong. C++ is not my strongest side though, so if you see something silly in my changes, then please let's fix it between us.
It compiles now and even starts up, although I can't at this very moment run it with my realsense camera- I will try that tomorrow.
I really need dense maps out of your orb_slam_2_ros for my research project. Is that all that was needed? Do you think it should work now? It looks like you are not really using your mt_task_queue. The relevant lines have been commented out in DenseMap.cc, DenseMap::AddFrameToMap function:
//task_queue_->AddTask (current_task_id_, 1, DenseMap::FitFrame); //task_queue_->AddTask (current_task_id_, 1, test);Do you think we could get this working? I'm sure it will be faster to do with your support.
Thanks,
Arturs Elksnis