robotology / robots-configuration

Contains robots configuration files
BSD 3-Clause "New" or "Revised" License
14 stars 69 forks source link

Feature/temperature reading #597

Closed MSECode closed 10 months ago

MSECode commented 10 months ago

This PR brings the following changes:

MSECode commented 10 months ago

cc: @valegagge

MSECode commented 10 months ago

The CI is failing with repeated errors of the type:

[ERROR] HasTempSensor parameter not found for board torso-eb5-j0_2 in bottle "2FOC" (Verbose 0 0 0) (AutoCalibration 0 0 0) (HasHallSensor 0 0 0) (TemperatureSensorType NONE NONE NONE) (HasRotorEncoder 1 1 1) (HasRotorEncoderIndex 1 1 1) (HasSpeedEncoder 0 0 0) (RotorIndexOffset 155 134 328) (MotorPoles 8 8 8)
[ERROR] BOARD torso-eb5-j0_2 (IP 10.0.1.5)  Missing motion control parameters in config file
[ERROR] |yarp.dev.PolyDriver|torso-eb5-j0_2-mc| Driver <embObjMotionControl> was found but could not open

Here's the complete log:

I saw that. Btw, just one info, on which superbuild version is running the CI. I mean, on which icub-main version it is build when running the CI? Because, on the last PR I developed for icub-main, I set that the configuration parameter HasTempSensor is not mandatory anymore, so it should not complain for that. Let me know.

pattacini commented 10 months ago

Because, on the last PR I developed for icub-main, I set that the configuration parameter HasTempSensor is not mandatory anymore, so it should not complain for that.

Ok, this can explain the problem!

This CI makes use of a specific docker image that I'm just about to rebuild then, in order to include your latest mods. See:

Let's wait. I'll relaunch the CI once done.

MSECode commented 10 months ago

It should be now ready for the review

valegagge commented 10 months ago

Hi @MSECode, All is fine! 🚀 thank you!

cc @pattacini