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

[PTQ] Remove logs from autogenerated ignored node names #2815

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

kshpv
Copy link
Collaborator

@kshpv kshpv commented Jul 16, 2024

Changes

From:

The first log is printed from the user Ignored Scope.
The second and the third ones are from the inner algorithms source and should not be printed.

image

To:

image

Reason for changes

Improve UX by removing logs from autogenerated ignored node names.

Related tickets

N/A

Tests

Manually on a provided root cause

@kshpv kshpv requested a review from a team as a code owner July 16, 2024 14:05
@github-actions github-actions bot added NNCF PT Pull requests that updates NNCF PyTorch NNCF OpenVINO Pull requests that updates NNCF OpenVINO NNCF ONNX Pull requests that updates NNCF ONNX NNCF PTQ Pull requests that updates NNCF PTQ labels Jul 16, 2024
@kshpv kshpv requested a review from alexsu52 July 16, 2024 14:06
Copy link
Contributor

@alexsu52 alexsu52 left a comment

Choose a reason for hiding this comment

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

LGTM

@alexsu52 alexsu52 merged commit 60000e4 into openvinotoolkit:develop Jul 23, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants