Conversation
Codecov Report
@@ Coverage Diff @@
## development #264 +/- ##
============================================
Coverage 81.66% 81.66%
============================================
Files 151 151
Lines 8646 8646
Branches 1328 1328
============================================
Hits 7061 7061
- Misses 1108 1109 +1
+ Partials 477 476 -1
Continue to review full report at Codecov.
|
ravinkohli
left a comment
There was a problem hiding this comment.
Thanks for this PR. I think it can be merged
nabenabe0928
left a comment
There was a problem hiding this comment.
Thanks for the PR.
I put a small comment.
| class CosineAnnealingWarmRestarts(BaseLRComponent): | ||
| """ | ||
| Set the learning rate of each parameter group using a cosine annealing schedule, | ||
| where \eta_{max}ηmax is set to the initial lr, T_{cur} is the number of epochs |
There was a problem hiding this comment.
Why did you delete \ ?
The usage of the Inline math represetations is nicely described here.
where :math:`\eta_{\max}` is set to the initial lr, :math:`T_{\text{cur}}` is the number of epochs
since the last restart and :math:`T_{i}` is the number of epochs between two warm
There was a problem hiding this comment.
Hello, sorry this was not an intentional change.
My vim configuration changed this. It did so because this should be a raw string, as \e is not a valid escaped sequence.
This PR enables python 3.9