Skip to content

docs(evals): align model-judge calibration policy#13

Merged
aryeko merged 1 commit into
mainfrom
chore/eval-kit-v0.1.5-calibration-policy
Jul 4, 2026
Merged

docs(evals): align model-judge calibration policy#13
aryeko merged 1 commit into
mainfrom
chore/eval-kit-v0.1.5-calibration-policy

Conversation

@aryeko

@aryeko aryeko commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • pin @agentic-workflow-kit/eval-kit to v0.1.5 and refresh the lockfile
  • align define-product model-judge docs and calibration template with the shared calibration/reporting policy
  • clarify partial/unknown handling, expected-bad adverse evidence, and raw result bundle policy

Verification

  • pnpm install
  • pnpm install --frozen-lockfile
  • pnpm eval:doctor
  • pnpm eval:list
  • pnpm eval:validate-fixtures
  • pnpm eval:judge:doctor
  • pnpm eval:judge:list
  • pnpm eval:judge:validate-fixtures
  • pnpm check
  • git diff --check

Notes

Docs/package migration only. PRD/product judge semantics remain local. Provider-backed commands remain outside pnpm check and CI.

@aryeko aryeko merged commit 37e21ac into main Jul 4, 2026
1 check passed
@aryeko aryeko deleted the chore/eval-kit-v0.1.5-calibration-policy branch July 4, 2026 01:51

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 101772e872

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


- include deterministic evidence first;
- include pointwise evidence only after deterministic evidence for the same review context exists;
- include counts for `covered`, `partial`, `missing`, `contradicted`, and `unknown`;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Align report-count policy with generated reports

In this repo, pnpm eval:report uses compileReport, which builds the Item verdicts summary from only the verdicts present in pointwiseResult.items (see evals/adapter.mjs:997-1007), so zero-count categories are omitted. Under this new policy, a report with no unknown or no partial verdicts will not actually include counts for all five categories, making reviewers unable to distinguish zero from not reported under the documented v0.1.5 calibration policy. Please either emit all five categories with zeroes or relax the policy to observed verdict counts.

Useful? React with 👍 / 👎.

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