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.

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

Monitor processing time for huge topic or package #949

Closed
3 tasks done
taikitanaka3 opened this issue May 23, 2022 · 3 comments
Closed
3 tasks done

Monitor processing time for huge topic or package #949

taikitanaka3 opened this issue May 23, 2022 · 3 comments
Labels
type:new-feature New functionalities or additions, feature requests.

Comments

@taikitanaka3
Copy link
Contributor

Checklist

  • I've read the contribution guidelines.
  • I've searched other issues and no duplicate issues were found.
  • I've agreed with the maintainers that I can plan this task.

Description

Accuracy and processing time are in the balance and it's necessary which module is heavy.

Purpose

to monitor topic hz easily and more precisely without subscribing heavy topic
image

Possible approaches

  • run autoware with this PR and see in plot_juggler
  • ros2 topic hz ~ topic at every topic for each manually and remember the trend of topic hz change.
    • be careful that ros2 topic hz also needs subscription of topic so itself also needs computation cost and if you see heavy topic, for example hz of point cloud, you need to turn off for every time.

Definition of done

ros2 run plotjuggler plotjuggler and see */debug/processing_time_ms

@taikitanaka3 taikitanaka3 changed the title Monitor processing time for huge package Monitor processing time for huge topic or package May 23, 2022
@BonoloAWF BonoloAWF added type:new-feature New functionalities or additions, feature requests. priority:low Lower urgency, can be addressed later. labels May 23, 2022
@BonoloAWF
Copy link

@taikitanaka3
Copy link
Contributor Author

@BonoloAWF
Thank you for information, but this issue is for planing and sensing performance monitoring so what you mentioned are not related.

@taikitanaka3
Copy link
Contributor Author

close this issue resolved by #946

@taikitanaka3 taikitanaka3 removed the priority:low Lower urgency, can be addressed later. label Sep 16, 2022
YoshiRi pushed a commit to YoshiRi/autoware.universe that referenced this issue Oct 17, 2023
…r_pipeline

feat: add radar far detection pipeline
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:new-feature New functionalities or additions, feature requests.
Projects
None yet
Development

No branches or pull requests

2 participants