Fix issue where predefined ignore layers incorrectly apply to GGUF#1340
Fix issue where predefined ignore layers incorrectly apply to GGUF#1340wenhuach21 merged 16 commits intomainfrom
Conversation
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Pull request overview
This PR fixes an issue related to predefined GGUF configuration by deferring the application of predefined ignore layers and auto-scheme generation until after the export format is determined.
Changes:
- Moved predefined ignore layers logic from
__init__toconfigure_layer_configwith GGUF format check - Deferred auto-scheme generation to occur in
configure_layer_configinstead of during initialization - Removed unused imports (
os, type hintsAny,Dict,List,Tuple, andcopy_python_files_from_model_cache)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| auto_round/compressors/utils.py | Removed unused imports and added blank lines |
| auto_round/compressors/base.py | Moved predefined ignore layers logic and auto-scheme generation from __init__ to configure_layer_config method, added orig_scheme field to preserve original scheme |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
for more information, see https://pre-commit.ci
…ix_0126 Signed-off-by: Wenhua Cheng <wenhua.cheng@intel.com>
for more information, see https://pre-commit.ci
wenhuach21
changed the title
Description
Please briefly describe your main changes, the motivation.
Type of Change
Related Issues
Fixes or relates to #
Checklist Before Submitting