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

velocity_controller sync to .iv develop latest #126

Merged

Conversation

tkimura4
Copy link
Collaborator

@tkimura4 tkimura4 commented Nov 22, 2021

PR Type

  • add/remove parameter

Related Links

tier4/AutowareArchitectureProposal.iv#699

Description

Add parameters corresponding with tier4/AutowareArchitectureProposal.iv#699

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.

@tkimura4 tkimura4 changed the title add param velocity_controller sync to .iv develop latest Nov 22, 2021
@tkimura4 tkimura4 requested a review from rej55 November 22, 2021 07:47
@tkimura4 tkimura4 merged commit 35a0575 into use-autoware-auto-msgs Nov 22, 2021
@tkimura4 tkimura4 deleted the velocity-controller-sync-to-iv-develop branch November 22, 2021 08:09
j4tfwm6z added a commit that referenced this pull request Feb 1, 2022
* FIx backward fixing distance

* Disable lane departure temporarily

* Disable lidar temporarily

* Enable multi thread in control container (#126)

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

* Feature/change acc param (#131)

* change acc param

* add space

* Enable avoidance without foa (#127)

* Enable avoidance without foa

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

* Update planning_launch/launch/scenario_planning/lane_driving/behavior_planning/behavior_planning.launch.xml

Co-authored-by: Fumiya Watanabe <rej55.g@gmail.com>

Co-authored-by: Fumiya Watanabe <rej55.g@gmail.com>

* Revert "Disable lidar temporarily"

This reverts commit 81359d412292b5396cef7aceb53909cb38cf1d95.

Co-authored-by: Daisuke Nishimatsu <42202095+wep21@users.noreply.github.com>
Co-authored-by: tkimura4 <tomoya.kimura@tier4.jp>
Co-authored-by: Fumiya Watanabe <rej55.g@gmail.com>
Co-authored-by: j4tfwm6z <proj-jpntaxi@tier4.jp>
kazuki527 pushed a commit to kazuki527/autoware_launch that referenced this pull request Apr 25, 2022
* modify rviz panel (tier4#110)

* add voltage

* fix status panel

* fix typo

* fix system monitor diag (tier4#109)

* tmp commit

* some fix

* comment out

* add file and comment out

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

* behavior path planner: parameter tuning and some fix (tier4#108)

* delete auto approval

* update rviz

* parameter tuning

* update rviz

* tuning tracking parameter

* tuning avoidance

* tuning obstacle avoidance

* delete unused

* change stop margin

* parameter fix

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

* fix Random filter (tier4#114)

* modify engage velocity + obstacle avoidance (tier4#113)

* modify engage velocity

* tuning avoidance

* parameter tuning

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

* Fix heartbeat (tier4#126) (tier4#129)

* fix: external_emergenxy_stop_heartbeat_timeout to 3.0

* feat: add heartbeat to meter panel

* feat: use_external_emergency_stop to false in psim

* fix: comment out

* fix: change timeout sec

Co-authored-by: Hiroki OTA <hiroki.ota@tier4.jp>

* fix: delete space

* fix: delete space

* fix: for pre-commit

* fix: typo

* fix typo

Co-authored-by: prds1lg <prod-s1-github@tier4.jp>
Co-authored-by: tier4-autoware-private-bot[bot] <87871706+tier4-autoware-private-bot[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants