Skip to content

Conversation

@daniil-lyakhov
Copy link
Collaborator

@daniil-lyakhov daniil-lyakhov commented Jun 25, 2025

Changes

Ignored patterns are being introduced to the WeightCompression algorithm

Reason for changes

ROPE pattern breaks the logic of the WeightCompression algorithm, required to be automatically ignored

Related tickets

huggingface/optimum-intel#1295
164548

Tests

test_rope_weight_compression

@github-actions github-actions bot added NNCF Common Pull request that updates NNCF Common NNCF PTQ Pull requests that updates NNCF PTQ NNCF OpenVINO Pull requests that updates NNCF OpenVINO NNCF ONNX Pull requests that updates NNCF ONNX labels Jun 25, 2025
@daniil-lyakhov daniil-lyakhov marked this pull request as ready for review June 26, 2025 12:19
@daniil-lyakhov daniil-lyakhov requested a review from a team as a code owner June 26, 2025 12:19
@daniil-lyakhov daniil-lyakhov changed the title WIP[WeightCompression] Ignored patterns introduction [WeightCompression] Ignored patterns introduction Jun 26, 2025
Copy link
Collaborator

@nikita-savelyevv nikita-savelyevv left a comment

Choose a reason for hiding this comment

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

Looks good!

@daniil-lyakhov daniil-lyakhov force-pushed the dl/wc/ignored_patterns branch from fc3ba76 to 1b01a78 Compare July 7, 2025 12:11
@github-actions github-actions bot removed NNCF Common Pull request that updates NNCF Common NNCF ONNX Pull requests that updates NNCF ONNX NNCF PTQ Pull requests that updates NNCF PTQ labels Jul 7, 2025
@daniil-lyakhov
Copy link
Collaborator Author

@AlexanderDokuchaev, I removed my approach and introduced your suggestion, please check

@daniil-lyakhov daniil-lyakhov force-pushed the dl/wc/ignored_patterns branch from 6a309a4 to 4462ef2 Compare July 10, 2025 16:32
@daniil-lyakhov daniil-lyakhov force-pushed the dl/wc/ignored_patterns branch from 4462ef2 to f4ef390 Compare July 10, 2025 16:33
Co-authored-by: Alexander Dokuchaev <[email protected]>
@AlexanderDokuchaev AlexanderDokuchaev merged commit d9fc39a into openvinotoolkit:develop Jul 10, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NNCF OpenVINO Pull requests that updates NNCF OpenVINO

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants