Misuse of quit() in api.py; Use of SystemExit(1) + narrow exception handling
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 50/100
Línea de trabajo
Comienza en face_recognition/api.py, alrededor de las líneas 8-13, donde se importa face_recognition_models y el controlador de excepciones general muestra instrucciones de instalación antes de llamar a quit(). Limita el manejo de excepciones y reemplaza el comportamiento de salida según lo solicitado. Estará terminado cuando las dependencias faltantes sigan mostrando el mensaje de instalación indicado y el módulo finalice con el estado 1 sin ocultar errores de importación no relacionados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
In face_recognition/api.py:
- The try/except block on lines 8-13 catches a broad Exception and then calls quit. Code below:
try:
import face_recognition_models
except Exception:
print("Please install face_recognition_models with this command before using face_recognition:\n")
print("pip install git+https://github.com/ageitgey/face_recognition_models")
quit()
Using ImportError instead of except Exception can prevent masking of unrelated errors, like syntax errors, that may happen during an import. Since the block calls an import and specifically checks for missing dependencies, use of ImportError is more focused on the codes specific purpose.
For the quit() --> SystemExit(1) suggestion, quit() isn't guaranteed to exist in all execution envs. Raising SystemExit(1) is more universal and serves the same purpose, preventing any disruption in existing code. SystemExit(1) also has the benefit of not needing an import to function.
- Lenguaje dominante
- Python
- Estrellas
- 56.8k
- Forks
- 13.7k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de ageitgey/face_recognition
-
CLI image-folder regex missing end-anchor picks up non-image files (foo.jpg.bak, notes.jpegdoc)Posiblemente ocupada @Vic-41148 la tomó hace 63 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
ageitgey/face_recognition#1678 ·
-
[Security] Automated static analysis findings for ageitgey/face_recognition (academic research)Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
ageitgey/face_recognition#1681 · 1 comentario ·
-
Honest-FeedbackAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
ageitgey/face_recognition#1669 · 3 comentarios ·
-
AiSoulAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
ageitgey/face_recognition#1668 · 1 comentario ·
-
Logical Error: Image display (.show()) placed inside loop, causing multiple windows for multi-face imagesPosiblemente ocupada @pavansai018 la tomó hace 317 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 55/100
ageitgey/face_recognition#1666 ·
Todos los issues de ageitgey/face_recognition
Issues similares
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
yunaremaia/proto-drift#22 ·
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
yunaremaia/memwatch#96 ·
-
bug good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
yunaremaia/ci-gate-watch#34 ·
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
yunaremaia/agent-capability-attestation#64 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
yunaremaia/agent-call-graph#43 ·
Los mantenedores suelen responder en 1 día