[callidomus] is not None

[hoggle]

Hallo zusammen,

ich check's gerad' ned:

Ich habe folgende Stelle im Code:

292: if high_vent_since is not None and high_vent_since != '':
293:  logger.info('high vent status since: ' + str(high_vent_since) + '(floor: ' + str(floor['floor']) + ')')
294:  delta = now - datetime.datetime.strptime(high_vent_since, "%d.%m.%Y %H:%M:%S")
295:  delta_in_minutes = delta.seconds / 60

und folgenden Fehler:

2017-03-06 19:21:42 INFO     ventilation_automation high vent status since: None(floor: basement)
2017-03-06 19:21:42 ERROR    ventilation_automation time data 'None' does not match format '%d.%m.%Y %H:%M:%S'
Traceback (most recent call last):
  File "/mfs/core/lib/config.py", line 263, in __call__
    exec(self.bytecode, self.locals)
  File "/data/callidomus/local/logics/ventilation_automation.py", line 294, in <module>
    delta = now - datetime.datetime.strptime(high_vent_since, "%d.%m.%Y %H:%M:%S")
  File "/usr/local/lib/python3.6/_strptime.py", line 565, in _strptime_datetime
    tt, fraction = _strptime(data_string, format)
  File "/usr/local/lib/python3.6/_strptime.py", line 362, in _strptime
    (data_string, format))
ValueError: time data 'None' does not match format '%d.%m.%Y %H:%M:%S'

Da high_vent_since None ist dürfte diese Stelle im Code eigentlich nicht erreicht werden.

Hat jemand ne Idee ?

[Stereofeld]

Hi Christian,

kann es sein, dass die Einrückung in den Zeilen 293 und 294 unterschiedlich ist (einmal Tabs und einmal Leerzeichen)? Dann würde er die Zeile 293 nicht mehr zur if-Abfrage zählen und die Zeile 293 in jedem Fall durchlaufen. Nur so ein Gedanke.

Und high_vent_since ist auch wirklich None und nicht vielleicht "None"?

Gruß
Thomas

[hoggle]

Hi Thomas,

nee die Einrückung passt.
Und es ist wirklich kein String..

> ls basement.ventilation.high_vent_mode_since
Nodes:
======
basement.ventilation.high_vent_mode_since = None

[Stereofeld]

... und wenn du den zweiten der Teil der Abfrage

and high_vent_since != ''

mal weglässt?

Thomas

[mumpf]

Hi Christian,

ich würde um die beiden Teilvergleiche noch Klammern setzen:

 
 if (high_vent_since is not None) and (high_vent_since != ''):

Liegt aber daran, dass ich nicht genau weiß, welcher Operator in Python stärker bindet. Spontan hätte ich auch gedacht, dass das funktionieren sollte...

Gruß, Waldemar

[callidomus]

Hallo Christian,

aus dem Codefragment ist schwierig nachzuvollziehen was da genau passiert. Insbesondere wie high_vent_since gesetzt wird.
An anderer Stelle bringst Du den "Beweis" vor, das das Item über das CLI None anzeigt.

Ich bin mir ziemlich sicher, das Item basement.ventilation.high_vent_mode_since ist von dem Typ 'Zeichenkette'.
Das kann man nicht mit dem nativen Python None belegen. Das wird in den String 'None' umgewandelt.
Die Vermutung von Thomas ist daher richtig.

Ich werde das CLI Plugin wahrscheinlich abändern, damit dieses in Zukunft repr verwendet um die Werte anzuzeigen.
(Dann sind datetime Objekte aber nicht mehr so einfach zu lesen.)

Du kannst in Deinem Code in Zeile 291 folgendes einfügen:

logger.warning('WERT: {}'.format(repr([FONT=courier new]high_vent_since[/FONT])))

Dann wird der wahre Wert angezeigt.

Bis bald

Marcus

[hoggle]

Ich hab' den Logger Mal eingebaut. Da die Stelle im Code jetzt gerad nicht durchlaufen wird schaue ich es mir morgen nochmal an.

Danke erstmal für die Rückmeldungen !