Skip to content

feat: encode ranking policy contract - #122

Merged
cschanhniem merged 2 commits into
mainfrom
agent/oamp-loop-20260702-1
Jul 5, 2026
Merged

cschanhniem merged 2 commits into
mainfrom
agent/oamp-loop-20260702-1

Conversation

@cschanhniem

Copy link
Copy Markdown
Collaborator

Summary

  • add a machine-readable ranking policy contract for ensemble vs expert
  • surface the policy in CLI rank output, batch reports, and metrics snapshot
  • add tests and doc updates so score choice stays benchmark-honest

Verification

  • ./.venv/bin/pytest tests/test_expert_ranking_integration.py tests/test_docs_consistent.py -q
  • pre-push hook: ./.venv/bin/python -m pytest -q (1499 passed, 7 skipped)
  • ./.venv/bin/openamp-foundry bench metrics-snapshot --out outputs/metrics_snapshot.json

@cschanhniem
cschanhniem merged commit 0253560 into main Jul 5, 2026
@cschanhniem
cschanhniem deleted the agent/oamp-loop-20260702-1 branch July 5, 2026 00:51
cschanhniem added a commit that referenced this pull request Jul 5, 2026
- Rewrote CONTRIBUTING.md with safety-first contribution checklist, claim policy, PR requirements, and code standards
- Created .github/PULL_REQUEST_TEMPLATE.md with safety impact checklist
- Updated docs/50_LOOP_PLAN.md: all 8 Phase 0 loops marked complete
- Fixed extraneous '}' in metrics_snapshot.py (leftover from PR #122 merge conflict)
- Regenerated outputs/metrics_snapshot.json with ranking_policy entry

Phase 0 exit criteria met:
  - make ci passes with benchmark gate
  - from openamp_foundry.calibration import GateVerdict works
  - New contributor can open safe first PR in <30 min

1701 tests passing.
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