From 1a3e86973753e84112c3d0bd7cd852d3cca1a00c Mon Sep 17 00:00:00 2001 From: Vincent STRAGIER Date: Fri, 23 Jun 2023 17:37:19 +0200 Subject: [PATCH] Better type checking. --- deepface/DeepFace.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/deepface/DeepFace.py b/deepface/DeepFace.py index e1d1eac87..d38a81094 100644 --- a/deepface/DeepFace.py +++ b/deepface/DeepFace.py @@ -296,17 +296,17 @@ def analyze( if isinstance(actions, str): actions = (actions,) - actions = list(actions) - - # Check if actions have been passed correctly - if not actions: + # check if actions is not an iterable or empty. + if not hasattr(actions, "__getitem__") or not actions: raise ValueError("`actions` must be a list of strings.") + actions = list(actions) + # For each action, check if it is valid for action in actions: if action not in ("emotion", "age", "gender", "race"): raise ValueError( - f"Invalid action passed ({action})). " + f"Invalid action passed ({repr(action)})). " "Valid actions are `emotion`, `age`, `gender`, `race`." ) # ---------------------------------