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

Update TF QAT docs. Deprecate TF create_compressed_model method #3217

Merged
merged 19 commits into from
Feb 5, 2025

Conversation

andrey-churkin
Copy link
Contributor

Changes

  • Update TF QAT docs
  • Deprecate the create_compressed_model() method for the TF backend.

Reason for changes

Ref: 158980

Related tickets

Ref: 158980

Tests

N/A

@andrey-churkin andrey-churkin requested a review from a team as a code owner January 28, 2025 10:38
@github-actions github-actions bot added documentation Improvements or additions to documentation NNCF TF Pull requests that updates NNCF TensorFlow labels Jan 28, 2025
@daniil-lyakhov
Copy link
Collaborator

daniil-lyakhov commented Jan 28, 2025

nncf/tensorflow/helpers/model_creation.py Outdated Show resolved Hide resolved
@github-actions github-actions bot added the NNCF PT Pull requests that updates NNCF PyTorch label Jan 30, 2025
Comment on lines 109 to 113
data = {
"quantization": {
"quantizer_setup": TFQuantizationSetup.from_state(self.config["quantizer_setup"]).get_state(),
}
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is necessary to unwrap TensorFlow wrappers for lists and dictionaries into built-in types.

@andrey-churkin
Copy link
Contributor Author

@alexsu52 Please review

@andrey-churkin
Copy link
Contributor Author

andrey-churkin commented Jan 31, 2025

nncf/tensorflow/utils/state.py Outdated Show resolved Hide resolved
nncf/tensorflow/utils/state.py Outdated Show resolved Hide resolved
nncf/tensorflow/utils/state.py Outdated Show resolved Hide resolved
nncf/tensorflow/quantization/quantize_model.py Outdated Show resolved Hide resolved
@alexsu52 alexsu52 merged commit e4c6678 into openvinotoolkit:develop Feb 5, 2025
17 checks passed
alexsu52 pushed a commit to alexsu52/nncf that referenced this pull request Feb 5, 2025
…vinotoolkit#3217)

### Changes

- Update TF QAT docs
- Deprecate the create_compressed_model() method for the TF backend.

### Reason for changes

Ref: 158980

### Related tickets

Ref: 158980

### Tests

N/A
KodiaqQ pushed a commit that referenced this pull request Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation NNCF PT Pull requests that updates NNCF PyTorch NNCF TF Pull requests that updates NNCF TensorFlow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants