Skip to content
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

ergoCubSN001/GazeboV1_1*: add ft imu via sensor #222

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Conversation

@Nicogene Nicogene self-assigned this Feb 21, 2024
@GiulioRomualdi
Copy link
Contributor

Hi @Nicogene I visualized the frames and they are placed where expected as you can notice the in following picture

image

Now I will do a check similarly to #217 (comment)

@GiulioRomualdi
Copy link
Contributor

I checked the IMU data with https://github.com/ami-iit/xcub-sensors-check and all of the readouts are correct. I added the gazebo model with the root fixed and I moved the robot from the motorgui

🧪 Running 12 tests...
├── Running test L_FOOT_FRONT_ORIENTATION...
├── Running test L_FOOT_REAR_ORIENTATION...
├── Running test L_FOOT_FRONT_GYRO...
├── Running test L_FOOT_REAR_GYRO...
├── Running test L_FOOT_REAR_ACC...
├── Running test L_FOOT_FRONT_ACC...
├── Running test R_FOOT_FRONT_ORIENTATION...
├── Running test R_FOOT_REAR_ORIENTATION...
├── Running test R_FOOT_FRONT_GYRO...
├── Running test R_FOOT_REAR_GYRO...
├── Running test R_FOOT_REAR_ACC...
└── Running test R_FOOT_FRONT_ACC...

📊 Test outcomes:
├── 🟢 Test L_FOOT_FRONT_ORIENTATION: Passed
├── 🟢 Test L_FOOT_REAR_ORIENTATION: Passed
├── 🟢 Test L_FOOT_FRONT_GYRO: Passed
├── 🟢 Test L_FOOT_REAR_GYRO: Passed
├── 🟢 Test L_FOOT_REAR_ACC: Passed
├── 🟢 Test L_FOOT_FRONT_ACC: Passed
├── 🟢 Test R_FOOT_FRONT_ORIENTATION: Passed
├── 🟢 Test R_FOOT_REAR_ORIENTATION: Passed
├── 🟢 Test R_FOOT_FRONT_GYRO: Passed
├── 🟢 Test R_FOOT_REAR_GYRO: Passed
├── 🟢 Test R_FOOT_REAR_ACC: Passed
└── 🟢 Test R_FOOT_FRONT_ACC: Passed
Signal Acc Gyro Orientation
r_foot_front R_FOOT_FRONT_ACC R_FOOT_FRONT_GYRO R_FOOT_FRONT_ORIENTATION
r_foot_rear R_FOOT_REAR_ACC R_FOOT_REAR_GYRO R_FOOT_REAR_ORIENTATION
l_foot_front L_FOOT_FRONT_ACC L_FOOT_FRONT_GYRO L_FOOT_FRONT_ORIENTATION
l_foot_rear L_FOOT_REAR_ACC L_FOOT_REAR_GYRO L_FOOT_REAR_ORIENTATION

Here you can find the dataset and the config file to run the tests
config_dataset_test_imu.zip

@GiulioRomualdi
Copy link
Contributor

I conducted a test on the model using an updated version of the walking controller, which aims to utilize the IMUs to enhance the system's overall robustness. The test yielded the expected results.

walking_with_imu.mp4

The only adjustment I made was increasing the frequency of the wrappers to match that of the robot (from 100Hz to 500Hz). However, this adjustment may warrant further discussion in a separate issue.

🟢 flag! @Nicogene you can merge it. Thank you very much for handling this 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants