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

rc_cessna: increase drag and reduce motor torque #59

Merged
merged 2 commits into from
Oct 31, 2024

Conversation

StefanoColli
Copy link
Contributor

Improved rc_cessna model simulation behaviour:

  • increased wings drag for faster deceleration when there is no throttle
  • increased vertical stabilizer area for a more realistic air resistance during banking manouvers
  • decreased motor power to reduce the plane top speed at full throttle (from over 100m/s to a more reasonable 40-50m/s)

@Jaeyoung-Lim
Copy link
Member

@StefanoColli Can you share flight logs that compares the previous and fixed behavior for reference?

@StefanoColli
Copy link
Contributor Author

Do you mean some graphs showing the different behavior (e.g., for the speed at max throttle and deceleration at 0 throttle) or complete ulg logs? Because the latter also depends on the tuning on PX4 side and not only on the Gazebo models.

@Jaeyoung-Lim
Copy link
Member

@StefanoColli Both. The description of the PR only discusses changes in the code without explaining the effects on the behavior of the model

- Add wheels friction
- Motor power tweak
@StefanoColli StefanoColli marked this pull request as draft October 10, 2024 14:24
@StefanoColli
Copy link
Contributor Author

Top speed at full throttle and deceleration at min throttle

  • With the previous model the plane was reaching 110m/s airspeed in a few seconds and the deceleration at throttle=0 was very slow
    topspeed_old
  • With the new model the top speed is around 56m/s and the deceleration due to air drag is much more effective
    topspeed_new

@StefanoColli
Copy link
Contributor Author

Vertical stabilizer area increased for banking maneuvers

  • In the old model a banking maneuver would induce some oscillations in pitch and yaw
    tail_wiggle_old
  • Increasing the vertical stabilizer area (and thus its effects) smoothed those oscillations
    tail_wiggle_new

@StefanoColli
Copy link
Contributor Author

Tracking performance

Old model tracking log
New model tracking log

@StefanoColli StefanoColli marked this pull request as ready for review October 10, 2024 15:21
@Jaeyoung-Lim
Copy link
Member

@StefanoColli Sorry for the late review. Thanks!

@Jaeyoung-Lim Jaeyoung-Lim merged commit 448ef6f into PX4:main Oct 31, 2024
@StefanoColli StefanoColli deleted the rc_cessna_drag branch October 31, 2024 14:24
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.

2 participants