[Don't Merge] Testing automatic eval - #12
Closed
yl231 wants to merge 10 commits into
Closed
Conversation
- Add automatic PR comment posting with evaluation results - Add extract_metrics.py helper to parse evaluation output - Add upfront model cost validation in check_config_prediction_files.py - Auto-detect dataset split based on prediction file size (809=sub_10, 8400=full) - Always prepare dataset in workflow to ensure availability
📊 Router Evaluation ResultsRouter: Metrics
Evaluation completed by RouterArena automated workflow |
- Add router config with glm-4-air model - Add prediction file with 8400 entries (full dataset) - Remove test1.json prediction file - Update llm_evaluation/run.py to simplify load_ground_truth_dataset - Update prep_datasets.py to support private repo dataset loading
- Add --check-generated-result flag to check_config_prediction_files.py - Flag defaults to False (for pre-inference validation) - When True, validates that all generated_result fields are populated - Add error handling in compute_arena_score to prevent math domain errors - Validates cost and accuracy inputs - Provides clear error messages for invalid values - Add checks in compute_router_metrics to detect when no entries evaluated - Prevents silent failures when all entries are skipped
- Fix populate script to create generated_result as dictionary (not string) - generated_result now matches format expected by evaluation code - Update README to use PR submission instead of email contact - Add instructions for submitting routers via Pull Request
- Update validation to check for dictionary format (not string) - Validate required fields: generated_answer, success, token_usage - Allow empty generated_answer when success is False (failed entries)
📊 Router Evaluation ResultsRouter: Metrics
Evaluation completed by RouterArena automated workflow |
- Remove test configs (test1.json, glm-4-air-router.json) - Remove test predictions (glm-4-air-router.json) - Keep only your-router.json config as in main branch
yl231
added a commit
that referenced
this pull request
Jul 14, 2026
…er (#159); Cross-Router update to #1 (#163) (#165) Adds three newly-merged router submissions and refreshes the Cross-Router row from its updated submission. Official /evaluate metrics: - Cross-Router (#163): Arena 76.12 · Acc 78.14% · $0.30/1K · Robust 67.14 -> now #1 - Nadir-Tumbler (#159): Arena 75.17 · Acc 75.34% · $0.08/1K · Robust 66.43 -> #4 - chuzom-solo-v32 (#161): Arena 70.61 · Acc 70.59% · $0.10/1K · Robust 100.00 -> #12 - BARouter (#157): Arena 67.09 · Acc 68.80% · $0.63/1K · Robust 52.38 -> #15 Manifest entries added for the three new routers so the website sync regenerates their derived data. Co-authored-by: yl231 <jamesbond19925@gmail.com> Co-authored-by: Claude Opus 4.8 (1M context) <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
1 participant
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.
Testing automatic eval