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

Extent the configuration for Smart Meter and Batteries #285

Open
s-allius opened this issue Feb 14, 2025 · 1 comment
Open

Extent the configuration for Smart Meter and Batteries #285

s-allius opened this issue Feb 14, 2025 · 1 comment
Labels
help wanted Extra attention is needed
Milestone

Comments

@s-allius
Copy link
Owner

s-allius commented Feb 14, 2025

To support devices other than inverters, we have several approaches. At the moment, the configuration in config.toml always starts with the keyword inverters. Now we can:

  1. simply add the new device types under inverters as well. Disadvantage, we do not check the grammar for the individual devices, as this depends on the data entered (e.g. serial number)
  2. add separate keywords like, smart_meters, batteries. Then we could customize the grammar for each type specifically.

I would prefer the second approach.

@mime24 : What do you think? How can we map this for the add-on?

Example:

[inverters."R170000000000001"]  # must start with R17, Y17 or Y47
node_id = 'inv1'                 # MQTT replacement for inverters serial number  
suggested_area = 'Roof'  # suggested installation place for home-assistant

[batteries."4100000000000001"]  # must start with 410
monitor_sn = 2070233888
node_id = 'batt1'                    # MQTT replacement for battery serial number  
suggested_area = 'Garage'  # suggested installation place for home-assistant
@s-allius s-allius added the help wanted Extra attention is needed label Feb 14, 2025
@s-allius s-allius added this to the Version 0.13 milestone Feb 14, 2025
@mime24
Copy link
Collaborator

mime24 commented Feb 14, 2025

separate keywords sounds defiantly more meaningful. It should be possible to adjust the config.yaml accordingly. but I'm not available for the next 8 days to create a test config.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants