ArduPilot/ardupilot

Write autotest for the no-short-failsafe option in auto mode for Plane

Ouverte

#19 190 ouverte le 9 nov. 2021

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)C++ (15 603 forks)batch import
Planegood 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

Feature request

Is your feature request related to a problem? Please describe. We had a regression where a user set the ignore-short-failsafe-in-auto option and it didn't have an effect

Describe the solution you'd like An autotest to ensure the option works

https://discuss.ardupilot.org/t/fs-long-actn-and-fs-short-actn/77900/8

Describe alternatives you've considered Having the feature break again and we don't know about it

Platform [ ] All [ ] AntennaTracker [ ] Copter [ X ] Plane [ ] Rover [ ] Submarine

Guide contributeur

Direction de recherche
Examinez la structure de test automatique existante pour Plane, probablement dans Tools/autotest/ ou similaire. Consultez la discussion du forum à l'adresse https://discuss.ardupilot.org/t/fs long actn and fs short actn/77900/8 pour les détails sur l'option no short failsafe. Examinez les tests existants pour des options de sécurité (failsafe) similaires afin de comprendre le modèle. Il n'y a pas de PR liées ; clarifiez avec les mainteneurs si nécessaire.
Stack technique
Aucun
Domaine
testing
Type d'issue
Test
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
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65