Skip to content

Compass HUD: Add Point of Interest markers - direction and distance #2401

Merged
rafaellehmkuhl merged 1 commit into
bluerobotics:masterfrom
ArturoManzoli:2371b-add-POIs-to-compassHUD
Feb 27, 2026
Merged

Compass HUD: Add Point of Interest markers - direction and distance #2401
rafaellehmkuhl merged 1 commit into
bluerobotics:masterfrom
ArturoManzoli:2371b-add-POIs-to-compassHUD

Conversation

@ArturoManzoli

@ArturoManzoli ArturoManzoli commented Feb 6, 2026

Copy link
Copy Markdown
Contributor
  • Add point of interest reference on Compass HUD, with direction and distance;
  • Three modes to display target's distance;
  • Warns visually on target reaching;
  • Tap to show target distance;
pois.mp4

Helps with #2371

@rafaellehmkuhl

Copy link
Copy Markdown
Member

Suuuuper cool

@ES-Alexander ES-Alexander added the docs-needed Change needs to be documented label Feb 8, 2026

@rafaellehmkuhl rafaellehmkuhl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super cool, and working super well here.

About the highlighted info in the image below, it sometimes appears and sometimes doesn't. What is the logic being used to control that?

Image

Comment thread src/components/widgets/CompassHUD.vue Outdated
min-height: 60px;
}

.poi-icons-container {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This appears to be dead code.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Comment thread src/components/widgets/CompassHUD.vue Outdated
Comment on lines +211 to +216
poi: {
showPoiOnHUD: true,
showDistances: 'onHudSide',
},
}
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the defaultOptions object merging approach, so those options are added to existing widget instances.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>
@ArturoManzoli ArturoManzoli force-pushed the 2371b-add-POIs-to-compassHUD branch from 2358c9b to cf70bda Compare February 27, 2026 10:44
@ArturoManzoli

Copy link
Copy Markdown
Contributor Author

Super cool, and working super well here.

About the highlighted info in the image below, it sometimes appears and sometimes doesn't. What is the logic being used to control that?

That info appears whenever the POI is on heading, on a 3 degree range. So if you are aiming to reach it, whenever your'e in course, the distance left to reach it will be displayed.

The logic is based mainly on isMarkerOnHeading, that on a marker update by updatePoiMarkers, will set the marker as highlighted if its on course.

@rafaellehmkuhl

Copy link
Copy Markdown
Member

Super cool, and working super well here.
About the highlighted info in the image below, it sometimes appears and sometimes doesn't. What is the logic being used to control that?

That info appears whenever the POI is on heading, on a 3 degree range. So if you are aiming to reach it, whenever your'e in course, the distance left to reach it will be displayed.

The logic is based mainly on isMarkerOnHeading, that on a marker update by updatePoiMarkers, will set the marker as highlighted if its on course.

I believe we can improve this logic so the card don't keep appearing and disappearing when the vehicle makes turns, but it's not a blocker, so I will approve and open an issue around that.

@rafaellehmkuhl rafaellehmkuhl merged commit f7b6e89 into bluerobotics:master Feb 27, 2026
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-needed Change needs to be documented

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants