robotology / idyntree

Multibody Dynamics Library designed for Free Floating Robots
BSD 3-Clause "New" or "Revised" License
177 stars 67 forks source link

Sensors cleanup #36

Closed traversaro closed 9 years ago

traversaro commented 9 years ago

There are some tasks necessary to finish the cleanup of the sensors iDynTree system.

Things necessary for this week, to produce the regressor bindings for @ommac and @DiegoRomeres :

(relatively) Low priority:

cc @naveenoid can you look into this?

naveenoid commented 9 years ago

im on it

naveenoid commented 9 years ago

SensorsTree renamed to SensorsList in 600659ab077393e705e7b47feae3ff5349a70b28

traversaro commented 9 years ago

@naveenoid I was thinking about the solution we discussed yesterday and I was wondering how you can deal with different type of sensors (such as internal six-axis ft sensors as opposed to a six-axis force plate, in a sense a "sensor of external force") that have the "measurement type".

In that case you cannot simply use the sensor type that you encoded for your measurement type, because there is not unequivocal mapping between the "measurement types" and the "sensor types".

Perhaps leaving the SensorType enum in the interface is slightly more insecure (only runtime checking as opposed to compile) but easier (and quicker) to implement. : )

naveenoid commented 9 years ago

My idea was that each sensor type has a unique measurement type but its not one to one. Each sensor has only one of the measurements. But let me think a bit after the first push. So we can debate better.

naveenoid commented 9 years ago

First version (incomplete still) is in https://github.com/robotology/idyntree/commit/793ad4efa87bc430cc9efdcfe019a8afb0d12b2f

traversaro commented 9 years ago

@naveenoid any updates?

traversaro commented 9 years ago

Solved in https://github.com/robotology/idyntree/pull/89 .