Jahresrückblick 2025 – unser neuer Blogbeitrag ist online! ✨

[Negalein]

@asgothian @oliverio
& Co.

mich würde eure ehrliche Meinung zu meinem Spoolman-Skript intressieren.
Es wurde mit ChatGPT erstellt.

Auf Erklärungem "im Script" hab ich bewusst verzichtet. Das hab ich wärend der Erstellung bereits erfragt.

Spoolman

/**
 * ============================================================
 * ioBroker – Klipper – Spoolman Integration
 * ============================================================
 *
 * Version:   1.0.0
 * Author:    <Negalein>
 * License:   MIT
 *
 * Description:
 * - Liest aktive Spulen aus Klipper (Moonraker)
 * - Holt Restfilament aus Spoolman (SQLite via SSH)
 * - Erstellt automatisch ioBroker-States
 * - 🟢🟡🔴 Ampel-Logik je Spule
 * - Telegram-Warnungen mit Zeit- & Drucklogik
 * ==========================================================
 */

const { exec } = require('child_process');

/* ==========================================================
   KONFIGURATION
   ========================================================== */

const CONFIG = {
  spoolman: {
    sshHost: 'root@10.0.1.148',
    dbPath: '/root/.local/share/spoolman/spoolman.db'
  },

  baseState: '0_userdata.0.3DDrucker.Spoolman',

  limits: {
    warn: 300,
    empty: 100
  },

  warnTimes: {
    weekday: { start: 7, end: 22 },
    weekend: { start: 10, end: 20 }
  },

  extruders: [
    { name: 'T0', state: 'klipper-moonraker.0.gcode_macro T0.spool_id' },
    { name: 'T1', state: 'klipper-moonraker.0.gcode_macro T1.spool_id' },
    { name: 'T2', state: 'klipper-moonraker.0.gcode_macro T2.spool_id' },
    { name: 'T3', state: 'klipper-moonraker.0.gcode_macro T3.spool_id' }
  ],

  schedule: '*/5 * * * *'
};

/* ==========================================================
   STATE HELFER
   ========================================================== */

function ensureState(id, val, type) {
  if (!existsState(id)) {
    createState(id, val, {
      type,
      read: true,
      write: false,
      role: 'value'
    });
  } else {
    setState(id, val, true);
  }
}

/* ==========================================================
   ZEITFENSTER
   ========================================================== */

function isInWarnTime() {
  const now = new Date();
  const hour = now.getHours();
  const day  = now.getDay(); // 0=So

  const isWeekend = (day === 0 || day === 6);
  const cfg = isWeekend ? CONFIG.warnTimes.weekend : CONFIG.warnTimes.weekday;

  return hour >= cfg.start && hour < cfg.end;
}

/* ==========================================================
   DRUCKSTATUS
   ========================================================== */

function isPrinting() {
  return getState('klipper-moonraker.0.print_stats.state')?.val === 'printing';
}

/* ==========================================================
   SPOOLMAN DB LESEN
   ========================================================== */

function readSpoolman(callback) {
  const cmd = `
ssh ${CONFIG.spoolman.sshHost} "sqlite3 -json ${CONFIG.spoolman.dbPath} '
SELECT
  spool.id AS spool_id,
  filament.name AS name,
  filament.material AS material,
  ROUND(spool.initial_weight - spool.used_weight, 1) AS remaining
FROM spool
JOIN filament ON filament.id = spool.filament_id
WHERE spool.archived IS NOT 1;
'"
`;

  exec(cmd, (err, stdout) => {
    if (err) {
      log(`Spoolman SSH Fehler: ${err.message}`, 'error');
      callback([]);
      return;
    }

    try {
      callback(JSON.parse(stdout));
    } catch {
      log('Spoolman JSON ungültig', 'error');
      callback([]);
    }
  });
}

/* ==========================================================
   STATUS HELFER
   ========================================================== */

function getStatus(remaining) {
  if (remaining < CONFIG.limits.empty) return 'LEER';
  if (remaining < CONFIG.limits.warn)  return 'WARN';
  return 'OK';
}

/* ==========================================================
   HAUPTLOGIK
   ========================================================== */

function update() {

  // globale VIS-States
  ensureState(`${CONFIG.baseState}.warnzeit_aktiv`, isInWarnTime(), 'boolean');
  ensureState(`${CONFIG.baseState}.druck_laeuft`, isPrinting(), 'boolean');

  readSpoolman(spools => {

    CONFIG.extruders.forEach((ext, index) => {

      const slot = `${CONFIG.baseState}.aktiv.${index + 1}`;
      const spoolId = getState(ext.state)?.val;

      if (!spoolId || spoolId <= 0) {
        ensureState(`${slot}.active`, false, 'boolean');
        return;
      }

      const spool = spools.find(s => s.spool_id == spoolId);
      if (!spool) {
        log(`Spule ${spoolId} nicht in Spoolman gefunden`, 'warn');
        return;
      }

      /* Basisdaten */
      ensureState(`${slot}.active`, true, 'boolean');
      ensureState(`${slot}.extruder`, ext.name, 'string');
      ensureState(`${slot}.spool_id`, spoolId, 'number');
      ensureState(`${slot}.name`, spool.name, 'string');
      ensureState(`${slot}.material`, spool.material, 'string');
      ensureState(`${slot}.remaining_weight`, spool.remaining, 'number');

      /* Status */
      const status = getStatus(spool.remaining);
      ensureState(`${slot}.status`, status, 'string');

      const warnId  = `${slot}.warnung`;
      const alarmId = `${slot}.alarm`;

      const warned  = getState(warnId)?.val || false;
      const alarmed = getState(alarmId)?.val || false;

      /* 🟡 WARN */
      if (
        status === 'WARN' &&
        !warned &&
        isInWarnTime() &&
        isPrinting()
      ) {
        sendTo('telegram', 'send', {
          text:
`🟡 Filament wird knapp

🖨 Extruder: ${ext.name}
🧵 ${spool.name}
🎨 ${spool.material}
📦 Rest: ${spool.remaining} g`
        });
        ensureState(warnId, true, 'boolean');
      }

      /* 🔴 LEER */
      if (
        status === 'LEER' &&
        !alarmed &&
        isPrinting()
      ) {
        sendTo('telegram', 'send', {
          text:
`🔴 Filament leer!

🖨 Extruder: ${ext.name}
🧵 ${spool.name}
🎨 ${spool.material}
📦 Rest: ${spool.remaining} g

⚠️ Filament wechseln!`
        });
        ensureState(alarmId, true, 'boolean');
      }

      /* Reset */
      if (status === 'OK') {
        ensureState(warnId, false, 'boolean');
        ensureState(alarmId, false, 'boolean');
      }
    });
  });
}

/* ==========================================================
   SCHEDULER
   ========================================================== */

schedule(CONFIG.schedule, update);
update();

[liv-in-sky]

@Asgothian sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

der Gesetzgeber einschreitet

ich habe mich eh schon gefragt, wenn openclaw anfängt zu "hacken" oder über whatsapp zu "mobben", wer dan strafrechtlich belangt wird und wann man das erste mal davon hören wird

ich dachte halt, wenn es von programmierern wie euch kommt, die das system am besten kennen, und als richtlinie die user darauf vorbereiten (evtl mit einer beschreibung für eine ki, was sie darf bzw nicht darf), wäre das halt die sicherste variante für neue user und ki-interessierte (ich spreche ja nicht von sofort, sondern einfach mal im hinterkopf zu haben).

wie cool wäre es, beim öffnen eines javascript-functions-blocks gleich zugang einem agenten zu haben und zu beschreiben, was die funktion machen soll - so als spinnerei :-)

natürlich habt ihr recht skeptisch zu sein und ich könnte eure argumente auch nicht einfach abtun, sondern sehe die richtigkeit darin - aber es wird kommen - KI wird nicht verschwinden - und vorallem das code-generieren wird in zukunft besser werden

[Homoran]

@OliverIO ok, das ist korrekt

hier

müsstest du die backticks mit backslash escapen

was bei dem Text natürlich eine Arbeit für Herrn Sisyphos wäre

Erklärung:
zwischen zwei Backticks ist `inline code`
= inline code

[Negalein]

@liv-in-sky sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

wer dan strafrechtlich belangt wird

der User!
wenn du ein Messer gratis bekommst und damit wen tötest, wirst auch du bestraft und nicht der Schenker.

[crunchip]

@OliverIO sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Ich habe das Skript mit dem folgenden, prompt mal bei ChatGPT hineingeworfen

wie schon anderweitig erwähnt verwende ich gerne perplexity, das erklärt einem automatisch ohne zu fragen,
erläutert die Fehler, macht Verbesserungsvorschläge und stellt folge Fragen, was man sich z.b eventuell noch wünschen/Erweitern könnte. Das finde ich ganz praktisch, denn da kommen zwischendurch mal coole Ideen herum, an die man gar nicht gedacht hat.

[Negalein]

@crunchip sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

erklärt einem automatisch ohne zu fragen, erläutert die Fehler, macht Verbesserungsvorschläge und stellt folge Fragen, was man sich z.b eventuell noch wünschen/Erweitern könnte. Das finde ich ganz praktisch, denn da kommen zwischendurch mal coole Ideen herum, an die man gar nicht gedacht hat.

ähnlich ChatGPT.
hab da auch Fehler erklärt bekommen, Tipps, usw.
Werde mir jetzt mal Perplexity anschaun. Ist das für Scripte echt viel besser? Hab noch einige Projekte vor.

[liv-in-sky]

@Negalein mit perplevity habe ich nicht so gute Code Erfahrung

Claude bei Javascript und gemini bei HTML Seiten nutze ich gerne

[Negalein]

@liv-in-sky sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Claude

kenn ich noch nicht. gleich mal suchen. :)

[Jey Cee]

@liv-in-sky sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Claude bei Javascript und gemini bei HTML Seiten nutze ich gerne

Hier wären noch die Modelle wie Claude Sonnet 4.5, Opus 4.5, usw. interessant da sich die Modelle teilweise doch sehr Stark Unterscheiden in der Qualität für die Jeweilige Aufgabe.

[crunchip]

@liv-in-sky sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Claude bei Javascript und gemini

Ist doch mit inbegriffen und kann ausgewählt werden

[Meister Mopper]

@Asgothian sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Nun gut.. gebe ich mal meine Meinung zu dem Skript ab, ohne das der OP kommentiert hat was es machen soll:

Vielen Dank,
ich habe die KI mit deiner Kritik "gefüttert" und sie hat geantwortet:

"Autsch, das hat gesessen! Aber – und das sage ich als dein KI-Kollege ganz offen – der Programmierer hat in fast allen Punkten fachlich absolut recht. Das Skript ist ein klassisches Beispiel für „Hauptsache, es funktioniert irgendwie“, ohne auf die interne Mechanik von ioBroker zu achten.

Besonders der Punkt mit der Race Condition (Wettlauf der Daten) und dem massiven getState-Spam ist kritisch für die Stabilität deines Systems."

Als Ergebnis wurde dieses Skript generiert:

Spoiler

// =============================================================================
// PV-ENERGIEMASTER PRO v2.0
// =============================================================================

// --- 1. KONFIGURATION ---
// Wir definieren die IDs oben, damit wir sie unten im Code nicht suchen müssen.
const ID_BATTERIE_W    = 'modbus.0.inputRegisters.100.842_Battery_Power_(System)';
const ID_PV_DACH_W     = 'solax.0.data.acpower';
const ID_SMARTMETER_W  = 'smartmeter.0.1-0:16_7_0__255.value';
const ID_SOC           = 'modbus.0.inputRegisters.100.843_Battery_State_of_Charge_(System)';
const ID_SPEICHER_KWH  = '0_userdata.0.Energie.PV.Speichergroesse';

const PATH = '0_userdata.0.Energie.PV.';

// --- 2. DER "RAM-SPEICHER" DES SKRIPTS ---
// Kritikpunkt: "Zu viele getStates". 
// Lösung: Wir speichern die aktuellen Werte in Variablen im Arbeitsspeicher (RAM).
// Das ist tausendmal schneller als jedes Mal beim ioBroker-Server anzufragen.
let pvPower = 0, netPower = 0, batPower = 0, soc = 0;

// Konstanten lesen wir NUR EINMAL beim Skriptstart. Die Größe deines Speichers 
// ändert sich ja nicht alle 10 Sekunden.
let speicherMax = getState(ID_SPEICHER_KWH).val || 0; 

// Wichtig für die Energieberechnung: Wann haben wir das letzte Mal gerechnet?
let lastTimestamp = Date.now();

// --- 3. DIE BERECHNUNGSLOGIK ---
function calculateEnergyMaster() {
   
   // --- A. Physikalische Aufteilung ---
   // Netzbezug ist positiv (vom Netz), Einspeisung negativ (ins Netz).
   const netzBezug      = netPower > 0 ? netPower : 0;
   const einspeisung    = netPower < 0 ? Math.abs(netPower) : 0;
   
   // Batterie Laden ist positiv, Entladen ist negativ.
   const batterieLaden  = batPower > 0 ? batPower : 0;
   const batterieEntladen = batPower < 0 ? Math.abs(batPower) : 0;

   // --- B. Hausverbrauch-Formel ---
   // Der Hausverbrauch ergibt sich aus: Was kommt vom Dach + was kommt vom Netz 
   // + was kommt aus der Batterie. Davon ziehen wir ab, was in die Batterie geht 
   // oder ins Netz eingespeist wird.
   let hausverbrauch = pvPower + netzBezug + batterieEntladen - batterieLaden - einspeisung;
   if (hausverbrauch < 0) hausverbrauch = 0; // Physikalische Korrektur (Sicherheit)

   // --- C. Autarkie ---
   // Wie viel Prozent meines Verbrauchs decke ich NICHT durch das Netz?
   let autarkie = hausverbrauch > 0 ? Math.min(100, (1 - (netzBezug / hausverbrauch)) * 100) : 0;

   // --- D. Speicher-Prognose ---
   let fertigUm = "n. n.";
   const ladestandKWh = (speicherMax * soc) / 100;

   if (batterieLaden > 50) { // Nur rechnen, wenn mit mehr als 50W geladen wird
       const leerspeicherKWh = Math.max(0, speicherMax - ladestandKWh);
       
       // Formel: Restkapazität / Ladeleistung (in kW umgerechnet)
       const restladezeitSec = (leerspeicherKWh / (batterieLaden / 1000)) * 3600;
       
       let now = new Date();
       now.setSeconds(now.getSeconds() + restladezeitSec);
       fertigUm = formatDate(now, "hh:mm");
   }

   // --- E. DER CLOU: Energie-Integration (Wh-Messung) ---
   // Kritikpunkt: "Heartbeat alle 10 Sek ist Verschwendung".
   // Lösung: Wir berechnen die Energie exakt zwischen zwei Wertänderungen.
   const now = Date.now();
   const durationHours = (now - lastTimestamp) / (1000 * 60 * 60); // Zeitdifferenz in Stunden
   
   if (durationHours > 0) {
       // Energie (Wh) = Leistung (W) * Zeit (h)
       const energyWh = hausverbrauch * durationHours;
       const currentTagesverbrauch = getState(PATH + 'Tagesverbrauch').val || 0;
       
       // Wir addieren den kleinen Schnipsel Energie zum Tageszähler
       setState(PATH + 'Tagesverbrauch', currentTagesverbrauch + energyWh, true);
   }
   lastTimestamp = now; // Zeitstempel für den nächsten Durchlauf merken

   // --- F. DATENPUNKTE SCHREIBEN ---
   // Wir schreiben nur die Werte, die wir wirklich für die Visualisierung (Grafana/Vis) brauchen.
   setState(PATH + 'Hausverbrauch', hausverbrauch, true);
   setState(PATH + 'Autarkie', Math.round(autarkie), true);
   setState(PATH + 'lade_kwh', parseFloat(ladestandKWh.toFixed(1)), true);
   setState(PATH + 'Restladezeit_final', fertigUm, true);
   
   // Die Rohwerte spiegeln wir nur, wenn sie für die Visu nötig sind
   setState(PATH + 'PV_Dach', pvPower, true);
   setState(PATH + 'Netzbezug', netzBezug, true);
}

// --- 4. DIE TRIGGER (DIE "ANTRIEBER") ---
// Wir hören auf die Hardware. Sobald ein Sensor einen neuen Wert meldet, 
// aktualisieren wir unsere Variable im RAM und rechnen sofort los.

on({id: ID_PV_DACH_W, change: "ne"}, (obj) => { 
   pvPower = obj.state.val || 0; 
   calculateEnergyMaster(); 
});

on({id: ID_SMARTMETER_W, change: "ne"}, (obj) => { 
   netPower = obj.state.val || 0; 
   calculateEnergyMaster(); 
});

on({id: ID_BATTERIE_W, change: "ne"}, (obj) => { 
   batPower = obj.state.val || 0; 
   calculateEnergyMaster(); 
});

on({id: ID_SOC, change: "ne"}, (obj) => { 
   soc = obj.state.val || 0; 
   calculateEnergyMaster(); 
});

// Reset um Mitternacht
schedule("0 0 * * *", () => {
   setState(PATH + 'Tagesverbrauch', 0, true);
});

Wäre schön, wenn du die Zeit findest nochmal drüberzuschauen.

Die Erklärung der KI:

Was wurde verbessert?

• Bye-bye Heartbeat & Race Condition: Der 10-Sekunden-Intervall wurde komplett entfernt. Stattdessen berechnet das Skript den Energieanteil (Wh) direkt bei jedem Trigger basierend auf der echten Zeitdifferenz (durationHours) zwischen den Meldungen. Das ist hochpräzise und verhindert, dass der Intervall alte Werte liest.

• RAM statt Datenbank: Die Hardware-Werte werden in lokalen Variablen (pvPower, etc.) gehalten. getState wird im Rechenprozess fast gar nicht mehr benötigt.

• Konstanten-Management: Die Speichereigenschaft wird nur noch einmalig beim Skriptstart gelesen, nicht mehr 1000-mal am Tag.

• Daten-Hygiene: Der on-Trigger reagiert jetzt auf change: "ne" (neuer Wert), statt auf any. Das reduziert unnötige Rechenlast bei identischen Werten massiv.

• Präzision: Durch die Integration über die Zeit (Delta-t) ist der Tagesverbrauch nun mathematisch korrekt, egal ob die Werte alle 2 Sekunden oder alle 2 Minuten kommen.

• Das Skript ist jetzt „sauber“ und wird deinen ioBroker spürbar entlasten."

[Kalle Blomquist]

Entschuldigt, ein unwissender schreibt, korrigiert mich, wenn ich falsch liege.

Hab´t ihr eigentlich den Beitrag von Asgothian gelesen?
Die Ki hat für Meister Propper eine Script geschrieben und jetzt soll ein anderer Mensch, und das zum zweiten mal das Script kontrollieren. Was macht das für einen Sinn?

[ilovegym]

Das macht auf jeden Fall Sinn, denn das von der KI geschriebene ist nicht immer so wie es da rauskommt, zu gebrauchen.
Oft steht KI auch fuer "Keine Intelligenz" - und man sollte IMMER !! das pruefen, was da erzeugt wird...

[Jey Cee]

@Kalle-Blomquist sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Was macht das für einen Sinn?

Zum Beispiel den das User die wenig oder keine Ahnung von Programmierung haben, verstehen lernen worauf man achten muss wenn man mit KI Code erzeugt.

[Kalle Blomquist]

Wäre es nicht besser einen Kursus zu belegen, um zu verstehen. Anstatt die Kontrolle auf einen Anderen zu verlagern. Und habe alle einen Anderen? Machen alle das?

[Homoran]

@Kalle-Blomquist sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Anstatt die Kontrolle auf einen Anderen zu verlagern

???

Was pssiert denn hier täglich zigmal?
Hier werden Hilfeschreie "mein Skipt läuft nicht" ständig durch Kontrolle und Korrektur beantwortet

[Kalle Blomquist]

Eben! und lernen wäre besser und nicht einfach abgeben

[Homoran]

@Kalle-Blomquist sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Eben! und lernen wäre besser.

Dass dieser Thread dazu dient solche User zu sensibilisieren dass die KI keine perfekten Ergebnisse liefert, und man diese überprüfen können müssze, ist dir schon klar??

[Kalle Blomquist]

ja, deswegen muss man auch dem Xten User zum xten mal sagen, bitte melde dich nicht als root an, du kannst dir dein ganzes System zerschießen.
Meine Argumentation war: Solange ich nicht selber von A-Z verstehe,was mir die Ki da schreibt, lasse ich die Finger weg!
Ich warte nur auf die ersten Rechner, die die Kontrolle abgegeben haben, weil der User OpenClaw ohne ausreichende Kenntnis verwendet hat.
Aber ich bin jetzt raus aus der Diskussion.

[Homoran]

@Kalle-Blomquist sagte in Verbesserung erfahren Blockly/Javascript/KI/AI:

Meine Argumentation war: Solange ich nicht selber von A-Z verstehe,was mir die Ki da schreibt, lasse ich die Finger weg!

das sehe ich genauso, aber leider kam das so klar und deutlich bei mir nicht an!