[Feature] Support G-Pass@k and LiveMathBench#1772
Merged
MaiziXiao merged 15 commits intoopen-compass:mainfrom Dec 30, 2024
Merged
[Feature] Support G-Pass@k and LiveMathBench#1772MaiziXiao merged 15 commits intoopen-compass:mainfrom
MaiziXiao merged 15 commits intoopen-compass:mainfrom
Conversation
tonysy
requested changes
Dec 23, 2024
| gen_config['temperature'] = temperature | ||
| # gen_config['top_k'] = 40 | ||
| # gen_config['temperature'] = temperature | ||
| pass # use the parameters passed from gen_config |
Collaborator
There was a problem hiding this comment.
Does this modification will introduce BC? @MaiziXiao
Contributor
There was a problem hiding this comment.
The old implementation will overwrite top_k and temperature if 'do_sample','top_k' and 'temperature' are all set. This change will make sure 'top_k' and 'temperature' will not be overwritten.
@jnanliu We should remove these lines here
liushz
reviewed
Dec 25, 2024
stephen-nju
pushed a commit
to stephen-nju/opencompass
that referenced
this pull request
May 14, 2025
* support G-Pass@k and livemathbench * fix bugs * fix comments of GPassKEvaluator * update saved details of GPassKEvaluator * update saved details of GPassKEvaluator * fix eval api configs & update openai_api for ease of debugging * update huggingface path * fix method name of G-Pass@k * fix default value of eval_model_name * refactor G-Pass@k evaluator * log generation params for each backend * fix evaluation resume * add notimplementerror
zyc140345
pushed a commit
to zyc140345/opencompass
that referenced
this pull request
Oct 23, 2025
* support G-Pass@k and livemathbench * fix bugs * fix comments of GPassKEvaluator * update saved details of GPassKEvaluator * update saved details of GPassKEvaluator * fix eval api configs & update openai_api for ease of debugging * update huggingface path * fix method name of G-Pass@k * fix default value of eval_model_name * refactor G-Pass@k evaluator * log generation params for each backend * fix evaluation resume * add notimplementerror
iamkaia
pushed a commit
to iamkaia/opencompass
that referenced
this pull request
Feb 4, 2026
* support G-Pass@k and livemathbench * fix bugs * fix comments of GPassKEvaluator * update saved details of GPassKEvaluator * update saved details of GPassKEvaluator * fix eval api configs & update openai_api for ease of debugging * update huggingface path * fix method name of G-Pass@k * fix default value of eval_model_name * refactor G-Pass@k evaluator * log generation params for each backend * fix evaluation resume * add notimplementerror
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
4 participants
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.
Motivation
Support the evaluation using G-Pass@k metric and update the configurations of LiveMathBench.
Modification
turbomind_with_tf_above_v4_33.pyto ensure that gen_cfg of model_cfg is passed into the lmdeploy pipeline.openai_api.pyto print error url for ease of debugging.Checklist
Before PR: