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

HWDEV-2246 add profile subcommand #37

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ar90n
Copy link
Contributor

@ar90n ar90n commented Aug 22, 2024

ref: HWDEV-2246

This PR is motivated to add a rosserial transport profiling feature. This PR contians following modifications.

  • Calculate follwoing topics
    • tx_bytes: tx bytes after system launched
    • rx_bytes: rx bytes after system launched
    • drop_tx_bytes: bytes which is failed to insert tx uart fifo after system launched
    • drop_rx_bytes: bytes which is failed to insert rx ring buffer after system launched
  • Add shell command whose name is ros
  • Add shell sub command whose name is profile for ros to show transport profile

This PR is checked in robot and I got following logs.

uart:~$ ros profile
TX bytes per sec: 24389.00
RX bytes per sec: 1126.00
Max TX bytes per sec: 24993.00
Max RX bytes per sec: 1166.00
TX drop bytes: 0
RX drop bytes: 0

This reverts commit e244398305ba40df34faf98dbe4e308a5ff73a2e.
@ar90n ar90n requested a review from shujima August 23, 2024 01:41
@ar90n ar90n marked this pull request as ready for review August 23, 2024 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants