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

[Feature Request]: Add the absolute amount of received Meshtastic packets to telemetry #4563

Closed
dm5tt opened this issue Aug 26, 2024 · 4 comments · Fixed by #5414
Closed
Assignees
Labels
enhancement New feature or request pinned Exclude from stale processing

Comments

@dm5tt
Copy link

dm5tt commented Aug 26, 2024

Platform

NRF52, ESP32, RP2040, Linux Native, other

Description

Problem: It's at the moment it's impossible to determine the utilization of a Router remotely. Without these data we cannot determine the health of the HF chain nor if the setup/position of the entire device can be improved.

Right now following data can be gathered using the Python script:

~$ /home/had/.local/bin/meshtastic --host 192.x.x.x --request-telemetry --dest \!279xxxx
Connected to radio
Sending telemetry request to !279xxxx on channelIndex:0 (this could take a while)
Telemetry received:
Battery level: 81.00%
Voltage: 4.00 V
Total channel utilization: 42.40%
Transmit air utilization: 13.42%
Uptime: 109625 s

It would be great to have an additional absolute counter of received, valid Meshtastic packets.

Example:

Received Packets: 20000

On the other side "Battery level" and "Voltage" is IMHO redundant.

@dm5tt dm5tt added the enhancement New feature or request label Aug 26, 2024
@garthvh
Copy link
Member

garthvh commented Aug 26, 2024

This is now in place available over the phone api for the node you are connected to.

#4475

@dm5tt
Copy link
Author

dm5tt commented Aug 26, 2024

Ah.. wasn't aware of that.

My use-case is a bit different as my station is a remote solar station and not directly accessible with BT/Wifi.

@thebentern
Copy link
Contributor

We could possibly make this one available over the wire as well. We need to break it out into its own module instead of leaning on Device Telemetry module as it exists now.

@thebentern thebentern self-assigned this Aug 26, 2024
@caveman99 caveman99 added the pinned Exclude from stale processing label Oct 30, 2024
@thebentern
Copy link
Contributor

thebentern commented Nov 21, 2024

After talking to @GUVWAF about this one, we've decided to make this a response to request instead of first class Telemetry module w/ interval. PR up shortly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pinned Exclude from stale processing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants
@caveman99 @garthvh @thebentern @dm5tt and others