Skip to content

Add Conversation Coherence evaluator#4959

Merged
AliMahmoudzadeh merged 7 commits into
mainfrom
selshafey/cc_multiturn
May 1, 2026
Merged

Add Conversation Coherence evaluator#4959
AliMahmoudzadeh merged 7 commits into
mainfrom
selshafey/cc_multiturn

Conversation

@salma-elshafey
Copy link
Copy Markdown
Contributor

@salma-elshafey salma-elshafey commented Apr 23, 2026

This PR upgrades the coherence evaluator to support conversation-level evaluation (not just single-turn), aligned with the recent output schema updates.

It adds a dedicated multi-turn prompty with explicit skip-gating rules, standardizes multi-turn outputs (coherence, coherence_score, coherence_result, coherence_status, coherence_reason,
coherence_properties), uses MessagesOrQueryResponseInputValidator for messages/query+response input handling, and includes concrete examples for skip behavior and each coherence level (1–5).

Quality is tested and verified to be accurate and reliable. link to report

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

Test Results for assets-test

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 62166d2.

♻️ This comment has been updated with latest results.

Comment thread assets/evaluators/builtin/coherence/evaluator/_coherence.py
Comment thread assets/evaluators/builtin/coherence/evaluator/_coherence.py
@AliMahmoudzadeh AliMahmoudzadeh merged commit fac4e7e into main May 1, 2026
37 checks passed
@AliMahmoudzadeh AliMahmoudzadeh deleted the selshafey/cc_multiturn branch May 1, 2026 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants