-
Notifications
You must be signed in to change notification settings - Fork 0
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
TODOs and improvement suggestions #7
Comments
I'd love to finally have a standard for adding modbus support to home assistant for different devices. For instance, most modbus-based heatpumps, ftx/ventilation systems etc should use mostly the same approach to their units (in temperatures, out temperatures, brine in/out, etc). If the standard would be set it would be so much easier to just add things to HA instead of having to write something up each time. |
Some thoughts:
Good question and I'm maybe not the right person to comment on this. However, in the case of "just downloading something that is posted on some forum" it may be a risk if the actual code could do "anything". In that case pure code may not be the best, unless you actually can do little review yourself or at least be able to trust that someone else have done it. Just a general question; have in any way been in contact with "mainline/official" HA? I really think this should be included (and maybe eventually replace the existing modbus integration). |
Based on the Casa-drivers, this should be easy to implement. But I don’t understand how you propose to do this without any kind of modbus-list / plugin? All different devices typically use different addresses, scaling etc… |
I'm not quite sure where to go further with this integration, but I have some thoughts:
I guess I more people start to use this integration, then these points needs to be addressed. Let me know what you think :)
The text was updated successfully, but these errors were encountered: