Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aktor wird nicht deaktiviert, wenn Zeitfenster gesetzt ist #79

Closed
dg1rtf opened this issue Oct 15, 2022 · 9 comments
Closed

Aktor wird nicht deaktiviert, wenn Zeitfenster gesetzt ist #79

dg1rtf opened this issue Oct 15, 2022 · 9 comments
Labels

Comments

@dg1rtf
Copy link

dg1rtf commented Oct 15, 2022

Das Abschalten eines batteryAktor am Ende des vorgegebenen Zeitintervalls erfolgt nicht (betrifft ext. auch die anderen Aktor-Typen.
Ich denke der Fehler liegt in External/Aktor.h, wo in

  if (BatterySOC < AktorPercentOn && BatterySOC > AktorPercentOff){
    writeAktor(AktorPosition + counterOn, 0);
    writeAktor(AktorPosition + counterOff, 0);
  }

counterOff = 0 immer unabhaengig von der Zeit gesetzt wird.
Damit kann AktorCounterOff >= 5 (ein paar Zeilen später) eigentlich nie true werden.
Mit

  if (TimerState == true && BatterySOC < AktorPercentOn && BatterySOC > AktorPercentOff){
    writeAktor(AktorPosition + counterOn, 0);
    writeAktor(AktorPosition + counterOff, 0);
  }

sollte das Setzen auf 0 nur im Zeitfenster passieren. Gilt glaube ich auch fuer die anderen Aktoren.

VG
Thomas

@nischram
Copy link
Owner

Hallo Thomas,
danke für den Hinweis. Ich werde es mir anschauen. Der Fehler ist bei mir nicht aufgefallen weil ich die "Aktoren" nicht nutze, denn ich steuere alles mit Programmen über die Homematic.
Du kannst ja ein Feedback geben ob die Änderung bei dir erfolgreich war und ggf. deine Version der Aktor.h hier zeigen.

Gruß Nico

@nischram nischram added the bug label Oct 16, 2022
@dg1rtf
Copy link
Author

dg1rtf commented Oct 16, 2022

Hallo Nico,

ich gebe Bescheid. Ich nutze die Out-PINs auch bloss als Semaphor und hab ein Python-Script drumgebaut, was den Status abfragt und dann Steckdosen mit Tasmota Firmware per HTTP-Request ein- und ausschaltet.
Ich weiss auch nicht, welchen Einfluss mein Patch auf die Mindesteinschalt bzw. Ausschaltdauer hat.

VG
Thomas

@HorstGF
Copy link

HorstGF commented Feb 9, 2023

Hallo zusammen,
ich habe das gleiche Problem bei Überschuss: Schaltet richtig ein, aber nie mehr aus.
Reine Zeitsteuerung funktioniert.
VG
Horst

@nischram
Copy link
Owner

nischram commented Feb 9, 2023

Hallo Horst,

ich hatte doch glatt aus den Augen verloren, dass hier noch ein Problem abgestellt werden muss.
Ich schaue mal ob ich die Empfehlung von Thomas einbauen kann, es wird nur gerade aktuell ein paar Tage dauern.
VG Nico

@dg1rtf
Copy link
Author

dg1rtf commented Feb 9, 2023 via email

@HorstGF
Copy link

HorstGF commented Feb 14, 2023

Habe zwar geändert, aber leider den Teil nicht neu gemacht (make). Muss also noch einen Tag Sonne abwarten.
Mein Fehler.
VG
Horst

@dg1rtf
Copy link
Author

dg1rtf commented Feb 15, 2023 via email

@nischram
Copy link
Owner

Hallo zusammen,

ich habe mir das jetzt mehrfach angesehen, ich denke das die Anpassung mit "TimerState == true" tatsächlich in der Schleife fehlt. Wie schon erwähnt, aktuell habe ich nicht wirklich die Möglichkeit das zu testen (mache alles über die HM und habe nichts an den GPIOs hängen). Ich werde es trotzdem einbauen.

Sollte dies keine Abhilfe bringen, könnte es vielleicht auch sein, dass z.B. der Batterieaktor als AktorTyp=3 konfiguriert ist. Dann wäre die Zeiteinstellung grundsätzlich inaktiv, erst wenn der AktorTyp=13 eingestellt ist, wird zu der Betterievorgbe auch die Zeiteinstellung genutzt.

Neben der Batterievorgabe und der Zeiteinstellung gibt es noch die "AktorMinOn" Einstellung, damit wird verhindert, dass der Aktor im Grenzwert laufend hin und her schaltet (oder Beispiel Spülmaschine, damit diese ca. 3 Std läuft vorm abschalten). Diese kann auch dazu führen das zum Zeitpunkt noch nicht abgeschaltet wird. erst wenn die Zeit "AktorMinOn" abgelaufen ist.

@dg1rtf Hallo Thomas, deine letzt Nachricht hast du wohl per Mail gesendet, der Anhang wurde nicht korrekt übernommen und du hast einige Daten mitgebetet die du vielleicht bearbeitet solltet. Das Python script für die Funksteckdosen kann ich nicht einbauen oder mich dazu äußern, da ich mich mit Python nicht auskenne oder beschäftigt habe. Die Steckdosen kenne ich auch nicht.

Viele Grüße Nico

@nischram
Copy link
Owner

Hey,
Änderung ist hochgeladen, ich hoffe es klappt.
VG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants