-
Notifications
You must be signed in to change notification settings - Fork 7.1k
Added annotation typing to googlenet #2858
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2858 +/- ##
=======================================
Coverage 73.36% 73.37%
=======================================
Files 99 99
Lines 8787 8788 +1
Branches 1387 1387
=======================================
+ Hits 6447 6448 +1
Misses 1916 1916
Partials 424 424
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor nitpicks. Given that we do the PRs incrementally, I didn't marked missing -> None
on the constructors. Please add them everywhere. Otherwise LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
* style: Added annotation typing for googlenet * fix: Removed duplicate typing * refactor: Moved factory function after class definition to fix typing * fix: Fixed annotation typing * refactor: Removed un-necessary import * fix: Fixed typing * refactor: Moved back up helper function and quote typed it
* style: Added annotation typing for googlenet * fix: Removed duplicate typing * refactor: Moved factory function after class definition to fix typing * fix: Fixed annotation typing * refactor: Removed un-necessary import * fix: Fixed typing * refactor: Moved back up helper function and quote typed it
Hi there!
As per #2025, annotation typing are welcome in torchvision. So, this PR focuses on
torchvision.models.googlenet
this PR!Any feedback is welcome!