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

add Darija (Moroccan dialects) tasks including darijammlu. darijahellaswag and darija_bench #2521

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

Conversation

hadi-abdine
Copy link

No description provided.

@CLAassistant
Copy link

CLAassistant commented Nov 28, 2024

CLA assistant check
All committers have signed the CLA.

@baberabb
Copy link
Contributor

baberabb commented Dec 4, 2024

Hi! looks good to commit if you could run the pre-commit. Also are all the benchmarks machine translated? maybe we could add that to the language column in lm_eval/tasks/README.md, the same as other tasks. Thanks!

@baberabb
Copy link
Contributor

Hi! LGTM! Could you merge to the current main to resolve the conflict.

@baberabb
Copy link
Contributor

Hi @hadi-abdine. The recursion error is due to the summarization tasks. I think the changes i identified should fix it

@hadi-abdine
Copy link
Author

Hello @baberabb. Thank you for the review, I just fixed the issue causing the recursion error in the summarization task.

@hadi-abdine hadi-abdine requested a review from baberabb January 31, 2025 15:45
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.

3 participants