ArduPilot/ardupilot

AC_Avoidance: Dijkstra's VisGraph should hold distances in meters

Aperta

#11.582 aperta il 13 giu 2019

Vedi su GitHubGuida contributor
 (13 commenti) (0 reazioni) (1 assegnatario)C++ (15.603 fork)batch import
CopterRovergood first issue

Metriche repository

Star
 (9336 stelle)
Metriche merge PR
 (Merge medio 26g 16h) (166 PR mergiate in 30 g)

Descrizione

Dijkstra's VisGraph class should hold distances in meters. It currently holds the distances in cm perhaps because this saves a small amount in computation but even so we should change this to be meters.

Guida contributor

Direzione di ricerca
Il problema consiste nel modificare l'unità di memorizzazione della distanza nella classe VisGraph di Dijkstra da centimetri a metri, come indicato nella riga 85 del file di intestazione `AP OADijkstra.h`. La modifica implica la rappresentazione interna e qualsiasi conversione. I commenti discutono i potenziali impatti sulle prestazioni e la necessità di aggiornare i calcoli correlati. Il maintainer assegnato non ha fornito una PR collegata, quindi un contributore dovrebbe rivedere la discussione e proporre una patch semplice per la conversione delle unità.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85