New test for IMU orientation

.github/workflows/ci.yml

@@ -20,7 +20,7 @@ jobs:
     strategy:
       matrix:
         build_type: [Release]
-        os: [windows-latest, ubuntu-latest, macOS-latest]
+        os: [windows-latest, ubuntu-latest]
 
     steps:
     - uses: actions/checkout@main
@@ -67,12 +67,6 @@ jobs:
                          qtbase5-dev qtdeclarative5-dev qtmultimedia5-dev libtinyxml-dev libgsl-dev wget curl autoconf \
                          autogen automake libtool mlocate libopencv-dev
 
-    - name: Dependencies [macOS]
-      if: matrix.os == 'macOS-latest'
-      run: |
-        brew update
-        brew upgrade
-        brew install ace cmake eigen gsl ipopt opencv pkg-config qt5
     - name: Source-based Dependencies [Windows] 
       if: matrix.os == 'windows-latest'
       shell: bash
@@ -111,7 +105,7 @@ jobs:
                      -DENABLE_icubmod_gazecontrollerclient=ON ..
         cmake --build . --config ${{ matrix.build_type }} --target INSTALL 
 
-    - name: Source-based Dependencies [Ubuntu/macOS]
+    - name: Source-based Dependencies [Ubuntu]
       if: matrix.os == 'ubuntu-latest' || matrix.os == 'macOS-latest'
       shell: bash
       run: |
@@ -160,7 +154,7 @@ jobs:
                      -DCMAKE_PREFIX_PATH=${GITHUB_WORKSPACE}/install -DCMAKE_INSTALL_PREFIX=${GITHUB_WORKSPACE}/install \
                      -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} ..
 
-    - name: Configure [Ubuntu/macOS]
+    - name: Configure [Ubuntu]
       if: matrix.os == 'ubuntu-latest' || matrix.os == 'macOS-latest'
       shell: bash
       run: |

.gitignore

@@ -3,4 +3,5 @@ build
 *.*~
 CMakeLists.txt.user
 result.txt
-.DS_Store
+.DS_Store
+.vscode

CMakeLists.txt

@@ -33,6 +33,7 @@ set(CMAKE_SHARED_MODULE_PREFIX "")
 # options
 option(ICUB_TESTS_USES_ICUB_MAIN "Turn on to compile the tests that depend on the icub-main repository" ON)
 option(ICUB_TESTS_USES_CODYCO    "Turn on to compile the test that depend on the codyco-superbuil repository" OFF)
+option(ICUB_TESTS_USES_IDYNTREE "Turn on to compile the test that depend on the idyntree repository" OFF)
 
 # Build examples?
 add_subdirectory(example/cpp)
@@ -99,4 +100,9 @@ add_subdirectory(src/skinWrapperTest)
 # Build system status tests (needs yarp newer than f0c9e83a66d1e2685361f82dd98b20ed6479ec04)
 #add_subdirectory(src/system-status)
 
+# Build IMU test
+if(ICUB_TESTS_USES_IDYNTREE)
+    find_package(iDynTree REQUIRED)
+    add_subdirectory(src/imu)
+endif()
 

src/imu/CMakeLists.txt

@@ -0,0 +1,22 @@
+cmake_minimum_required(VERSION 3.5)
+project(imu)
+
+add_definitions(-D_USE_MATH_DEFINES)
+
+include_directories(${CMAKE_SOURCE_DIR}
+                    ${RobotTestingFramework_INCLUDE_DIRS})
+
+robottestingframework_add_plugin(${PROJECT_NAME} HEADERS ${PROJECT_NAME}.h
+                                                 SOURCES ${PROJECT_NAME}.cpp)
+
+# add required libraries
+target_link_libraries(${PROJECT_NAME} RobotTestingFramework::RTF
+                                      RobotTestingFramework::RTF_dll
+                                      YARP::YARP_robottestingframework
+                                      iDynTree::idyntree-high-level 
+                                      iDynTree::idyntree-estimation)
+# set the installation options
+install(TARGETS ${PROJECT_NAME}
+        EXPORT ${PROJECT_NAME}
+        COMPONENT runtime
+        LIBRARY DESTINATION lib)

src/imu/imu.cpp

@@ -0,0 +1,242 @@
+#include <iostream>
+#include <cmath>
+#include <numeric>
+
+#include <robottestingframework/TestAssert.h>
+#include <robottestingframework/dll/Plugin.h>
+
+#include <yarp/os/Property.h>
+#include <yarp/os/Stamp.h>
+
+#include "imu.h"
+
+using namespace robottestingframework;
+ROBOTTESTINGFRAMEWORK_PREPARE_PLUGIN(Imu)
+
+Imu::Imu() : TestCase("Imu") { }
+
+Imu::~Imu() { }
+
+bool Imu::setup(yarp::os::Property& property) {
+
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(property.check("robot"), "The robot name must be given as the test parameter!");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(property.check("port"), "The port name must be given as the test parameter!");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(property.check("model"), "Please, provide the urdf model path.");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(property.check("frame"), "Please, provide the frame name.");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(property.check("sensor"), "Please, provide the sensor name.");
+
+    robotName = property.find("robot").asString(); // robot name
+    portName = property.find("port").asString(); // name of the port from which the data are streamed
+    modelPath = property.find("model").asString(); // urdf model path
+    frameName = property.find("frame").asString(); // frame on which the sensor is attached
+    sensorName = property.find("sensor").asString(); // sensor name within urdf
+
+    ROBOTTESTINGFRAMEWORK_TEST_REPORT("Running IMU test on "+robotName+"...");
+
+    yarp::os::Property MASclientOptions;
+    MASclientOptions.put("device", "multipleanalogsensorsclient");
+    MASclientOptions.put("remote", portName);
+    MASclientOptions.put("local", "/imuTest/"+portName);
+
+    MASclientDriver = new yarp::dev::PolyDriver(MASclientOptions);
+
+    yarp::os::Property controlBoardOptions;
+    controlBoardOptions.put("device", "remotecontrolboardremapper");
+    yarp::os::Bottle axesNames;
+    yarp::os::Bottle & axesList = axesNames.addList();
+    axesList.addString("neck_pitch");
+    axesList.addString("neck_roll");
+    axesList.addString("neck_yaw");
+    axesList.addString("torso_pitch");
+    axesList.addString("torso_roll");
+    axesList.addString("torso_yaw");
+    axesList.addString("l_shoulder_pitch");
+    axesList.addString("l_shoulder_roll");
+    axesList.addString("l_shoulder_yaw");
+    axesList.addString("l_elbow");
+    axesList.addString("r_shoulder_pitch");
+    axesList.addString("r_shoulder_roll");
+    axesList.addString("r_shoulder_yaw");
+    axesList.addString("r_elbow");
+
+    controlBoardOptions.put("axesNames", axesNames.get(0));
+
+    yarp::os::Bottle remoteControlBoards;
+    yarp::os::Bottle & remoteControlBoardsList = remoteControlBoards.addList();
+    remoteControlBoardsList.addString("/"+robotName+"/torso");
+    remoteControlBoardsList.addString("/"+robotName+"/head");
+    remoteControlBoardsList.addString("/"+robotName+"/left_arm");
+    remoteControlBoardsList.addString("/"+robotName+"/right_arm");
+    controlBoardOptions.put("remoteControlBoards", remoteControlBoards.get(0));
+    controlBoardOptions.put("localPortPrefix", "/test");
+
+    controlBoardDriver = new yarp::dev::PolyDriver(controlBoardOptions);
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(MASclientDriver->isValid(), "Device driver cannot be opened");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(MASclientDriver->view(iorientation), "Unable to open orientation interface");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(controlBoardDriver->isValid(), "Device driver cannot be opened");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(controlBoardDriver->view(ipos), "Unable to open position control interface");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(controlBoardDriver->view(ienc), "Unable to open encoder interface");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(controlBoardDriver->view(iaxes), "Unable to open axes interface");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(controlBoardDriver->view(ilim), "Unable to open limits interface");
+
+
+    outputPort.open("/test-imu/out");
+    ROBOTTESTINGFRAMEWORK_TEST_REPORT("Opening port "+outputPort.getName()+"...");
+
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(ienc->getAxes(&axes), "Cannot get number of controlled axes");
+    std::string axisName;
+    std::vector<std::string> axis;
+
+    for (int i = 0; i < axes; i++)
+    {
+        ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(iaxes->getAxisName(i, axisName), "Cannot get the name of controlled axes");
+        axis.push_back(axisName);
+    }
+
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(model.loadReducedModelFromFile(modelPath.c_str(), axis), Asserter::format("Cannot load model %s", modelPath.c_str()));
+    kinDynComp.loadRobotModel(model.model());
+
+    iDynTree::Vector3 baseLinkOrientationRad;
+    baseLinkOrientationRad.zero();
+
+    baseLinkOrientation = iDynTree::Rotation::RPY(baseLinkOrientationRad[0], baseLinkOrientationRad[1], baseLinkOrientationRad[2]);
+    I_T_base = iDynTree::Transform(baseLinkOrientation, iDynTree::Position::Zero());
+    iDynTree::Twist baseVelocity = iDynTree::Twist::Zero();
+
+    unsigned int dofs = kinDynComp.model().getNrOfDOFs();
+    s.resize(dofs);
+    ds.resize(dofs);
+
+    positions.resize(axes);
+    velocities.resize(axes);
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(ienc->getEncoders(positions.data()), "Cannot get joint positions");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(ienc->getEncoderSpeeds(velocities.data()), "Cannot get joint positions");
+
+    for(auto i = 0; i < axes; i++)
+    {
+        s.setVal(i, iDynTree::deg2rad(positions[i]));
+        ds.setVal(i, iDynTree::deg2rad(velocities[i]));
+    }
+
+    gravity.zero();
+    gravity(2) = -9.81;
+
+    kinDynComp.setRobotState(
+        I_T_base,
+        s,
+        baseVelocity,
+        ds,
+        gravity
+    );
+
+    return true;
+}
+
+void Imu::tearDown() {
+    outputPort.interrupt();
+    outputPort.close();
+
+    if(MASclientDriver)
+    {
+        delete MASclientDriver;
+        MASclientDriver = 0;
+    }
+
+    if(controlBoardDriver)
+    {
+        delete controlBoardDriver;
+        controlBoardDriver = 0;
+    }
+}
+
+void Imu::run() {
+    ROBOTTESTINGFRAMEWORK_TEST_REPORT("Starting reading IMU orientation values...");
+    ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(iorientation->getOrientationSensorMeasureAsRollPitchYaw(0, rpyValues, timestamp), "Unable to obtain rpy measurements.");
+
+    iDynTree::Rotation I_R_FK = kinDynComp.getWorldTransform(frameName).getRotation(); 
+    I_R_I_IMU = (I_R_FK * ((iDynTree::Rotation::RPY(iDynTree::deg2rad(rpyValues[0]), iDynTree::deg2rad(rpyValues[1]), iDynTree::deg2rad(rpyValues[2]))).inverse())); 
+
+    double minLim;
+    double maxLim;
+
+    for (int i = kinDynComp.model().getJointIndex("neck_pitch"); i <= kinDynComp.model().getJointIndex("neck_yaw"); i++)
+    {
+        ilim->getLimits(i, &minLim, &maxLim);
+
+        moveJoint(i, minLim + 5);
+        yarp::os::Time::delay(1.);
+        moveJoint(i, maxLim - 5);
+        yarp::os::Time::delay(1.);
+    }
+}
+
+bool Imu::sendData(iDynTree::Vector3 expectedValues, iDynTree::Vector3 imuSignal)
+{
+    yarp::os::Bottle& out = outputPort.prepare();
+    static yarp::os::Stamp stamp;
+
+    stamp.update();
+    outputPort.setEnvelope(stamp);
+    out.clear();
+
+    out.addFloat64(iDynTree::rad2deg(expectedValues[0]));
+    out.addFloat64(iDynTree::rad2deg(expectedValues[1]));
+    out.addFloat64(iDynTree::rad2deg(expectedValues[2]));
+
+    out.addFloat64(iDynTree::rad2deg(imuSignal[0]));
+    out.addFloat64(iDynTree::rad2deg(imuSignal[1]));
+    out.addFloat64(iDynTree::rad2deg(imuSignal[2]));
+
+    out.addFloat64(stamp.getTime());
+
+    outputPort.writeStrict();    
+
+    return true;
+}
+
+bool Imu::moveJoint(int ax, double pos)
+{
+    bool done = false;
+    double refPos;
+
+    yarp::os::Time::delay(.1);
+
+    ipos->positionMove(ax, pos);
+    ipos->getTargetPosition(ax, &refPos);
+
+    while(!done)
+    {
+        iorientation->getOrientationSensorMeasureAsRollPitchYaw(0, rpyValues, timestamp);
+
+        ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(ienc->getEncoders(positions.data()), "Cannot get joint positions");
+        ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(ienc->getEncoderSpeeds(velocities.data()), "Cannot get joint positions");
+
+        for (auto i = 0; i < axes; i++)
+        {
+            s.setVal(i, iDynTree::deg2rad(positions[i]));
+            ds.setVal(i, iDynTree::deg2rad(velocities[i]));
+        }  
+
+        kinDynComp.setRobotState(
+        I_T_base,
+        s,
+        baseVelocity,
+        ds,
+        gravity);
+
+        iDynTree::Rotation expectedImuSignal = kinDynComp.getWorldTransform(frameName).getRotation();
+        iDynTree::Rotation imuSignal = (I_R_I_IMU * iDynTree::Rotation::RPY(iDynTree::deg2rad(rpyValues[0]), iDynTree::deg2rad(rpyValues[1]), iDynTree::deg2rad(rpyValues[2]))); 
+
+        auto error = (expectedImuSignal * imuSignal.inverse()).log();
+        double err_mean = (std::accumulate(error.begin(), error.end(), 0)) / error.size();
+        ROBOTTESTINGFRAMEWORK_ASSERT_ERROR_IF_FALSE(err_mean < 0.1, "Error > 0.1! Aborting ...");
+
+        sendData(expectedImuSignal.asRPY(), imuSignal.asRPY());
+        ipos->checkMotionDone(&done);
+
+    }
+
+    ipos->positionMove(ax, 0.0);
+
+    return true;
+}

