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

Support Float type #118

Open
ChristianKuehnel opened this issue Dec 20, 2017 · 1 comment
Open

Support Float type #118

ChristianKuehnel opened this issue Dec 20, 2017 · 1 comment

Comments

@ChristianKuehnel
Copy link

If would be nice, if pysnmp would support the Float data type out of the box: https://tools.ietf.org/html/draft-perkins-opaque-01

For Home Assistant I implemented a workaround:
https://github.com/ChristianKuehnel/home-assistant/blob/b8cf07e116d539767503c7c73ead0ef6399a4107/homeassistant/components/sensor/snmp.py#L195

But this is not the most elegant solution. It would be nice, if that type would be available as part of pysnmp.

If you're looking for a test case: The system load in snmpd is a Float

@etingof
Copy link
Owner

etingof commented Dec 20, 2017

Thank you for rising this FR! It totally makes sense to me.

It is not hard to code up, however I need to think how to fit this second-stage Opaque payload en/decoding within the existing API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants