Skip to content

switched bias initialization to initialize to zeros per standard Xavi…#358

Open
eknag wants to merge 1 commit into
facebookresearch:mainfrom
eknag:dev/eknag/fix_dlrm_mlp_bias_initialization
Open

switched bias initialization to initialize to zeros per standard Xavi…#358
eknag wants to merge 1 commit into
facebookresearch:mainfrom
eknag:dev/eknag/fix_dlrm_mlp_bias_initialization

Conversation

@eknag
Copy link
Copy Markdown

@eknag eknag commented Sep 25, 2023

…er initialization. See pdf page 251 in https://proceedings.mlr.press/v9/glorot10a/glorot10a.pdf. As set before, the bias may be initialized to a large negative value, leading to a negative input to Relu. This prevents any training, as all derivatives are zero afterward.

See downstream issue here: pytorch/benchmark#1927

…er initialization. See pdf page 251 in https://proceedings.mlr.press/v9/glorot10a/glorot10a.pdf. As set before, the bias may be initialized to a large negative value, leading to a negative input to Relu. This prevents any training, as all derivatives are zero after
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants