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.

Sign up for GitHub

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

Jump to bottom

feat(obstacle_avoidance_planner): fix optimization at end of trajectory #309

Merged
merged 10 commits into from
Mar 14, 2023
Merged

feat(obstacle_avoidance_planner): fix optimization at end of trajectory #309

merged 10 commits into from
Mar 14, 2023

Conversation

mkuri
Copy link

@mkuri mkuri commented Mar 9, 2023 •
edited
Loading

Description

Pre-review checklist for the PR author

Add refactoring of the obstacle avoidance planner to fix a vehicle pose problem when arriving at bus stops.

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

purewater0901 and others added 7 commits February 27, 2023 07:02
@purewater0901 @mkuri
feat(behavior_path_planner, obstacle_avoidance_planner): add new driv…
57661cc 
…able area (autowarefoundation#2472)

* update

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

* update obstacle avoidance planner

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

* clean code

Signed-off-by: yutaka <[email protected]>

* uddate

Signed-off-by: yutaka <[email protected]>

* clean code

Signed-off-by: yutaka <[email protected]>

* remove resample

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

* add orientation

Signed-off-by: yutaka <[email protected]>

* change color

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

* remove drivable area

Signed-off-by: yutaka <[email protected]>

* add flag

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

* update color

Signed-off-by: yutaka <[email protected]>

* fix some codes

Signed-off-by: yutaka <[email protected]>

* change to makerker array

Signed-off-by: yutaka <[email protected]>

* change avoidance utils

Signed-off-by: yutaka <[email protected]>

Signed-off-by: yutaka <[email protected]>
@purewater0901 @mkuri
feat(motion_utils): add points resample function (autowarefoundation#…
0bf8468 
…2465)

Signed-off-by: yutaka <[email protected]>

Signed-off-by: yutaka <[email protected]>
@purewater0901 @mkuri
fix(motion_utils): rename sampling function (autowarefoundation#2469)
f006b54 
Signed-off-by: yutaka <[email protected]>

Signed-off-by: yutaka <[email protected]>
@mkuri
fix(behavior_path_planner): fix a merge failure
cdad633 
Signed-off-by: Makoto Kurihara <[email protected]>
@takayuki5168 @mkuri
feat(interpolation): add curvature calculation (autowarefoundation#2801)
86ae796 
* feat(interpolation): add curvature calculation

Signed-off-by: Takayuki Murooka <[email protected]>

* update

Signed-off-by: Takayuki Murooka <[email protected]>

* fix

Signed-off-by: Takayuki Murooka <[email protected]>

* add test

Signed-off-by: Takayuki Murooka <[email protected]>

* fix

Signed-off-by: Takayuki Murooka <[email protected]>

---------

Signed-off-by: Takayuki Murooka <[email protected]>
@takayuki5168 @mkuri
update obstacle avoidance planner, static centerline optimizer, tier4…
2a814e8 
…_planning_launch

Signed-off-by: Takayuki Murooka <[email protected]>
@takayuki5168 @mkuri
update velocity on joint and correct trajectory z
33e7ac6 
Signed-off-by: Takayuki Murooka <[email protected]>
@github-actions github-actions bot added common documentation Improvements or additions to documentation launch planning labels Mar 9, 2023
@TakaHoribe TakaHoribe removed their request for review March 9, 2023 04:03
@mkuri mkuri mentioned this pull request Mar 14, 2023
Closed
@mkuri mkuri changed the title Experiment/fix optimization at end of trajectory feat(obstacle_avoidance_planner): fix optimization at end of trajectory Mar 14, 2023
@mkuri mkuri mentioned this pull request Mar 14, 2023
Closed
7 tasks
@mkuri
Copy link
Author

mkuri commented Mar 14, 2023

@takayuki5168
obstacle avoidance plannerのリファクタをX2のv2.1.0 (= universe v0.6.1)に入れる件です。
code ownerレビューをもらえると！
（drivable areaのrviz pluginも入れようと思いますが、conflictsあったので機能だけ入れてしまいます）

@TomohitoAndo
Copy link

I confirmed this works with Planning Simulator.
image
image
image

@mkuri mkuri merged commit ba6ff2d into beta/v0.6.1 Mar 14, 2023
@mkuri mkuri deleted the experiment/fix-optimization-at-end-of-trajectory branch March 14, 2023 11:23
@shmpwk
Copy link
Collaborator

shmpwk commented Mar 14, 2023 •
edited
Loading

@mkuri
3/17の半自動生成リリースノート説明会で申し上げる予定ですが，
今回のようなbeta/v0.6.1ブランチへのhotfixのmergeは，squashではなくmerge commitとしていただくことは可能でしょうか？
というのも，半自動生成リリースノートでは，beta/v0.6.1のコミット毎に生成されるためです．
現状の仕様ですと，squash mergeされた場合は１コミット分しか自動的に記載できないため，手動でリリースノートに記載していただくことになります．

@mkuri
Copy link
Author

mkuri commented Mar 14, 2023 •
edited
Loading

@shmpwk
なるほどです。
mainへの複数commitを1つにまとめたPRはmerge commitするようにいたします。
revertはリリースノート的には大丈夫ですよね？

@mkuri
Copy link
Author

mkuri commented Mar 15, 2023

@shmpwk
今回のこのPRはcreate a merge commitすると雑多なcommitも含まれそうなので手動で対応しようかなと思います。次からはリリースノート自動化の観点も考慮してhotfixのmergeをするようにいたします。

@shmpwk
Copy link
Collaborator

shmpwk commented Mar 15, 2023 •
edited
Loading

@mkuri
ありがとうございます。
revert問題ないです。その場合、squashコミットとrevertコミットがリリースノートに記載されますが、スプレッドシート上でフィルタリングして除外できます。

雑多な時コミットが挟まった場合は、手動で記載した方が楽かもです。
（そのあたりも今週末に説明しますね。）

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TomohitoAndo TomohitoAndo TomohitoAndo approved these changes

@takayuki5168 takayuki5168 takayuki5168 approved these changes

@rej55 rej55 Awaiting requested review from rej55 rej55 is a code owner

@kosuke55 kosuke55 Awaiting requested review from kosuke55 kosuke55 is a code owner

@kyoichi-sugahara kyoichi-sugahara Awaiting requested review from kyoichi-sugahara kyoichi-sugahara is a code owner

@soblin soblin Awaiting requested review from soblin soblin is a code owner

@satoshi-ota satoshi-ota Awaiting requested review from satoshi-ota satoshi-ota is a code owner

@shmpwk shmpwk Awaiting requested review from shmpwk shmpwk is a code owner

@taikitanaka3 taikitanaka3 Awaiting requested review from taikitanaka3 taikitanaka3 is a code owner

@tkimura4 tkimura4 Awaiting requested review from tkimura4 tkimura4 is a code owner

@purewater0901 purewater0901 Awaiting requested review from purewater0901

@zulfaqar-azmi-t4 zulfaqar-azmi-t4 Awaiting requested review from zulfaqar-azmi-t4 zulfaqar-azmi-t4 is a code owner

@yukkysaito yukkysaito Awaiting requested review from yukkysaito yukkysaito is a code owner

Assignees
No one assigned
Labels
common documentation Improvements or additions to documentation launch planning
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@mkuri @TomohitoAndo @shmpwk @takayuki5168 @purewater0901