Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NaN values in telemetry #82

Closed
quimnut opened this issue Sep 2, 2024 · 3 comments
Closed

NaN values in telemetry #82

quimnut opened this issue Sep 2, 2024 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@quimnut
Copy link
Contributor

quimnut commented Sep 2, 2024

A new node had some invalid telemetry today on .328

meshinfo_1 | | File "templates/static/node.html.j2", line 43, in block 'content'
meshinfo_1 | | {{ node.telemetry.temperature | round(1) }}°C
meshinfo_1 | | ^^^^^^^^^^^^^^^^^
meshinfo_1 | | File "/usr/local/lib/python3.12/site-packages/jinja2/filters.py", line 1169, in do_round
meshinfo_1 | | return round(value, precision)
meshinfo_1 | | ^^^^^^^^^^^^^^^^^^^^^^^
meshinfo_1 | | TypeError: type str doesn't define round method

nodes.json had;

"telemetry": {
  "relative_humidity": "NaN",
  "temperature": "NaN"
}

It also ended up in telenetry.json

meshinfo_1 | | File "templates/static/telemetry.html.j2", line 173, in block 'content'
meshinfo_1 | | {{ item.payload.temperature | round(2) }} °C

@kevinelliott
Copy link
Collaborator

@quimnut Good report. Yes, confirmed, I'm seeing these too.

@kevinelliott
Copy link
Collaborator

@quimnut Do you want to take a stab at this one?

@kevinelliott kevinelliott added the bug Something isn't working label Sep 6, 2024
@kevinelliott kevinelliott added this to the 0.1.0 milestone Sep 6, 2024
@kevinelliott
Copy link
Collaborator

Completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants
@kevinelliott @quimnut and others