ArduPilot / ArduPilot/MissionPlanner

Plan: Loiter Radius field should not be used for multicopters

Open
#3,305 2 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
C#
Stars
2.4k
Forks
2.9k
Avg merge
19h 16m
Merged PRs (30d)
4

Description

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](https://github.com/ArduPilot/MissionPlanner/assets/1498098/9e5c8f63-3661-4de4-8198-9222d2d2a5db)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in the Plan screen where the Loiter Radius field is populated and in the map logic that draws circles for LOITER_TURNS commands. For multicopters, the field and radius must not be used; when the command radius is zero, no circle should be displayed.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
desktop
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.