Skip to content

fix: parse satellite fields as floats - #172

Merged
CramBL merged 1 commit into
AeroRust:mainfrom
connectedcars:mbk/quectel-bg96-gsv-support
Sep 30, 2026
Merged

CramBL merged 1 commit into
AeroRust:mainfrom
connectedcars:mbk/quectel-bg96-gsv-support

Conversation

@CramBL

@CramBL CramBL commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Resolves the first part of #171

Parse GSV elevation, azimuth, and SNR directly as floating-point values to support receivers such as the Quectel BG96 that emit fractional satellite data.

Add coverage for fractional GSV fields and update the documentation to describe the accepted field formats.

@CramBL
CramBL requested a review from elpiel September 29, 2026 14:32
@codecov

codecov Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.30%. Comparing base (5556b91) to head (2a57cf2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #172      +/-   ##
==========================================
- Coverage   81.34%   81.30%   -0.04%     
==========================================
  Files          39       39              
  Lines        1576     1573       -3     
==========================================
- Hits         1282     1279       -3     
  Misses        294      294              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@CramBL
CramBL force-pushed the mbk/quectel-bg96-gsv-support branch 2 times, most recently from 7f36bac to 6703084 Compare September 29, 2026 14:42
Parse GSV elevation, azimuth, and SNR directly as floating-point
values to support receivers such as the Quectel BG96 that emit
fractional satellite data.

Add coverage for fractional GSV fields and update the documentation
to describe the accepted field formats.
@CramBL
CramBL force-pushed the mbk/quectel-bg96-gsv-support branch from 6703084 to 2a57cf2 Compare September 30, 2026 06:58
@CramBL
CramBL merged commit 6184679 into AeroRust:main Sep 30, 2026
15 checks passed
@CramBL
CramBL deleted the mbk/quectel-bg96-gsv-support branch September 30, 2026 08:15
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.

1 participant