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
Include type hash in topic endpoint info (rep2011) #1104
Conversation
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 - some small nitpicks you could address
3b439dc
to
6ba0ec1
Compare
4200a55
to
df8837a
Compare
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.
This one looks good to me.
@achim-k Am I correct in thinking that this one has no other dependencies? If that's the case, then I'll go ahead and run CI on it.
Yes, that's correct. |
@achim-k There were a bunch of test failures in here; can you take a look? |
I was wrong. It depends on ros2/ros2cli#816 which adds marshalling support for the added |
There also seems to be a |
Signed-off-by: Hans-Joachim Krauch <achim.krauch@gmail.com>
df8837a
to
0c94401
Compare
It looks like there are still test failures. |
Pushed a fix to ros2/ros2cli#816 |
@Mergifyio backport iron |
✅ Backports have been created
|
Signed-off-by: Hans-Joachim Krauch <achim.krauch@gmail.com> (cherry picked from commit ecf8958)
Part of ros2/ros2#1159
Features
TypeHash
classTopicEndpointInfo
classThis will print the type hash when printing out verbose topic information with
rostopic info --verbose
. Note that without the--no-daemon
flag, ros2/ros2cli#816 is required which adds xmlrpc marshalling support for the newrclpy.type_hash.TypeHash
class