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

add e2e tests for Unsloth qlora and test the builds #2093

Merged
merged 15 commits into from
Nov 30, 2024
Merged

Conversation

winglian
Copy link
Collaborator

No description provided.

@winglian winglian added the hold don't merge this yet label Nov 20, 2024
@winglian winglian removed the hold don't merge this yet label Nov 22, 2024
@winglian winglian changed the title see if unsloth installs cleanly in ci add e2e tests for Unsloth qlora and test the builds Nov 22, 2024
@@ -67,6 +67,7 @@ jobs:
run: |
pip3 show torch
pip3 install -U -e .
python scripts/unsloth_install.py | sh
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this be added to pytest-sdist and to the nightly workflow too?

src/axolotl/utils/models.py Outdated Show resolved Hide resolved
src/axolotl/utils/models.py Show resolved Hide resolved
src/axolotl/utils/models.py Outdated Show resolved Hide resolved
tests/e2e/patched/test_unsloth_qlora.py Outdated Show resolved Hide resolved
tests/e2e/patched/test_unsloth_qlora.py Outdated Show resolved Hide resolved
tests/e2e/patched/test_unsloth_qlora.py Outdated Show resolved Hide resolved
tests/e2e/patched/test_unsloth_qlora.py Show resolved Hide resolved
@winglian winglian merged commit 5f1d98e into main Nov 30, 2024
15 checks passed
@winglian winglian deleted the unsloth-tests branch November 30, 2024 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants