Skip to content

Conversation

@muramura
Copy link
Contributor

@muramura muramura commented Sep 29, 2024

The GCS can download a MISSION from the FC before executing it.
The MISSION contains sequence numbers. Each time the FC completes a MISSION item, it sends a MISSION_ITEM_REACHED notification.
The GCS can obtain commands and other information from the notified sequence numbers.
Therefore, I want to prevent the FC from sending messages.

MP displays the current sequence number on the HUD.

AFTER
Screenshot from 2024-09-29 12-59-35

BEFORE
Screenshot from 2024-09-29 11-52-56

MISSION_ITEM_REACHED
Screenshot from 2024-09-29 13-01-18

@tpwrules
Copy link
Contributor

This will break RC telemetry systems capable of conveying messages such as ELRS.

@andyp1per
Copy link
Contributor

This will break RC telemetry systems capable of conveying messages such as ELRS.

Yes agreed. Closing on that basis.

@andyp1per andyp1per closed this Sep 29, 2024
@muramura
Copy link
Contributor Author

muramura commented Sep 29, 2024

@andyp1per san.
ELRS is less common than SBUS. Should we configure it to send when using ELRS?

I would like this PR to be reopened. ELRS is used by some people, but many more use SBUS

@peterbarker
Copy link
Contributor

@muramura We could move to sending the same data via an actual protocol rather than text - but it seems some people are relying on the text currently.

Note also #27779 which would be a nice replacement for the text messages, but GCS support will be years in the making!

@andyp1per
Copy link
Contributor

There are lots of systems that rely on the text that the flight controller sends (not just this text but lots of messages) - this PR appears to break that assumption - I can't see how that is useful. I mean maybe a config option to turn down the spam?

@timtuxworth
Copy link
Contributor

timtuxworth commented Sep 29, 2024

@muramura We could move to sending the same data via an actual protocol rather than text - but it seems some people are relying on the text currently.

Note also #27779 which would be a nice replacement for the text messages, but GCS support will be years in the making!

Having an interface depend the structure and information contained in a text message is a very, very bad idea. These 2 PR's combined seem like a much better idea.

If we make it an option we can start now and in a few years when the GCS support is there, we will have a much better system.

@timtuxworth
Copy link
Contributor

Having an interface depend the structure and information contained in a text message is a very, very bad idea. These 2 PR's combined seem like a much better idea.

Every time I read this I want to add another "very". :-)

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.

5 participants