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

Drop MSP_ANALOG frame support #9920

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

DzikuVx
Copy link
Member

@DzikuVx DzikuVx commented Apr 10, 2024

All clients are required to switch to MSP2_INAV_ANALOG frame instead that provides all relevant data

@DzikuVx DzikuVx added this to the 8.0 milestone Apr 10, 2024
@stronnag
Copy link
Collaborator

Appreciate your point about "All clients are required to switch to MSP2_INAV_ANALOG"; but this is exactly how we lost third party tools like 'Ezgui' and 'Mobile Flight'.

If the 3rd party client wants to support other MSP based FCs / older INAV, rather than testing for different versions of "ANALOG", history suggests that they may just not bother.

@sensei-hacker
Copy link
Collaborator

I'm curious how many third-party applications exist at this point. I know two third parties, and one of those is rather close to being first party.

If there are only one or two of any consequence, perhaps we could consider those two specifically?

@DzikuVx
Copy link
Member Author

DzikuVx commented Apr 10, 2024

The list of clients is rather short indeed. EzGui was lost for multiple reasons, changing API was one of them.
I do wonder how long we should keep support for frames that have better alternatives

@b14ckyy
Copy link
Collaborator

b14ckyy commented Apr 11, 2024

SB app and MWP are the most important still maintained ones and FlyMe2TheMoon LUA is the third one currently implementing INAV support. I assume he will go for up2date stuff anyway. And don't forget impload.

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.

None yet

4 participants