Add support for room humidity VRC700

[muraus]

Added the support to read the actual room humidity provided by the internal sensor of the vrc700

[Miecholek]

VRC720 and VR92 - I use latest config files and thy dont support humidity. Could you add the support as well?

[pulla2908]

@Miecholek I think this is only for VCR700. That would be a another pull request in my opinion.

My question here is: Why is there no review happening? The change looks straight forward to me.

[DanielTheCoder]

Is this a similar configuration which is also present in the typespec definition file?
https://github.com/john30/ebusd-configuration/blob/aadc7ec1c4c7616ec5a423fb56f512baaee16271/src/vaillant/15.700.tsp#L73C2-L73C3

@pulla2908, @muraus may we should also add the current value to the typespec definition file. Would love to help but never had made changes to the typespec definitions

[pulla2908]

@DanielTheCoder I would say yes, this is the new approach: Maintain typespec definition file. CSV files are then generated based on this definition files (as far I understood). But I still have to deep in this topic a bit more.

[muraus]

Unfortunately, I don't have any experience with TypeSpec files yet either - I am using the old CSV version of ebusd.

Looks like the new approach is to maintan the TypeSpec Files and then the CDN Repository is automatically updated.
If this is the way to go (which i assume) this PR should be closed and the value should be added in the TypeSpec files.

@john30 It would be good to add a section about community contribution in the README or the wiki, describing how pull requests should look and what the process is.

[Miecholek]

@Miecholek I think this is only for VCR700. That would be a another pull request in my opinion.

My question here is: Why is there no review happening? The change looks straight forward to me.

I think Its a good idea to open a new pull request. Where do you get communication protocol info from? like addresses, variable types, etc. Tried to use VCR700 setup for VCR720 but it doesnt work.

[pulla2908]

New pull request is also my suggestion here. Then there start with type script definition changes.

[pulla2908]

Maybe I can help here a bit, at least I can support or do a review, at the moment it is a matter of time for me :-(

[Miecholek]

I managed to read the data by trials and errors. For VRC720 and VC92 all humidity data are stored here "02000300, 301,302..." depending how many controllers we have. The only problem is refresh rate of this file -humidity updates in +/- 1hr periods so this probalby should be changed. I am new to ebusd (just few days, dont know how the files work yet), but tried adding this to 15700.csv with no success. Would be good to have similar refresh rate like VR_71 file (every 5 seconds or at least minutes).
In my case zone 1 and 3 are VR92s, zone 2 is VRC720.

Regarding files I just added to ctlv2.csv:
*r,,,,,,B524,02000300,,,IGN:4,,,
r,,z1RoomHumidity2800clt,,,,,2800,,,EXP,,%,room humidity zone 1
*r,,,,,,B524,02000301,,,IGN:4,,,
r,,z2RoomHumidity2800clt,,,,,2800,,,EXP,,%,room humidity zone 2
*r,,,,,,B524,02000302,,,IGN:4,,,
r,,z3RoomHumidity2800clt,,,,,2800,,,EXP,,%,room humidity zone 3

Also, tried reading addr 06000901 - it only works for VRC720.
*r,,,,,,B524,06000901,,,IGN:4,,,
r,,z2901RoomHumidity0700clt,,,,,0700,,,EXP,,%,room humidity zone 2

My conclusion:
WRC720 stored humidity data under: 02000300,301,302.. 2800 and 06000900, 901,902... 0700.
VR92 stored data only under: 02000300,301,302... 2800.

Hope this helps.

Regarding refresh rate- I will try to pull it manually with mqtt request anyway, but maybe you have better solution doing this with csv.

[chrizzzp]

Much of the work has already been done/started for the VRC 720/x line of regulators. Please have a look at this PR:

#482

and this (most complete) Repo:

https://github.com/jonesPD/ebusd-configuration/blob/master/ebusd-2.1.x/en/vaillant/15.ctlv2.csv

The VRC700 is different in many registers, so this PR is really only for this older unit.

[garageboxnv]

Is the PR for typespec on humidity for vrc700 already existing?

[garageboxnv]

Is the PR for typespec on humidity for vrc700 already existing?

Added this in my pull request in tsp definition