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

Enhancement: --request-environmental #596

Closed
joneyes opened this issue Jun 15, 2024 · 7 comments · Fixed by #687
Closed

Enhancement: --request-environmental #596

joneyes opened this issue Jun 15, 2024 · 7 comments · Fixed by #687

Comments

@joneyes
Copy link

joneyes commented Jun 15, 2024

It is possible to request the following via CLI for a remote node:
--request-position (returns position co-ords & alt)
--request-telemetry (returns current device metrics)

Is it possible to add --request-environmental to query environmental metric data, when enabled?

@rcarteraz
Copy link
Member

I'm pretty sure this would actually be a firmware request first.

@joneyes
Copy link
Author

joneyes commented Jun 16, 2024

OK - I'll raise and enhancement request

@joneyes joneyes closed this as completed Jun 16, 2024
@joneyes
Copy link
Author

joneyes commented Jun 16, 2024

OK - I'll raise and enhancement request

@joneyes joneyes reopened this Jun 23, 2024
@joneyes
Copy link
Author

joneyes commented Jun 23, 2024

Firmware to support this has now been updated.

@ianmcorvidae
Copy link
Contributor

Excellent, thanks for shepherding that. I'll try to get it implemented over here sometime soon, hopefully.

@joneyes
Copy link
Author

joneyes commented Oct 12, 2024

Did you make any progress on this? - specifically looking for the power metrics telemetry

@ianmcorvidae
Copy link
Contributor

I totally lost track of it, I'm sorry. I put together a PR to make this possible, which should go into the next release. I made it work by passing a second argument along with --request-telemetry to specify the subtype desired, rather than individual arguments for each type.

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 a pull request may close this issue.

3 participants