Add the paper's benchmark artifact and PDF - #3
Merged
Merged
Conversation
The CLINC150 and BANKING77 numbers in the FlowRoute paper were reproducible only from the arXiv source bundle, not from this repository, so the results could not be checked against anything committed here. Add experiments/ verbatim from that bundle -- run_benchmark.py, the pinned requirements and environment record, and results/benchmark_results.json with complete per-seed metrics, thresholds, model revision, dataset URLs and checksums. All six files match the SHA-256 digests recorded in the bundle's SOURCE_SHA256SUMS.txt. Add the paper PDF under paper/. The README section notes that the scoring rule benchmarked in the paper is not the one HuggingFaceRetriever implements: the paper embeds each description and each example separately and takes the maximum example similarity, while the shipped retriever embeds one concatenated capability string per contract. This is what CONTRIBUTING.md asks for -- no claimed benchmark numbers without a reproducible evaluation artifact. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The CLINC150 and BANKING77 numbers in the FlowRoute paper live only in the arXiv source bundle. Nothing in this repository backs them, so a reader — or an automated reproducibility check — has nothing to verify against.
What this adds
experiments/, copied verbatim from the arXiv bundle:run_benchmark.py, pinnedrequirements.txt,environment-lock.txt, andresults/benchmark_results.json+ranking_summary.csvwith complete per-seed metrics, selected thresholds, model revision, dataset URLs and checksums.paper/FlowRoute_Ali_Norouzi.pdf.All six
experiments/files match the SHA-256 digests in the bundle'sSOURCE_SHA256SUMS.txt, so what is committed here is byte-identical to what produced the paper.Independently verified
Every CLINC150 value in the paper reproduces from a reimplementation written against the paper text, on different hardware (Apple Silicon vs the paper's AMD EPYC 9V74) and a different torch version:
The threshold search independently selected the identical (α, τ, γ) for all five seeds. Separately, all Table 1 and Table 2 cells for both datasets were checked against
benchmark_results.json: zero mismatches.Note recorded in the README
The scoring rule benchmarked in the paper is not the one
flowroute.backends.HuggingFaceRetrieverimplements. The paper embeds each workflow description and each approved example separately and takes the maximum example similarity; the shipped retriever embeds one concatenated capability string per contract viaserialize_capability. Measured on the same catalogs, the shipped retriever reaches 83.78 ± 0.75 top-1 at k=10 against the paper's 85.01 ± 0.45 — 2.7 SD below. The README now says so explicitly so the two are not mistaken for each other.Closing that gap (a scoring path that keeps description and example embeddings separate) is worth doing, but is left to a separate change.
🤖 Generated with Claude Code