Skip to content

Correcció de SyntaxWarnings, millora de scripts i millora en la recàrrega de dades de la demo.#358

Merged
amorilla merged 10 commits into
ctrl-alt-d:masterfrom
rafatecno1:neteja-warnings
Mar 13, 2026
Merged

Correcció de SyntaxWarnings, millora de scripts i millora en la recàrrega de dades de la demo.#358
amorilla merged 10 commits into
ctrl-alt-d:masterfrom
rafatecno1:neteja-warnings

Conversation

@rafatecno1

Copy link
Copy Markdown
Contributor

Aquesta PR és la continuació de la millora de l'entorn que vaig iniciar amb la demo de Docker però afecta a algun aspecte més. L'objectiu d'aquests canvis és fer que la instal·lació i la càrrega de dades siguin molt més estables, especialment en sistemes moderns i entorns amb recursos limitats.

🛠 Canvis introduïts:

  • 🐍 Python i Càrrega de dades:
    • Correcció de SyntaxWarning:
      S'han corregit les 'invalid escape sequences' i s'han substituït les comparacions d'identitat (is) per comparacions de valor (==) on calia, assegurant la compatibilitat amb Python 3.12. Ara, quan es fa el migrate o la càrrega de dades de la demo no apareixen missatges d'error que embruten la terminal.
    • Refactor de carrega.py:
      Ara el script permet executar load_demo_data múltiples vegades sense errors d'integritat de dades. Això facilita fer proves i el desplegament inicial.
  • 🐚 Scripts de sistema i Bash:
    • Funció esperar_apt(): S'ha afegit una gestió intel·ligent per detectar si el gestor de paquets apt està bloquejat per processos de fons, evitant que la instal·lació falli en el primer inici del servidor si el sistema operatiu està intentant actualitzar el sistema en segon pla.
    • Gestió d'usuaris: S'ha corregit un error en la inclusió automàtica de l'usuari instal·lador al grup docker per evitar problemes de permisos.
    • Millores en setup_cron.sh: Petites correccions decoratives en el missatge final sobre les tasques programades.

⚙️ Altres:

  • S'ha afegit git i make al script d'instal·lació de l'entorn de docker per evitar que el script instal·lador de la demo demani el password de sudo per instal·lar make si troba que no està instal·lat.
  • Correcció d'errors tipogràfics menors i normalització de noms de variables per a una millor llegibilitat del codi.

✅ Proves realitzades:
He verificat que la instal·lació no es bloqueja i que la càrrega de dades és consistent en ambdós sistemes. Els canvis s'han testejat en entorns nets de Ubuntu 24.04 LTS Server i Debian 13 Server, constatant que només cal tenir 1GB de RAM per ambdós sistemes, tot i que per Ubuntu calen 15GB de disc mentre que a Debian amb només 10GB és suficient.

@ctrl-alt-d ctrl-alt-d left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGFM

@rafatecno1

Copy link
Copy Markdown
Contributor Author

Up! que ja ha passat la 2a avaluació! ;-)

@ctrl-alt-d

Copy link
Copy Markdown
Owner

Per mines ok, @amorilla , fes el merge si a tu et sembla be

@amorilla amorilla self-requested a review March 13, 2026 18:20
@amorilla amorilla merged commit 0954ca2 into ctrl-alt-d:master Mar 13, 2026
1 check passed
@rafatecno1 rafatecno1 deleted the neteja-warnings branch March 15, 2026 07:21
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.

3 participants