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 openvino/test_training.py #548

Merged
merged 2 commits into from
Feb 5, 2024

Conversation

daniil-lyakhov
Copy link
Contributor

@daniil-lyakhov daniil-lyakhov commented Feb 1, 2024

What does this PR do?

  • Ignored scope is updated for CUSTOMIZED_QUANTIZATION_CONFIG
  • Number of expected quantizers is decreased for the impacted cases.

Reasons:

After nncf update (2.7.0 -> 2.8.0), NNCFGraph was fixed for the Bert model: now head masks path is visible for the quantization algorithm:
Screenshot 2024-02-01 121631
This led to head masks path quantization:
Screenshot 2024-02-01 115618
I've updated Ignored scope with IS from the default config, and now amount of quantizers was decreased.

CC: @AlexKoff88

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@AlexKoff88
Copy link
Collaborator

It looks good and fixes a subset of tests. Let's merge. The rest of them are fixed in #538 and #535.

@AlexKoff88 AlexKoff88 merged commit 5120f75 into huggingface:main Feb 5, 2024
7 of 10 checks passed
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.

3 participants