Skip to content

[Collectors] Histogram Aggregator #3626

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.

Sign up for GitHub

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

Jump to bottom
Merged
AlexanderDokuchaev merged 12 commits into from
Oct 2, 2025
Merged

[Collectors] Histogram Aggregator #3626

AlexanderDokuchaev merged 12 commits into from
Oct 2, 2025

Conversation

@daniil-lyakhov
Copy link
Collaborator

@daniil-lyakhov daniil-lyakhov commented Aug 19, 2025 •
edited
Loading

Changes

Histogram aggregator (like HistogramObserver) is introduced in NNCF

Reason for changes

Histogram observer show better results for MinMax quantization of some image classification models (especially for custom quantizers like XNNPACK and etc)

Related tickets

144804

Tests

Conformance tests: manual/job/post_training_quantization/717

  • tests/common/experimental/test_reducers_and_aggregators.py::test_histogramm_aggregator
  • tests/cross_fw/test_templates/test_ptq_params.py is updated to check the brand new PTQ params
  • tests/common/experimental/test_statistic_collector.py is updated to check new MinMaxStatistic.MIN_MAX_STAT attribute

@github-actions github-actions bot added NNCF Common Pull request that updates NNCF Common NNCF PTQ Pull requests that updates NNCF PTQ labels Aug 19, 2025
@daniil-lyakhov daniil-lyakhov force-pushed the dl/histogram_aggregator branch 3 times, most recently from 1806d27 to 0deaf81 Compare August 20, 2025 13:46
@github-actions github-actions bot added NNCF OpenVINO Pull requests that updates NNCF OpenVINO NNCF ONNX Pull requests that updates NNCF ONNX labels Aug 22, 2025
@daniil-lyakhov daniil-lyakhov force-pushed the dl/histogram_aggregator branch 4 times, most recently from edd46fb to 52e919e Compare August 27, 2025 09:21
@daniil-lyakhov daniil-lyakhov force-pushed the dl/histogram_aggregator branch 7 times, most recently from 306b4bd to a19c5b1 Compare September 10, 2025 07:14
@github-actions github-actions bot added the NNCF PT Pull requests that updates NNCF PyTorch label Sep 16, 2025
@daniil-lyakhov daniil-lyakhov marked this pull request as ready for review September 17, 2025 09:07
@daniil-lyakhov daniil-lyakhov requested a review from a team as a code owner September 17, 2025 09:07
AlexanderDokuchaev
AlexanderDokuchaev requested changes Sep 26, 2025
View reviewed changes
@github-actions github-actions bot added the NNCF TF Pull requests that updates NNCF TensorFlow label Sep 29, 2025
@daniil-lyakhov daniil-lyakhov force-pushed the dl/histogram_aggregator branch 2 times, most recently from e2faffc to e008279 Compare September 29, 2025 14:38
@AlexanderDokuchaev AlexanderDokuchaev merged commit 04e6b5b into openvinotoolkit:develop Oct 2, 2025
20 checks passed
@daniil-lyakhov daniil-lyakhov mentioned this pull request Nov 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AlexanderDokuchaev AlexanderDokuchaev AlexanderDokuchaev approved these changes

Assignees

No one assigned

Labels

NNCF Common Pull request that updates NNCF Common NNCF ONNX Pull requests that updates NNCF ONNX NNCF OpenVINO Pull requests that updates NNCF OpenVINO NNCF PT Pull requests that updates NNCF PyTorch NNCF PTQ Pull requests that updates NNCF PTQ NNCF TF Pull requests that updates NNCF TensorFlow

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@daniil-lyakhov @AlexanderDokuchaev