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

airspeed_selector noisy startup #14807

Closed
dagar opened this issue May 1, 2020 · 4 comments
Closed

airspeed_selector noisy startup #14807

dagar opened this issue May 1, 2020 · 4 comments
Assignees

Comments

@dagar
Copy link
Member

dagar commented May 1, 2020

INFO  [airspeed_selector] No airspeed sensor detected. Switch to non-airspeed mode.
WARN  [airspeed_selector] Airspeed: switched from sensor 0 to -1

WARN  [airspeed_selector] Airspeed: switched from sensor -1 to 1

@sfuhrer let's adjust this so there aren't unnecessary warnings in good setups.

@dagar dagar self-assigned this May 1, 2020
@Antiheavy
Copy link
Contributor

keep all that information as INFO for the logs though!

@dagar
Copy link
Member Author

dagar commented May 1, 2020

keep all that information as INFO for the logs though!

To clarify, I only want to get rid of that unnecessary output on startup. The airspeed selector should be able to cleanly initialize into airspeed mode at first unless the sensors are actually missing.

@sfuhrer
Copy link
Contributor

sfuhrer commented May 2, 2020

Agree, that's annoying, not sure when it started. @dagar have you seen it also outside of SITL?
#14811 would be a potential fix, though a bit hacky.

@sfuhrer
Copy link
Contributor

sfuhrer commented May 7, 2020

Let's close this, as it has been fixed with #14811

@sfuhrer sfuhrer closed this as completed May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants