Skip to content
This repository has been archived by the owner on Mar 27, 2023. It is now read-only.

Auto/fix topic name in rviz config #145

Merged
merged 1 commit into from
Dec 2, 2021
Merged

Auto/fix topic name in rviz config #145

merged 1 commit into from
Dec 2, 2021

Conversation

kyoichi-sugahara
Copy link
Contributor

PR Type

  • Bug Fix

Description

change display topic name in rviz from /control/trajectory_follower/predicted_trajectory to /control/trajectory_follower/lateral/predicted_trajectory

Review Procedure

Remarks

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

  • Commits are properly organized and messages are according to the guideline
  • Code follows coding guidelines
  • (Optional) Unit tests have been written for new behavior
  • PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

  • All open points are addressed and tracked via issues or tickets
  • Write release notes

CI Checks

  • Build and test for PR: Required to pass before the merge.
  • Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@github-actions github-actions bot added the main label Dec 2, 2021
@takayuki5168 takayuki5168 changed the base branch from main to use-autoware-auto-msgs December 2, 2021 07:35
takayuki5168
takayuki5168 approved these changes Dec 2, 2021
View reviewed changes
Copy link
Contributor

@takayuki5168 takayuki5168 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@takayuki5168 takayuki5168 merged commit d50be70 into use-autoware-auto-msgs Dec 2, 2021
@takayuki5168 takayuki5168 deleted the auto/fix_topic_name_in_rviz_config branch December 2, 2021 08:42
kazuki527 pushed a commit to kazuki527/autoware_launch that referenced this pull request Apr 25, 2022
@h-ohta
fix: comment out map/node_alive_monitoring (tier4#145)
b2f9cdd
takayuki5168 pushed a commit that referenced this pull request May 26, 2022
change use start request to true (#145)
c7b5df3
tzhong518 pushed a commit that referenced this pull request Aug 1, 2022
@tier4-autoware-private-bot
Merge pull request #145 from tier4/sync-upstream
42bb7a1 
chore: sync upstream
kosuke55 pushed a commit that referenced this pull request Jan 17, 2023
@yukkysaito
fix: avoidance parameter file (#145)
ae57ded 
* fix: avoidance parameter file

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>

* Update autoware_launch/config/planning/scenario_planning/lane_driving/behavior_planning/behavior_path_planner/avoidance/avoidance.param.yaml

Signed-off-by: Yukihiro Saito <yukky.saito@gmail.com>
This was referenced Jan 21, 2023
Merged
Merged
Merged
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@takayuki5168 takayuki5168 takayuki5168 approved these changes

Assignees
No one assigned
Labels
main
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kyoichi-sugahara @takayuki5168