Skip to content

Commit 2808e5a

Browse files
m7md7sienCopilot
andauthored
Potential fix for pull request finding
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent 6c81f6a commit 2808e5a

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

sdk/evaluation/azure-ai-evaluation/azure/ai/evaluation/_evaluators/_common/_validators/_evaluation_level_utils.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,12 @@ def _merge_query_response_messages(query: List[dict], response: List[dict]) -> L
5353

5454
def _split_messages_at_latest_user(messages: List[dict]) -> Tuple[List[dict], List[dict]]:
5555
"""Split messages into query/response slices at the latest user turn."""
56-
latest_user_index = max(i for i, message in enumerate(messages) if message["role"] == MessageRole.USER)
56+
latest_user_index = max(
57+
(i for i, message in enumerate(messages) if message.get("role") == MessageRole.USER.value),
58+
default=-1,
59+
)
60+
if latest_user_index == -1:
61+
raise ValueError("messages must contain at least one message with role 'user'.")
5762
return messages[: latest_user_index + 1], messages[latest_user_index + 1 :]
5863

5964

0 commit comments

Comments
 (0)