Skip to content

A scenario that never reached the model is not scored - #3

Open
veer0608 wants to merge 1 commit into
mainfrom
not-scored-is-not-zero
Open

A scenario that never reached the model is not scored#3
veer0608 wants to merge 1 commit into
mainfrom
not-scored-is-not-zero

Conversation

@veer0608

Copy link
Copy Markdown
Owner

A provider outage arrives shaped exactly like an agent failure: no tools, no answer, scores zero. 9 of 80 scenario-runs died that way on a provider-side 400 during yesterday's eight-run series, and every one was averaged into the reported F1. The number measured the provider, not the agent.

Change

  • The aggregate runs over scenarios that actually reached the model.
  • Unreached ones are counted and named: scenarios: 10 (8 scored) and NOT SCORED: 2 never reached the model (plan-limits, known-issue-rate-limit).
  • The report table marks them NOT SCORED instead of printing them as failures with F1 0.00.
  • run_error is carried on the row, so the count is recoverable from the JSON scorecard.

Effect

A live run that scored 0.77 under the old accounting scores 0.958 over the eight scenarios that ran. The previously published mean of 0.78 across eight runs is therefore a floor, not an estimate.

38 tests, up from 36. The two new ones assert that an errored row does not drag the mean and that the count is reported rather than hidden.

It arrives shaped exactly like one the agent failed: no tools, no answer,
scores zero. Nine of eighty scenario-runs died that way on a provider-side
400, and every one of them was averaged into the reported F1, so the number
measured the provider rather than the agent.

The aggregate now runs over the scenarios that actually reached the model.
The rest are counted, named in the summary as NOT SCORED, and marked in the
report table rather than quietly printed as failures. A run where nothing
reached the model now reports nothing rather than zero.

The effect is not small: a run scoring 0.77 under the old accounting scores
0.958 over the eight scenarios that ran.
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.

1 participant