ArduPilot/ardupilot

AC_Avoidance: Dijkstra's VisGraph should hold distances in meters

Offen

#11.582 geöffnet am 13.06.2019

Auf GitHub ansehenContributor Guide
 (13 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (15.603 Forks)batch import
CopterRovergood first issue

Repository-Metriken

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

Beschreibung

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.

Contributor Guide

Research-Richtung
Das Problem besteht darin, die Speichereinheit für Entfernungen in der VisGraph Klasse von Dijkstra von Zentimetern in Meter zu ändern, wie in der Header Datei `AP OADijkstra.h` Zeile 85 vermerkt. Die Änderung betrifft die interne Darstellung und alle Umrechnungen. In den Kommentaren werden mögliche Leistungsauswirkungen und die Notwendigkeit der Aktualisierung verwandter Berechnungen diskutiert. Der zugewiesene Maintainer hat keinen verknüpften PR bereitgestellt, daher sollte ein Mitwirkender die Diskussion überprüfen und einen einfachen Einheitenumrechnungspatch vorschlagen.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
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++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85