Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added rangefinder "GY-US42(v2) Ultrasonic Range Sensor" #6792

Merged
merged 2 commits into from
Apr 9, 2021

Conversation

emtrax-ltd
Copy link
Contributor

Add the ultrasonic range sensor GY-US42 as a new rangefinder to inav.

Because of the sensor's i2c trigger write command (only point to the register and restart), there was the need to patch the Method "drivers/bus_ic2_hal.c-> i2cWriteBuffer(...)" for passing through null values to HAL_I2C_Master_Transmit (raw access).

The module and this software change was successfully testet with inav 3.0.0 (master) and the modified target "matekf405se" for the matek f405 wing.

This rebased pull request replaces the old one #5698

@JulioCesarMatias
Copy link
Collaborator

what is the maximum reliable altitude you have achieved with it?

@JulioCesarMatias
Copy link
Collaborator

JulioCesarMatias commented Apr 6, 2021

adding an average filter seems to me to be interesting >> https://github.com/JulioCesarMatias/RangeFinder

@DzikuVx
Copy link
Member

DzikuVx commented Apr 6, 2021

@emtrax-ltd build if failing on several targets, please fix

@emtrax-ltd
Copy link
Contributor Author

emtrax-ltd commented Apr 6, 2021

@emtrax-ltd build if failing on several targets, please fix

@DzikuVx ... fixed!

@JulioCesarMatias
Copy link
Collaborator

add V2 after US42?

@emtrax-ltd
Copy link
Contributor Author

add V2 after US42?

not necessary -> v1 and v2 are compatible - "v1" is no longer available on the market ... "v2", because i
tested with this version

@emtrax-ltd
Copy link
Contributor Author

what is the maximum reliable altitude you have achieved with it?

specs says [email protected] or 7m@5v -> so 5m is the limit because i2c runs at 3.3v ... BUT i limited it to 4m because ardupilot also recommends this as maximum.

@JulioCesarMatias
Copy link
Collaborator

qual é a altitude máxima confiável que você alcançou com ele?

as especificações dizem [email protected] ou 7m @ 5v -> então 5m é o limite porque o i2c funciona a 3,3v ... MAS eu limitei a 4m porque o ardupilot também recomenda isso como máximo.

for multirotors I think it's cool, but for the wing not much, because the altitude is very low, who knows in the future iNav may work with, for example, the Benewake TF02

@DzikuVx DzikuVx added this to the 3.0 milestone Apr 7, 2021
@DzikuVx DzikuVx added the Release Notes Add this when a PR needs to be mentioned in the release notes label Apr 9, 2021
@DzikuVx DzikuVx merged commit 5dbcc9b into iNavFlight:master Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants