Skip to content

Conversation

@rokosbasilisk
Copy link

No description provided.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.


# Language Model Evaluation Harness

![](https://github.com/EleutherAI/lm-evaluation-harness/workflows/Build/badge.svg)
Copy link

Choose a reason for hiding this comment

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

This image is missing a text alternative (alt attribute). This is a problem for people using screen readers.

@leogao2
Copy link
Contributor

leogao2 commented Dec 29, 2021

If I remember correctly the reason we never merged the original PR was because the gpt2 results didnt line up with the numbers in the paper; do you have eval numbers for the new PR?

@rokosbasilisk
Copy link
Author

rokosbasilisk commented Dec 29, 2021

the paper does'nt mention any LLM results.

@leogao2
Copy link
Contributor

leogao2 commented Dec 29, 2021

Oh, I must have gotten it mixed up with another task then

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

👏 You fixed the issue(s)! Great work.

@rokosbasilisk
Copy link
Author

Is there an available implementation of this to try for this task?

@StellaAthena StellaAthena linked an issue Dec 31, 2021 that may be closed by this pull request
@StellaAthena StellaAthena requested a review from leogao2 December 31, 2021 15:01
leogao2
leogao2 previously approved these changes Jan 1, 2022
Copy link
Contributor

@leogao2 leogao2 left a comment

Choose a reason for hiding this comment

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

tested locally

@leogao2 leogao2 merged commit 8728710 into EleutherAI:master Jan 4, 2022
qmdnls pushed a commit to qmdnls/lm-evaluation-harness that referenced this pull request Aug 17, 2023
LZY-the-boys pushed a commit to LZY-the-boys/lm-evaluation-harness-fast that referenced this pull request Sep 12, 2023
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.

Implement the ASDiv Evaluation

3 participants