Fix md5 sum of array field type #24
Conversation
|
@kt3k We've been trying to keep our fork https://github.com/fetchrobotics/rosgo more up to date because I don't think @akio is accepting PRs. (@akio feel free to correct me and we will start contributing back). I think we already fixed this but we'd welcome your change on our fork |
|
@cjds |
kt3k
force-pushed
the
feature/fix-md5-array-type
branch
from
64ed95b to
2e44ecb
Compare
|
We moved to our own fork https://github.com/seqsense/rosgo I'm sorry that we can't collaborate well on this. @akio We can help this project if you restart the development and maintainance. |
|
@kt3k if you're interested in PRing to our fork, we'd be interested in being the cannonical source for rosgo. I noticed you put in .13 support to your fork which we haven't done yet. But if we have a single source tree between us we can get each others changes |
This PR tries to fix md5 sum calculation. If the message has the field
float64[36] covariance, the current implementation usesfloat64 covariancefor its md5 text. This PR fixes it tofloat64[36] covariance.I got
c23e848cf1b7533a8d7c259073a97e6fforgeometry_msgs/PoseWithCovariancemessage type with this change, which corresponds torosmsg md5output of the type.fixes #23