-
Notifications
You must be signed in to change notification settings - Fork 246
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
Add logs for user guiding better #2523
Conversation
Greetings, @UsingtcNower! Thank you for your interest in the NNCF Filter Pruning algorithm! I've checked your changes, and it looks like this warning is ambiguous: this information is present in the pruning statistics. Please check the filter pruning statistics for identity pruning masks and confirm it shows that pruning masks are identity. In case you are using pruning algorithm in your custom pipeline, you can print statistics using following code:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2523 +/- ##
============================================
+ Coverage 46.73% 84.63% +37.90%
============================================
Files 498 498
Lines 46178 45540 -638
============================================
+ Hits 21579 38541 +16962
+ Misses 24599 6999 -17600
... and 350 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Thanks for your reply @daniil-lyakhov !
|
@UsingtcNower, could you please attach NNCF logs with this warning turned on? |
@daniil-lyakhov , sure! Here I attach the NNCF logs. |
@UsingtcNower Thanks for the contribution! |
Changes
Reason for changes
Related tickets
Tests