AOSSIE-Org / AOSSIE-Org/PictoPy

fix: improve SQLite error handling and logging in face_clusters.py

Abierto
#1,230 3 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Python
Estrellas
283
Forks
679
Merge medio
7 d 2 h
PR fusionados (30 d)
3

Descripción

`face_clusters.py` has multiple problems:

- `db_delete_all_clusters` uses broad `except Exception` and `print()` instead of logging
- `db_insert_clusters_batch` catches no exceptions at all
- 5 other functions have no exception handling

### Proposed fix
- Replace `except Exception` with `sqlite3.Error` across all functions
- Replace `print()` with `logger.error()`
- Add `sqlite3.Error` handling to unprotected functions
- Add `import logging` and `logger = logging.getLogger(__name__)`

I'd like to fix this.

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.