search

LavermanJJ / home-assistant-solarfocus

🏡 Solarfocus eco manager touch integration for Home Assistant
Apache License 2.0
14 stars 4 forks source link

Add Solar component #20

Closed saschaguilliard closed 1 year ago

saschaguilliard commented 1 year ago

Hallo,

kannst du bitte noch die Solarsensoren rein machen? Adresse 2100-2113 in der Modbus TCP - Registerdaten pdf.

Grüße Sascha

LavermanJJ commented 1 year ago

Hi @saschaguilliard, The component was missing as up to now there was no one for testing. I've created a PR at the pysolarfocus lib to add the component: https://github.com/LavermanJJ/pysolarfocus/pull/15/

However, I will first finalize the integration of the multiple-components feature in the HA-integration, afterwards I'll add the changes of the solar.

It would be great if you could already now check out the pull-request of the pysolarfocus lib and run it to see if the readings are correct. Do you feel confident enough to do so?

saschaguilliard commented 1 year ago

Ich bin nur normaler User ohne großartige Programmierkenntnisse. Ich schau was ich tun kann...

Jeroen Laverman @.***> schrieb am Sa., 29. Okt. 2022, 19:52:

Hi @saschaguilliard https://github.com/saschaguilliard, The component was missing as up to now there was no one for testing. I've created a PR at the pysolarfocus lib to add the component: LavermanJJ/pysolarfocus#15 https://github.com/LavermanJJ/pysolarfocus/pull/15

However, I will first finalize the integration of the multiple-components feature in the HA-integration, afterwards I'll add the changes of the solar.

It would be great if you could already now check out the pull-request of the pysolarfocus lib and run it to see if the readings are correct. Do you feel confident enough to do so?

— Reply to this email directly, view it on GitHub https://github.com/LavermanJJ/home-assistant-solarfocus/issues/20#issuecomment-1295917578, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3R4JFNTB4GECWM2VIFIOF3WFVP5JANCNFSM6AAAAAARR3PGWE . You are receiving this because you were mentioned.Message ID: @.***>

saschaguilliard commented 1 year ago

Ich bin nur normaler User ohne großartige Programmierkenntnisse. Ich schau was ich tun kann... Jeroen Laverman @.> schrieb am Sa., 29. Okt. 2022, 19:52: … Hi @saschaguilliard https://github.com/saschaguilliard, The component was missing as up to now there was no one for testing. I've created a PR at the pysolarfocus lib to add the component: LavermanJJ/pysolarfocus#15 <LavermanJJ/pysolarfocus#15> However, I will first finalize the integration of the multiple-components feature in the HA-integration, afterwards I'll add the changes of the solar. It would be great if you could already now check out the pull-request of the pysolarfocus lib and run it to see if the readings are correct. Do you feel confident enough to do so? — Reply to this email directly, view it on GitHub <#20 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3R4JFNTB4GECWM2VIFIOF3WFVP5JANCNFSM6AAAAAARR3PGWE . You are receiving this because you were mentioned.Message ID: @.>

Ok, ich hab keine Ahnung wie ich die Sensoren checken kann. Sorry...

LLukas22 commented 1 year ago

@saschaguilliard I created an executable from the solar implementation branch.

You could download and execute it. It will ask you for the IP address of your heating system and the try to read the solar sensor values. Would be great if you could post the output here and compare it to the values on your heating systems display.

saschaguilliard commented 1 year ago

image Du bist genial Sieht sehr gut aus. collector temperatur 1 stimmt, buffer sensor 1 stimmt, Statur (aktuell 209) stimmt auch. Ich werde noch einige male testen, wenn die Zirkulation läuft.