Closed ccoffrin closed 2 years ago
Merging #15 (4b61714) into main (c1aa1dc) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## main #15 +/- ##
=======================================
Coverage 94.58% 94.58%
=======================================
Files 5 5
Lines 443 443
=======================================
Hits 419 419
Misses 24 24
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update c1aa1dc...4b61714. Read the comment docs.
@ccoffrin This all looks good to me. The linear schedule is also probably more intuitive for a beginner
@zmorrell after reviewing various example in the literature, I propose to update the canonical examples in the readme to use the linear annealing schedule as this is fairly common in the literature.