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

Fix bug for DJI O3's altitude when using imperial #9076

Merged
merged 1 commit into from
May 26, 2023

Conversation

MrD-RC
Copy link
Collaborator

@MrD-RC MrD-RC commented May 24, 2023

Fix for an issue discovered where the altitude symbol would not show on DJI O3 setups.

If anyone with DJI O3 could confirm, it would be most appreciated.

@MrD-RC MrD-RC added this to the 7.0 milestone May 24, 2023
MrD-RC referenced this pull request May 24, 2023
Fixes #9005

Tested on other systems via HITL. All fine. Should be tested by someone with the O3 system for final confirmation.
@mmosca
Copy link
Collaborator

mmosca commented May 24, 2023

I will try to check it out this weekend.

@psmitty7373
Copy link
Contributor

psmitty7373 commented May 24, 2023

Can confirm on my O3 Air and Goggles 2, the units display correctly now.

image

@DzikuVx DzikuVx merged commit e14efda into master May 26, 2023
@DzikuVx DzikuVx deleted the MrD_Fix-for-the-DJI-OSD-Imperial-altitude-hack branch May 26, 2023 06:47
@DzikuVx
Copy link
Member

DzikuVx commented May 26, 2023

awesome, thanks a lot

@MrD-RC MrD-RC mentioned this pull request Jun 4, 2023
MrD-RC pushed a commit that referenced this pull request Jun 8, 2023
…rial-altitude-hack

Fix bug for DJI O3's altitude when using imperial
@MrD-RC MrD-RC modified the milestones: 7.0, 6.1.1 Jun 8, 2023
sensei-hacker pushed a commit to sensei-hacker/inav_unofficial_targets that referenced this pull request Oct 4, 2023
…I-OSD-Imperial-altitude-hack

Fix bug for DJI O3's altitude when using imperial
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.

4 participants