UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[heinzie]

@Peoples

Hallo @Peoples ,

vielen Dank! Geht wieder wie verückt :-)
Das hätte ich ohne Deine Hilfe definitiv nicht hinbekommen.

Gruß
heinzie

[AST64]

Danke ... das Script speichert nun wieder Bilder :-) Ich habe aber ein seltsames Verhalten.
Bei mir wird mit einem Trigger das Script ausgelöst ... aber das Bild mit xxx_0.jpg ist erst sichtbar, wenn ich nochmal triggere.
Der Namen stimmt allerdings im Ordner. Es ist so, als wäre das aktuellste Bild im "NICHTS" ;-) also im Cache oder so ...
Ich merke das daran, weil meine CAM einen Zeitstempel erzeugt und ich immer nur das 2 aktuellste Bild gezeigt bekomme.
Das Script habe ich 1:1 kopiert und nur den Ordner und den Namen geändert. Im Funktionsaufruf dann noch den Trigger rein.
Kann das einer bestätigen/nachvollziehen?

Danke Andreas

Hier mal ein Bild ... wie ihr sehen könnt ist der erste Abschnitt ein Bild von 14.27 Uhr obwohl ich 14.30 Uhr ausgelöst habe.
Der 2te Abschnitt ist von 14.31 Uhr und zeigt das Bild vom Trigger um 14.30 Uhr, der letzte Abschnitt zeigt wiederum das um 14.32 Uhr getriggerte Bild.
Es ist das Bild vom Trigger um 14.31 Uhr!!!

[Peoples]

@AST64

Den habe ich auch gehabt

kommentiere mal das

//    deleteImage(fname);

aus Zeile 77 ist das bei mir hab das im Post oben schon geändert. Danach gings bei mir

[LoxDUS]

@Peoples

Du hast mir ja in einem anderen post den tipp gegeben das wie hier beschrieben zu machen. Danach funktionierte zumindest das Zugreifen auf die Bilder schon mal.... ABER

Ich habe auch das Problem wie @AST64 das sich das ganze gespeichere um ein Bild verschiebt. D.h. erst beim wiederholten triggern kommt das eigentliche Bild richtig an.

Das

//deleteImage(fname);

habe ich auch schon auskommentiert.... hilft aber nix. Wenn ich den ganzen Code so umbaue das ich das asynchrone lesen nutze mit

fs.readFile(...);

werden zwar die Bilder direkt richtig ins Filesystem geschrieben aber ich habe keinen Zugriff mit VIS mehr da drauf.
Wenn ich die

 var data = fs.readFileSync('/tmp/image.jpg');

Funktion nutze habe ich zwar mit VIS zugriff auf die Dateien aber die sind dann immer "verspätet" im System.

Hast Du noch eine Idee was man machen kann damit zum richtigen Zeitpunkt die Daten ins filesystem gesdchrieben werden oder wie man ins Filesystem schriebt damit VIS auch darauf zugreifen kann?

Mittlerweile steige ich da, was javascript im Hintergrund macht, aus und weiß nicht mehr weiter....

Viele Grüße,

Artur

[Peoples]

@LoxDUS
habe auch gemerkt dass es nur ein paarmal funktioniert, ich such mal weiter den Fehler wenn ich was habe gebe ich Bescheid

[Peoples]

@LoxDUS

Versuch mal bitte folgendes hab da mal ein setTimeout eingebaut, vielleicht hat die Zeit einfach nicht gereicht um die 50 Dateien umzubenennen.

// *******************************************************************************************************
//
// Bilder Speichern und in DB aufnehmen
// Autor: Peoples
//
// v2.0.0 - 23.11.2019  Neue Version - Auf Grund der Änderungen in Js-controller 2x
// v2.0.1 - 12.12.2019  setTimeout mit 10 Sekunden eingebaut
// *******************************************************************************************************

// -----------------------------------------------------------------------------
// allgemeine Variablen
// -----------------------------------------------------------------------------
var fs = require('fs');
var logging = false;         // Log Meldungen ein/aus schalten

// Kamera Bildquelle muss beim Funktionsaufruf mit übergeben werden
const source_url_backyard = 'http://192.168.2.8:8765/picture/1/current/';       
const source_url_trash = 'http://192.168.2.8:8765/picture/2/current/';
const source_url_main_door = 'http://192.168.2.8:8765/picture/3/current/';

const imageCountMax = 50;                            // Anzahl der Bilder, die vorgehalten werden sollen

