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

add quick setup guide for return to home behaviours #5456

Merged
merged 1 commit into from
Oct 1, 2023

Conversation

Hwurzburg
Copy link
Contributor

@timtuxworth maybe this will help someone setting up a Qplane....at least you wont get surprise behavior... maybe not "tuned" for distances and alts, but at least the functions are consciously being chosen

@timtuxworth
Copy link
Contributor

This is a great idea @Hwurzburg - finding the "Q" versions of things is currently scattered all over the place as notes to standard "non Q" behavior. I'll start a review and I'll go through it over the weekend.

@Hwurzburg
Copy link
Contributor Author

review carefully...still have anesthesia-head

Copy link
Contributor

@timtuxworth timtuxworth left a comment

Choose a reason for hiding this comment

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

Initial thoughts.


1.If you lose RC link for greater than :ref:`FS_LONG_TIMEOUT<FS_LONG_TIMEOUT>` in fixed wing flight set :ref:`FS_LONG_ACTN<FS_LONG_ACTN>` value below to obtain various behavior options:
- "0" : do nothing if in AUTO mode, otherwise switch to RTL mode
- "1" : switch to RTL mode
Copy link
Contributor

Choose a reason for hiding this comment

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

This should mention the Q_RTL_MODE parameter which will mean (if I understand correctly) that even in fixed wing flight, a long failsafe will actually QRTL if enabled (value > 0).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

unless an option to switch to RTL is selected that(q_rtl_mode) has no impact....further down implications of RTL mode for a QPlane are discussed...will add note referencing #3,4 below this

- "2" : Transition to fixed wing, and do a special approach to home, transition back to VTOL mode, move to over home, switch to QLAND and land at home in VTOL
- "3" :Switch to QRTL mode. See below.

5. When switching to QRTL, instead of transitioning to fixed wing (assuming you are not close to home already), flying back to home, then switching to VTOL mode as you approach home, then switching to QLAND and landing at home (see :ref:`qrtl-mode` for more information), you can disable the fixed wing approach and return home and land in only VTOL mode if :ref:`Q_OPTIONS<Q_OPTIONS>` bit 16 set.
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems very convoluted. Try rewording it based on a positive statement .. i.e. don't start with "instead of" ... something like:
The standard QRTL will transition to fixed wing, fly back home, switch to VTOL then switch to QLAND to land at the home location. You can disable the fixed wing approach and return home and land in only VTOL mode if :ref:Q_OPTIONS<Q_OPTIONS> bit 16 set.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

reworded a bit...confused even me with a clearer head

- set :ref:`Q_OPTIONS<Q_OPTIONS>` bit 20 to switch to RTL mode instead. (see #3,4 below)

3. Anytime you switch to RTL (due either to manual mode change or failsafe action), do you want to execute an autoland sequence (does not need to actually have a land command, if some other action is desired), rather than just return and loiter around home?
- If yes,then set up a DO_LAND_START mission sequence and enable the :ref:`RTL_AUTOLAND<RTL_AUTOLAND>` parameter. See :ref:`do_land_start` for details of setup.
Copy link
Contributor

Choose a reason for hiding this comment

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

Sure - but when you click the link and get to do_land_start you will find no reference to VTOL/Q modes. I think this is confusing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added note on that page.....the feature itself is VTOL/FW agnostic....the NAV_LAND command evokes the do_land_start tag itself....

Copy link
Contributor

@timtuxworth timtuxworth Sep 30, 2023

Choose a reason for hiding this comment

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

Ok - this is more of a question, does this auto land behavior apply to RTL only, or would it also apply to QRTL? Either way it might be good to call it out explicitly - or is that what #4 is telling me?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

its named RTL_AUTOLAND....RTL mode (a fixed wing mode of flight) normally just returns and loiters..but can do the do_land_start sequence instead
and QRTL mode always lands in the end...with or without a fixed wing approach
if you are in VTOL flight switching to RTL mode can do the behaviors as selected in #4...if its selected to be RTL mode, the RTL_AUTOLAND can affect it, since its RTL

Copy link
Contributor

Choose a reason for hiding this comment

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

its named RTL_AUTOLAND....RTL mode (a fixed wing mode of flight) normally just returns and loiters..but can do the do_land_start sequence instead
and QRTL mode always lands in the end...with or without a fixed wing approach
if you are in VTOL flight switching to RTL mode can do the behaviors as selected in #4...if its selected to be RTL mode, the RTL_AUTOLAND can affect it, since its RTL

I'd take exactly those words and put them on the page.

@@ -29,6 +29,37 @@ Key Parameters
throttle while flying or you risk disarming your motors.
- The default :ref:`SCHED_LOOP_RATE<SCHED_LOOP_RATE>` for a QuadPlane is to 300 (Hz). Most QuadPlanes do not need this to be rasied. Some very small vehicles (< 1Kg) might benefit from setting it to 400. In heavier vehicles, their higher inertia results in lower effective control response rates, so they do not benefit from a higher loop rate. Raising above 300 only leads to larger log files in these vehicles.

.. _return_behavior_setup:

Return Behavior QuickSetup Guide:
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems like a funny place to put this on the bottom of a page that is generally about Q parameters. Perhaps it should be on it's own page.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I disagree...this is basic QuadPlane setup....it is not related to the other topics like frame selection, etc in this major First Time Setup group....if the page expands more, then putting the other sections along with this, on individual pages might be warranted, but that may never happen


1.If you lose RC link for greater than :ref:`FS_LONG_TIMEOUT<FS_LONG_TIMEOUT>` in fixed wing flight set :ref:`FS_LONG_ACTN<FS_LONG_ACTN>` value below to obtain various behavior options:
- "0" : do nothing if in AUTO mode, otherwise switch to RTL mode (see #3,4 below)
- "1" : switch to RTL mode( see #3,4 below)
Copy link
Contributor

Choose a reason for hiding this comment

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

Spacing - "mode (see #3, 4 below)" (the space should be before the ( not after


While there are many parameters setting distances and heights for the various home return modes/behaviors (read each mode's wiki page), this is a quick setup guide for basic behaviors.

1.If you lose RC link for greater than :ref:`FS_LONG_TIMEOUT<FS_LONG_TIMEOUT>` in fixed wing flight set :ref:`FS_LONG_ACTN<FS_LONG_ACTN>` value below to obtain various behavior options:
Copy link
Contributor

@timtuxworth timtuxworth Sep 30, 2023

Choose a reason for hiding this comment

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

I think the word "below" doesn't make sense since it's actually right here. For "below" I'd expect to have to scroll to find the "below". OTOH - I'm not sure if it's how the items display in the preview, but I think a sub-list would be easier to understand instead of stringing 1, 2, 3 in as part of the paragraph.

@Hwurzburg Hwurzburg force-pushed the qplane-param branch 2 times, most recently from 8b5e2c2 to 7225354 Compare September 30, 2023 19:51
Copy link
Contributor

@timtuxworth timtuxworth left a comment

Choose a reason for hiding this comment

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

Didn't mean to click "start a review"

- set :ref:`Q_OPTIONS<Q_OPTIONS>` bit 20 to switch to RTL mode instead. (see #3,4 below)

3. Anytime you switch to RTL (due either to manual mode change or failsafe action), do you want to execute an autoland sequence (does not need to actually have a land command, if some other action is desired), rather than just return and loiter around home?
- If yes,then set up a DO_LAND_START mission sequence and enable the :ref:`RTL_AUTOLAND<RTL_AUTOLAND>` parameter. See :ref:`do_land_start` for details of setup.
Copy link
Contributor

Choose a reason for hiding this comment

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

its named RTL_AUTOLAND....RTL mode (a fixed wing mode of flight) normally just returns and loiters..but can do the do_land_start sequence instead
and QRTL mode always lands in the end...with or without a fixed wing approach
if you are in VTOL flight switching to RTL mode can do the behaviors as selected in #4...if its selected to be RTL mode, the RTL_AUTOLAND can affect it, since its RTL

I'd take exactly those words and put them on the page.

@Hwurzburg
Copy link
Contributor Author

@timtuxworth thanks for reviewing....added those words by adding a lead in section....

@Hwurzburg Hwurzburg merged commit d3d827c into ArduPilot:master Oct 1, 2023
@Hwurzburg Hwurzburg deleted the qplane-param branch October 1, 2023 00:02
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