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

Additional sensors for string 1 & 2 as well as L1,2,3 of relevant external source #112

Open
3 tasks done
wittimagic opened this issue Feb 26, 2024 · 4 comments
Open
3 tasks done
Labels
enhancement New feature or request

Comments

@wittimagic
Copy link

Checklist

  • I have filled out the template to the best of my ability.
  • This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
  • This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.

No

Describe the solution you'd like

It would be nice to have dedicated sensors for all solar fields for monitoring purposes.
The modbus gives separate values for the connected strings (string 1 and string 2) as well as for the external generator per phase (L1,2,3).
I have individual single phase generators connected to the external generator meter and can therefore monitor each individual pv field as well as both strings directly connected.

This might be a nice feature for others too and should be considered as nice to have.

Describe alternatives you've considered

Alternative is to not add the sensors with the addon and get them traditionally via modbus individually.

Additional context

This is my modbus yaml as an example:

  • name: "E3DC"
    type: tcp
    host: 192.168.1.5
    port: 502
    retry_on_empty: true
    sensors:
    • name: E3DC String 1 Power
      unique_id: e3dc_string_1_power
      data_type: uint16
      address: 40101
      device_class: power
      unit_of_measurement: W
      scan_interval: 2
    • name: E3DC String 2 Power
      unique_id: e3dc_string_2_power
      data_type: uint16
      address: 40102
      device_class: power
      unit_of_measurement: W
      scan_interval: 2
    • name: E3DC Powermeter 3 L1
      unique_id: e3dc_powermeter_3_L1
      data_type: int16
      address: 40117
      device_class: power
      unit_of_measurement: W
      scan_interval: 2
    • name: E3DC Powermeter 3 L2
      unique_id: e3dc_powermeter_3_L2
      data_type: int16
      address: 40118
      device_class: power
      unit_of_measurement: W
      scan_interval: 2
    • name: E3DC Powermeter 3 L3
      unique_id: e3dc_powermeter_3_L3
      data_type: int16
      address: 40119
      device_class: power
      unit_of_measurement: W
      scan_interval: 2

Diagnostics dump

No response

@wittimagic wittimagic added the enhancement New feature or request label Feb 26, 2024
@wittimagic
Copy link
Author

If you consider implementing this idea, it might be nice to add the data as additional attributes to the already given sensors.
That way the list of all sensors will not become too crowded.

@torbennehmer
Copy link
Owner

Please add the diagnostic dump

@wittimagic
Copy link
Author

The dump of your the e3dc integration:
config_entry-e3dc_rscp-7a29928e2c66dc30bb06b0ae4364c0cf.json

Not sure how to get a dump of the modbus internal integration.

@torbennehmer
Copy link
Owner

@wittimagic I do not need a modbus dump, it#s irrelevant, as I#m not using modbus. The one of me is perfectly fine. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants