perf(processing_time): add cyclic and processing time ms to pcd related package

[taikitanaka3]

Description

add processing time for debug

Related links

#949

tier4#46

Tests performed

by psim and real vehicle

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.

[codecov]

Codecov Report

Merging #946 (bdca1f1) into main (1168f14) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

❗ Current head bdca1f1 differs from pull request most recent head 12f4581. Consider uploading reports for the commit 12f4581 to get more accurate results

@@           Coverage Diff            @@
##            main    #946      +/-   ##
========================================
- Coverage   9.90%   9.84%   -0.07%     
========================================
  Files        879     879              
  Lines      56251   56621     +370     
  Branches   10638   10638              
========================================
  Hits        5572    5572              
- Misses     45960   46330     +370     
  Partials    4719    4719              

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <0.00%> (?)
total	9.90% <0.00%> (ø)		Carriedforward from 1168f14

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

Impacted Files	Coverage Δ
...l_based_approximate_compare_map_filter_nodelet.cpp	0.00% <0.00%> (ø)
...ion/src/voxel_based_compare_map_filter_nodelet.cpp	0.00% <0.00%> (ø)
...lter/occupancy_grid_map_outlier_filter_nodelet.hpp	0.00% <ø> (ø)
.../src/occupancy_grid_map_outlier_filter_nodelet.cpp	0.00% <0.00%> (ø)
...ssor/concatenate_data/concatenate_data_nodelet.hpp	0.00% <ø> (ø)
...ocessor/include/pointcloud_preprocessor/filter.hpp	0.00% <ø> (ø)
.../src/concatenate_data/concatenate_data_nodelet.cpp	0.00% <0.00%> (ø)
...or/src/crop_box_filter/crop_box_filter_nodelet.cpp	0.00% <0.00%> (ø)
.../src/distortion_corrector/distortion_corrector.cpp	0.00% <0.00%> (ø)
...src/outlier_filter/ring_outlier_filter_nodelet.cpp	0.00% <0.00%> (ø)
... and 9 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1168f14...12f4581. Read the comment docs.

[yukkysaito]

I think it is better to measure from the start of callback to the end of callback.

[taikitanaka3]

@yukkysaito

I think it is better to measure from the start of callback to the end of callback.

yes but in this case we can't measure how heavy "subscription / publish" of topic is right?

[yukkysaito]

I see.
It is reasonable and unavoidable.

[yukkysaito]

@taikitanaka3 Can you resolve conflicts?

[taikitanaka3]

yes i will do it later

[taikitanaka3]

@yukkysaito Done

[yukkysaito]

I think it is better to measure from the start of callback to the end of callback.

yes but in this case we can't measure how heavy "subscription / publish" of topic is right?

It is reasonable about heavy topics. But behavior planning topics are not heavy.

TODO (discussed @taikitanaka3 )

• rename from processing time to cyclic time
• remove debug code from behavior planning
• add processing time from the start of callback to the end of callback.

[taikitanaka3]

I think it is better to measure from the start of callback to the end of callback.

yes but in this case we can't measure how heavy "subscription / publish" of topic is right?

It is reasonable about heavy topics. But behavior planning topics are not heavy.

TODO (discussed @taikitanaka3 )

• rename from processing time to cyclic time
• remove debug code from behavior planning
• add processing time from the start of callback to the end of callback.

@yukkysaito

• rename from processing time to cyclic time
• remove debug code from behavior planning
• add processing time from the start of callback to the end of callback.

[yukkysaito]

LGTM

[taikitanaka3]

• tested by experiment
• release note