mirror of
https://github.com/encounter/adk-python.git
synced 2026-07-09 18:19:28 -07:00
feat: Data model for Rubric based metric and eval config
Details: - We plan on introducing Rubric based metrics in subsequent changes. This change introduces the data model needed that allows agent developer to provide rubrics. - We also introduce a data model for the config that the eval system has been using for quite some time. It was loosely and informally described as a dictionary of metric names and expected thresholds. In this change, we actually formalize it using a pydantic data model, and extend it allow developers to specify rubrics as a part of their eval config. What is a rubric based metric? A rubric based metric is the assessment of a Agent's response (final or intermediate) along some rubric. This evaluation of agent's response significantly differs from the strategy where one has to provide a golden response. PiperOrigin-RevId: 805488436
This commit is contained in:
committed by
Copybara-Service
parent
37228beddd
commit
e88e667770
@@ -0,0 +1,96 @@
|
||||
# Copyright 2025 Google LLC
|
||||
#
|
||||
# Licensed under the Apache License, Version 2.0 (the "License");
|
||||
# you may not use this file except in compliance with the License.
|
||||
# You may obtain a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing, software
|
||||
# distributed under the License is distributed on an "AS IS" BASIS,
|
||||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||
# See the License for the specific language governing permissions and
|
||||
# limitations under the License.
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
from unittest import mock
|
||||
|
||||
from google.adk.cli.cli_eval import _DEFAULT_EVAL_CONFIG
|
||||
from google.adk.cli.cli_eval import get_eval_metrics_from_config
|
||||
from google.adk.cli.cli_eval import get_evaluation_criteria_or_default
|
||||
from google.adk.evaluation.eval_config import EvalConfig
|
||||
from google.adk.evaluation.eval_rubrics import Rubric
|
||||
from google.adk.evaluation.eval_rubrics import RubricContent
|
||||
|
||||
|
||||
def test_get_evaluation_criteria_or_default_returns_default():
|
||||
assert get_evaluation_criteria_or_default("") == _DEFAULT_EVAL_CONFIG
|
||||
|
||||
|
||||
def test_get_evaluation_criteria_or_default_reads_from_file():
|
||||
eval_config = EvalConfig(
|
||||
criteria={"tool_trajectory_avg_score": 0.5, "response_match_score": 0.5}
|
||||
)
|
||||
mock_open = mock.mock_open(read_data=eval_config.model_dump_json())
|
||||
with mock.patch("builtins.open", mock_open):
|
||||
assert get_evaluation_criteria_or_default("dummy_path") == eval_config
|
||||
|
||||
|
||||
def test_get_eval_metrics_from_config():
|
||||
rubric_1 = Rubric(
|
||||
rubric_id="test-rubric",
|
||||
rubric_content=RubricContent(text_property="test"),
|
||||
)
|
||||
eval_config = EvalConfig(
|
||||
criteria={
|
||||
"tool_trajectory_avg_score": 1.0,
|
||||
"response_match_score": 0.8,
|
||||
"final_response_match_v2": {
|
||||
"threshold": 0.5,
|
||||
"judge_model_options": {
|
||||
"judge_model": "gemini-pro",
|
||||
"num_samples": 1,
|
||||
},
|
||||
},
|
||||
"rubric_based_final_response_quality_v1": {
|
||||
"threshold": 0.9,
|
||||
"judge_model_options": {
|
||||
"judge_model": "gemini-ultra",
|
||||
"num_samples": 1,
|
||||
},
|
||||
"rubrics": [rubric_1],
|
||||
},
|
||||
}
|
||||
)
|
||||
eval_metrics = get_eval_metrics_from_config(eval_config)
|
||||
|
||||
assert len(eval_metrics) == 4
|
||||
assert eval_metrics[0].metric_name == "tool_trajectory_avg_score"
|
||||
assert eval_metrics[0].threshold == 1.0
|
||||
assert eval_metrics[0].criterion.threshold == 1.0
|
||||
assert eval_metrics[1].metric_name == "response_match_score"
|
||||
assert eval_metrics[1].threshold == 0.8
|
||||
assert eval_metrics[1].criterion.threshold == 0.8
|
||||
assert eval_metrics[2].metric_name == "final_response_match_v2"
|
||||
assert eval_metrics[2].threshold == 0.5
|
||||
assert eval_metrics[2].criterion.threshold == 0.5
|
||||
assert (
|
||||
eval_metrics[2].criterion.judge_model_options["judge_model"]
|
||||
== "gemini-pro"
|
||||
)
|
||||
assert eval_metrics[3].metric_name == "rubric_based_final_response_quality_v1"
|
||||
assert eval_metrics[3].threshold == 0.9
|
||||
assert eval_metrics[3].criterion.threshold == 0.9
|
||||
assert (
|
||||
eval_metrics[3].criterion.judge_model_options["judge_model"]
|
||||
== "gemini-ultra"
|
||||
)
|
||||
assert len(eval_metrics[3].criterion.rubrics) == 1
|
||||
assert eval_metrics[3].criterion.rubrics[0] == rubric_1
|
||||
|
||||
|
||||
def test_get_eval_metrics_from_config_empty_criteria():
|
||||
eval_config = EvalConfig(criteria={})
|
||||
eval_metrics = get_eval_metrics_from_config(eval_config)
|
||||
assert not eval_metrics
|
||||
@@ -840,6 +840,7 @@ def test_run_eval(test_app, create_test_eval_set):
|
||||
"threshold": 0.5,
|
||||
"score": 1.0,
|
||||
"evalStatus": 1,
|
||||
"details": {},
|
||||
}],
|
||||
}
|
||||
for k, v in expected_eval_case_result.items():
|
||||
|
||||
@@ -15,6 +15,7 @@
|
||||
from __future__ import annotations
|
||||
|
||||
from google.adk.evaluation.eval_case import Invocation
|
||||
from google.adk.evaluation.eval_metrics import BaseCriterion
|
||||
from google.adk.evaluation.eval_metrics import EvalMetric
|
||||
from google.adk.evaluation.eval_metrics import JudgeModelOptions
|
||||
from google.adk.evaluation.eval_metrics import PrebuiltMetrics
|
||||
@@ -130,9 +131,8 @@ def _create_test_evaluator_gemini(
|
||||
EvalMetric(
|
||||
metric_name="final_response_match_v2",
|
||||
threshold=threshold,
|
||||
judge_model_options=JudgeModelOptions(
|
||||
judge_model="gemini-2.5-flash",
|
||||
num_samples=3,
|
||||
criterion=BaseCriterion(
|
||||
threshold=0.5,
|
||||
),
|
||||
),
|
||||
)
|
||||
|
||||
@@ -20,6 +20,7 @@ from unittest.mock import MagicMock
|
||||
from google.adk.evaluation.eval_case import Invocation
|
||||
from google.adk.evaluation.eval_metrics import EvalMetric
|
||||
from google.adk.evaluation.eval_metrics import JudgeModelOptions
|
||||
from google.adk.evaluation.eval_metrics import LlmAsAJudgeCriterion
|
||||
from google.adk.evaluation.evaluator import EvalStatus
|
||||
from google.adk.evaluation.evaluator import EvaluationResult
|
||||
from google.adk.evaluation.evaluator import PerInvocationResult
|
||||
@@ -60,15 +61,19 @@ class MockLlmAsJudge(LlmAsJudge):
|
||||
@pytest.fixture
|
||||
def mock_llm_as_judge():
|
||||
return MockLlmAsJudge(
|
||||
EvalMetric(
|
||||
eval_metric=EvalMetric(
|
||||
metric_name="test_metric",
|
||||
threshold=0.5,
|
||||
judge_model_options=JudgeModelOptions(
|
||||
judge_model="gemini-2.5-flash",
|
||||
judge_model_config=genai_types.GenerateContentConfig(),
|
||||
num_samples=3,
|
||||
criterion=LlmAsAJudgeCriterion(
|
||||
threshold=0.5,
|
||||
judge_model_options=JudgeModelOptions(
|
||||
judge_model="gemini-2.5-flash",
|
||||
judge_model_config=genai_types.GenerateContentConfig(),
|
||||
num_samples=3,
|
||||
),
|
||||
),
|
||||
),
|
||||
criterion_type=LlmAsAJudgeCriterion,
|
||||
)
|
||||
|
||||
|
||||
@@ -94,10 +99,11 @@ def test_get_eval_status():
|
||||
assert get_eval_status(score=None, threshold=0.8) == EvalStatus.NOT_EVALUATED
|
||||
|
||||
|
||||
def test_llm_as_judge_init_missing_judge_model_options():
|
||||
def test_llm_as_judge_init_missing_criterion():
|
||||
with pytest.raises(ValueError):
|
||||
MockLlmAsJudge(
|
||||
EvalMetric(metric_name="test_metric", threshold=0.8),
|
||||
criterion_type=LlmAsAJudgeCriterion,
|
||||
)
|
||||
|
||||
|
||||
@@ -107,10 +113,16 @@ def test_llm_as_judge_init_unregistered_model():
|
||||
EvalMetric(
|
||||
metric_name="test_metric",
|
||||
threshold=0.8,
|
||||
judge_model_options=JudgeModelOptions(
|
||||
judge_model="unregistered_model",
|
||||
criterion=LlmAsAJudgeCriterion(
|
||||
threshold=0.5,
|
||||
judge_model_options=JudgeModelOptions(
|
||||
judge_model="unregistered_model",
|
||||
judge_model_config=genai_types.GenerateContentConfig(),
|
||||
num_samples=3,
|
||||
),
|
||||
),
|
||||
),
|
||||
criterion_type=LlmAsAJudgeCriterion,
|
||||
)
|
||||
|
||||
|
||||
|
||||
Reference in New Issue
Block a user