johanmeijer / grott

Growatt inverter monitor
https://github.com/johanmeijer/grott/wiki
397 stars 111 forks source link

SPF to PVoutput wrong values #163

Open onzifonzi opened 2 years ago

onzifonzi commented 2 years ago

Hi, Here are the data received from SPF, beta docker image.

but pvoutput shows image

I think ppv1 have to be in Power column and op_watt to go in Power Used.

Regards

onzifonzi commented 2 years ago

The problem solved Here is the last output image

Here are my modifications

        if header[14:16] != "20" :
            pvdata = {
                "d"     : pvodate,
                "t"     : pvotime,
            #2.7.1    "v1"    : definedkey["pvenergytoday"]*100,
                **"v2"    : definedkey["ppv1"]/10,**
                **"v3"    : definedkey["eacDischarToday"]*100,**
                **"v4"    : definedkey["pvpowerout"]/10,**
                **"v6"    : definedkey["vpv1"]/10**
                }
            if not conf.pvdisv1 :
                pvdata["v1"] = definedkey["pvenergytoday"]*100
            else:
                if conf.verbose :  print("\t - " + "Grott PVOutput send V1 disabled")

            if conf.pvtemp :
                pvdata["v5"] = **definedkey["dcdctemp"]/10**

Thanks

johanmeijer commented 2 years ago

HI @onzifonzi. Thanks for your remark.

We can have a discussion on if we should have ppv1 or powerout being sent to pvouput.org. I personally think that powerout is a better value because that is the real (netto) power delivered to the grid (my case) or your battery (= pvv1 - the loss of energy in the inverter).

I never thought about sending energy usage information from the inverter to pvoutput because I do not have a SPF or other inverter with battery connected. My energy usage is being sent bij my energy meter.

I do think it is a good idea to sent energy usage for some inverter to pvoutput like already being done in case a smart meter (SDM630) is connected to the inverter.

I see you use eacDischarToday and not op_watt is that right? It is not that easy for me to change (as for you) because I have to take into account that multiple inverter types are being used so I have to make it possible to enable/disable this with a parameter in grott.ini for compatibility reasons.

But will implement it in the next version.

onzifonzi commented 2 years ago

Hi, @johanmeijer, Thank you for comments,

Here are the values that SPF logger sends

I have no battery and cant see what is difference between ppv1 and pvoutput, by me these values are the same :) I'm using eacDischarToday because this value is resetting at midnight.

I'll monitor values and its changing through the day and will update the thread.

Regards.

onzifonzi commented 2 years ago

Pvstatus is 10, vpv1 and ppv1 are 0 when the sun went down, but there is something strange. When op_watt and pvpowerout are below 3000 eacDischarToday increases all the day, but when they are beyond 3500 eacDischarToday stops increasing after vpv1 and ppv1 get 0 and start increasing again when sun rising up.

I think this is growatt problem and I've reported to them by email.

For now last my modifications are fine. You can include them in next version.

Regards.