Skip to content

Conversation

@rmaia3d
Copy link
Contributor

@rmaia3d rmaia3d commented May 14, 2024

This PR adds two things to the current OSD 2D map implementation:
1 - configurable horizontal and vertical margins
2 - configurable, fixed, reference line at the center of the screen

Feature (2) draws a dashed line on the center of the map display, if using "up is north" setting. The line is aligned at the heading set by a configurable parameter. The idea is to set this heading to the same heading as an easily recognizable and static feature nearby the place you are flying, for example the runway (if flying from a club), a nearby street or fence. So now you are able to compare the craft's position on the OSD map with this reference, making it easier to orientate yourself, better align directional antennas and so on. If set to -1 (the default setting), this line is disabled.

For now, these new parameters are only configurable via the CLI. If this PR gets accepted, the parameters could probably be added to the configurator UI. But since JScript is not my area of expertise, I will leave such a PR to be made by someone more comfortable with that.

A screenshot of the line enabled and configured (this shows being used with a Walksnail video system, but it's universal, working with analog and all digital setups supported by Inav, including DJI):

Inav_Map2dLineRef_WS_Annotated

The light blue circle is the arrow which represents the craft's current position and heading (where the arrow is pointed to). The magenta ellipse marks the AHI line (artificial horizon, which works just fine together with the features from this PR). The red arrow points the reference line (feature added by this PR), configured to be aligned with the main runway (pointed to by the purple arrow).

@DzikuVx DzikuVx added this to the 8.0 milestone Jun 8, 2024
@DzikuVx
Copy link
Member

DzikuVx commented Jun 8, 2024

@rmaia3d please resolve conflicts

rmaia3d added 3 commits June 10, 2024 12:14
Adds a configurable reference line in the center of the screen, when 2D map with North as reference heading is enabled.
@rmaia3d rmaia3d force-pushed the pr/map2d_ref_line branch from 8535e68 to b9fcd40 Compare June 10, 2024 15:22
@rmaia3d
Copy link
Contributor Author

rmaia3d commented Jun 10, 2024

@rmaia3d please resolve conflicts

Done!! 👍

@b14ckyy
Copy link
Collaborator

b14ckyy commented Nov 15, 2024

Hey @rmaia3d
This is a really cool feature. Unfortunately it got missed in the right time. Can you please resolve the conflicts one more time and we can merge in master for a potential 8.1 release? (As long as no existing parameters get changed)

@b14ckyy b14ckyy modified the milestones: 8.0, 8.1 Nov 15, 2024
@MrD-RC MrD-RC modified the milestones: 8.1, 9.0 Oct 26, 2025
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.

5 participants