Skip to content

fix(reward): bipartite-match BFCL multi-call grading (#180)#218

Open
kiannidev wants to merge 1 commit into
mini-router:mainfrom
kiannidev:sn74-kiannidev-bfcl-match
Open

fix(reward): bipartite-match BFCL multi-call grading (#180)#218
kiannidev wants to merge 1 commit into
mini-router:mainfrom
kiannidev:sn74-kiannidev-bfcl-match

Conversation

@kiannidev

@kiannidev kiannidev commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #180: BFCL multi-call grading sorted candidate (scalar args) and gold (allowed-value lists) independently, then zip-paired mismatched calls — correct multi-call answers scored 0.

Change

  • Replace sort-and-zip with _bfcl_match_all_calls (backtracking bipartite perfect matching)
  • Offline tests in tests/test_reward_bfcl.py
  • JOURNAL entry

Test plan

  • pytest tests/test_reward_bfcl.py — 3 passed

Replace sort-and-zip alignment with a perfect bipartite matching so
correct multi-call answers are not scored 0 when candidate scalars and
gold allowed-value lists serialize in different orders.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions github-actions Bot added docs Documentation changes miner Miner contribution PR train Training or routing changes labels Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation changes miner Miner contribution PR train Training or routing changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

reward: BFCL multi-call grading misaligns candidate/gold calls, scoring correct answers 0

1 participant