const visInstance = 0;
const imageFolder = 'camImages/';                   // Speicherort/Ordner unter Vis - Muss existieren!

// -----------------------------------------------------------------------------
// Objekte
// -----------------------------------------------------------------------------

//zu Testzwecken
//createState('System.Iobroker.Angezeigtes_Bild_Haustuere',  {def: '/_temp/eingangstuer_0.jpg',type: 'string',name: 'Angezeigtes Bild der Kamera Haustuere'});

// #############################################################################
// #                                                                           #
// #  Funktion zum löschen der angegebenen Bilddateien                         #
// #                                                                           #
// #############################################################################
function deleteImage(imageFile) {
    delFile('vis.'+visInstance, imageFolder+imageFile, function (error) {
        log('file' +imageFile +' gelöscht')
    }); 
}

// #############################################################################
// #                                                                           #
// #  Funktion zum umbenennen der angegebenen Bilddateien                      #
// #                                                                           #
// #############################################################################
function renameImage(oldName, newName) {
    try{
        readFile('vis.'+visInstance, imageFolder+oldName+'.jpg', function (error, data) {
                writeFile('vis.'+visInstance, imageFolder+newName+'.jpg', data, function (error) {
                if(logging) log(oldName+'.jpg renamed to '+newName+'jpg');
// Nicht benötigt da überschrieben wird
//                    delFile('vis.0', imageFolder+oldName+'.jpg', function (error) {
//                        if(logging) log(oldName+'.jpg deleted after rename');
//                    });
                });
        }); 
    }
    catch(error) { if (error.code != "ENOENT") log(error); }
}

// #############################################################################
// #                                                                           #
// #  Funktion zum speichern der angegebenen Bilddateien                       #
// #                                                                           #
// #############################################################################
function saveImage(imageNamePre, camUrl) {
    deleteImage(imageNamePre+ (imageCountMax-1))

    for (var i=imageCountMax-2; i >= 0; i-- ) { 
        renameImage(imageNamePre + i,imageNamePre + (i+1)); 
        if(logging) log('renameImage: '+imageNamePre + i+'.jpg , to '+imageNamePre + (i+1)+'jpg')
    }
    setTimeout(function () {
        var fname = imageNamePre + "0.jpg";
    //    deleteImage(fname);
        exec('wget --output-document /tmp/image.jpg \''+camUrl+'\'');
        var data = fs.readFileSync('/tmp/image.jpg');
        writeFile('vis.'+visInstance, imageFolder + fname, data, function (error) {
            if(logging) log('file '+fname+' neu gespeichert');
        });
        fname = '';
    }, 10000);
}

// Funktionsaufruf 
// "eingangstuer" wird hier den nummerierten Bildern vorangestellt
//saveImage("eingangstuer_", source_url_main_door);

Und bitte wieder ne Rückinfo

[LoxDUS]

@Peoples
Hat bei mir leider nicht geholfen.
Ich habe auch mal versucht die ganze Funktion als "Asynchron" zu markieren:

async function saveImage_2() {

und dann das schreiben in das Verzeichnis mit

await writeFile('vis.'+visInstance, imageFolder + Name, data, function (error) {

irgendwie dazu zu bewegen das gewartet wird bis das vom Betriebssystem abgearbeitet ist. Funktioniert aber auch nicht.

.... ich forsche weiter.....

[LoxDUS]

So, habe nochmal ein wenig "rumgeforscht".

Anscheinend geht hier durch das asynchrone Verhalten von Node.js so einiges durcheinander. Die Versuche das durch "process.nextTick" oder "await" und "premises" hinzubekommen sind bei mir fehlgeschlagen. Ich habe aber einen Weg gefunden wie es zumindest bei mir jetzt funktioniert die Bilder Zeitgerecht und so abzulegen das ich mit VIS darauf zugreifen kann (auch wenn sie auf einer gemounteten FAT32 SD-Karte liegen).

Wenn ich das ganze asynchrone IO-Filehandling von Node.js richtig verstanden habe, sollte man in den Callback Funktionen das Filehandling einbauen. Denn nur dort ist sichergestellt das die File-Operation auch wirklich gelaufen ist. Wenn man das sequentiell im Quellcode schreibt kann es sein, dass, wegen der asynchronität von Node.js, sich Funktionen einander überholen.

Meine Lösung sieht nun so aus:

var KlingelBilder = ["","",""];
function saveImage() {
    var time = new Date();

    // Bildnamen erzeugen
    var Name = 'Eing_'+time.getFullYear()+time.getMonth()+time.getDate()+'_'+time.getHours()+time.getMinutes()+time.getSeconds()+'.jpg';
    request.get({url: cam_img, encoding: 'binary'}, function (err, response, body) {
        if(err) {
            log('Beim holen des Bildes ist ein Fehler aufgetreten!');
        } else
        {   fs.writeFile('/home/USB/bellpics/tmp.jpg', body, 'binary', function(err) {   
                if (err) {
                    log('Fehler beim Speichern von Bild /home/USB/bellpics/tmp.jpg   Fehler:' + err, 'warn');
                 } else {
                    var data = fs.readFileSync('/home/USB/bellpics/tmp.jpg');
                    writeFile('vis.0', 'KameraBilder/' + Name, data, function (error) {
                        if(error) {
                            log('Fehler beim Speichern von Bild KameraBilder/' + Name +'   Fehler:' + err, 'warn');
                        } else {
                            // Bilder durchrotieren. An position 0 steht immer das letzte Bild
                            log('Bild wurde als KameraBilder/' + Name +' gespeichert');
                            KlingelBilder[2] = KlingelBilder[1];
                            KlingelBilder[1] = KlingelBilder[0];
                            KlingelBilder[0] = '/vis.0/KameraBilder/' + Name;
                            setState('javascript.0.Sonderfunktionen.KlingelBild1', KlingelBilder[0]);
                            setState('javascript.0.Sonderfunktionen.KlingelBild2', KlingelBilder[1]);
                            setState('javascript.0.Sonderfunktionen.KlingelBild3', KlingelBilder[2]);
                        }
                    });
                 }
            });
        }
    });

}

@Peoples
Ich habe auch die exec Funktion wieder gegen das request.get (Zeile 7) ausgetauscht weil ich ungerne die exec-Funktion im Javascript-Adapter freischalte.

Die Verzeichnisse kann man sich ja nach belieben anpassen....

Viele Grüße,

Artur

[apollon77]

@LoxDUS sagte in gelöst-wie writeFile() in js script nutzen:

Wenn ich das ganze asynchrone IO-Filehandling von Node.js richtig verstanden habe, sollte man in den Callback Funktionen das Filehandling einbauen. Denn nur dort ist sichergestellt das die File-Operation auch wirklich gelaufen ist. Wenn man das sequentiell im Quellcode schreibt kann es sein, dass, wegen der asynchronität von Node.js, sich Funktionen einander überholen.

100% korrekt!

[Peoples]

Ich baue jetzt mal meines um so wie du es gesagt hast und werde morgen berichten ob es dann immernoch läuft :-)

[heinzie]

@AST64
Kannst Du ggf. mal dein gesamtes Script hier zur Verfügung stellen. Habe leider auch das Problem von "AST64 26 Nov 2019, 13:37" das immer wieder alte Bilder eingespielt werden.

[heinzie]

Hallo,
habe leider wieder Probleme mit dem Script und dem speichern der Bilder. Es scheint als wenn er irgendwie Bilder aus dem Cache lädt.
Hat denn von euch jemand eine laufende Ausführung und könnte das Script mal posten?

[Peoples]

@heinzie

// *******************************************************************************************************
//
// Bilder Speichern und in DB aufnehmen
// Autor: Peoples
//
// v2.0.0 - 23.11.2019  Neue Version - Auf Grund der Änderungen in Js-controller 2x
// v2.0.1 - 13.12.2019  Wieder geändert mit callback 
// *******************************************************************************************************

// -----------------------------------------------------------------------------
// allgemeine Variablen
// -----------------------------------------------------------------------------
var fs = require('fs');
var logging = false;         // Log Meldungen ein/aus schalten

// Kamera Bildquelle muss beim Funktionsaufruf mit übergeben werden
const source_url_main_door =    'http://192.168.XX.XX:xxxx/picture/1/current/?_username=admin&_signature=xxxxx';
const source_url_trash =        'http://192.168.XX.XX:xxxx/picture/1/current/?_username=admin&_signature=xxxxx';
const source_url_backyard =     'http://192.168.XX.XX:xxxx/picture/1/current/?_username=admin&_signature=xxxxx';  
const source_url_front =        'http://192.168.XX.XX:xxxx/picture/1/current/?_username=admin&_signature=xxxxx';

const imageCountMax = 50;                            // Anzahl der Bilder, die vorgehalten werden sollen

const visInstance = 0;
const imageFolder = 'camImages/';                   // Speicherort/Ordner unter Vis - Muss existieren!

// -----------------------------------------------------------------------------
// Objekte
// -----------------------------------------------------------------------------


// #############################################################################
// #                                                                           #
// #  Funktion zum löschen der angegebenen Bilddateien                         #
// #                                                                           #
// #############################################################################
function deleteImage(imageFile) {
    delFile('vis.'+visInstance, imageFolder+imageFile, function (error) {
        if(logging)log('file' +imageFile +' gelöscht')
    }); 
}

// #############################################################################
// #                                                                           #
// #  Funktion zum umbenennen der angegebenen Bilddateien                      #
// #                                                                           #
// #############################################################################
function renameImage(oldName, newName) {
    
        try{
            readFile('vis.'+visInstance, imageFolder+oldName+'.jpg', function (error, data) {
                    writeFile('vis.'+visInstance, imageFolder+newName+'.jpg', data, function (error) {
                    if(logging) log(oldName+'.jpg renamed to '+newName+'jpg');
    // Nicht benötigt da überschrieben wird
    //                    delFile('vis.0', imageFolder+oldName+'.jpg', function (error) {
    //                        if(logging) log(oldName+'.jpg deleted after rename');
    //                    });
                    });
            }); 
        }
    catch(error) { if (error.code != "ENOENT") log(error); }
    
}

// #############################################################################
// #                                                                           #
// #  Funktion zum speichern der angegebenen Bilddateien                       #
// #                                                                           #
// #############################################################################
function saveImage(imageNamePre, camUrl) {
     deleteImage(imageNamePre+ (imageCountMax-1))
    for (var i=imageCountMax-2; i >= 0; i-- ) {
        renameImage(imageNamePre + i,imageNamePre + (i+1)); 
     if(logging) log('renameImage: '+imageNamePre + i+'.jpg , to '+imageNamePre + (i+1)+'jpg')
    }   
    setTimeout(function(){ 
        request.get({url: camUrl, encoding: 'binary'}, function (err, response, body) {
            if(err) {
                log('Beim holen des Bildes ist ein Fehler aufgetreten!');
            } else
            {   fs.writeFile('/tmp/image.jpg', body, 'binary', function(err) {   
                    if (err) {
                        log('Fehler beim Speichern von Bild /home/USB/bellpics/tmp.jpg   Fehler:' + err, 'warn');
                    } else {
                        var data = fs.readFileSync('/tmp/image.jpg');
                        var fname = imageNamePre + "0.jpg";
                        writeFile('vis.'+visInstance, imageFolder + fname, data, function (error) {
                            if(error) {
                                log('Fehler beim Speichern von Bild camImages' + fname +'   Fehler:' + err, 'warn');
                            } else {
                                // Bilder durchrotieren. An position 0 steht immer das letzte Bild
                                log('Bild wurde als camImages/' + fname +' gespeichert');
                            }
                        });
                    }
                });
            }
        });
    }, 5000);
}

// Funktionsaufruf 
// "eingangstuer" wird hier den nummerierten Bildern vorangestellt
//saveImage("eingangstuer_", source_url_main_door);

So sieht meines aus und das läuft bisher problemlos

[heinzie]

@Peoples Vielen Dank, die Version läuft bei mir auch. Eine Frage noch zu der Zeit bei der Timeoutfunktion. Bei dir steht dort 5000, ist das so technisch notwendig oder ist die Zeit bei dir so gewollt?

[apollon77]

@heinzie sagte in gelöst-wie writeFile() in js script nutzen:

Es scheint als wenn er irgendwie Bilder aus dem Cache lädt.

Was ist im Web Adapter beim Caching (glaube heisste da blöd Puffer) eingestellt? Sonst ist die Frage was dein Browser macht

[heinzie]

@apollon77 sagte in gelöst-wie writeFile() in js script nutzen:

Web Adapter

meinst du die Stelle, da ist bei mir nichts aktiviert.

[liv-in-sky]

@heinzie
@apollon77

könnte es das hier sein - das hat damals für dateien funktioniert

https://forum.iobroker.net/post/296618

[heinzie]

@liv-in-sky Habe das nochmal geprüft, leider kein Erfolg. Nur die Version von Peoples läuft bei mir.

[Peoples]

@heinzie

die 5000 steht ja für 5 Sekunden, ich glaube ich hatte damals Probleme da ja bei mir auch die Historie mit 50 Bildern immer aktuallisiert werden muss.

[heinzie]

@Peoples Danke für die Info