-
Notifications
You must be signed in to change notification settings - Fork 240
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
Mypy checks for NNCF common pruning code #2613
base: develop
Are you sure you want to change the base?
Conversation
@vshampor @p-wysocki Hello! Please take a look at my PR and let me know if i would need to do anything more. Thank you :) |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #2613 +/- ##
============================================
- Coverage 91.17% 28.01% -63.17%
============================================
Files 494 494
Lines 45428 45379 -49
============================================
- Hits 41420 12713 -28707
- Misses 4008 32666 +28658
... and 386 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Greeting, @RitikaxShakya! |
@daniil-lyakhov Hello! |
Looks like an environment problem, our internal CI fails with following error:
please try to reinstall nncf and launch torch tests |
@daniil-lyakhov Hello! I reinstalled and run all tests |
Hello! Sorry for a delay, but as the error states your python interpreter has no site packages directory (please see following call https://github.com/openvinotoolkit/nncf/blob/develop/tests/shared/paths.py#L23). Could you please try to run following commands to check your python is working correctly?
|
@daniil-lyakhov Hello! Thank you! There were some packages missing, after running those commands and after
|
@RitikaxShakya , Let's take a look on one of the error:
here you are trying to use
|
Hello @daniil-lyakhov Sorry for so late reply. I made the changes as you suggested seems like its still not passing checks. Also the page for details of NNCF TF precommit (Ubuntu 20.04), NNCF Torch + common precommit are not opening. |
Hi @RitikaxShakya, no problems! Please try to rebase your PR. Besides following error appeared in TF pre-commit
Please try to run pytorch and tensorflow pre-commit locally, as pre-commit works fine on develop branch you should be able to find bugs in your pr |
@RitikaxShakya , do you plan to continue to work on this PR? |
@MaximProshin Hello! sorry for late reply, I was busy with my semester exams, so slowly working on issues, its almost completed, there are few more errors i need to fix. |
hey @RitikaxShakya, will you have a time to finish it? |
Yes, i will try to finish it this week, sorry for being so late. |
thanks! do not worry :) |
@RitikaxShakya please rebase your PR. |
Thank you for informing, and i am trying to complete this issue soon |
@daniil-lyakhov @RitikaxShakya what is the status of this PR? Is it still active? |
Changes
This PR closes issue #2494 by addressing various mypy checks
# type:ignore
for cases requiring significant refactoring due to untyped packages, attributes not defined etc..mypy.ini
to include additional files for type checking.Tests
Pytest
andmake pre-commit
test were run to ensure that the changes did not change any logic and that the codebase remained functional.