ArduPilot/ardupilot

AC_Avoidance: Dijkstra's VisGraph should hold distances in meters

开放

#11,582 创建于 2019年6月13日

在 GitHub 查看贡献者指南
 (13 条评论) (0 个反应) (1 位负责人)C++ (15,603 个派生)batch import
CopterRovergood first issue

仓库指标

星标
 (9,336 个星标)
PR 合并指标
 (平均合并 26天 16小时) (30 天内合并 166 个 PR)

描述

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.

贡献者指南

研究方向
该问题要求将 Dijkstra 的 VisGraph 类中的距离存储单位从厘米改为米，如头文件 `AP OADijkstra.h` 第 85 行所述。此更改涉及修改内部表示以及任何相关的单位转换。评论中讨论了潜在的性能影响以及更新相关计算的需要。指定的维护者尚未提供关联的 PR，因此贡献者应审阅讨论并提交一个直接的单位转换补丁。
技术栈
cpp
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
1
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1 小时以内
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
85