Skip to content

feat(test): fixed wing test cards #26824

Open
afwilkin wants to merge 4 commits intomainfrom
andrew/fixed_wing_cards
Open

feat(test): fixed wing test cards #26824
afwilkin wants to merge 4 commits intomainfrom
andrew/fixed_wing_cards

Conversation

@afwilkin
Copy link
Copy Markdown
Contributor

Solved Problem

We are starting to do more fixed wing test cards for flight testing. This is an update to include them in the test cards for the flight test team.

We split them up between multicopter and fixed wing. In the future we will want to add the VTOL and Rover models as well

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 19, 2026

💡 Commit messages could be improved

Not blocking, but these commit messages could use some cleanup.

Commit Message Suggestion
8d4de0e8d3 addded fixed wing test cards Missing conventional commit format (e.g. "feat(ekf2): add something")
a1ab9f3239 updated flight test cards Missing conventional commit format (e.g. "feat(ekf2): add something")
0663d3b1ae Update docs/en/test_cards/fw_01_manual_modes.md Missing conventional commit format (e.g. "feat(ekf2): add something")
9170c3d3f3 Update docs/en/test_cards/fw_02_full_autonomous.md Missing conventional commit format (e.g. "feat(ekf2): add something")

See the commit message convention for details.


This comment will be automatically removed once the issues are resolved.

@github-actions
Copy link
Copy Markdown

/en/test_and_ci/test_flights.md

@afwilkin afwilkin changed the title Andrew/fixed wing cards feat(test): fixed wing test cards Mar 19, 2026
@afwilkin
Copy link
Copy Markdown
Contributor Author

/en/test_and_ci/test_flights.md

This was defined like that before.

@hamishwillee do you have any comment on my documentation update? or need me to change anything?

@afwilkin afwilkin requested a review from mrpollo March 30, 2026 19:18
mrpollo
mrpollo previously approved these changes Mar 31, 2026
@mrpollo
Copy link
Copy Markdown
Contributor

mrpollo commented Mar 31, 2026

@hamishwillee ill wait for you to merge


    ❏ Throttle automatically managed to maintain airspeed

    ❏ Roll/Yaw response 1:1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it 1:1 in FW altitude mode?


    ❏ Enable Mission End RTL

    ❏ Duration of 1 to 2 minutes
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This doesn't match the duration of the mission you set in preflight.


    ❏ Auto takeoff (hand launch or runway)

    ❏ Changes in altitude throughout the mission
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These repeat the main preflight steps. Should we prefix with "Verify" for these steps?

Copy link
Copy Markdown
Contributor

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

@afwilkin Apologies for the delay. For some reaon I am no longer getting mail notifications for this project - I only just realised that so haven't been reliably checking issues.

Love the structural changes. Just some nits.

Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants