dsmrreader / dsmr-reader

DSMR-telegram reader & data visualizer for hobbyists. Free for non-commercial use.
https://dsmr-reader.readthedocs.io
Other
459 stars 95 forks source link

SyntaxWarning: invalid escape sequence '\d' #1965

Closed jicho closed 4 months ago

jicho commented 4 months ago

Communicatietaal / Language

🇳🇱 Nederlandstalig

Help yourself

Inquiry

Report a bug

Description

Zojuist heb ik de laatste versie van DSMR-reader geïnstalleerd via de update van xirixiz/dsmr-reader-docker.

Alles draait soepel, maar bij het opstarten kreeg ik deze melding in de docker CLI logs:

dsmr-reader  | /app/dsmr_parser/parsers.py:41: SyntaxWarning: invalid escape sequence '\d'
dsmr-reader  |   """

Dit is niet iets wat me afkomstig lijkt te zijn door de container, maar meer vanuit de Python code.

Wou het even melden, heb geen idee of de melding een probleem gaat veroorzaken in de juiste werking. Alles lijkt in ieder geval zijn ding te doen.

Of ligt het aan de Python versie in de container?

root@dsmr_reader_docker:/app# python -V
Python 3.12.2

DSMR-reader version

v5.11

DSMR-reader platform

Docker (e.g. Xirixiz's DSMR-reader Docker)

Optional: Debug info dump (of DSMR-reader)

No response

Optional: Smart meter telegram

No response

jicho commented 4 months ago

Dank voor de snelle reactie. 'k Zie hier idd wel iets https://docs.python.org/3/whatsnew/3.12.html rondom de SyntaxWarning

Eigenlijk wel apart dat er "gepiept" wordt over een regex in de documentatie van een functie in parsers.py. Zou je eigenlijk niet verwachten... Toch?

dennissiemensma commented 4 months ago

Ja grote kans dat het Python 3.12 is, want Python-versies zijn niet perse compatible met vorige versies.

Het staat wel op de planning om te upgraden overigens, maar dat is een grote klus omdat ik ook het onderliggende Django-framework moet upgraden. En daardoor moet ik ook weer oude database-versies droppen.

Dus grote kans dat het tzt een grote bump gaat worden, waarbij 3 Python-versies gedropt worden en een hoop oude databases.

En omdat het upgraden zoveel gedoe is voor veel gebruikers, wordt dat vermoedelijk ook het moment dat ik niet meer "native" DSMR-reader ondersteun, maar alleen nog via Docker.