Skip to content

Remove clear_ml#1261

Merged
dsikka merged 1 commit intomainfrom
rm_clear_ml
Mar 17, 2025
Merged

Remove clear_ml#1261
dsikka merged 1 commit intomainfrom
rm_clear_ml

Conversation

@dsikka
Copy link
Copy Markdown
Collaborator

@dsikka dsikka commented Mar 17, 2025

SUMMARY:

  • Remove requirement for tokens and the one test which uses them

@dsikka dsikka added the ready When a PR is ready for review label Mar 17, 2025
@github-actions
Copy link
Copy Markdown

👋 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.

Copy link
Copy Markdown
Collaborator

@andy-neuma andy-neuma left a comment

Choose a reason for hiding this comment

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

thanks!

Copy link
Copy Markdown
Collaborator

@brian-dellabetta brian-dellabetta left a comment

Choose a reason for hiding this comment

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

🧹

@dsikka dsikka enabled auto-merge (squash) March 17, 2025 18:52
@andy-neuma
Copy link
Copy Markdown
Collaborator

putting comment here as well. we'll need to remove usage of tj-actions/changed-files@v45 as well.

@dsikka dsikka disabled auto-merge March 17, 2025 19:27
@dsikka dsikka merged commit cf6a304 into main Mar 17, 2025
8 of 9 checks passed
@dsikka dsikka deleted the rm_clear_ml branch March 17, 2025 19:27
dsikka pushed a commit that referenced this pull request Mar 18, 2025
## Purpose ##
* Complete the removal of clear_ml #1261 by deleting the clear_ml test
from the transformers gha

```
============================= test session starts ==============================
platform linux -- Python 3.9.21, pytest-8.3.5, pluggy-1.5.0 -- /home/runner/_work/_tool/Python/3.9.21/x64/bin/python
cachedir: .pytest_cache
rootdir: /home/runner/_work/llm-compressor/llm-compressor
configfile: pyproject.toml
plugins: mock-3.14.0, rerunfailures-15.0
ERROR: file or directory not found: tests/llmcompressor/transformers/test_clear_ml.py
collecting ... collected 0 items
```

Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>
kylesayrs pushed a commit that referenced this pull request Mar 27, 2025
SUMMARY:
- Remove requirement for tokens and the one test which uses them

Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>
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.

4 participants