Skip to content

Support for more sensor values #3

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

Open
jacksgt opened this issue Oct 5, 2024 · 11 comments
Open

Support for more sensor values #3

jacksgt opened this issue Oct 5, 2024 · 11 comments

Comments

@jacksgt
Copy link

jacksgt commented Oct 5, 2024

Hello,

first of all many thanks for putting together this awesome HomeAssistant integration!
I can confirm that the integration can be installed via HACS and connects correctly to the HAN port of my SmartMeterGateway.

Would it be possible to expose the rest of the OBIs values as entities in HomeAssistant?
I have attached a manual (in German) that documents the full list of OBIs values on page 27.
Handbuch-fuer-Verbraucher.pdf

I had a brief look at the source code and would be happy to help with this.

@jannickfahlbusch
Copy link
Owner

Thank you for the suggestion! Adding the values would be possible and should be doable relatively easy.

I'm currently short on time and am working on some refactorings of the component so adding more OBIS codes will probably take some time

@jacksgt
Copy link
Author

jacksgt commented Oct 8, 2024

Thanks for the quick reply!

I'd be happy to try my hand at PR, though if you're planning to refactor the code we might end up with conflicts.
Let me know what you think.

@dl6dbh

This comment was marked as off-topic.

@jannickfahlbusch

This comment was marked as off-topic.

@dl6dbh

This comment was marked as off-topic.

@jannickfahlbusch

This comment was marked as off-topic.

@AppLeYArD2901
Copy link

@dl6dbh please do not use this issue for support requests. That's not how GitHub works. Create your own issue, and maybe someone will help you there.

@jannickfahlbusch
Copy link
Owner

Hi @jacksgt,

it took way too long to refactor the initial version for better maintainability.

I just pushed #16 which should simplify the handling of additional values.

Could you send me a list of values that you'd like to have supported? I'll add them to the exposed sensors then :)

@jacksgt
Copy link
Author

jacksgt commented Dec 22, 2024

@jannickfahlbusch Thanks for your work! I put together a quick prototype at #20 , though unfortunately I cannot test it currently because my HAN port credentials don't work anymore / changed.
I'll let you know once I get the opportunity to test it locally :-)

@dl6dbh

This comment has been minimized.

@dl6dbh

This comment has been minimized.

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

No branches or pull requests

4 participants