mavsdk_tests: add multicopter alt hold test #24396
Conversation
dakejahl
force-pushed
the
pr-multicopter_altitude_hold_mavsdk_test
branch
2 times, most recently
from
37eee44 to
2e18156
Compare
If I increase the sleep time from 30 seconds to say 2 mins I bet we'd see failures at 2m. In gz sim with x500 after a few minutes of hovering I see up to 2.5m error. |
|
This PR also fixes log upload on failed tests. However the logs can be pretty big, so maybe we want to reduce SDLOG_PROFILE for SITL tests. 135MB log file from the latest failure |
|
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: |
|
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: |
|
Can you please update the models on this branch so the tests pass? |
dakejahl
force-pushed
the
pr-multicopter_altitude_hold_mavsdk_test
branch
from
f02e02a to
4e4f35b
Compare
dakejahl
force-pushed
the
pr-multicopter_altitude_hold_mavsdk_test
branch
from
4e4f35b to
a46d254
Compare
dakejahl
force-pushed
the
pr-multicopter_altitude_hold_mavsdk_test
branch
from
a46d254 to
49f58c7
Compare
|
Looks removing the random walk from the gazebo-classic IMU fixes all of the mavsdk-sitl tests. So indeed the altitude issue was likely causing the failures. Shall we remove the random walk from all of the airframes now? @bresch maybe an explanation would be helpful on how things have changed, @Jaeyoung-Lim had some concerns about why the sim worked well before |
CI SITL test for multicopter takeoff and hold. This is for testing EKF height estimation issues.