search

fsaris / home-assistant-zonneplan-one

Unofficial Zonneplan integration for Home Assistant
https://github.com/fsaris/home-assistant-zonneplan-one
MIT License
108 stars 18 forks source link

Issue sensor for returning #30

Closed vbakker closed 1 year ago

vbakker commented 1 year ago

I put the extra sensor in configuratief.yaml sensor:

I got this error: Invalid config for [sensor.integration]: The 'unit' option has been removed, please remove it from your configuration. Got OrderedDict([('platform', 'integration'), ('source', 'sensor.zonneplan_p1_electricity_production'), ('name', 'Zonneplan P1 electricity production kWh'), ('unit', 'kWh'), ('unit_prefix', 'k'), ('round', 2)]). (See ?, line ?).

fsaris commented 1 year ago

unit: kWh isn't needed anymore. This line can be removed. Will update the readme

vbakker commented 1 year ago

Okay, I deleted that line, but it doesn't provide any data.

Op wo 30 nov. 2022 07:00 schreef Frans Saris @.***>:

unit: kWh isn't needed anymore. This line can be removed. Will update the readme

— Reply to this email directly, view it on GitHub https://github.com/fsaris/home-assistant-zonneplan-one/issues/30#issuecomment-1331681071, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZPFQKKWC3DV76RJQLL47GTWK3UO7ANCNFSM6AAAAAASO5FKCM . You are receiving this because you authored the thread.Message ID: @.***>

fsaris commented 1 year ago

Did you check it after a few minutes? An Integration sensor needs multiple data points before it can give a value. What do you see in the Developer tools - States overview?

vbakker commented 1 year ago

Sorry, i forgot that the sun was not shining. Working great now

Op wo 30 nov. 2022 15:34 schreef Frans Saris @.***>:

Did you check it after a few minutes? An Integration sensor needs multiple data points before it can give a value. What do you see in the Developer tools - States overview?

— Reply to this email directly, view it on GitHub https://github.com/fsaris/home-assistant-zonneplan-one/issues/30#issuecomment-1332245177, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZPFQKKDYO4PAGLCQR6RG7DWK5QV3ANCNFSM6AAAAAASO5FKCM . You are receiving this because you authored the thread.Message ID: @.***>

mvdzwaan commented 1 year ago

I would advise to also add 

method: left

without that I got some strange jumps in the calculation after long periods of no increase

vbakker commented 1 year ago

I just put it the end of the sensor lines in the config......yaml?

Op wo 30 nov. 2022 21:15 schreef mvdzwaan @.***>:

I would advise to also add

method: left

without that I got some strange jumps in the calculation after long periods of no increase

— Reply to this email directly, view it on GitHub https://github.com/fsaris/home-assistant-zonneplan-one/issues/30#issuecomment-1332688599, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZPFQKKQ6P7RHM4D2FAQ56DWK6YWTANCNFSM6AAAAAASO5FKCM . You are receiving this because you authored the thread.Message ID: @.***>

mvdzwaan commented 1 year ago
vbakker commented 1 year ago

Perfect. Thank you.

Issue can be closed now

Op wo 30 nov. 2022 21:50 schreef mvdzwaan @.***>:

  • platform: integration unique_id: zonneplanproduction source: sensor.zonneplan_p1_electricity_production name: "Zonneplan P1 electricity production kWh" unit_prefix: k method: left round: 2

— Reply to this email directly, view it on GitHub https://github.com/fsaris/home-assistant-zonneplan-one/issues/30#issuecomment-1332720043, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZPFQKMTYMCQEAZBMYZUQQTWK64YXANCNFSM6AAAAAASO5FKCM . You are receiving this because you authored the thread.Message ID: @.***>