semuconsulting/PyGPSClient

Graphview and Skyview widges omitting some NMEA GSV SVIDs

Closed this issue · 0 comments

PyGPSClient Bug Report Template

Incorrect index value for NMEA GSV records in nmea_handler.py (the parsed GSV message is correct - the issue is just with the display):

Lines 214-216 read:

        for i in range(4):
            idx = f"_{i:02d}"
            svid = getattr(data, "svid" + idx, "")

which is causing the 4th SVID in each GSV message to be omitted from the graphview and skyview widgets. The lines should read:

        for i in range(4):
            idx = f"_{i+1:02d}"
            svid = getattr(data, "svid" + idx, "")

To Reproduce

  1. Process any NMEA GSV sentence with 4 valid SVID entries.

Expected Behaviour

Should include the 4th valid SVID entry in GSV messages.

Additional context

Add any other context about the problem here.