-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Simplified if-expression. #267
Conversation
Codecov Report
@@ Coverage Diff @@
## master #267 +/- ##
==========================================
+ Coverage 83.03% 83.09% +0.05%
==========================================
Files 112 112
Lines 6448 6441 -7
Branches 1029 1029
==========================================
- Hits 5354 5352 -2
+ Misses 901 896 -5
Partials 193 193
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -235,7 +235,7 @@ def __init__(self, | |||
|
|||
self.layers = nn.ModuleList() | |||
for i, num_blocks in enumerate(self.stage_blocks): | |||
first_block = True if i == 0 else False | |||
first_block = bool(i == 0) |
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.
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.
Modified.
74df57e
to
724da6f
Compare
* Simplified if-expression. * Simplified following mentor's convention.
* Simplified if-expression. * Simplified following mentor's convention.
* Simplified if-expression. * Simplified following mentor's convention.
* impl lr and momentum visualizer * provide fakerun
* Simplified if-expression. * Simplified following mentor's convention.
Fix #247