ShunChengWu / SceneGraphFusion

BSD 2-Clause "Simplified" License
160 stars 26 forks source link

Is it ok without entering "pose"? The paper says that only RGBD is required #8

Closed why-freedom closed 3 years ago

why-freedom commented 3 years ago

If I annotate this paragraph, i.e. do not assign pose to inseg, the result will be messy,Why?Looking forward to your reply。

void GraphSLAM::ProcessFrame(int idx, const cv::Mat &colorImage, const cv::Mat &depthImage,const Eigen::Matrix4f pose) { // if(pose){ // 此处pose给inseg,否则没有pose,inseg结果较差? hy // pose_ = pose; // inseg_->setpose(pose); // } mTimeStamp = idx; CTICK("[SLAM][ProcessFrame]1.ProcessFrame"); ......

ShunChengWu commented 3 years ago

the front-end estimates pose with a relatively naive approach. you can replace the pose estimation from the InSeg system with other methods.

stevenAce13 commented 2 years ago

Hi! Thanks for the work.

I've a similar question to this pose and looking forward to your comments :).

For this "pose" variable here, may we assume it's checking whether a ground-truth campose pose is given/provided outside to be set for the inseg? More specifically,

Keller et al. [19]: Real-Time 3D Reconstruction in Dynamic Scenes Using Point-Based Fusion. ([19] referenced from the paper [50] in SceneGraphFusion, which is "RealTime and Scalable Incremental Segmentation on Dense SLAM".)

ShunChengWu commented 2 years ago

exactly.