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

HD OSD #15

Closed
wants to merge 3 commits into from
Closed

HD OSD #15

wants to merge 3 commits into from

Conversation

Scavanger
Copy link

@Scavanger Scavanger commented Jun 2, 2023

Some changes compared to the first version:

#12

  • OSD no longer flickers (except on the statistics page after the flight, but this is due to INAV).

  • Analogue OSD still present, plugin automatically switches to HD if INAV reports HD support and is enabled, otherwise analogue is used. The CLI switch has to stay in, it is currently not possible to change the OSD displayport in INAV after initilation.

  • Full compatibility to INAV without HITL HD OSD

  • Ring buffer for HD OSD to prevent the MSP packets from becoming too large.

Adapted INAV version:
https://github.com/Scavanger/inav/tree/HITL-HD-OSD

Known issues:

  • X-Plane crashes randomly when using the map. I don't know if this bug is caused by my changes or if it was already there. The flight path from the HD OSD works.
    When I want to debug the error, the map works. :(

  • For HD OSD a dummy "MSP Displayport" must be configured on a UART in the Configurator, otherwise "WTFOS" cannot be selected in the OSD tab to configure the OSD correctly.
    Requires a change in the configurator.

  • The HD OSD needs a few seconds after connecting to be displayed, but then reacts quickly.

@RomanLut
Copy link
Owner

RomanLut commented Jun 3, 2023

Give me a green light when you think you done all changes and I can start integrating it into master.

@RomanLut
Copy link
Owner

HD OSD support is implemented in main branch.
Related PR to inav: iNavFlight/inav#9327

@RomanLut RomanLut closed this Sep 30, 2023
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.

2 participants