ArduPilot/ardupilot

Copter: move RC_Channels_Copter::save_trim to Copter class

Ouverte

#32 279 ouverte le 24 févr. 2026

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)C++ (15 603 forks)batch import
CopterNon-Functional Changegood first issue

Métriques du dépôt

Stars
 (9 336 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Feature Request

As part of reviewing PR https://github.com/ArduPilot/ardupilot/pull/32233 @peterbarker and @rmackay9 both thought that it would be best if this was moved to the Copter class. The means the code itself should be moved from RC_Channel_Copter to Copter.cpp (I think)

This shouldn't be any change in behaviour but it should be at least lightly tested in SITL.

Guide contributeur

Direction de recherche
Examinez la PR #32233 pour comprendre le contexte du changement prévu. Déplacez la fonction `save trim` de `RC Channel Copter` vers `Copter.cpp`, en vous assurant qu'il n'y a pas de changement de comportement. Testez le changement dans la simulation SITL.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
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.
Une demi journée
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.
Plutôt claire
Prérequis
C++ArduPilot codebase familiarity
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50