Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] Fix AGIEval chinese sets #972

Merged
merged 7 commits into from
May 6, 2024
Merged

Conversation

xu-song
Copy link
Contributor

@xu-song xu-song commented Mar 14, 2024

Motivation

Fix AGIEval chinese sets: 'jec-qa-kd' and 'jec-qa-ca' are chinese datasets.

'jec-qa-kd':
'以下是一道中国司法考试基础知识题,请选择正确的答案。',
'jec-qa-ca':
'以下是一道中国司法考试案例分析题,请选择正确的答案。',

Changes

 agieval_chinese_sets = [
     'gaokao-chinese',
     'gaokao-english',
     'gaokao-geography',
     'gaokao-history',
     'gaokao-biology',
     'gaokao-chemistry',
     'gaokao-physics',
     'gaokao-mathqa',
     'logiqa-zh',
     'gaokao-mathcloze',
+    'jec-qa-kd',
+    'jec-qa-ca',
 ]

Before PR

以下是一道中国司法考试基础知识题,请选择正确的答案。\n中国商务部决定对...辖权的法院起诉\nThe answer is 

After PR

以下是一道中国司法考试基础知识题,请选择正确的答案。\n中国商务部决定对...辖权的法院起诉\n答案是: 

@xu-song xu-song reopened this Mar 14, 2024
@tonysy tonysy requested a review from Leymore March 19, 2024 08:11
@Leymore Leymore merged commit d501710 into open-compass:main May 6, 2024
4 checks passed
liuyaox pushed a commit to liuyaox/opencompass that referenced this pull request Jun 26, 2024
* [Fix] Fix AGIEval chinese sets

* Create agieval_gen_617738.py

* [Fix] Fix AGIEval chinese sets

* Restore agieval_gen_64afd3.py

* Update agieval_gen.py

* Create agieval_mixed_0fa998.py

* Update agieval_mixed.py
Leymore pushed a commit to Leymore/opencompass that referenced this pull request Jul 12, 2024
* [Fix] Fix AGIEval chinese sets

* Create agieval_gen_617738.py

* [Fix] Fix AGIEval chinese sets

* Restore agieval_gen_64afd3.py

* Update agieval_gen.py

* Create agieval_mixed_0fa998.py

* Update agieval_mixed.py
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.

None yet

3 participants