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

point to upstream autoawq for transformers fix #2070

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

winglian
Copy link
Collaborator

@winglian winglian commented Nov 16, 2024

Description

There's an upstream breaking change in transformers that breaks autoawq and it breaks our nightlies that check against upstream transformers main: https://github.com/axolotl-ai-cloud/axolotl/actions/runs/11865507782/job/33071826272

There's a fix in autoawq, casper-hansen/AutoAWQ@a28c747, but we have to have a direct dependency on it which will prevent us from releasing 0.5.1 to PyPi next week. This is a placeholder PR for now to verify the functionality and hopefully we'll get a patch release of autoawq soon. @casper-hansen

EDIT: I've triggered the nightly test against this branch here for validation: https://github.com/axolotl-ai-cloud/axolotl/actions/runs/11867074628

@winglian winglian added the hold don't merge this yet label Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold don't merge this yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant