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.

Sign up for GitHub

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

Jump to bottom

Battery #14

Open
Pe245 opened this issue Apr 23, 2022 · 13 comments
Open

Battery #14

Pe245 opened this issue Apr 23, 2022 · 13 comments
Labels
enhancement New feature or request

Comments

@Pe245
Copy link

Pe245 commented Apr 23, 2022

It would be great to see batt level (%) and batt power (W) for automation purposes
@JanJaapKo JanJaapKo added the enhancement New feature or request label Apr 28, 2022
@JanJaapKo
Copy link
Owner

Hi @Pe245 ,

Good idea. I'm gonna need to dive into that. I'll need to figure out which values make sense to read in (Goodwe spits out a lot of stuff). Could you send me some debug logging? Just set the plugin's log level to debug and filter in the Domoticz log window for your solar panels. Hope this works (it's been a while since working on this plugin) and Domoticz doesn't truncate the log lines......

@JanJaapKo JanJaapKo mentioned this issue Apr 28, 2022
Merged
@PvdGulik
Copy link

Goodmoring,

Trying to update with "git pull", but get this error.
I don't recall that I changed anything...

Updating e1326f0..ea1284c
error: Your local changes to the following files would be overwritten by merge:
pycache/GoodWe.cpython-37.pyc
pycache/plugin.cpython-37.pyc
Please commit your changes or stash them before you merge.
Aborting

What to do?

Thanks, Pieter

@JanJaapKo
Copy link
Owner

JanJaapKo commented Apr 29, 2022 •
edited
Loading

Hey the .pyc files are binaries that python creates when running a script locally (you probably ran plugin_test.py). In the last updates they were added to the ignore file, but you'll need to pull that first ;) This should help:
git checkout .

I normally refer to this page:
start using git

@PvdGulik
Copy link

Thanks, worked for me!
I'll test the changes.

Have a nice day

@Pe245
Copy link
Author

Pe245 commented Apr 29, 2022 via email •
edited by JanJaapKo
Loading

@JanJaapKo
Copy link
Owner

JanJaapKo commented Apr 30, 2022 •
edited
Loading

Hi @Pe245 , I've got 2 questions, basically:

  1. Interesting observation that Domoticz crashes after update. Which version of Domoticz, python and OS are you running?

  2. unit E/14 is the input of a certain string of panels. This is not very accurate (I calculate it by multiplying the string's current and Voltage value so it is a snapshot and not averaged over some time) it is meant to give an idea of the generated power when more than 1 string is used (I've got a string facing East and one facing West). The inverter's output power is given in unit 4. This is the same power as you see in the Goodwe SEMS portal app. I'm assuming this it the power supplied from the inverter to the net, so either from the panels or the battery. I'm not sure how this is reported. The GoodWe API returns about 26 kB of info so there is quite some details, not everything has a very meaningful name however, so it is sometimes a bit guessing what is used for what. With my setup I could check values received with what I was seeing on the app at that moment....

So basically question number 2 is: is unit 4 not usable for your automation?

@JanJaapKo
Copy link
Owner

@Pe245 ,
I just pushed a version that logs some fields that I think are interesting to see about the battery. Can you check if they actually log something usefull?

@Pe245
Copy link
Author

Pe245 commented May 2, 2022 via email

@Pe245
Copy link
Author

Pe245 commented May 2, 2022 via email

@JanJaapKo
Copy link
Owner

Hi @Pe245 ,

Sorry for late reply, was away for holiday and work.......

Yes you need to set it to debug. Which works fine on buster but perhaps not on higher versions of Raspian. Without debug, I can't do much, I need those logs (as I don't have a battery installed I can't use my own logging)

@JanJaapKo
Copy link
Owner

ok, due to a crashed RPi, I had to upgrade to bullseye. The plugin runs fine with debug logging on Bullseye. So if you want to have the battery added then I need the data structures that are returned on the GetMonitorDetailByPowerstationId endpoint. I see I missed logging the full response in the previous version, I just updated it. So please make sure to pull the latest version on the addBattery branch. Look vor the line which sais: response station data request

@Pe245
Copy link
Author

Pe245 commented Jun 18, 2022 via email

@Pe245
Copy link
Author

Pe245 commented Jun 29, 2022 via email

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

No branches or pull requests
3 participants
@PvdGulik @JanJaapKo @Pe245