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

Enable Code Carbon by default in evaluation runs #1427

Closed
isaac-chung opened this issue Nov 10, 2024 · 7 comments · Fixed by #1428
Closed

Enable Code Carbon by default in evaluation runs #1427

isaac-chung opened this issue Nov 10, 2024 · 7 comments · Fixed by #1428

Comments

@isaac-chung
Copy link
Collaborator

isaac-chung commented Nov 10, 2024

[Edit] Was discussing with @KennethEnevoldsen about this suggestion as a way to combine run eval time + hardware info so we don't have to keep track of both. Enabling this would be a convenient step towards that before we decide to use something else that is more representative (such as FLOPS).

@Muennighoff
Copy link
Contributor

What's the motivation for enabling it by default? I think @imenelydiaker intentionally made it optional in #712 (comment) 🤔

@isaac-chung
Copy link
Collaborator Author

Was discussing with @KennethEnevoldsen about this suggestion as a way to combine run eval time + hardware info so we don't have to keep track of both. Enabling this would be a convenient step towards that before we decide to use something else that is more representative (such as FLOPS).

@Muennighoff
Copy link
Contributor

Makes sense, thanks for the context!!

@isaac-chung
Copy link
Collaborator Author

No worries, my bad for not filling out the description in the first place. Updated!

@isaac-chung
Copy link
Collaborator Author

FYI all: (copied from the PR)
Based one this issue, it seems like we can* allow multiple runs, but the tracking will be somewhat skewed. It won't matter in the tests, but will for sure affect real runs if multiple runs are done on the same machine. I think we would also add a line in README and/or a warning to let users know.

@Samoed
Copy link
Collaborator

Samoed commented Nov 14, 2024

CO² added to open llm leaderboard https://huggingface.co/spaces/open-llm-leaderboard/open_llm_leaderboard

@isaac-chung
Copy link
Collaborator Author

Merged into branch v2.0.0

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 a pull request may close this issue.

3 participants