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

Plan: Loiter Radius field should not be used for multicopters #3305

Open
rmackay9 opened this issue Mar 8, 2024 · 1 comment
Open

Plan: Loiter Radius field should not be used for multicopters #3305

rmackay9 opened this issue Mar 8, 2024 · 1 comment
Labels

Comments

@rmackay9
Copy link
Contributor

rmackay9 commented Mar 8, 2024

The Plan screen has a "Loiter Radius" field which is probably populated from a parameter if using Plane. Copter doesn't have this parameter though so we should hide this field and never use it (for multicopters).

This field should also NOT be used (for multicopters) when drawing the circle on the map for LOITER_TURNS commands. If the LOITER_TURNS command's radius is zero then the circle should not be displayed because the vehicle will do a panorama shot (e.g. a pirouette, e.g. circle in place).

image

@casrya
Copy link

casrya commented Mar 8, 2024

@rmackay9
I took a look at this because I use LOITER_TURNS in missions and had never noticed any problems, and found it is fact working correctly for me with Copter.

I believe it is because my value shown in the disabled loiter radius textfield (which you have highlighted in red) is set to 0.

As a workaround you can try changing it in config.xml as follows (I suspect your existing value will be 80 which you may have set using a different vehicle type?)

<TXT_loiterrad>0</TXT_loiterrad>

P.S I agree this appears to be a vehicle type bug but thought this might be helpful in the meantime?

@rmackay9 rmackay9 changed the title Plan: Loiter Radius field should be be used for multicopters Plan: Loiter Radius field should not be used for multicopters Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants