Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Conversation

mehmetdogru
Copy link
Contributor

@mehmetdogru mehmetdogru commented Jan 24, 2023

Signed-off-by: Mehmet Dogru 42mehmetdogru42@gmail.com

closes autowarefoundation/autoware.universe#2125

Description

Related links

Tests performed

Notes for reviewers

Pre-review checklist for the PR author

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.

  • The PR follows the pull request guidelines.
  • The PR has been properly tested.
  • The PR has been reviewed by the code owners.

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.
  • The PR is ready for merge.

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

@codecov
Copy link

codecov bot commented Jan 24, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.09 ⚠️

Comparison is base (b30849e) 11.23% compared to head (0f0b6f8) 11.15%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #148      +/-   ##
==========================================
- Coverage   11.23%   11.15%   -0.09%     
==========================================
  Files          33       33              
  Lines        2385     2403      +18     
  Branches     1260     1273      +13     
==========================================
  Hits          268      268              
- Misses       1672     1690      +18     
  Partials      445      445              
Flag Coverage Δ *Carryforward flag
differential 9.27% <0.00%> (?)
total 11.23% <ø> (ø) Carriedforward from b30849e

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
tmp/lanelet2_extension/lib/query.cpp 7.61% <0.00%> (-0.38%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mehmetdogru mehmetdogru force-pushed the 2125-consider-lane-ego-angle-diff-in-lane-following-module branch from d90483e to dad9b92 Compare March 14, 2023 06:55
@mehmetdogru mehmetdogru marked this pull request as ready for review March 14, 2023 07:13
tmp/lanelet2_extension/lib/query.cpp Outdated Show resolved Hide resolved
tmp/lanelet2_extension/lib/query.cpp Outdated Show resolved Hide resolved
mehmetdogru and others added 6 commits March 23, 2023 14:11
…following module

Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>
Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>
Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>
Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>
Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>
@mehmetdogru mehmetdogru force-pushed the 2125-consider-lane-ego-angle-diff-in-lane-following-module branch from 5af8183 to 0f0b6f8 Compare March 23, 2023 12:18
@mehmetdogru mehmetdogru requested a review from mitsudome-r March 23, 2023 12:18
@mitsudome-r mitsudome-r merged commit d8444ee into autowarefoundation:main Mar 28, 2023
@mehmetdogru mehmetdogru deleted the 2125-consider-lane-ego-angle-diff-in-lane-following-module branch March 28, 2023 07:33
satoshi-ota pushed a commit to satoshi-ota/autoware_common that referenced this pull request Apr 27, 2023
…following module (autowarefoundation#148)

* feat(lanelet2_extension): consider lane-ego angle difference in lane following module

Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>

* remove debug prints

Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>

* include limits

Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>

* style(pre-commit): autofix

* fix: use bg:distance

Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>

* refactor: replace with getLaneletAngle

Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>

---------

Signed-off-by: Mehmet Dogru <42mehmetdogru42@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[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.

behavior_path_planner generating invalid path orientation
2 participants