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

fix(compare_map_segmentation): initialize timer_callback_group_ #3914

Merged
merged 1 commit into from
Jun 8, 2023
Merged

fix(compare_map_segmentation): initialize timer_callback_group_ #3914

merged 1 commit into from
Jun 8, 2023

Conversation

tkimura4
Copy link
Contributor

@tkimura4 tkimura4 commented Jun 7, 2023 •
edited
Loading

Description

Currently, the variable timer_callback_group_ is used before initialization.
In this case map_update_timer_ may be assigned to the default callback group.
I fixed it.

Related links

x

Tests performed

We confirm that the node works correctly on the actual vehicle

Notes for reviewers

x

Interface changes

none

Effects on system behavior

none(?)

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.

@tkimura4
fix(compare_map_segmentation): initialize timer_callback_group_
05bfacc 
Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>
@github-actions github-actions bot added the component:perception Advanced sensor data processing and environment understanding. (auto-assigned) label Jun 7, 2023
yukkysaito
yukkysaito approved these changes Jun 7, 2023
View reviewed changes
Copy link
Contributor

@yukkysaito yukkysaito left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Jun 7, 2023 •
edited
Loading

Codecov Report

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

Comparison is base (b2f4517) 14.08% compared to head (05bfacc) 14.08%.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #3914      +/-   ##
==========================================
- Coverage   14.08%   14.08%   -0.01%     
==========================================
  Files        1449     1449              
  Lines      102310   102311       +1     
  Branches    27293    27293              
==========================================
  Hits        14412    14412              
- Misses      72114    72115       +1     
  Partials    15784    15784
Flag Coverage Δ *Carryforward flag
differential 0.00% <0.00%> (?)
total 14.08% <ø> (ø) Carriedforward from b2f4517

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

Impacted Files Coverage Δ
...are_map_segmentation/src/voxel_grid_map_loader.cpp 0.00% <0.00%> (ø)

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

badai-nguyen
badai-nguyen approved these changes Jun 7, 2023
View reviewed changes
Copy link
Contributor

@badai-nguyen badai-nguyen left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for your PR.

