ArduPilot/ardupilot

Rover: Guided mode's wp_bearing is incorrect for some sub modes

Ouverte

#11 284 ouverte le 7 mai 2019

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)C++ (15 603 forks)batch import
BUGRovergood first issue

Métriques du dépôt

Stars
 (9 336 étoiles)
Métriques de merge PR
 (Merge moyen 26j 16h) (166 PRs mergées en 30 j)

Description

While working on this PR: https://github.com/ArduPilot/ardupilot/pull/11283 I found that Guided mode's wp_bearing valueis incorrect for some submodes.

Guided uses the mode.cpp's wp_bearing method which calls the navigation library. This works when _guided_mode == Guided_WP but is incorrect when it's Guided_HeadingAndSpeed, Guided_TurnRateAndSpeed and Guided_Loiter.

We should override the wp_bearing for Guided mode and then handle these other sub modes. For example, for Guided_HeadingAndSpeed we should use the target heading. For Guided_TurnRateAndSpeed we should return 0.0f. For Guided_Loiter we should call ModeLoiter's wp_bearing.

Guide contributeur

Direction de recherche
Remplacez wp bearing en mode Guided pour gérer les sous modes : pour Guided HeadingAndSpeed, utilisez le cap cible ; pour Guided TurnRateAndSpeed, renvoyez 0.0f ; pour Guided Loiter, appelez wp bearing de ModeLoiter.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75