Skip to content

Support overriding metric configs - #223

Open
JosephMarinier wants to merge 1 commit into
mainfrom
pr/joseph/support-overriding-metric-configs
Open

Support overriding metric configs#223
JosephMarinier wants to merge 1 commit into
mainfrom
pr/joseph/support-overriding-metric-configs

Conversation

@JosephMarinier

Copy link
Copy Markdown
Collaborator

Support overriding metric configs via CLI arguments or environment variables.

Examples:

EVA_METRIC_CONFIGS='{"faithfulness": {"judge_model": "gpt-5.6-terra", "judge_params": {"reasoning_effort": "none"}}}'
EVA_METRIC_CONFIGS__FAITHFULNESS__JUDGE_MODEL=gpt-5.6-terra
EVA_METRIC_CONFIGS__FAITHFULNESS__JUDGE_PARAMS__REASONING_EFFORT=none

via CLI arguments or environment variables.
@JosephMarinier
JosephMarinier force-pushed the pr/joseph/support-overriding-metric-configs branch from a61b552 to 89f6cde Compare September 2, 2026 23:32
Comment thread src/eva/run_benchmark.py
Comment on lines +58 to +59
if config.metric_configs:
runner.config.metric_configs = config.metric_configs

@JosephMarinier JosephMarinier Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can debate this. Should it be possible to override metric configs in a rerun? It was useful to me for testing, but it might be too dangerous to accidentally override metric configs.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that's a debate we could have about multiple config fields. Instead, we could consider adding an option to explicitly enable config overrides (or disable reading the config saved on disk), something like:
eva --run-id ... --ignore-previous-config

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i like the --ignore-previous-config idea. For most config values right now I don't think we can override them on re-run, so I wonder if we should enable overriding all of them at the same time and introduce --ignore-previous-config at the same time

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.

2 participants