Raspi GPIO wird gesetzt, wie Ursache herausfinden?

[paul53]

@laser sagte: Das gehe ich noch mal intensiv durch.

Auszug aus der Datei main.js des Adapters "rpi2":

function writeGpio(port, value) {
    port = parseInt(port, 10);
    if (!adapter.config.gpios[port] || !adapter.config.gpios[port].enabled) {
        adapter.log.warn('Port ' + port + ' is not writable, because disabled.');
        return;
    } else if (adapter.config.gpios[port].input === 'in' || adapter.config.gpios[port].input === 'true' || adapter.config.gpios[port].input === true) {
        return adapter.log.warn('Port ' + port + ' is configured as input and not writable');
    }

    if (value === 'true')  value = true;
    if (value === 'false') value = false;
    if (value === '0')     value = false;
    value = !!value;

Zahlen werden in Logikwerte gewandelt.

[ticaki]

@laser

bin doch nicht weg:

Em ja, das Skript macht nicht das was du willst.

bitte achte beim Schreiben von Skripten auf das Einrücken, das verhindert schon mal ne Menge Fehler

{
  code hier
  {
    code hier
    {
      code hier
    }
    code hier
  }
  code hier
}

dann hast du in Deinem Skript sowas

if (netzeinspeise <= (-0.01))
if (netzeinspeise <= (-0.1))

das erste If kannste auch weg lasen da jede Zahl kleiner -0.01 auch kleiner -0.1 ist.

du wolltest wohl:

if (netzeinspeise <= (-0.01)){
} else if (netzeinspeise <= (-0.1)){
} else if (netzeinspeise >= (0.1)){
} usw.

[Laser]

@ticaki Damit wollte ich etwas gestaffelt schalten. Wenn eine kleine Netzeinspeisung vorhanden ist, verändere den Wert (Eigenverbrauch) in 5 er Schritten. Wenn ein großer Wert vorliegt, ändere den Wert in großen Schritten. Z.B. wenn morgens der Toaster eingeschaltet wird, passe den Wert schneller an. Sonst in kleinen Schritten. Ist erst mal ein Versuch, ob ich so damit leben kann. Ist beim "Brot-Toasten" am Küchentisch entstanden. Besser geht immer.
Das Einrücken muß ich noch verbessern.

[ticaki]

@laser
Ich zeige nur nicht optimalen Code auf. Das geht echt nicht darum irgendeine Bewertung vor zu nehmen.

Zum Code, eine gestaffelte Abfrage in der nur 1 von x Möglichkeiten ausgewählt werden soll, wird mit If () else if() else usw. aufgebaut, dann trifft immer nur 1 Möglichkeit zu.

Ich dachte du wolltest das erreichen.

[Laser]

@ticaki Werde ich anpassen. Das war wirklich mit der sehr heißen Nadel gestrickt. Das gefällt mir auch nicht.
@Homoran und die Logeinträge werden auch weniger. Bin da mal mit der Keule durch gegangen.

[Homoran]

@laser sagte in Raspi GPIO wird gesetzt, wie Ursache herausfinden?:

die Logeinträge werden auch weniger. Bin da mal mit der Keule durch gegangen.

dann wird"s auch bald übersichtlicher im Log

[Laser]

@ticaki ist das so besser?

// über MQQT Datenpunkt beschreiben- alt, jetzt Adapter
let ZielwertM  = 80;
let ZielwertA = 180;

schedule({hour: [19], minute:[10] },   function () {
    setState('rpi2.0.gpio.22.state'/**/,true);  //  1 ist einschalten
});
 // Abends einen anderen Sollwert zur Einspeisung setzen
schedule({hour: [19], minute:[15] },   function () {
//ZielwertA = getState('javascript.0.variables.SollwertHoy330Abend'/*SollwertHoy330Abend*/).val;
setState('opendtu.1.112184711203.power_control.limit_nonpersistent_absolute'/**/,ZielwertA);
});

 // Morgens einen anderen Sollwert zur Einspeisung setzen
schedule({hour: [02], minute:[15] },   function () { 
//ZielwertM = getState('javascript.0.variables.SollwertHoy300'/*SollwertHoy300*/).val;
setState('opendtu.1.112184711203.power_control.limit_nonpersistent_absolute'/**/, ZielwertM);
});

schedule({hour: [09, 23 ], minute:[20] },   function () { 
setState('opendtu.1.112184711203.power_control.limit_nonpersistent_absolute'/**/,10); // vor dem Ausschalten runterfahren
});

schedule({hour: [09,10 ], minute:[27] },   function () { 
setState('rpi2.0.gpio.22.state'/**/,false);  //  0 ist ausschalten
});

// Anpassung der Einspeiseleistung (getaktet):
setInterval(Einspeisen,20000 );
function Einspeisen () {
    let netzeinspeise = getState('javascript.0.variables.DP-kW-ges-Zaehler'/*DP-kW-ges-Zaehler*/).val;
    let SWLeistung    = getState('opendtu.1.112184711203.power_control.limit_nonpersistent_absolute'/**/).val;
    // console.log(" netzeinspeise: " +netzeinspeise);

    if (netzeinspeise <= (-0.01)){
    SWLeistung = (SWLeistung -10);  // Wenn eingespeist wird, takte den SW Umrichter runter

    // Und schnell runterfahren bei großer Leistung ins Netz:
    } else if (netzeinspeise <= (-0.1)) {
    SWLeistung = (SWLeistung -100); // Wenn eingespeist wird, takte den SW Umrichter runter
    // hochfahren:
    } else if (netzeinspeise >= (0.01)){
    SWLeistung = (SWLeistung +10);  // Wenn n. eingespeist wird, takte den SW Umrichter hoch

    // schnell hochfahren bei Verbraucher >100 Watt ein:
    } else if (netzeinspeise >= (0.1)){
    SWLeistung = (SWLeistung +100);  // Wenn n. eingespeist wird, takte den SW Umrichter hoch

    };   // Ende Else if

        if (SWLeistung <=10)  {
        SWLeistung = 10;   // Begrenzer
        } else if (SWLeistung >=230) {
        SWLeistung = 230;  // Begrenzer    
        } else {SWLeistung = SWLeistung};
        console.log("Sollwert Lstg: " +SWLeistung);
        setState('opendtu.1.112184711203.power_control.limit_nonpersistent_absolute'/**/,SWLeistung);
   
};  //Ende function Einspeisen

[ticaki]

@laser

Eine kleine Optimierung am Ende: <= 10 braucht nur <10 wenns == 10 ist, ist es ja 10 auch für 230

und eine Formsache: ich würde das letzte else auch

} else {
  SWLeistung = SWLeistung
}

