-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
2 changed files
with
4 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
# Some remarks | ||
|
||
* Han et al. 2020 uses 1.0 for alpha3 in the article however in the code a alpha3 of 0.1 is found. | ||
* Han et al. 2020 uses 50 for steps_per_update while other authors use 100. It is a bit strange that Han collects 100 steps and only performs 50 SGD updates. | ||
* Han et al. 2020 uses 50 for steps\_per\_update while other authors use 100. It is a bit strange that Han collects 100 steps and only performs 50 SGD updates. |