-
Notifications
You must be signed in to change notification settings - Fork 38
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
[RobotInterface] Fix MAS sensors in YarpSensorBridge and remove cameras from SensorBridge #182
[RobotInterface] Fix MAS sensors in YarpSensorBridge and remove cameras from SensorBridge #182
Conversation
{ | ||
if (sensorInterface == nullptr) | ||
{ | ||
return -1; |
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.
size_t
is an unsigned integer. I'm afraid that this will return the maximum number
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.
Thank you! Fixed in d4dcb8a.
@@ -809,7 +712,7 @@ struct YarpSensorBridge::Impl | |||
* Attach generic IMU sensor types and MAS inertials | |||
*/ | |||
bool attachAllInertials(const yarp::dev::PolyDriverList& devList) | |||
{ | |||
{ |
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.
White spaces
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.
Fixed in d4dcb8a.
@@ -824,7 +727,7 @@ struct YarpSensorBridge::Impl | |||
return false; | |||
} | |||
} | |||
|
|||
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.
White spaces
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.
Fixed in d4dcb8a.
m_pimpl->bridgeInitialized = true; | ||
return true; | ||
} | ||
|
||
bool YarpSensorBridge::setDriversList(const yarp::dev::PolyDriverList& deviceDriversList) | ||
{ | ||
{ |
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.
White spaces
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.
Fixed in d4dcb8a.
@@ -106,13 +94,12 @@ bool YarpSensorBridge::setDriversList(const yarp::dev::PolyDriverList& deviceDri | |||
<< std ::endl; | |||
return false; | |||
} | |||
|
|||
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.
White spaces
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.
Fixed in d4dcb8a.
19c9f53
to
d4dcb8a
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.
@prashanthr05 tell me if you want to reorganize the commits otherwise I can merge
@GiulioRomualdi I cherry-picked a commit from another branch, so I think it will be ok to merge to prevent conflicts while merging the other branch. |
Done |
This PR includes,