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

Update arabic_evals.py: Fix custom arabic tasks [2nd attempt] #444

Merged
merged 8 commits into from
Dec 20, 2024

Conversation

alielfilali01
Copy link
Contributor

Fix alghafa prompt function by explicitly determining the list of choices based on task_name.
(Not all subsets of AlGhafa Native share same columns)
cc: @clefourrier

Fix alghafa prompt function by explicitly determining the list of choices based on task_name.
Not all subsets of AlGhafa Native share same columns
@HuggingFaceDocBuilderDev
Copy link
Collaborator

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

alielfilali01 and others added 3 commits December 13, 2024 12:39
Co-authored-by: Clémentine Fourrier <22726840+clefourrier@users.noreply.github.com>
Fix to `KeyError: 'sol3'` and `TypeError: Doc.__init__() got an unexpected keyword argument 'target_for_fewshot_sorting'`
@alielfilali01
Copy link
Contributor Author

Ready to be merged [√]
fixing :

KeyError: 'sol3'

And:

TypeError: Doc.__init__() got an unexpected keyword argument 'target_for_fewshot_sorting'

@alielfilali01
Copy link
Contributor Author

@clefourrier I believe this PR is ready to be merged and it is tested 👍

@clefourrier
Copy link
Member

Yep, will merge as soon as tests pass ^^ thanks for the ping!

@clefourrier
Copy link
Member

@alielfilali01 you'll need to make sure code qual passes

@alielfilali01
Copy link
Contributor Author

Damn i keep forgetting that 🥲 sorry will run it later ounce i can ! Thanks @clefourrier

@NathanHB NathanHB merged commit fbca143 into huggingface:main Dec 20, 2024
3 checks passed
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.

4 participants