Benzinpreis-Plugin

[psilo]

Msinn kann ich reproduzieren. ich checke das

Offenbar nimmt er LAT und LON nicht.. ich übergebe korrekt, bekomme dann aber
2022-12-05 12:13:19 ERROR plugins.tankerkoenig {'status': 'error', 'ok': False, 'message': 'lng nicht angegeben, oder ausserhalb der gültigen Grenzen\nlat nicht angegeben, oder ausserhalb der gültigen Grenzen'}

Nach dieser Zuweisung ist lat = None:

self.logger.error(lat)
self.logger.error(lon)
lat = self._lat if not lat else None
lon = self._lon if not lon else None
self.logger.error(lat)​

Sisamiwe ich habe mal wie folgt umgebaut. Deckt das das ab, was Du an der Stelle wolltest?

if lat is None:
    lat = self._lat
if lon is None:
    lon = self._lon​

Damit gehts bei mir wieder...

[Msinn]

Die Fehler sind raus. Allerdings bekomme ich noch regelmäßig eine Warnung:

2022-12-05  10:12:22 WARNING  plugins.tankerkoenig Plugin 'tankerkoenig': Tankerkönig didn't return any station

Ich habe den Radius auf 25 km hoch gesetzt, aber die Warnung bleibt.
Ich habe in die Logik noch ein Logging eingebaut und die Meldung kommt aus dem Aufruf, mit dem ich die billigste Tankstelle bestimmen möchte:

cheapest = sh.tankerkoenig.get_petrol_stations(sh._lat, sh._lon, 'e10', 'price', rad=25)

und len(cheapest) ist 0

[psilo]

Habs umgebaut und gepusht. Gerne nochmal testen.

[psilo]

Ich schreibs morgen früh um. Der PR zieht sich..

[Msinn]

Ich gehe/ging davon aus, dass das Plugin unverändert funktionieren sollte. Da ist das umbenennen einer Funktion eher ungünstig.

Bei dem stolpern über den String Wert: Public Functions eines Plugins sollten fehlertoleranter sein als normale Methoden eines Plugins und nicht unbedingt sofort eine Exception schmeißen.

Bei einem unpassenden Datentyp sollte entweder ein verständlicher Log Eintrag erfolgen oder (was hier einfacher ist) ein Casting auf den richtigen Typ vorgenommen werden.

[psilo]

Msinn ich habe die funktion in get_petrol_station_full umbenannt (plugin.yaml sollte angepasst sein). Sisamiwe hatte eine 2te ohne Benzinpreise gebaut, die nutzt jetzt die "_full" und eleminiert danach die Benzinpreise. Kann auch "_full" wieder rausnehmen. Wie es gewünscht ist. Wobei mir grade kommt, dass ich einfach nen bool übergeben kann und den Originalnamen wieder verwenden kann.

PS: Der Check rad > 25 ist neu, daher ist das Plugin bisher mit deinem str nie auf die Nase geflogen.

[Msinn]

Ich habe die Zeile wie oben beschrieben eingetragen -> Keine Änderung

Dann bin ich auf die Suche gegangen. Ich habe vor der Exception Zeile den rad Wert ausgeben lassen.
Der stammt aus einer Logik. In der Logik, wird er auch als String angegeben (was früher keine Probleme machte), obwohl er in der Konfig-Doku als Float dokumentiert ist.

Nachdem ich in der Logik den Wert als numerischen Wert übergebe, ist der Fehler weg. Dafür habe ich einen neuen:

2022-12-04  20:04:05 ERROR    logics.tankerkoenig  In der Logik ist ein Fehler aufgetreten:
   Logik 'tankerkoenig', Datei '/usr/local/shng_dev/logics/tankerkoenig.py', Zeile 27
   Hauptroutine der Logik, Exception: 'TankerKoenig' object has no attribute 'get_petrol_station_detail'

Da scheint eine Funktion auf der Strecke geblieben zu sein.

[psilo]

Sisamiwe dem Fehler nach landet im rad>25 ein "str". Die Frage ist warum. Drum mal auf die Antwort von Msinn abwarten.

[Sisamiwe]

Ohne das tiefer zu prüfen aber ich glaube die Syntax macht Mist.

Das glaube ich nicht.

[psilo]

Probier bitte mal folgende Zeile:
def get_petrol_stations(self, lat=None, lon=None, price='diesel', sort='price', rad=4):
anstatt
def get_petrol_stations(self, lat: float = None, lon: float = None, price: str = 'diesel', sort: str = 'price', rad: float = 4) -> list:

Ohne das tiefer zu prüfen aber ich glaube die Syntax macht Mist.

Noch ne Frage: es kracht ja wenn "rad>25" geprüft wird. Was übergibst Du als rad? Vielleicht macht das Probleme?
https://github.com/smarthomeNG/plugi...init__.py#L191

[Msinn]

Python 3.8.3

[psilo]

Msinn python version? habs eigentlich mit 3.7 und 3.9 jetzt getestet gehabt.

[Msinn]

Ich habe eben gepullt und neu gestartet. Nun bekomme ich beim Start folgenden Fehler:

2022-12-04  16:15:13 ERROR    logics.tankerkoenig  In der Logik ist ein Fehler aufgetreten:
   Logik 'tankerkoenig', Datei '/usr/local/shng_dev/plugins/tankerkoenig/__init__.py', Zeile 191
   function get_petrol_stations(), Exception: '>' not supported between instances of 'str' and 'int'

[Sisamiwe]

wenn du auch mal gegenchecken könntest.

Kann ich gern machen. Gib mir einfach Bescheid.

[psilo]

Sisamiwe ich versuche gerade methoden zusammenzulegen und passe dafür auch meine logiken an. wäre klasse, wenn du auch mal gegenchecken könntest.