@tkimura4 tkimura4 merged commit f6e6c79 into autowarefoundation:main Jun 8, 2023
@tkimura4 tkimura4 deleted the fix/timer_callback_in_voxel_grid_map branch June 8, 2023 08:34
tkimura4 added a commit to tier4/autoware.universe that referenced this pull request Jun 8, 2023
@tkimura4
fix(compare_map_segmentation): initialize timer_callback_group_ (auto…
eae2ec1 
…warefoundation#3914)

Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>
@tkimura4 tkimura4 mentioned this pull request Jun 8, 2023
Merged
4 tasks
tkimura4 added a commit to tier4/autoware.universe that referenced this pull request Jun 8, 2023
@tkimura4
fix(compare_map_segmentation): initialize timer_callback_group_ (auto…
6dc0f19 
…warefoundation#3914) (#563)

Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>
kyoichi-sugahara pushed a commit to kyoichi-sugahara/autoware.universe that referenced this pull request Jun 8, 2023
@tkimura4 @kyoichi-sugahara
fix(compare_map_segmentation): initialize timer_callback_group_ (auto…
880216a 
…warefoundation#3914)

Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>
@tier4-autoware-public-bot tier4-autoware-public-bot bot mentioned this pull request Jun 9, 2023
Merged
kyoichi-sugahara added a commit that referenced this pull request Jun 14, 2023
@kyoichi-sugahara @tkimura4 @zulfaqar-azmi-t4
refactor(behavior_path_planner): generalize safety check function (#3968
9e7dc27 
)

* temp

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* temp

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* fix(compare_map_segmentation): initialize timer_callback_group_ (#3914)

Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>

* fix(behavior_path_planner): small refactoring for resamplePathWithSpline (#3922)

Signed-off-by: Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>

* fix(behavior_path_planner): avoid invalid access (#3923)

Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>

* temo

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* create PredictedPath debug marker

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* delete debug print

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* refactoring

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* refactor

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* refactoring

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* refactoring

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* update

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* add param for predicted path generation

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* refactoring

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* temp

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* generalize safety check function

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* delete unnecessary difference

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* modify call refactored function

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

* Update planning/behavior_path_planner/include/behavior_path_planner/utils/safety_check.hpp

Co-authored-by: Zulfaqar Azmi <93502286+zulfaqar-azmi-t4@users.noreply.github.com>

* Update planning/behavior_path_planner/src/utils/safety_check.cpp

Co-authored-by: Zulfaqar Azmi <93502286+zulfaqar-azmi-t4@users.noreply.github.com>

---------

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>
Signed-off-by: Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
Co-authored-by: Tomoya Kimura <tomoya.kimura@tier4.jp>
Co-authored-by: Zulfaqar Azmi <93502286+zulfaqar-azmi-t4@users.noreply.github.com>
kyoichi-sugahara added a commit to kyoichi-sugahara/autoware.universe that referenced this pull request Jun 16, 2023
@kyoichi-sugahara
Squashed commit of the following:
bf08a58 
commit 3dcfb6e
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Tue Jun 13 14:41:19 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit eeb884c
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Tue Jun 13 14:13:34 2023 +0900

    refactoring

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 50e8346
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Tue Jun 13 13:50:49 2023 +0900

    refactoring

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 9063019
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Tue Jun 13 13:22:49 2023 +0900

    refactor

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 5fe015a
Merge: ab7d208 f09e575
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Tue Jun 13 12:56:28 2023 +0900

    Merge branch 'main' into feature/create_predicted_path_from_target_velcoity

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit ab7d208
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Sat Jun 10 01:17:41 2023 +0900

    refactoring

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 6eb0dae
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Sat Jun 10 00:54:05 2023 +0900

    delete debug print

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 57e75c4
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Sat Jun 10 00:52:06 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 89bfa91
Merge: abaf563 fc08b4f
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 21:26:18 2023 +0900

    Merge branch 'main' into feature/create_predicted_path_from_target_velcoity

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit abaf563
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 20:37:08 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 1b32573
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 20:00:41 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 9db1416
Merge: 1ae32a3 21b6403
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 19:23:51 2023 +0900

    Merge branch 'main' into feature/create_predicted_path_from_target_velcoity

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 1ae32a3
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 18:45:19 2023 +0900

    create PredictedPath debug marker

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 1bc6451
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 15:45:09 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 6c78054
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 14:55:22 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit df7cbc1
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 14:43:52 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 0210dc1
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 14:31:58 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit f01e8e1
Merge: d9f8edb 35f94d0
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 12:17:24 2023 +0900

    Merge branch 'main' into feature/add_safety_checker_pull_out

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit d9f8edb
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Fri Jun 9 12:07:51 2023 +0900

    temo

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit b206444
Author: Tomoya Kimura <tomoya.kimura@tier4.jp>
Date:   Thu Jun 8 22:32:40 2023 +0900

    fix(behavior_path_planner): avoid invalid access (autowarefoundation#3923)

    Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>

commit 6aa9f02
Author: Zulfaqar Azmi <93502286+zulfaqar-azmi-t4@users.noreply.github.com>
Date:   Thu Jun 8 18:02:30 2023 +0900

    fix(behavior_path_planner): small refactoring for resamplePathWithSpline (autowarefoundation#3922)

    Signed-off-by: Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>

commit 880216a
Author: Tomoya Kimura <tomoya.kimura@tier4.jp>
Date:   Thu Jun 8 17:34:31 2023 +0900

    fix(compare_map_segmentation): initialize timer_callback_group_ (autowarefoundation#3914)

    Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>

commit 920a0cb
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Thu Jun 8 19:14:32 2023 +0900

    update

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit a75cf49
Merge: e27face 69e89d9
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Thu Jun 8 16:30:09 2023 +0900

    Merge branch 'main' into feature/add_safety_checker_pull_out

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit e27face
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Tue Jun 6 19:36:32 2023 +0900

    temp

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

commit 7ec623e
Author: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Date:   Tue Jun 6 16:29:51 2023 +0900

    add function to create PredictedPath from target velocity

    Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yukkysaito yukkysaito yukkysaito approved these changes

@badai-nguyen badai-nguyen badai-nguyen approved these changes

@amc-nu amc-nu Awaiting requested review from amc-nu

Assignees
No one assigned
Labels
component:perception Advanced sensor data processing and environment understanding. (auto-assigned)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tkimura4 @yukkysaito @badai-nguyen