Modulentwicklung für Rhasspy Sprachassistent

[JensS]

Auf der Suche nach der Außentemperatur bin ich über leere @matchesInRoom gestolpert und habe RHASSPY_getDevicesByIntentAndType etwas angepasst.

Code Auswählen Erweitern

# Sammelt Geräte über Raum, Intent und optional Type
sub RHASSPY_getDevicesByIntentAndType {
    my $hash   = shift // return;
    my $room   = shift;
    my $intent = shift;
    my $type   = shift; #Beta-User: any necessary parameters...?

my @matchesInRoom; my @matchesOutsideRoom;
    my $prefix = $hash->{prefix};
   
    return if !defined $hash->{helper}{devicemap};
    for my $devs (keys %{$hash->{helper}{devicemap}{devices}}) {
        my $mapping = RHASSPY_getMapping($hash, $devs, $intent, $type, 1, 1) // next;
        my $mappingType = $mapping->{type};
        my @rooms = $hash->{helper}{devicemap}{devices}{$devs}{rooms};

        # Geräte sammeln
        if ( !defined $type ) {
            #any { m{\A$room\z}ix } @rooms
            any { $_ eq $room } @rooms
                ? push @matchesInRoom, $devs
                : push @matchesOutsideRoom, $devs;
        }
if ( defined $type && $mappingType && $type =~ m{\A$mappingType\z}ix ) {
my @Hilfsarray = @{ $rooms[0] };
            for (my $i=0;$i < @Hilfsarray;$i++){
if ( $room eq  $rooms[0][$i]){
            push @matchesInRoom, $devs}else{
            push @matchesOutsideRoom, $devs};
    }
        }
    }
    return (\@matchesInRoom, \@matchesOutsideRoom);
}

$rooms[0][$i] passt bei mir. Habe aber jeweils nur einen Raum pro Gerät. Hab's nochmal überarbeitet...
In sentence.ini musste zusätzlich, wie in den vorigen Beiträgen beschrieben, Type:Temperatur in Type:temperature geändert werden.

Gruß Jens

p.s.

Code Auswählen Erweitern

any { $_ eq $room } @roomsWas macht "any"? G??gle findet bei "any" many...

[Beta-User]

Danke vorab mal für den Schubs betr. "inRoom".

"any" gehört zu List::Util, und sollte dasselbe machen wie zuvor das "grep", siehe https://perldoc.perl.org/List::Util.pm#any. Leider hat aber auch schon das grep nicht das geliefert, was es sollte .

Hab's daher jetzt umgebaut und dabei auch gleich die Rückmeldung im Fall des "matchoutside" etwas angepaßt, so dass jetzt wenigstens der richtige Raum angesagt werden sollte, falls sich das device nicht im Raum befindet.

[Cordula]

@Beta-User
Wollte dir noch Rückmeldung geben. Ich habe jetzt die 0.4.7a im Einsatz. Damit funktioniert der rhasspyIntent ohne Parameter und der rhasspyIntent mit undef-Parametern. Das Problem mit dem Intent SetOnOff ist auch gelöst, allerdings erst nachdem ich die Räume und die Device händisch auf Kleinschreibung umgestellt habe und die Slots händisch angepasst habe. Dabei ist mir aufgefallen, dass der Update der Slots nicht mehr funktioniert, da hier jetzt nicht mehr das Attribut rhasspyMaster verwendet wird, sondern http://localhost:12101.

[drhirn]

Das ist leider noch nicht dokumentiert. Wir haben die URL zur Base in die DEF übernommen, damit man das Attribut nicht vergessen kann. Du müsstest also die DEF um

Code Auswählen Erweitern

IP_Port=<url zur base:port> ergänzen.

Und danach das Modul neu laden

Code Auswählen Erweitern

reload 10_RHASSPY.pm

[JensS]

Danke für any!

Wieso schiebst Du die Räume überhaupt als Array in rooms? Als einzelne Listenelemente des Hashes würden sie eine Ebene höher liegen und man könnte auch direkt darauf zugreifen.

Gruß Jens

[Beta-User]

Hier nochmal ein update, ich habe das jetzt nach WebIF umbenannt und die cref entsprechend erweitert.
Weiter statt "localhost" die nummerische Entsprechung eingearbeitet (Namensauflösungen können blockieren, daher sollte das hier m.E. beispielhaft auch gleich nummerisch sein...).

Ob rooms (und ggf. auch andere Infos wie rhasspyNames etc.) als Array Sinn machen, muss ich mal schauen, aber das wäre eh' nur ein internes Ding. Klingt mir aber immer mehr so...

[Cordula]

@drhirn
Alles klar, habs angepasst. Funktioniert. Danke.

Hab jetzt erst gesehen, dass du eine neue Version gemacht hast. Bei der neuen Version wird WebIf übernommen. Aber für den Rhasspy-Update scheinbar immer noch IP_port genommen, der auf localhost steht. Also kein Update.

[Beta-User]

Das Internal IP_Port verschwindet erst nach einem Neustart, und für das neue Internal braucht es auch entweder einen Neustart oder das Anfassen der DEF (würde letzteres empfehlen, um gleich die korrekte Info unter dem aktuellen Namen einzugeben).

[Cordula]

Passt.

[Beta-User]

Ob rooms (und ggf. auch andere Infos wie rhasspyNames etc.) als Array Sinn machen, muss ich mal schauen, aber das wäre eh' nur ein internes Ding. Klingt mir aber immer mehr so...

"Eigentlich" ist es im list als Komma-separierte Liste schöner, da diese Angaben dann auch gleich vorne zusammen erscheinen...

Allerdings ist der Umbau "gefahrgeneigt", weil unpassende Datentypen FHEM in den Abgrund reißen. Ich hoffe, alle relevanten Stellen erwischt zu haben, aber ausdrücklich: Experimentelle Version! (also noch experimenteller als üblich... )

Nachtrag: Da sich der erwartete devicemap-Hash ändert, muss dieser erneuert werden! (Neustart oder reload devicemap)

[JensS]

Die experimentelle Version teste ich heute Abend. Hab ich etwas spät gelesen, da ich beim experimentieren war.   

Code Auswählen Erweitern

$hash->{helper}{devicemap}{devices}{Sensor3}{raueme}{aussen} = 'aussen';
$hash->{helper}{devicemap}{devices}{Sensor3}{raueme}{Garten} = 'Garten';
if ( exists $hash->{helper}{devicemap}{devices}{$devs}{raueme}{$room}){push @matchesInRoom, $devs}else{push @matchesOutsideRoom, $devs;}
        }
Gruß Jens

[Beta-User]

Für's Analysieren noch: Es gibt jetzt @verbose 5 ein paar neue Log-Einträge, damit man besser nachvollziehen kann, was jeweils als @matches rausgekommen ist...

[Beta-User]

Dann gleich noch ein update der experimentellen Version...

- der in der cref enthaltene Filter für die language und fhemId-Geschichte ist jetzt nicht nur Teil der cref, sondern hoffentlich auch im Code aktiv...
- überflüssige Attribute (defaultRoom und rhasspyMaster) sind aus der Attributliste raus; kann also sein, dass da entsprechende (einmalige) Logeinträge beim nächsten Start kommen...

[JensS]

Prima, GetNumeric läuft mit {Type:temperature} für Room und Device. 
Info: {Type:Temperatur} funktioniert ausschließlich mit Device.

[Beta-User]

Prima, keine Klagen bisher ...

Prima, GetNumeric läuft mit {Type:temperature} für Room und Device. 

Es scheint auch mit nur Raum zu klappen, allerdings eben nach wie vor mit der Einschränkung, dass potentiell jedes Gerät vom Type:temperature ausgelesen wird... Da das mit der Solltemperatur ein Spezialthema für "thermostat" ist, neige ich im Moment dazu, ggf. eine Sonderbehandlung für diesen Type vorzusehen; mal sehen...

Info: {Type:Temperatur} funktioniert ausschließlich mit Device.

Das ist interessant, und evtl. auch hilfreich, um Sonderfälle abzufangen. Vom Bauchgefühl her würde ich aber behaupten, dass du damit auch die Helligkeit von einem Dimmer  und/oder den Öffnungsgrad eines Rollladens abfragen kannst, weil bei "bekanntem" Device eben abgerufen wird, was via GetNummeric verfügbar ist, wenn es nichts spezifisches gibt (und "Temperatur" wird ggf. - im Unterschied zu den meisten anderen Keywords - auch weiter übersetzt).


Nachtrag:
Schaut auch bitte mal in eure Logs. Hin und wieder habe ich auch Warnings über "uninitialized value" bei diversen split-Aktionen etc.. Einen Teil davon sollte ich über die letzten Änderungen erwischt haben, aber über manches stolpert man erst, wenn man bestimmte Dinge einstellt oder Optionen nutzt. Die Fixe dazu sind meistens nichts großes, aber die Warnings eben unschön...