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.

Sign up for GitHub

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

Jump to bottom

Remove olmo as a dependency #1148

Merged

dakinggg merged 1 commit into main from saaketh/remove_olmo

Apr 29, 2024

Contributor

snarayan21 commented Apr 29, 2024

Removes olmo as a dependency since we don't use it at all, and it requires torch < 2.3.0 which is incompatible with us upgrading.


          remove olmo

ac1480f

snarayan21 requested a review from a team as a code owner

April 29, 2024 17:13

dakinggg approved these changes

View reviewed changes

Collaborator

dakinggg left a comment

Note: The olmo models are now in the transformers library, so this package should not be required to use them any more.

dakinggg enabled auto-merge (squash)

April 29, 2024 17:25

dakinggg merged commit fbcf311 into main

9 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment