Update information about TeraRanger rangefinders #192
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hamishwillee
reviewed
Apr 4, 2018
|
|
||
| The sensors are enabled using the parameter [SENS_EN_TRANGER](../advanced_config/parameter_reference.md#SENS_EN_TRANGER) (you can set the type of sensor or that PX4 should auto-detect the type). | ||
|
|
||
| > **Note** In case of auto-detect for Evo sensors the min_range and max_range are set to the lowest and highest possible reading across the Evo family (currently 0.5 - 60 m). If a true values of min_range and max_range are required it is advised to select the appropriate model of the Evo sensor instead of using autodetect. | ||
|
|
There was a problem hiding this comment.
FYI, for rendering reasons, if two notes follow each other you have to prepend a span on the second one, as shown:
> **Note** Blah blah
<span></span>
> **Tip** Blah blah
|
@msadowski Thanks - looks excellent to me. This can be merged when PX4/PX4-Autopilot#9169 is approved. |
hamishwillee
reviewed
Apr 4, 2018
There was a problem hiding this comment.
Merged thanks. I also added the span and did minor editorial changes in 6c21103
|
@hamishwillee thanks a lot! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Last week we created a pull request for a support of new TeraRanger sensor (Evo 600Hz).
Here is a promised update to the documentation regarding the sensor types and auto-detect.
Please let me know if I should do any further changes.