Add basic support for b516 energy statistics#504
Add basic support for b516 energy statistics#504filippz wants to merge 1 commit intojohn30:masterfrom
Conversation
|
One should add that the
Depending on the device functionality, these statistics will report in all cases the electrical power consumption of the individual device (general consumption, even if low compared to heating power consumptions). In addition to that, the VRC720 regulator can also provide the It is expected also other devices support the |
|
I've added appropriate includes to |
|
I've removed |
|
Any progress on this PR to merge changes? |
|
@john30 can you please have a look into it? And maybe merge? |
I'm on it, question here |
…and add 'Stat' prefix, add StatEnergySumYear with input params
|
would be great if someone could have a look at 91a3e2f |
|
reworked+merged |
ok here is the ctlv2 (since this is still the main source on the current HEAD commit): rename it to whatever you have |
|
Tested the above 15.ctlv2.csv (which generally works, see ebusd log below), but somehow the new b516 definitions with the 'Stat' prefix are not incorporated (neither appear in HA nor when read from ebusd via TCP client). All other (preceeding) definitions of the CSV seem to be OK (e.g. Errorhistory). Any advice? ebusd log: |
|
@chrizzzp |
|
Correct. Sorry, there was a typo in the config-path and a wrong CSV was used. With the proper CSV the new b516 definitions (Stat*) work (albeit not all make sense for all circuits), with the exception of the This will throw an error I understand this definition is used to explain the message composition for b516 queries, but is it also funtional? How is this supposed to be used, e.g. via TCP client? |
I've added that one in order not to having to send some hex codes for getting any of the other values. if you want to use it, you need to hand in the source, usage, and the year, e.g. |
BTW usage should be: 0=all;3=heat;4=water;5=cool
I see, a nice example for passing additional message parameters to the read command. The hex message would be: I'm still trying to understand the StatEnergySumYear definition. Does it include the zeros preceding 'source', 'usage' and 'year'? |
thanks for pointing that out!
ah my bad, the input needs another ";", so it should be
which zeroes do you mean? |
Thanks, now it works. I had to use the -f option when reading several energy values shortly after another.
Sorry, my fault, question is obsolete. BTW would adding a definition to retrieve the monthly energy sum in a similar fashion be possible? The third parameter byte (translated to 0x00 in the statenergysumyear hex message) would have to be set in combination with the fourth byte indicating the half-year. I don't see how to achieve this with a divider, as it was possible with the year byte for the annual energy sum. Would one then have to define discrete values for the month and for the year, which would result in quite complex definitions? |
This PR introduces basic support for "b516" statistics present in some
bai(possibly only present in HW: 7603) gas boilers andhmuheat pumps which don't supportPrEnergyCount*/PrEnergySum*values.Details are explained in the first post here: #490 (comment) and during the discussions there we came to an agreement that only 14 of most useful values are to be defined, although the script (https://gist.github.com/filippz/c0d04e4d607e27fc6b89d127db0ee941) can generate 10794 values based on our findings.
Testing and comments are welcome.