FT7 series Wind/Airflow sensor Driver #22471
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
henrykotze
force-pushed
the
windsensor-driver
branch
from
b768a6c
to
15f6e2e
Compare
dagar
reviewed
Dec 1, 2023
dagar
reviewed
Dec 1, 2023
dagar
reviewed
Dec 1, 2023
src/drivers/wind_sensor/ft_technologies/ft_technologies_serial.cpp
Outdated
Show resolved
Hide resolved
dagar
reviewed
Dec 1, 2023
src/drivers/wind_sensor/ft_technologies/ft_technologies_serial.hpp
Outdated
Show resolved
Hide resolved
dagar
reviewed
Dec 1, 2023
src/drivers/wind_sensor/ft_technologies/ft_technologies_serial_main.cpp
Outdated
Show resolved
Hide resolved
dagar
reviewed
Dec 1, 2023
dagar
changed the title
|
Would FT7 be the right generalized driver naming? |
dagar
reviewed
Dec 1, 2023
dagar
reviewed
Dec 1, 2023
dagar
reviewed
Dec 1, 2023
src/drivers/wind_sensor/ft_technologies/ft_technologies_serial.hpp
Outdated
Show resolved
Hide resolved
henrykotze
force-pushed
the
windsensor-driver
branch
from
7342633
to
eec9d73
Compare
|
|
Jaeyoung-Lim
reviewed
Dec 4, 2023
copied paste along with formatting change Some cleanup of driver Some formatting and more debugging info year updates, remove private vars, rm CMakeList naming updates deleted files removed unnecessary parameters improved naming Resolved documentation workflow issue uorb topic name: sensor_wind -> sensor_airflow log sensor_airflow optionally
|
Flight tested it: Unfortunately, I don't have a photo with the windsensor on the drone during a flight test, but have some leading up to that test. We will need to make some adjustments to get better info
|
henrykotze
force-pushed
the
windsensor-driver
branch
from
bcf34f7
to
ee312d7
Compare
dagar
approved these changes
Jan 22, 2024
dagar
changed the title
jwwaite
pushed a commit
to AIVS-Inc/PX4-Autopilot
that referenced
this pull request
Feb 12, 2024
- new uorb topic sensor_airflow - log sensor_airflow optionally
Peize-Liu
pushed a commit
to Peize-Liu/PX4-Autopilot
that referenced
this pull request
Mar 24, 2024
- new uorb topic sensor_airflow - log sensor_airflow optionally
timzarhansen
pushed a commit
to timzarhansen/PX4-Autopilot
that referenced
this pull request
Sep 3, 2024
- new uorb topic sensor_airflow - log sensor_airflow optionally
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.
New Sensor Driver: Windsensor/Airflow Sensor
This brings support to communicate with an ultrasonic wind sensor, produced by FT technologies: FT742. It provides measurements about the speed and direction of the airflow flowing through its cavity. Its generally used when stationary, and thus will measure the wind conditions, however, there are use cases for mounting on a multicopter to do inspection of the airflow produced by wind turbines. Usecases
The sensor communicates over RS485, and thus a RS485 to UART signal transfer module is needed to communicate with the sensor, as seen in the below image
Part of a larger plan to bring the measurements into the EKF to improved its wind estimates, and hence getting the sensor in Gazebo @dagar :
gazebosim/gz-sim#2256
The code was originally based on the lightware lidar serial driver. It queries the sensor at its maximum rate of 10Hz, and publishes its data on the sensor_wind uorb topic.
The PR is an WIP, and opened for any comments or suggested changes to improved. There are still a tad amount of cleanup to do, but thought I get it out rather sooner than later.
Changelog Entry
For release notes: