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

Scrolls branch #2309

Open
wants to merge 125 commits into
base: main
Choose a base branch
from
Open

Scrolls branch #2309

wants to merge 125 commits into from

Conversation

blitzionic
Copy link

fix(scrolls): update load_metric import to use evaluate library

Hugging Face moved the load_metric function from the datasets library to the evaluate library.
Updated scrolls.py to reflect this change to avoid ImportError.

Niklas Muennighoff and others added 30 commits February 12, 2023 10:47
[triviaqa] The ground truth must be a *substring* of the generated an…
Sort descending as intended for `greedy_until`
daishen and others added 25 commits August 2, 2023 22:34
add bnb_4bit_use_double_quant and low_cpu_mem_usage
Removes the `_remove_prefixes` optimization as it's invalid for exact match metrics. Closes EleutherAI#726
…tch-1

Update crowspairs.py higher_is_better
feature: add vllm interface
fix(scrolls): update load_metric import to use evaluate library

Hugging Face moved the load_metric function from the datasets library to the evaluate library.
Updated scrolls.py to reflect this change to avoid ImportError.
@CLAassistant
Copy link

CLAassistant commented Sep 16, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
5 out of 10 committers have signed the CLA.

✅ baberabb
✅ StellaAthena
✅ haileyschoelkopf
✅ haonan-li
✅ lintangsutawika
❌ daishen
❌ jiminHuang
❌ tothemoon96
❌ blitzionic
❌ Me1oyy


daishen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@baberabb
Copy link
Contributor

Hi! Thanks for this, but can you make another PR with a clean git history?

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.