Skip to content

Conversation

syphernl
Copy link

Upgraded

  • Python 3.11 =>3.12
  • Alpine 3.21 => 3.22

Changed

  • Switch pip to uv as it is much faster
  • Minor tweaks here and there

Notes

@xirixiz
Copy link
Owner

xirixiz commented Sep 19, 2025

Hi @syphernl, ik zie de PR nu pas. Ik heb een tijd issues gehad met het ontvangen van mail van GitHub. Uiteindelijk is het opgelost met Microsoft gelukkig 😓. Ik heb er zelf ook een keer mee gespeeld met Python 3.13 en ik heb daarover toen ook contact gehad met Dennis (DSMR Reader eigenaar). Dennis is bezig met een nieuwe versie maar door andere zaken op prive vlak heb ik mij daar niet mee bezig gehouden. Ik kijk dit weekend even naar jouw PR, maar er viel mij iig al op dat LD_LIBRARY_PATH bijvoorbeeld dubbel gedefinieerd staat. Verder vallen mij nog wat andere zaken op die op het eerste gezicht vreemd ogen. Heb je mogelijk gebruik gemaakt van AI? Wellicht beledig ik jou nu, en dat is dan ook zeker niet de bedoeling ✌️ , maar ik wil het graag begrijpen waarom je sommige keuzes hebt gemaakt. Een andere is bijvoorbeeld het toevoegen van TAR en extra comments bij de verschillende STEPS.

@syphernl
Copy link
Author

Hi @xirixiz,

Thanks voor je feedback 🙏. Je hebt helemaal gelijk: LD_LIBRARY_PATH stond inderdaad dubbel en er zijn wat extra dingen toegevoegd die misschien niet strikt noodzakelijk zijn (zoals tar).
Ik heb bij het opstellen van de Dockerfile inderdaad gebruikgemaakt van een LLM om wat sneller door bepaalde stappen heen te komen. Vandaar ook de extra comments en het opnemen van packages als tar.

Belangrijk om te vermelden: deze versie draait hier ondertussen al ruim 3 weken stabiel, zonder issues. Heb de Dockerfile zojuist een beetje opgeschoond.

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

Successfully merging this pull request may close these issues.

2 participants