microsoft-foundry / microsoft-foundry/Model-Router-Auto-Evaluation

Feedback: exporting JudgeResult/AbsoluteScore output as EvalPort ResultSet for portable comparison

Abierto
#9 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
HTML
Estrellas
29
Forks
11
Merge medio
2 h 48 min
PR fusionados (30 d)
2

Descripción

Hi maintainers — feedback/idea, not a bug report, filed under the "general feedback" path from the README.

Context on me: I maintain EvalPort, an open (Apache 2.0) interchange spec for portable LLM eval test cases, graders, suites, and results, with a Python/TS SDK (evalport-sdk / openeval) and 30+ real, tested framework adapter packages under adapters/. I'm not affiliated with Microsoft — just an outside contributor who read through src/judge.py and thinks this tool's judge output maps cleanly onto EvalPort's Result/ResultSet shape.

Why I'm opening this: This toolkit's anti-bias judge design is genuinely good — dual-ordering pairwise (pairwise_router_first / pairwise_baseline_first) reconciled into a single pairwise_winner, with disagreement collapsed to a tie rather than picked arbitrarily, plus independent 1–5 AbsoluteScore scoring (accuracy, completeness, clarity, helpfulness, and the overall average). That's exactly the kind of structured judge result EvalPort's Result schema was built to carry portably — so a team benchmarking Model Router here could also run the same prompts through DeepEval, Ragas, or Azure AI Evaluation and diff results side by side, without hand-writing glue code each time.

Concretely, what I'm suggesting (not asking anyone to build — happy to draft it myself as a PR if there's interest): a small, optional exporter — e.g. src/exporters/evalport.py — that takes a JudgeResult and emits an EvalPort-shaped result record. Sketch, using the real fields from judge.py:

# src/exporters/evalport.py
from .judge import JudgeResult

def judge_result_to_evalport(result: JudgeResult) -> dict:
    """Map a JudgeResult onto an EvalPort Result record."""
    return {
        "test_case_id": result.prompt_id,
        "grader_results": [
            {
                "grader_id": "pairwise_dual_order",
                "type": "llm_judge",
                "value": result.pairwise_winner,  # "model_router" | "baseline" | "tie"
                "metadata": {
                    "router_first_winner": result.pairwise_router_first.winner if result.pairwise_router_first else None,
                    "baseline_first_winner": result.pairwise_baseline_first.winner if result.pairwise_baseline_first else None,
                },
            },
            {
                "grader_id": "absolute_score_router",
                "type": "llm_judge",
                "value": result.router_score.overall if result.router_score else None,
                "metadata": result.router_score.__dict__ if result.router_score else {},
            },
            {
                "grader_id": "absolute_score_baseline",
                "type": "llm_judge",
                "value": result.baseline_score.overall if result.baseline_score else None,
                "metadata": result.baseline_score.__dict__ if result.baseline_score else {},
            },
        ],
        "metadata": {
            "judge_model": result.judge_model,
            "latency_ms": result.latency_ms,
            "error": result.error,
        },
    }

Wrapping a batch of these plus a suite_id and grader definitions into an EvalPort ResultSet and validating it with evalport-sdk's validate_suite/validate_result_set would be maybe 20-30 more lines, following the shape already used by two adapters in the repo I maintain: azure-ai-evaluation-openeval-adapter (closest analog — same Azure AI ecosystem) and deepeval-openeval-adapter (comparable dual-scoring LLM-judge output).

Benefit for this repo specifically: it'd let compare_results.py-style comparisons extend beyond two local runs to any EvalPort-emitting eval tool, without this project taking on a hard dependency — the export could be fully optional (pip install -e ".[evalport]") and touch nothing in the core judge.py pipeline.

Happy to open a draft PR behind an extras flag if that's useful, or just leave this as food for thought if it's not a direction you want to take the tool. Either way, appreciate the clean dataclass design in judge.py — made this easy to reason about from the outside.

— Sahi, independent contributor (not affiliated with Microsoft)

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza con src/judge.py para revisar JudgeResult y sus campos de score; después, inspecciona compare_results.py y el punto de entrada propuesto src/exporters/evalport.py. Comprueba cómo un extra opcional de EvalPort podría empaquetar el exportador y validar un lote con validate_suite y validate_result_set sin cambiar la canalización principal; se considera terminado cuando el ResultSet asignado es válido y opcional.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
tooling
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Activo
Claridad
Bastante claro
Aptitud para principiantes
52/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.