ArduPilot/ardupilot

AC_Avoidance: Dijkstra's VisGraph should hold distances in meters

Open

#11,582 opened on Jun 13, 2019

View on GitHub
 (13 comments) (0 reactions) (1 assignee)C++ (15,603 forks)batch import
CopterRovergood first issue

Repository metrics

Stars
 (9,336 stars)
PR merge metrics
 (Avg merge 26d 16h) (166 merged PRs in 30d)

Description

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 direction
Examine the VisGraph class and its usage of centimeter units; convert all related distance calculations and variables to meters.
Tech stack
cpp
Domain
backend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C++
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
85