ArduPilot / ArduPilot/ardupilot

RangeFinder: set RNGFND_DIST_MAX parameter defaults based on driver

Open
#9,745 3 comments 0 reactions 0 assignees View on GitHub
Enhancement Library
Dominant language
C++
Stars
15.9k
Forks
21.4k
Avg merge
3d 17h
Merged PRs (30d)
119

Description

We can make the configuration of range finders a little easier for the users by setting the default RNGFND_MAX_CM and RNGFND_MIN_CM values to appropriate values based on the range finder driver.

So for example, the [teraranger one wiki page](http://ardupilot.org/copter/docs/common-teraranger-one-rangefinder.html) states:
- RNGFND_MAX_CM = 1400
- RNGFND_MIN_CM = 20

So we could probably add a couple of lines like this into the AP_RangeFinder_TeraRangerI2C::init() method:

- state.min_distance_cm.set_default(20)
- state.max_distance_cm.set_default(1400)

Contributor guide

Open the contributing guide

Research direction

Start with AP_RangeFinder_TeraRangerI2C::init() and compare the documented TeraRanger One values in its wiki page. Check how driver state min_distance_cm and max_distance_cm defaults are set, then verify that each applicable driver uses appropriate documented defaults when configuration is created.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.