src/imu/imu.h

@@ -0,0 +1,62 @@
+#ifndef _IMU_H_
+#define _IMU_H_
+
+#include <yarp/dev/PolyDriver.h>
+#include <yarp/dev/MultipleAnalogSensorsInterfaces.h>
+#include <yarp/dev/IPositionControl.h>
+#include <yarp/dev/IEncoders.h>
+#include <yarp/dev/IAxisInfo.h>
+#include <yarp/dev/IControlLimits.h>
+#include <yarp/robottestingframework/TestCase.h>
+
+#include <iDynTree/KinDynComputations.h>
+#include <iDynTree/ModelLoader.h>
+#include <iDynTree/Model.h>
+
+class Imu : public yarp::robottestingframework::TestCase {
+    public:
+        Imu();
+        virtual ~Imu();
+        virtual bool setup(yarp::os::Property& property);
+        virtual void tearDown();
+        virtual void run();
+
+    private:
+        std::string robotName;
+        std::string portName;
+        std::string modelPath;
+        std::string frameName;
+        std::string sensorName;
+
+        yarp::dev::PolyDriver *MASclientDriver;
+        yarp::dev::PolyDriver *controlBoardDriver;
+        yarp::dev::IOrientationSensors* iorientation;
+        yarp::dev::IPositionControl* ipos;
+        yarp::dev::IEncoders* ienc;
+        yarp::dev::IAxisInfo* iaxes;
+        yarp::dev::IControlLimits* ilim;
+
+        yarp::os::BufferedPort <yarp::os::Bottle> outputPort;
+        yarp::sig::Vector rpyValues;
+        yarp::sig::Vector positions;
+        yarp::sig::Vector velocities;
+
+        int axes;
+        double timestamp;
+
+        iDynTree::ModelLoader model;
+        iDynTree::KinDynComputations kinDynComp;
+
+        iDynTree::VectorDynSize s;
+        iDynTree::VectorDynSize ds;
+        iDynTree::Vector3 gravity;
+        iDynTree::Rotation baseLinkOrientation;
+        iDynTree::Twist baseVelocity;
+        iDynTree::Transform I_T_base;
+        iDynTree::Rotation I_R_I_IMU;
+
+        bool sendData(iDynTree::Vector3 expectedValues, iDynTree::Vector3 imuSignal);
+        bool moveJoint(int ax, double pos);
+    };
+
+#endif //_IMU_H_