Skip to content

Round telemetry precision#9594

Closed
fresh3nough wants to merge 1 commit intomeshtastic:developfrom
fresh3nough:fix-9542-telemetry-precision
Closed

Round telemetry precision#9594
fresh3nough wants to merge 1 commit intomeshtastic:developfrom
fresh3nough:fix-9542-telemetry-precision

Conversation

@fresh3nough
Copy link
Copy Markdown

Summary

  • round telemetry float values to fixed precision before send/serialize
  • add shared telemetry precision helper
  • update telemetry serializer tests for rounded values

Testing

  • pio test -e native -f test_meshpacket_serializer (fails: pio not installed)

Co-Authored-By: Warp agent@warp.dev

Co-Authored-By: Warp <agent@warp.dev>
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions Bot added the needs-review Needs human review label Feb 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

@fresh3nough, Welcome to Meshtastic!

Thanks for opening your first pull request. We really appreciate it.

We discuss work as a team in discord, please join us in the #firmware channel.
There's a big backlog of patches at the moment. If you have time,
please help us with some code review and testing of other PRs!

Welcome to the team 😄

@github-actions github-actions Bot added first-contribution enhancement New feature or request labels Feb 10, 2026
@fifieldt fifieldt added the ai-generated Possible AI-generated low-quality content label Feb 10, 2026
@caveman99
Copy link
Copy Markdown
Member

Sorry, this is pointless. Over the air, floating point values are always the same size, as noted in the discussion.

@caveman99 caveman99 closed this Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated Possible AI-generated low-quality content enhancement New feature or request first-contribution needs-review Needs human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants