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.

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

Skip accelerate tests #208

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Skip accelerate tests #208

merged 1 commit into from
Nov 21, 2024

Conversation

kylesayrs
Copy link
Contributor

Purpose

  • Better user experience for users who run
python3 -m pip install -e .
python3 -m pytest tests

Changes

  • Skip tests which require accelerate if accelerate is not installed

Testing

  • CI tests

@kylesayrs kylesayrs mentioned this pull request Nov 19, 2024
Merged
@kylesayrs kylesayrs self-assigned this Nov 19, 2024
@dsikka dsikka merged commit 9010372 into main Nov 21, 2024
1 check passed
@dsikka dsikka deleted the kylesayrs/skip-accelerate-tests branch November 21, 2024 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dsikka dsikka dsikka approved these changes

@mgoin mgoin mgoin approved these changes

Assignees

@kylesayrs kylesayrs

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kylesayrs @mgoin @dsikka