ArduPilot/ardupilot

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

Offen

#19.190 geöffnet am 09.11.2021

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (15.603 Forks)batch import
Planegood first issue

Repository-Metriken

Stars
 (9.336 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 26T 16h) (166 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandene Autotest Struktur für Plane, wahrscheinlich in Tools/autotest/ oder ähnlich. Überprüfen Sie die Forumsdiskussion unter https://discuss.ardupilot.org/t/fs long actn and fs short actn/77900/8 für Details zur Option no short failsafe. Sehen Sie sich vorhandene Tests für ähnliche Failsafe Optionen an, um das Muster zu verstehen. Es gibt keine verknüpften PRs; klären Sie bei Bedarf mit den Maintainern ab.
Tech Stack
Keine
Domain
testing
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65