-
Notifications
You must be signed in to change notification settings - Fork 112
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
fuse -> ROS 2 fuse_variables: Linting #296
fuse -> ROS 2 fuse_variables: Linting #296
Conversation
bd9d0f6
to
5dee971
Compare
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
5dee971
to
d4ea263
Compare
0.0, 0.0, 0.0, | ||
0.0, 0.0, 0.0, | ||
0.0, 0.0, 0.0; | ||
0.0, 0.0, 0.0, |
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.
nit, indenting looked intentional, could use /* *INDENT-OFF* */
and /* *INDENT-OFF* */
here
0.0, 0.0, 0.0, | ||
0.0, 0.0, 0.0, | ||
0.0, 0.0, 0.0; | ||
0.0, 0.0, 0.0, |
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.
same comment about indentation
|
||
double x[4] = {qw, qx, qy, qz}; | ||
fuse_core::MatrixXd actual(3, 4); | ||
actual << 0.0, 0.0, 0.0, 0.0, | ||
0.0, 0.0, 0.0, 0.0, | ||
0.0, 0.0, 0.0, 0.0; | ||
0.0, 0.0, 0.0, 0.0, |
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.
nit, indenting
bool success = parameterization->ComputeMinusJacobian(x, actual.data()); | ||
|
||
fuse_core::MatrixXd expected(3, 4); | ||
expected << 0.0, 0.0, 0.0, 0.0, | ||
0.0, 0.0, 0.0, 0.0, | ||
0.0, 0.0, 0.0, 0.0; | ||
0.0, 0.0, 0.0, 0.0, |
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.
nit indenting
d4ea263
to
298a6ef
Compare
fuse_variables/include/fuse_variables/orientation_3d_stamped.hpp
Outdated
Show resolved
Hide resolved
fuse_variables/include/fuse_variables/orientation_3d_stamped.hpp
Outdated
Show resolved
Hide resolved
Signed-off-by: methylDragon <methylDragon@gmail.com>
298a6ef
to
8013ad1
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.
LGTM with green Rpr job!
@ros-pull-request-builder retest this please! |
See: #276
I also noticed that some package.xmls were missing the schema link (causing them to get skipped over by xmllint. That's been fixed for all packages completed so far.
Pinging @svwilliams for visibility.