Skip to content

Conversation

@HThuren
Copy link
Member

@HThuren HThuren commented Apr 9, 2023

Show Sonar altitude at Setup tab, if Sonar respond from active Sensor.
image

@github-actions

This comment has been minimized.

@blckmn
Copy link
Member

blckmn commented Apr 9, 2023

AUTOMERGE: (FAIL)

  • github identifies PR as mergeable -> PASS
  • assigned to a milestone -> PASS
  • cooling off period lapsed -> PASS
  • commit count less or equal to three -> PASS
  • Don't merge label NOT found -> PASS
  • at least one RN: label found -> PASS
  • Tested label found -> FAIL
  • assigned to an approver -> FAIL
  • approver count at least three -> FAIL

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 2 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions
Copy link
Contributor

github-actions bot commented Apr 9, 2023

Do you want to test this code? Here you have an automated build:
Betaflight-Configurator-Android
Betaflight-Configurator-Linux
Betaflight-Configurator-macOS
Betaflight-Configurator-Windows
WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

@nerdCopter
Copy link
Member

approving untested.

@haslinghuis haslinghuis merged commit db97687 into betaflight:master Apr 12, 2023
@HThuren HThuren deleted the showSonarStatus branch April 15, 2023 10:56
@ibrahimqazi
Copy link

Show Sonar altitude at Setup tab, if Sonar respond from active Sensor. image

Hello @HThuren , does this also display LiDAR range-finder data or only sonar sensor ?

@HThuren
Copy link
Member Author

HThuren commented Sep 13, 2025

Hi https://github.com/ibrahimqazi , LIDAR are covered and attitude are supposed to be there. But at time for development I did not have a LiDAR sensor, so for sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: COMPLETED

Development

Successfully merging this pull request may close these issues.

5 participants