so schreiben ist beim lesen einfacher zu sehen das alles abgedeckt ist.

Wobei das letzte else ja keine Funktion hat, dann kannst du es auch erstmal leer lassen

[Laser]

@ticaki Ist halt immer wieder was drangestrickt worden. Einfach mit copy/paste.
So gefällt mir das viel besser. Danke!

[Laser]

Das Script mußte ich noch etwas anpassen (die Staffelung der Grenzwerte). Funktioniert jetzt wie gewollt, nur übersichtlicher.
Auch das Log ist durch die Aufräumaktion übersichtlicher geworden. Aber 5:21 Uhr passiert immer noch etwas ungewolltes:
(Auch der VE-Error ist noch vorhanden. Genau um 5:21 Uhr. Da muß ich wohl noch etwas suchen.
Das Script SSH-Exec startet den VEdirect Adapter neu, wenn gestört. Da kommt aber nichts mit 5:21 Uhr vor
Das Modbus Gerät steht zu der Zeit.)

2023-07-24 05:21:52.759  - warn: javascript.0 (3320) You are assigning a number to the state "rpi2.0.gpio.22.state" which expects a boolean. Please fix your code to use a boolean or change the state type to number. This warning might become an error in future versions.
2023-07-24 05:21:52.761  - warn: javascript.0 (3320)     at Object.setState (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:1730:20)
2023-07-24 05:21:52.761  - warn: javascript.0 (3320)     at Timeout._onTimeout (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:1788:29)
2023-07-24 05:21:52.761  - warn: javascript.0 (3320)     at listOnTimeout (node:internal/timers:571:11)
2023-07-24 05:21:52.761  - warn: javascript.0 (3320)     at processTimers (node:internal/timers:512:7)
2023-07-24 05:21:52.770  - info: javascript.0 (3320) State value to set for "rpi2.0.gpio.22.state" has to be type "boolean" but received type "number" 
2023-07-24 05:21:53.074  - warn: modbus.0 (138498) Error: undefined
2023-07-24 05:21:53.076  - error: modbus.0 (138498) Request timed out.
2023-07-24 05:21:53.077  - error: modbus.0 (138498) Cannot write single register [1]: {"err":"timeout","timeout":5000}
2023-07-24 05:21:53.083  - info: javascript.0 (3320) script.js.Logik.SSH-Exec: VE Err: 1
2023-07-24 05:21:53.092  - info: javascript.0 (3320) script.js.Logik.SSH-Exec: VE Err: 1
2023-07-24 05:21:53.273  - warn: modbus.0 (138498) Poll error count: 2 code: "App Timeout"
2023-07-24 05:21:55.520  - info: javascript.0 (3320) script.js.Solar.Ges-Regler-_Lade-SWAnalog: Einspeise: 0.08
2023-07-24 05:21:55.521  - info: javascript.0 (3320) script.js.Solar.Ges-Regler-_Lade-SWAnalog:  Eigenverbr.Wert hoch

Allerdings schaltet der GPIO.22 nicht mehr ungewollt.
Was hat sich geändert? Das Script mit dem GPIO.22 habe ich kopiert (neuer Name) und angepasst (if Konstrukte durch else if ersetzt). Dem alten Script einen neuen Namen verpasst und nicht gestartet.

[ticaki]

@laser sagte in Raspi GPIO wird gesetzt, wie Ursache herausfinden?:

Allerdings schaltet der GPIO.22 nicht mehr ungewollt.
Was hat sich geändert?

Na alle Skripte. Ich denke der Fehler kam durch das erste Skript das du hier gepostet hast.

[Homoran]

@ticaki sagte in Raspi GPIO wird gesetzt, wie Ursache herausfinden?:

@laser sagte in Raspi GPIO wird gesetzt, wie Ursache herausfinden?:

Allerdings schaltet der GPIO.22 nicht mehr ungewollt.
Was hat sich geändert?

Na alle Skripte. Ich denke der Fehler kam durch das erste Skript das du hier gepostet hast.

(oder) durch die nichtverwendung von elseif,

[Laser]

@homoran evtl. war es wirklich das else if. Nur wie soll das bei einem gestoppten Script wirken?? Und den GPIO schalten?

[ticaki]

@laser sagte in Raspi GPIO wird gesetzt, wie Ursache herausfinden?:

@homoran evtl. war es wirklich das else if. Nur wie soll das bei einem gestoppten Script wirken?? Und den GPIO schalten?

2023-07-24 05:21:52.759  - warn: javascript.0 (3320) You are assigning a number to the state "rpi2.0.gpio.22.state" which expects a boolean. Please fix your code to

schedule({ hour: [19], minute: [10] }, function () {
  setState("rpi2.0.gpio.22.state" /**/, 1); //  1 ist einschalten
});

schedule({ hour: [09, 10], minute: [27] }, function () {
  setState("rpi2.0.gpio.22.state" /**/, 0); //  0 ist ausschalten
});

Wenn das alles im ersten Post gestanden hätte, wäre mir der Zusammenhang wohl direkt aufgefallen

Wenn du nochmal sowas hast machs wie ich:

schedule({ hour: [19], minute: [10] }, function () {
  log(1)
  setState("rpi2.0.gpio.22.state" /**/, 1); //  1 ist einschalten
});

schedule({ hour: [09, 10], minute: [27] }, function () {
  log(2)
  setState("rpi2.0.gpio.22.state" /**/, 0); //  0 ist ausschalten
});

und schau am nächsten Tag ins Log wer der Schuldige ist.

Auch wenn ich keine Ahnung habe wieso das um die Uhrzeit schalten sollte, aber sind die einzigen falsch Zuweisungen in deinem Script...

habs ausprobiert, wird richtig eingetragen:

script.js.Test.Skript_122: {"type":"cron","pattern":{"hour":[19],"minute":[10]},"scriptName":"script.js.Test.Skript_122","id":"cron_1690278872308_51349"}
script.js.Test.Skript_122: {"type":"cron","pattern":{"hour":[9,10],"minute":[27]},"scriptName":"script.js.Test.Skript_122","id":"cron_1690278872308_65468"}

[Laser]

@ticaki Es ist der zweite Tag, wo es wie gewollt schaltet. Kein GPIO wird mehr fehlgeschaltet. Ich kann es nicht mehr nachvollziehen. Aber für mich sieht es immer noch so aus, als wenn Teile eines Scriptes weiterarbeiten, obwohl sie gelöscht wurden. Durch das umkopieren und umbenennen des Scriptes, und das Neustarten des Raspi tritt der Effekt nicht mehr auf.
In einer früheren Variante des Scriptes habe ich den GPIO.22 um 23:00 Uhr aus und um 5:20 Uhr wieder eingeschaltet.

[paul53]

@laser sagte: In einer früheren Variant des Scriptes habe ich den GPIO.22 um 23:00 Uhr aus und um 5:20 Uhr wieder eingeschaltet.

Dann hat geholfen:
@laser sagte in Raspi GPIO wird gesetzt, wie Ursache herausfinden?:

Neustarten des Raspi

Neustart von ioBroker hätte wahrscheinlich auch genügt.

[Laser]

@paul53 Hatte eigenartigerweise nicht genügt. Auch kompletter Neustart des Raspi nicht. Erst mit der Umbenennerei ging es.