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

feat(ndt_scan_matcher): add nearest voxel transfromation probability param #198

Merged

Conversation

YamatoAndo
Copy link
Contributor

@YamatoAndo YamatoAndo commented Feb 8, 2022

PR Type

  • New Feature

Related Links

Description

Review Procedure

See autowarefoundation/autoware.universe#364.

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.

@YamatoAndo YamatoAndo force-pushed the feature/nearest_voxel_transfomation_probability branch from 40e2acd to 3def078 Compare March 9, 2022 09:23
@YamatoAndo YamatoAndo marked this pull request as ready for review March 9, 2022 09:23
@YamatoAndo YamatoAndo force-pushed the feature/nearest_voxel_transfomation_probability branch from 3def078 to f95f8ca Compare March 30, 2022 03:39
…param

Signed-off-by: YamatoAndo <yamato.ando@gmail.com>
Signed-off-by: YamatoAndo <yamato.ando@gmail.com>
@YamatoAndo YamatoAndo force-pushed the feature/nearest_voxel_transfomation_probability branch from 3900ce5 to 242dcf7 Compare March 30, 2022 05:56
@YamatoAndo
Copy link
Contributor Author

@RyuYamamoto Would you please check this PR as well?

Copy link
Contributor

@RyuYamamoto RyuYamamoto left a comment

Choose a reason for hiding this comment

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

LGTM

@YamatoAndo YamatoAndo merged commit d513999 into tier4/universe Mar 31, 2022
@YamatoAndo YamatoAndo deleted the feature/nearest_voxel_transfomation_probability branch March 31, 2022 06:24
takayuki5168 pushed a commit that referenced this pull request May 26, 2022
* Fix incorrect remap (#183)

* Fix incorrect remap

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

* Use set_remap

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

* add septentrio_gnss_driver launcher and switch(septentrio <-> ublox) (#196)

* add septentrio_gnss_driver launcher and switch(septentrio <-> ublox)

* rm arg(gnss_receiver) escalation and modify septentrio_gnss_driver_node option

* change gnss_receiver default septentrio to ublox

* remap all septentrio_gnss_driver topic names

* replace septentrio gnss driver launch type 'node' to 'include'

* Use set_parameter for use_sim_time (#198)

* Use set_parameter for use_sim_time

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

* Add default parameter for scenario simulator

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

* change system_monitor.launch.xml to launch.py (#203)

* Add control composed launch (#201)

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

* add param for psim v2 (#197)

* add param for psim v2

Signed-off-by: kosuke murakami <kosuke.murakami@tier4.jp>

* Update to pass var 'scenario_simulation' to dummy_perception_publisher

* Update dummy_perception_publisher's arg-name

Co-authored-by: yamacir-kit <httperror@404-notfound.jp>

* Feature/porting v0.9.1 master (#202)

* add simulation arg (#174)

* add simulation arg

* add comment

* change arg name

* add parameter use_empty_dynamic_object_publisher into perception.launch (#176)

Signed-off-by: Azumi Suzuki <azumi.suzuki@tier4.jp>

Co-authored-by: Yusuke FUJII <yusuke.fujii@tier4.jp>
Co-authored-by: s-azumi <38061530+s-azumi@users.noreply.github.com>

* Fix mistakes in control.launch.py (#209)

* Fix typo in control.launch.py

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

* Add remote_cmd_converter_component

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

* Feature/add external cmd selector (#206)

* Add remote_cmd_selector to control.launch (#172)

* Add remote_cmd_selector to control.launch

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

* Rename remote_cmd_selector to external_cmd_selector

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

* Change default external cmd source (#173)

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

* modify to use control.launch.xml

* add external_cmd_selector to control.launch.py

* Fix remapping in control.launch.py

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

* Fix remapping in control.launch.py again

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

Co-authored-by: Keisuke Shima <keisuke.shima@tier4.jp>

* 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>

* Add container launch for planning (#205)

* Add container launch for planning

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

* fix porting miss

* fix lane_driving.launch.xml

* Add missing parameters

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

Co-authored-by: taichiH <azumade.30@gmail.com>

* Add map.launch.py (#212)

* Add map.launch.py

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

* Update map.launch.xml

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

* Rename parameter for lanelet2 map path

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

* Rename AstarNavi to FreespacePlannerNode (#213)

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

* Add spell check action (#214)

* Unify YAML file extension of GitHub action to yml (#216)

* Fix PR template (#218)

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

* Fix PR template link (#219)

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

* Fix minor typos in PULL_REQUEST_TEMPLATE.md (#222)

* Fix depends in localization_launch to exec_depend (#223)

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

* Add build CI (#224)

* Add build CI

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

* Fix build_depend.repos

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

* Add private repository settings

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

* Remove self repo

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

* Install pip

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

* Use packages-select and packages-up-to

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

* Change job name

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

* Add build_and_test.yml

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

* Install kvaser library

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

* Build only internal packages in CI (#227)

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

* Fix typos in launch files (#231)

* Fix typos in launch files

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

* Fix lint

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

Co-authored-by: wep21 <border_goldenmarket@yahoo.co.jp>

* Format launch files (#228)

* Format launch files

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

* Format launch.py

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

* Fix lint

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

* Exclude dummy_perception_publisher.launch.xml if is scenario_simulation (#230)

* Fix unresolved conflicts

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

Co-authored-by: hiro-ya-iv <30652835+hiro-ya-iv@users.noreply.github.com>
Co-authored-by: Daisuke Nishimatsu <42202095+wep21@users.noreply.github.com>
Co-authored-by: Taichi Higashide <taichi.higashide@tier4.jp>
Co-authored-by: Kosuke Murakami <kosuke.murakami@tier4.jp>
Co-authored-by: yamacir-kit <httperror@404-notfound.jp>
Co-authored-by: Keisuke Shima <keisuke.shima@tier4.jp>
Co-authored-by: Yusuke FUJII <yusuke.fujii@tier4.jp>
Co-authored-by: s-azumi <38061530+s-azumi@users.noreply.github.com>
Co-authored-by: taichiH <azumade.30@gmail.com>
Co-authored-by: Kazuki Miyahara <kmiya@outlook.com>
Co-authored-by: wep21 <border_goldenmarket@yahoo.co.jp>
takayuki5168 pushed a commit that referenced this pull request May 26, 2022
tier4-autoware-public-bot bot pushed a commit that referenced this pull request Feb 10, 2023
…ance (#198)

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.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