ylabonte / proconip-hass

Home Assistant integration for the Pool Digital ProCon.IP swimming pool control system. All relevant pool controller info as sensors in Home Assistant and various options to switch all relays as desired.
MIT License
3 stars 3 forks source link

Redox Wert, gerundeter Wert fehlerhaft?! #15

Closed gisbertg closed 1 year ago

gisbertg commented 1 year ago
Bildschirmfoto 2023-06-06 um 18 12 40 Bildschirmfoto 2023-06-06 um 18 11 51

Der Wert springt hin und her, nach dem Komma.Das soll nicht so sein, oder? Geht es mit dem Projekt weiter? Sieht ansonsten ja schon ganz gut aus.

ylabonte commented 1 year ago

Sorry, die Integration ist noch in einem allerersten Test-Stadium... und als ich sie selber testen wollte, hat mein altes Notebook den Dienst verweigert. Damit bist du mir quasi sogar schon einen Schritt voraus. 😅 Dachte nicht, dass schon wer die Integration wahrnimmt und installiert. Werde das Projekt aber in den nächsten Wochen fortsetzen.

Ich lasse das Issue mal offen, bis ich ein erstes Release fertig habe, um dich auf diesem Kanal auf dem Laufenden zu halten bzw. dir hier mitzuteilen, wenn sich eine Aktualisierung bzw. ein erneuter Test/Einsatz der Integration lohnt.

gisbertg commented 1 year ago

Danke! https://www.poolsteuerung.de/viewtopic.php?f=29&start=20&t=1678&sid=dda7376f328568a5bd5ba0649e39f581 Vielleicht hilft dir der Link weiter?!

eseppy commented 1 year ago

Hi, ich habe mir die Integration auch installiert und bin an weiteren Relais interessiert ;-) Helfe gerne beim testen!

ylabonte commented 1 year ago

Die Integration ist jetzt grob überarbeitet und sollte so nun auch einigermaßen funktionieren (Redox + pH Sensoren, mehrere Binary Sensoren und vor allem Switches für alle Relays). Vorher solltet ihr aber die bereits angelegten Konfigurationen (unter Einstellungen -> Integrationen -> auf das kleine Zahnrad der Integration -> da dann im Kontextmenü den Eintrag löschen) sowie die zuvor installierte Integration löschen (_wirklich das Verzeichnis /config/custom_components/proconip löschen_). Ich musste die Integration umbenennen, weshalb sie jetzt unter custom_components/proconip_pool_controller abgelegt wird/werden muss.

gisbertg commented 1 year ago

Sieht schon "sehr gut" aus!!! Vielleicht könntest du die Relais noch in irgendeiner Config auswählbar machen? Im Moment werden ja alle in der Übersicht angezeigt. Aber ansonsten, Respekt!

Eins ist mir noch aufgefallen. Die Temperatursensoren werden nicht mit angezeigt?!

ylabonte commented 1 year ago

Das stimmt. Gibt da auch noch mehr Dinge, die aktuell noch fehlen:

Hab letzte Nacht nur den aktuellen Zwischenstand veröffentlicht, nachdem ich endlich überhaupt wieder einen lauffähigen Stand erreicht hatte, der mir zumindest schon halbwegs brauchbar vorkam. Im Endeffekt soll die Integration den selben Funktionsumfang bieten, wie mein ioBroker Adapter.

Ob ich kontrollieren kann, was nach der Installation im Standard-Dashboard (Übersicht) auftaucht, hab ich noch gar nicht raus. Kenne es aber von den meisten Integrationen so, dass sie die Übersicht ziemlich zumüllen, weshalb ich selber eigentlich auf allen Geräten ein eigenes Dashboard als Standard konfiguriert habe, auf dem nur das angezeigt wird, das ich auch wirklich sehen/nutzen will und diese Standard-Übersicht gar nicht mehr nutze. Aber ich schaue mal, ob ich es hinbekomme, dass standardmäßig nur Relais, Temperatur-Sensoren, etc. angezeigt werden, die nicht mit "n.a." benannt sind.

ylabonte commented 1 year ago

Ich hab gerade schon mal die Temperatur-Sensoren ergänzt...

gisbertg commented 1 year ago

funktioniert!

ylabonte commented 1 year ago

Jetzt gibt es auch

gisbertg commented 1 year ago

Und die Restinhalte funktionieren auch schon. 👍

gisbertg commented 1 year ago

https://community.home-assistant.io/t/how-to-show-3-states-on-a-button/525204 Hier vielleicht eine Idee für einen 3 Wege Button? On/Auto/Off

ylabonte commented 1 year ago

Das ist eine Konfiguration/Hilfestellung für die Verwendung von Conditional Cards, also eine Lösung, die man als Anwender nutzen kann. Oder mit anderen Worten, etwas das du selber machen kannst, aber nichts, was ich im Rahmen der Integration einbauen oder out-of-the-box bereitstellen könnte. Ich glaube aber, die Lösung bringt auch nur einen Button, der mehrere States anzeigen kann. Wünschenswert wäre ja aber ein Button bzw. eine Card, die auch alle drei Schaltflächen bietet. Hierzu müsste man eine Custom Card implementieren (https://developers.home-assistant.io/docs/frontend/custom-ui/custom-card/). Will nicht ausschließen, dass ich das noch tue, aber das wird dann ein separates kleines Projekt.

Komme aber auch erst nächste Woche wieder dazu, mich weiter mit dem Thema auseinanderzusetzen.

gisbertg commented 1 year ago

Genau, den Tri-State Button gibt es ja leider nicht. Da muß man tricksen. Hab keine Eile, super Arbeit!

ylabonte commented 1 year ago

Ich will nicht ausschließen, dass ich noch etwas eigenes mache. Aber vorerst würde ich Vorschlagen, die custom:button-card (gibt's auch über HACS oder direkt hier) zu nutzen, um Status und alle nötigen Buttons halbwegs schlank abzubilden. Das kann im einfachsten Fall dann etwa so aussehen:

image

Das ist dann zwar kein einzelner Button, der 3 States abbildet, aber zwei Buttons, über die man den aktuellen Status in Form von Auto/Manuell und An/Aus ablesen und umschalten kann.

square: false
type: grid
columns: 1
cards:
  - type: horizontal-stack
    cards:
      - type: custom:button-card
        color_type: label-card
        name: Filterpumpe
        color: transparent
      - type: custom:button-card
        entity: switch.relay_no_8_poolpumpe_auto_mode
        show_icon: false
        show_name: false
        show_state: true
        state:
          - value: 'on'
            state_display: Auto
          - value: 'off'
            state_display: Manuell
      - type: custom:button-card
        entity: switch.relay_no_8_poolpumpe
        show_icon: false
        show_name: false
        show_state: true
        state:
          - value: 'on'
            state_display: An
          - value: 'off'
            state_display: Aus

Die grid-Card hier auf oberster Ebene mit columns: 1 ist lediglich ein Wrapper, um mehrere custom:button-cards zu bündeln. Alles ab Zeile 5 kann dann beliebig oft bzw. je Relais kopiert werden. Die name und entity Attribute müssen natürlich dann jeweils entsprechend angepasst werden.

Das ist aber auch wirklich nur das einfachste Beispiel. Im Idealfall sollte man das wohl über die Konfigurations-Template-Funktion der custom:button-card lösen, um nicht immer die gesamte Konfiguration zu kopieren. Außerdem bietet die custom:button-card unzählige Möglichkeiten Icons und Farben zu nutzen, um Dinge aufzuhübschen...

Das Issue hier würde ich ansonsten aber mal ganz gerne schließen. Das ursprüngliche hiermit gemeldet Problem hat sich ja längst geklärt und funktional sollte momentan eigentlich alles im grünen Bereich sein. Sollte dir etwas auffallen, das nicht funktioniert oder wenn du noch einen Verbesserungsvorschlag einreichen möchtest, kannst du dafür gerne jederzeit ein neues Issue eröffnen.