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

Open Arabic LLM Leaderboard 2 #2661

Merged
merged 12 commits into from
Feb 10, 2025
Merged

Open Arabic LLM Leaderboard 2 #2661

merged 12 commits into from
Feb 10, 2025

Conversation

alielfilali01
Copy link
Contributor

@alielfilali01 alielfilali01 commented Feb 10, 2025

This blog introduces the new iteration of OALL.
cc: @rcojocaru @basma-b @AmzTheory @Manel-Hik

Copy link
Member

@pcuenca pcuenca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, make sure to review the checklist that was added to the PR template, which you can see here for example.

Specifically:

  • You need to add an entry at the end of _blog.yml
  • Images (except the thumbnail) must not be stored in this repo. We typically use this dataset, you are welcome to open a PR there.

Copy link
Contributor

@ariG23498 ariG23498 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reads like a breeze, really love it.

IMO the blog post is too dense (which I am not against) but I have seen blog posts being more impactful with more images and less text. I strongly recommend embedding the leaderboard in the introductory section and adding link to it.

All the other cosmic changes have to do with making the blog post more and more concise.

_blog.yml Show resolved Hide resolved
leaderboard-arabic-v2.md Show resolved Hide resolved
leaderboard-arabic-v2.md Outdated Show resolved Hide resolved
leaderboard-arabic-v2.md Outdated Show resolved Hide resolved
leaderboard-arabic-v2.md Show resolved Hide resolved
leaderboard-arabic-v2.md Show resolved Hide resolved
leaderboard-arabic-v2.md Show resolved Hide resolved
leaderboard-arabic-v2.md Outdated Show resolved Hide resolved
leaderboard-arabic-v2.md Show resolved Hide resolved
leaderboard-arabic-v2.md Outdated Show resolved Hide resolved
@alielfilali01
Copy link
Contributor Author

@pcuenca & @ariG23498 besides that bad looking table (which i have no idea how to fix), i think we are good to merge ? Any other comments ?

@ariG23498
Copy link
Contributor

@alielfilali01 does this work? #2661 (comment)

leaderboard-arabic-v2.md Outdated Show resolved Hide resolved
leaderboard-arabic-v2.md Outdated Show resolved Hide resolved
@alielfilali01
Copy link
Contributor Author

alielfilali01 commented Feb 10, 2025

@pcuenca & @ariG23498 Done

@pcuenca pcuenca merged commit d3f7881 into huggingface:main Feb 10, 2025
1 check 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