hultenvp / solis-sensor

HomeAssistant integration for the SolisCloud PV Monitoring portal via SolisCloud API
Apache License 2.0
191 stars 42 forks source link

Ha can't restart after updates. #117

Closed lex53 closed 2 years ago

lex53 commented 2 years ago

Hi,

I'm getting this error and cannot restart. Any ideas?

Logger: homeassistant.config Source: config.py:454 First occurred: 10:09:31 PM (1 occurrences) Last logged: 10:09:31 PM

Invalid config for [sensor.solis]: [inverter_serial] is an invalid option for [sensor.solis]. Check: sensor.solis->inverter_serial. (See /config/sensors.yaml, line 1). Please check the docs at https://github.com/hultenvp/solis-sensor/

hultenvp commented 2 years ago

Hi @lex53,

You have a very old config inverter_serial is no longer a valid config entry. The integration now autodetects all inverters present in a plant and reports their data fields as sensors. It has been deprecated for a while and the log produced warnings that the inverter serial was deprecated, so I guess you either skipped a few updates or missed the warnings. Just remove inverter_serial and your integration will work again.

Cheers.

lex53 commented 2 years ago

Hi thanks for the quick response.

I removed inverter_serial from sensors. Yaml file but it's still not rebooting. Is that the correct file or should I be looking somewhere else?

Cheers Alex

On Mon, 16 May 2022, 08:14 hultenvp, @.***> wrote:

Hi @lex53 https://github.com/lex53,

You have a very old config https://github.com/hultenvp/solis-sensor#ginlong-platform-v2 inverter_serial is no longer a valid config entry. The integration now autodetects all inverters present in a plant and reports their data fields as sensors. It has been deprecated for a while and the log produced warnings that the inverter serial was deprecated, so I guess you either skipped a few updates or missed the warnings. Just remove inverter_serial and your integration will work again.

Cheers.

— Reply to this email directly, view it on GitHub https://github.com/hultenvp/solis-sensor/issues/117#issuecomment-1127310959, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATDHTOYQVXWZX4N2ZRREIQLVKHYVXANCNFSM5V7XW47Q . You are receiving this because you were mentioned.Message ID: @.***>

hultenvp commented 2 years ago

Hey Alex

You need to remove it from the configuration.Yaml file. I assume your sensors.yaml is included from your configuration.yaml. if so, then that should be fine otherwise edit your configuration.yaml.

As per documentation the config should look as follows:

sensor:
  - platform: solis
    name: "My Solis Inverter"
    portal_domain: "m.ginlong.com" (replace for Solarman, Sofar)
    portal_username: "portal_username"
    portal_password: "portal_password"
    portal_plant_id: "plantId"

Home assistant can sometimes be hard to convince that the config has changed, but after a reboot it must for sure accept the config.

If still not working then please post the log.

hultenvp commented 2 years ago

Hi @lex53,

Does it work now?

Cheers

snigehere commented 2 years ago

Hi.. not so far .. the energy page shows I have generated 34kw today but solis says 17.. still a spike at start of day.. but it did reset just after the spike. Screenshot_20220521-123058_Home Assistant Screenshot_20220521-123111_SolisCloud

snigehere commented 2 years ago

Sorry.. looked again and it reset midday still... I need to spend a bit of time to check everything and maybe reset Screenshot_20220521-123628_Home Assistant

hultenvp commented 2 years ago

Closing due to inactivity from submitter. Feel free to reopen.