Skip to content

feat(gps): add full logging of UBX_MON_RF - #28174

Closed
ThomasRigi wants to merge 2 commits into
mainfrom
pr-ubx-mon-rf
Closed

ThomasRigi wants to merge 2 commits into
mainfrom
pr-ubx-mon-rf

Conversation

@ThomasRigi

Copy link
Copy Markdown
Member

Solved Problem

UBX_MON_RF contains useful information for debugging GNSS issues.

Solution

Log all data sent by the receiver on UBX_MON_RF

Test coverage

tbc

Context

Somewhat linked to #28173 : both give more possibilities to debug the GNSS signal.

@ThomasRigi
ThomasRigi requested a review from dakejahl August 6, 2026 16:42
Comment thread msg/SensorGnssRf.msg
# GNSS RF

uint64 timestamp # time since system start (microseconds)
uint64 timestamp_sample # time since system start (microseconds)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@dakejahl

dakejahl commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Can you combine this with the other PR? 1 PR for PX4 and 1 PR for the GPS submodule

@ThomasRigi

Copy link
Copy Markdown
Member Author

Ported to #28173 .

@ThomasRigi ThomasRigi closed this Aug 11, 2026
@ThomasRigi
ThomasRigi deleted the pr-ubx-mon-rf branch August 11, 2026 09:31
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