Misuse of quit() in api.py; Use of SystemExit(1) + narrow exception handling
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 50/100
Piste de recherche
Commencez dans face_recognition/api.py, autour des lignes 8 à 13, où face_recognition_models est importé et où le gestionnaire d’exceptions général affiche des instructions d’installation avant d’appeler quit(). Restreignez la gestion des exceptions et remplacez le comportement de sortie comme demandé. Le travail est terminé lorsque les dépendances manquantes affichent toujours le message d’installation indiqué et que le module se termine avec le statut 1 sans masquer les erreurs d’importation non liées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Python
- Étoiles
- 56.8k
- Forks
- 13.7k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ageitgey/face_recognition
-
CLI image-folder regex missing end-anchor picks up non-image files (foo.jpg.bak, notes.jpegdoc)Peut-être pris @Vic-41148 l’a pris il y a 63 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
ageitgey/face_recognition#1678 ·
-
[Security] Automated static analysis findings for ageitgey/face_recognition (academic research)Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
ageitgey/face_recognition#1681 · 1 commentaire ·
-
Honest-FeedbackOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
ageitgey/face_recognition#1669 · 3 commentaires ·
-
AiSoulOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
ageitgey/face_recognition#1668 · 1 commentaire ·
-
Logical Error: Image display (.show()) placed inside loop, causing multiple windows for multi-face imagesPeut-être pris @pavansai018 l’a pris il y a 317 jours. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 55/100
ageitgey/face_recognition#1666 ·
Toutes les issues de ageitgey/face_recognition
Issues similaires
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
yunaremaia/proto-drift#22 ·
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
yunaremaia/memwatch#96 ·
-
bug good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 91/100
yunaremaia/ci-gate-watch#34 ·
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
yunaremaia/agent-capability-attestation#64 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 86/100
yunaremaia/agent-call-graph#43 ·
Les mainteneurs répondent en général sous 1 jour