dsmrreader / dsmr-reader

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

Infra: Django 4.2 LTS update & drop support for Python 3.7 / 3.8 / 3.9 / 3.10 / 3.11 #1594

Open dennissiemensma opened 2 years ago

dennissiemensma commented 2 years ago

Django 4.x heeft geen ondersteuning meer voor Python 3.7, terwijl die versie nog meer dan een jaar ondersteund wordt.

Hiervoor is dus weer een versie bump nodig. Wellicht een wat lichtere major bump voor DSMR-reader. Ik kan overwegen om zelfs alleen Python 3.9+ te ondersteunen, hoewel het met Docker niet heel spannend is.

dennissiemensma commented 2 years ago

Ik denk dat Python 3.9 tzt prima is. Deze is standaard op Debian Bullseye zie ik.

dennissiemensma commented 1 year ago

Django 4.2 LTS komt volgende maand.

dennissiemensma commented 1 year ago

image

dennissiemensma commented 1 year ago

Ik denk zelfs Python 3.9 al overwegen om niet meer te ondersteunen

dennissiemensma commented 1 year ago

Django

image

dennissiemensma commented 1 year ago

Django mag 4.2 LTS worden, ruim 2 jaar support

dennissiemensma commented 7 months ago

Ik loop nu wel tegen issues aan met Python 3.7. Echter alleen nog met dev packages, dus ik zal kijken of ik die kan bumpen naar een hogere versie dan DSMR-reader minimaal ondersteunt.

De CI test sws telkens alle versies, dus dat zou goed genoeg moeten zijn voor wat eindgebruikers draaien (non-dev).

dennissiemensma commented 4 months ago

Inmiddels is Python 3.12 de standaard. Icm containers, lijkt het me dan ook de enige om te ondersteunen.

image

dennissiemensma commented 4 months ago

Ik zet development wel eerst naar 3.11 en later hoger. Ivm upgraden Django

dennissiemensma commented 4 months ago

Django: image

dennissiemensma commented 3 months ago
dennissiemensma commented 3 months ago

Ik moet eerst weer https://docs.djangoproject.com/en/5.0/releases/4.1/naar Pytnon 3.10, totdat ik alle Django upgrades gehad heb

Daarna kan ik direct naar 3.12, wat sws al het plan was.

dennissiemensma commented 3 months ago

De switch naar 4.0 en 4.1 leek ok. Naar 4.2 gaat stuk door wijzigingen in het ORM helaas:

 django.core.exceptions.FieldError: Cannot compute Sum('electricity2'): 'electricity2' is an aggregate
dennissiemensma commented 3 months ago

Gevonden. Het kwam neer op dat ik in het ORM geen aggregaties dezelfde naam mag geven als de velden waarop het gebaseerd is.

Ik dacht dat ik dat vaak had gedaan, maar het lijkt ook maar 1 plek, dus bij deze gefixt. Rest van de tests slagen, dus ik moet nog wel een checken wat ik nog mis.

dennissiemensma commented 3 months ago

Ik zal iig bumpen naar Python 3.12, daar komen ook vast nog wat incompatible changes uit.

dennissiemensma commented 3 months ago

Ik zal deze kopie ook bijwerken, waar mogelijk:

dennissiemensma commented 3 months ago

Dat laatste is toch wat teveel werk. Dat zeggende hebben kan ik ook kijken of ik het eindelijk als dependency kan includen. Gezien die lib stable is en ook regelmatig onderhoud krijgt.