AOSSIE-Org / AOSSIE-Org/PictoPy

fix: replace broad Exception and print() with proper error handling in faces.py

Aberta
#1,222 4 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Python
Estrelas
283
Forks
679
Merge médio
7d 2h
PRs com merge (30d)
3

Descrição

`db_update_face_cluster_ids_batch` in `faces.py` has two problems:

- Uses broad `except Exception` instead of `sqlite3.Error`
- Uses `print("Error updating face cluster IDs in batch.")` instead of proper logging

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

I'd like to fix this if maintainers agree.

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.