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

SITL: FlightAxis Improvements #28282

Merged
merged 3 commits into from
Oct 25, 2024

Conversation

robertlong13
Copy link
Collaborator

Adds a new options parameter for the FlightAxis SITL frame:

  • Option to automatically reset the aircraft position in RealFlight!!
  • Move the existing framestring-passed options to option bits (but keep the existing framestring parsing for now)
  • Option to silence the FPS reporting printf

@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Oct 2, 2024
libraries/SITL/SIM_FlightAxis.h Outdated Show resolved Hide resolved
Copy link
Contributor

@tridge tridge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good to merge after these changes

libraries/SITL/SITL.cpp Outdated Show resolved Hide resolved
libraries/SITL/SIM_FlightAxis.h Outdated Show resolved Hide resolved
@timtuxworth
Copy link
Contributor

It makes sense to not call it FlightAxis - RealFlight stopped calling it that in (I think) Evolution.

@robertlong13
Copy link
Collaborator Author

It makes sense to not call it FlightAxis - RealFlight stopped calling it that in (I think) Evolution.

Agreed. It's an unfortunate historical holdover that we should fix in a future PR. That said, the param name we should go ahead and set correctly now.

@peterbarker peterbarker merged commit 61593e2 into ArduPilot:master Oct 25, 2024
98 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants