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

Where to setup supported sensor list #38

Closed
andreondra opened this issue Mar 21, 2024 · 0 comments
Closed

Where to setup supported sensor list #38

andreondra opened this issue Mar 21, 2024 · 0 comments

Comments

@andreondra
Copy link
Member

  1. During device configuration (add integration)
    • on inverter's fw update the device must be re-added
    • no connection to the inverter needed during integration loading (async_setup_entry should not take too long, otherwise hass wil complain)
  2. During integration loading
    • no re-adding on fw update
    • very slow integration loading if the list is loaded from the inverter due to frequent timeouts from the inverter,
    • if the list can be generated without connection to the inverter (e.g. dry run of protocol parser based on cached PCU version), it will be very fast and cleaner than 1)
  3. Dynamically update sensor list in coordinator
    • I don't think this is even possible

The 2) looks the best if the connection to the inverter can be avoided. For this to work, sermatec_inverter script will need to support dry-run of the protocol parser.

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

1 participant