base documentation for lidar-lidar, lidar-camera, lidar-radar

vividf commented 2 months ago

Description

Base documentation for lidar-lidar, lidar-camera, lidar-radar are done.

Note: I also remove some unused variables.

Tests performed

Notes for reviewers

Pre-review checklist for the PR author

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

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

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.

knzo25 commented 2 months ago

@vividf As mentioned before in the internal channel, push documentation in the branch I prepared. Do not create a different branch

codecov-commenter commented 2 months ago

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (feature/new_api_documentation@a8a4b6a). Click here to learn what that means.

Files	Patch %	Lines
...ier4_ground_plane_utils/src/ground_plane_utils.cpp	0.00%	3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## feature/new_api_documentation #165 +/- ## =============================================================== Coverage ? 0.00% =============================================================== Files ? 69 Lines ? 7652 Branches ? 758 =============================================================== Hits ? 0 Misses ? 7652 Partials ? 0 ``` | [Flag](https://app.codecov.io/gh/tier4/CalibrationTools/pull/165/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tier4) | Coverage Δ | | |---|---|---| | [differential](https://app.codecov.io/gh/tier4/CalibrationTools/pull/165/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tier4) | `0.00% <0.00%> (?)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tier4#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tier4 / CalibrationTools