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
feat: add caret record CLI #38
Conversation
Signed-off-by: hsgwa <19860128+hsgwa@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hsgwa
I have one question.
I checked this PR ran correctly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! (Sorry to forget)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There seems a problem in Galactic.
We may need a new branch for galactic in ros2caret repository, but it makes us to maintain two branches...
ros2caret/verb/record.py
Outdated
base_path=args.path, | ||
ros_events=events_ust, | ||
kernel_events=events_kernel, | ||
context_fields=context_fields, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Runtime error occurs in Galactic (Ubuntu 20.04), because ros2_tracing in Galactic uses context_names
instead of context_fields
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@takeshi-iwanari
Thank you for pointing out!
fixed in: c3c3e98
Signed-off-by: hsgwa <19860128+hsgwa@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: hsgwa 19860128+hsgwa@users.noreply.github.com
This PR add new CLI:
This is just a wrapper of following:
ros2 trace -k -u "ros*"
I'm planning to add some features to
ros2 caret record
ros2 run caret_trace clock_recorder
.