Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SyntaxWarning: invalid escape sequence '\d' #1965

Closed
4 tasks done
jicho opened this issue Feb 23, 2024 · 2 comments
Closed
4 tasks done

SyntaxWarning: invalid escape sequence '\d' #1965

jicho opened this issue Feb 23, 2024 · 2 comments

Comments

@jicho
Copy link

jicho commented Feb 23, 2024

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
Copy link
Author

jicho commented Feb 23, 2024

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
Copy link
Member

dennissiemensma commented Feb 23, 2024

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.

@dennissiemensma dennissiemensma closed this as not planned Won't fix, can't repro, duplicate, stale Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants