Skip to content

refactor: extract LLM call logging into shared evaluator_utils helper#1520

Open
smflorentino wants to merge 1 commit intomainfrom
sflorentino/extract-llm-logging-helper
Open

refactor: extract LLM call logging into shared evaluator_utils helper#1520
smflorentino wants to merge 1 commit intomainfrom
sflorentino/extract-llm-logging-helper

Conversation

@smflorentino
Copy link
Collaborator

@smflorentino smflorentino commented Mar 27, 2026

Summary

  • Extract logging from llm_as_judge_evaluator into evaluator_utils.py for reuse across LLM evaluators
  • Apply to legacy_llm_as_judge_evaluator, and legacy_trajectory_evaluator

Test plan

  • Added unit tests for _call_llm_with_logging (success, error wrapping, HTTP error logging, JSON fallback)
  • All 1685 existing tests pass

This is critical to diagnose LLM errors in LLM-based evaluators.

🤖 Generated with Claude Code

@github-actions github-actions bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository labels Mar 27, 2026
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@smflorentino smflorentino force-pushed the sflorentino/extract-llm-logging-helper branch from 291be21 to c3379ec Compare March 27, 2026 05:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant