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

simulate airspeed sensor in NPS #1554

Merged
merged 7 commits into from
Mar 9, 2016
Merged

simulate airspeed sensor in NPS #1554

merged 7 commits into from
Mar 9, 2016

Conversation

flixr
Copy link
Member

@flixr flixr commented Mar 7, 2016

Get airspeed, pressure and temperature from FDM and add an airspeed sensor that directly pushes equivalent airspeed to the state interface.

Also fix some other NPS related things:

  • disable airspeed_adc module for NPS
  • fix NPS_BYPASS_NPS for fixedwings

switch to Lisa/MX with IMU and add NPS target
and convert subsystems to modules
@flixr
Copy link
Member Author

flixr commented Mar 7, 2016

@gautierhattenberger anything to add here?
While it might be better to get airspeed directly from the FDM (if possible), this seems to work ok for me..

@gautierhattenberger
Copy link
Member

It looks fine for me. At some point I think only air_data should update state interface and sensor driver only publish ABI messages. But it is for a later PR. Also, here it is TAS that is computed. Probably good enough for simulation, but actual aircraft are using EAS to fly.

@flixr
Copy link
Member Author

flixr commented Mar 8, 2016

Added airspeed, pressure and temperature to the FDM interface (and get it in JSBSim case).
Airspeed sensor now uses equivalent airspeed from FDM.

As you said it would make more sense to only update the state interface from air_data, but currently most airspeed sensor modules already compute airspeed from dynamic pressure and set the state interface directly.

flixr added a commit that referenced this pull request Mar 9, 2016
simulate airspeed sensor in NPS (for fixedwings)

Get airspeed, pressure and temperature from FDM and add an airspeed sensor that directly pushes equivalent airspeed to the state interface.

Also fix some other NPS related things:
- disable airspeed_adc module for NPS
- fix NPS_BYPASS_NPS for fixedwings
@flixr flixr merged commit 76a5e13 into master Mar 9, 2016
@flixr flixr deleted the nps_airspeed branch March 9, 2016 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants