Skip to content

Arm status#19

Open
r1000ru wants to merge 4 commits intotbs-fpv:mainfrom
skydevices-tech:arm_status
Open

Arm status#19
r1000ru wants to merge 4 commits intotbs-fpv:mainfrom
skydevices-tech:arm_status

Conversation

@r1000ru
Copy link

@r1000ru r1000ru commented Apr 1, 2025

I suggest adding an ARM status to the frame 0x21 Flight Mode, which can be used for voice alerts, timer resets, and other tasks.

Before that, I created an issue about it. It was proposed to supplement the status with reasons for failure, but they may vary and vary depending on the flight controllers.

For message about problem, i suggest add "message" frame

Ilidur and others added 4 commits March 25, 2025 08:40
Co-authored-by: bkleiner <benedikt.kleiner@gmail.com>
@haslinghuis
Copy link

Suggest splitting the PR into

  • adding arm status
  • adding airspeed sensor

In Betaflight we use Flight Mode flag to indicate arm status

@bkleiner
Copy link
Collaborator

bkleiner commented Apr 8, 2025

Airspeed changes are caused by a rebase on our end and were not originally introduced in this PR

@JimB40
Copy link
Contributor

JimB40 commented Apr 23, 2025

@bkleiner
Copy link
Collaborator

I dislike the idea of „glueing“ this information onto a existing frame. @JimB40 idea of creating a dedicated payload seems more suitable in my mind. @JimB40 would you like to come up with a PR for this?

@haslinghuis
Copy link

@JimB40
Copy link
Contributor

JimB40 commented Apr 24, 2025

  • See current implementation (in BF) for 0x21 Flight Mode payload

I know this. We worked together to improve it. :)
In above mentioned #17 we work to separate "Arming Status" from 0x21 frame and move it to a new unique CRSF frame.

@JimB40
Copy link
Contributor

JimB40 commented Apr 24, 2025

I dislike the idea of „glueing“ this information onto a existing frame. @JimB40 idea of creating a dedicated payload seems more suitable in my mind. @JimB40 would you like to come up with a PR for this?

Frame 0x23 Arming status?

@bkleiner
Copy link
Collaborator

I dislike the idea of „glueing“ this information onto a existing frame. @JimB40 idea of creating a dedicated payload seems more suitable in my mind. @JimB40 would you like to come up with a PR for this?

Frame 0x23 Arming status?

Jup. Basically put what you describe in #17 into a PR.

@r1000ru
Copy link
Author

r1000ru commented Jun 30, 2025

Hi. Is there an understanding of whether there will be a separate frame for the ARM status and the reason for its failure? I would like to draw your attention to the fact that if the reasons are given by a bit field, it would be nice to have a flight controller software identifier. Since the lua widget linked to the model can show errors, but behind the hardware (transmission of telemetry via Bluetooth/USB/UART), there is no longer any binding to the model and the application will not be able to process the bit mask correctly.

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

Successfully merging this pull request may close these issues.

6 participants