Skip to content
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

feat(autoware_planning_data_analyzer)!: replace tier4_debug_msgs with tier4_internal_debug_msgs #199

Conversation

mitsudome-r
Copy link
Member

@mitsudome-r mitsudome-r commented Jan 23, 2025

Description

This replaces tier4_debug_msgs with tier4_internal_debug_msgs.
This is a PR to resolve autowarefoundation/autoware#5580.

Interface Change

Topic Type Topic Name Old Message Type New Message Type
Pub ~/output/manual_metrics tier4_debug_msgs/Float32MultiArrayStamped autoware_internal_debug_msgs/Float32MultiArrayStamped
Pub ~/output/system_metrics tier4_debug_msgs/Float32MultiArrayStamped autoware_internal_debug_msgs/Float32MultiArrayStamped
Pub ~/output/manual_score tier4_debug_msgs/Float32MultiArrayStamped autoware_internal_debug_msgs/Float32MultiArrayStamped
Pub ~/output/system_score tier4_debug_msgs/Float32MultiArrayStamped autoware_internal_debug_msgs/Float32MultiArrayStamped

How was this PR tested?

I have confirmed that the node launches without crashing and publishes the data with the new topic type:

ros2 launch autoware_planning_data_analyzer behavior_analyzer.launch.xml bag_path:=<ROSBAG>

image

NOTE: There were errors regarding glog component from the same launch file, but that could be fixed in a separate PR.

Notes for reviewers

None.

Effects on system behavior

None.

… tier4_internal_debug_msgs

Signed-off-by: Ryohsuke Mitsudome <[email protected]>
Copy link

github-actions bot commented Jan 23, 2025

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@mitsudome-r mitsudome-r merged commit d401618 into autowarefoundation:main Jan 23, 2025
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Porting XXXStamped messages from tier4_autoware_msgs to autoware_internal_msgs
2 participants