Skip to content

feat: handle tiny resolution by tp manager and point cloud divider#240

Closed
anhnv3991 wants to merge 5 commits into
autowarefoundation:mainfrom
anhnv3991:feature/tiny_res_tp_manager
Closed

feat: handle tiny resolution by tp manager and point cloud divider#240
anhnv3991 wants to merge 5 commits into
autowarefoundation:mainfrom
anhnv3991:feature/tiny_res_tp_manager

Conversation

@anhnv3991

Copy link
Copy Markdown

Description

  • Add an option to generate a metadata file by point cloud divider. Sometimes we just want a metadata file without actually dividing PCD files.
  • Enable the tp manager to handle tiny resolution, which provides more accurate TP estimation, without sacrifying much processing time.

How was this PR tested?

Notes for reviewers

  • Run the tp manager to collect average TPs of map segments
  • Run the tp checker to compare rosbags TPs with the expected TPs computed from the map TPs
  • Visualize the results by rviz2

Effects on system behavior

None.

Signed-off-by: Anh Nguyen <anh.nguyen.2@tier4.jp>
@github-actions

github-actions Bot commented Apr 28, 2025

Copy link
Copy Markdown

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

Anh Nguyen and others added 3 commits April 28, 2025 12:29
@YamatoAndo

Copy link
Copy Markdown
Contributor

@anhnv3991 There is no dependency between the changes to tp manager and point cloud divider, so please split them into separate PRs.

@anhnv3991

Copy link
Copy Markdown
Author

@YamatoAndo Thank you for reviewing.
I splitted the PR into

@anhnv3991 anhnv3991 closed this Jun 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants