-
Notifications
You must be signed in to change notification settings - Fork 0
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
[UPD] Decouple LiDAR data from DR-SPAAM module #1
base: feature/OSS2
Are you sure you want to change the base?
Conversation
@@ -481,8 +490,10 @@ void multiple_sensor_person_tracking::SobitEduPersonTracker::onInit() { | |||
cloud_scan_.reset(new PointCloud()); | |||
marker_array_.reset(new visualization_msgs::MarkerArray); | |||
following_position_.reset( new multiple_sensor_person_tracking::FollowingPosition ); | |||
scan_msg_.reset( new sensor_msgs::LaserScan ); | |||
|
|||
sub_scan_ = nh_.subscribe(pnh_.param<std::string>( "scan_topic_name", "/scan"), 1, &SobitEduPersonTracker::scan_callback, this); |
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.
Scanデータをmessage_filtersとしてsubcribeする必要があるのではないか?
@Jiahao9
@@ -600,8 +610,10 @@ void multiple_sensor_person_tracking::SobitEduPersonTrackerId::onInit() { | |||
marker_array_.reset(new visualization_msgs::MarkerArray); | |||
no_exists_marker_array_.reset(new visualization_msgs::MarkerArray); | |||
following_position_.reset( new multiple_sensor_person_tracking::FollowingPosition ); | |||
scan_msg_.reset( new sensor_msgs::LaserScan ); | |||
|
|||
sub_scan_ = nh_.subscribe(pnh_.param<std::string>( "scan_topic_name", "/scan"), 1, &SobitEduPersonTracker::scan_callback, this); |
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.
Launch上でScanのTopicを記入できるようにした方がいいかも
@Jiahao9
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.
もし,基本的に/scan
というトピック名が変わらないのであれば,そのままでオッケー
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.
トラッキングする前に、/scan
データ前処理する可能性はなくはないので、Launchファイルに追加します.
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.
paramファイルでトピック名を変更できるようしました。
このリクエストをマージしたあと、また新しくPull をリクエストしたいと思います
…nd Odometry topics
No description provided.