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

Fix test which required accelerate, apply style #194

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

kylesayrs
Copy link
Contributor

Purpose

  • Fix test which was failing without accelerate installed

Changes

  • Reuse helper function to avoid passing device_map
  • Apply style

@dsikka
Copy link
Contributor

dsikka commented Oct 22, 2024

Which test was failing, the mse or apply test?
Was it just skipped in the ci?

@kylesayrs
Copy link
Contributor Author

@dsikka Test fails if accelerate is not installed. Small change but nicer experience for users who do the following

pip install compressed-tensors
pytest tests

@kylesayrs kylesayrs merged commit d3dea3f into main Oct 23, 2024
1 check passed
@kylesayrs kylesayrs deleted the kylesayrs/fix-tests-without-accelerate branch October 23, 2024 19:18
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 this pull request may close these issues.

4 participants