Enable moment estimation for quadrotor_model - Githubissues

ethz-asl / data-driven-dynamics

Data Driven Dynamics Modeling for Aerial Vehicles

Other

99 stars 14 forks source link

Enable moment estimation for quadrotor_model #102

Closed Jaeyoung-Lim closed 3 years ago

Jaeyoung-Lim commented 3 years ago

Problem Description Moment estimation has been disabled on the quadrotor_model. This PR enables moment estimation by default on the quadrotor_model, and fixes the issues that occured when trying to enable moment estimation.

Enabled moment estimation by default for quadrotor_model
Specified rotor type as pipeline was complaining no rotor type
Removed failure assertion when moment estimation was enabled

Additional Context

Attempt to fix https://github.com/ethz-asl/data-driven-dynamics/issues/101 and fix https://github.com/ethz-asl/data-driven-dynamics/issues/99

Jaeyoung-Lim commented 3 years ago

@manumerous I think I marked this as ready for review as a mistake: the moment parameters are still missing from the model results