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

fix rviz settings #77

Merged
merged 7 commits into from
Mar 4, 2021
Merged

fix rviz settings #77

merged 7 commits into from
Mar 4, 2021

Conversation

tkimura4
Copy link
Collaborator

@tkimura4 tkimura4 commented Mar 3, 2021 •
edited
Loading

  • Change the sytle of pointcloud (Points -> Squares).
  • Fix topic name of checkpoint etc.
  • Enable visualization of predicted paths of dynamic objects.

@wep21
Copy link
Contributor

wep21 commented Mar 3, 2021

image

@wep21
Copy link
Contributor

wep21 commented Mar 3, 2021 •
edited
Loading

image

wep21
wep21 reviewed Mar 3, 2021
View reviewed changes
autoware_launch/rviz/autoware.rviz Outdated
@@ -697,7 +697,7 @@ Visualization Manager:
Name: DynamicObjects
Namespaces:
label: true
path: false
path: true
Copy link
Contributor

@wep21 wep21 Mar 3, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

Path flag is false for detection.
https://github.com/tier4/autoware_launcher/blob/cdc3e2de5882445ad8d468a1788c95ed508a71c0/autoware_launch/rviz/autoware.rviz#L599-L604

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed at 0da8afb .

wep21
wep21 reviewed Mar 3, 2021
View reviewed changes
autoware_launch/rviz/autoware.rviz Outdated
@@ -718,7 +718,7 @@ Visualization Manager:
Name: DynamicObjects
Namespaces:
label: true
path: false
path: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Path flag is false for tracking.
https://github.com/tier4/autoware_launcher/blob/cdc3e2de5882445ad8d468a1788c95ed508a71c0/autoware_launch/rviz/autoware.rviz#L616-L621

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed at 0da8afb .

wep21
wep21 reviewed Mar 3, 2021
View reviewed changes
autoware_launch/rviz/autoware.rviz Outdated
@@ -739,7 +739,7 @@ Visualization Manager:
Name: DynamicObjects
Namespaces:
label: true
path: false
path: true
path confidence: true
position covariance: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Position covariance flag is false for prediction.
https://github.com/tier4/autoware_launcher/blob/cdc3e2de5882445ad8d468a1788c95ed508a71c0/autoware_launch/rviz/autoware.rviz#L633-L638

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed at 0da8afb .

@wep21
Copy link
Contributor

wep21 commented Mar 3, 2021

image

@wep21
Copy link
Contributor

wep21 commented Mar 3, 2021

image

@wep21
Copy link
Contributor

wep21 commented Mar 3, 2021

image

wep21
wep21 approved these changes Mar 4, 2021
View reviewed changes
Copy link
Contributor

@wep21 wep21 left a comment

Choose a reason for hiding this comment

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

LGTM
image

@wep21 wep21 merged commit 61993fb into tier4:ros2 Mar 4, 2021
tier4-autoware-private-bot bot pushed a commit that referenced this pull request Sep 14, 2021
@kenji-miyake @wep21
Remove vehicle info param server (#77)
12e0d39 
* Add vehicle info launch for setting vehicle info param (#199)

* Add vehicle info launch for setting vehicle info param

Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp>

* Add comment for global parameters section

Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp>

* Fix year and apply format

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Remove ready vehicle info param

Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp>

Co-authored-by: Kenji Miyake <kenji.miyake@tier4.jp>

* Remove vehicle info param server from logging simulator (#211)

Signed-off-by: wep21 <border_goldenmarket@yahoo.co.jp>

Co-authored-by: Daisuke Nishimatsu <42202095+wep21@users.noreply.github.com>
kazuki527 added a commit to kazuki527/autoware_launch that referenced this pull request Apr 25, 2022
@tier4-autoware-private-bot @h-ohta
@kazuki527 @prds1lg
sync rc rc/v0.1.1 (tier4#77)
2e8344b 
* fix center line resolution (tier4#76)

* planning vel limit = 4.5km/h (tier4#75)

* Fix/adjust experiment kashiwanoha (tier4#78)

* change to 5m

* Adjust obstacle stop parameters

Co-authored-by: prds1lg <prod-s1-github@tier4.jp>

Co-authored-by: Hiroki OTA <hiroki.ota@tier4.jp>
Co-authored-by: Kazuki Matsumoto <59071591+kazuki527@users.noreply.github.com>
Co-authored-by: prds1lg <prod-s1-github@tier4.jp>
tzhong518 pushed a commit that referenced this pull request Aug 1, 2022
@satoshi-ota
feat: update lat/lon control config (#77)
98a0a63 
* fix: update lat/lon control params

* disable smooth stop

* chore: remove unused config file
kosuke55 pushed a commit that referenced this pull request Jan 17, 2023
@awf-autoware-bot @kminoda @pre-commit-ci
chore: update sync-param-files.yaml (#77)
a8acfa7 
* chore: update sync-param-files.yaml

* ci(pre-commit): autofix

Co-authored-by: kminoda <kminoda@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: kminoda <44218668+kminoda@users.noreply.github.com>
This was referenced Jan 21, 2023
Merged
Merged
Merged
Merged
takayuki5168 pushed a commit that referenced this pull request Jan 21, 2023
@rej55 @UMiho
Fix/turn signal topic name (#77)
522718e
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@wep21 wep21 wep21 approved these changes

@mitsudome-r mitsudome-r Awaiting requested review from mitsudome-r

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

Successfully merging this pull request may close these issues.
2 participants
@tkimura4 @wep21