Skip to content

Update test_oneshot_and_finetune.py to use pytest.approx#1339

Merged
kylesayrs merged 6 commits intomainfrom
sparsity-tests-approx-fixes
Apr 22, 2025
Merged

Update test_oneshot_and_finetune.py to use pytest.approx#1339
kylesayrs merged 6 commits intomainfrom
sparsity-tests-approx-fixes

Conversation

@markurtz
Copy link
Copy Markdown
Collaborator

@markurtz markurtz commented Apr 9, 2025

SUMMARY:
Update to better surface errors and better adhere to pytest best practices

TEST PLAN:
Automated testing

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2025

👋 Hi! Thank you for contributing to llm-compressor. Please add the ready label when the PR is ready for review.

Note: This is required to complete the testing suite, please only add the label once the PR is code complete and local testing has been performed.

…rface errors and better adhere to pytest best practices

Signed-off-by: Mark Kurtz <mark.j.kurtz@gmail.com>
@markurtz markurtz force-pushed the sparsity-tests-approx-fixes branch from 5f2149a to 4a276f4 Compare April 9, 2025 14:17
@markurtz markurtz changed the title Update test_oneshot_and_finetune.py to use pytest.approx to better surface errors and better adhere to pytest best practices Update test_oneshot_and_finetune.py to use pytest.approx Apr 9, 2025
@markurtz markurtz added the ready When a PR is ready for review label Apr 9, 2025
@markurtz markurtz self-assigned this Apr 9, 2025
@kylesayrs kylesayrs enabled auto-merge (squash) April 22, 2025 21:57
@kylesayrs kylesayrs merged commit 1f2b796 into main Apr 22, 2025
8 checks passed
@kylesayrs kylesayrs deleted the sparsity-tests-approx-fixes branch April 22, 2025 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready When a PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants