CSP-pedestrian-detection-in-pytorch icon indicating copy to clipboard operation
CSP-pedestrian-detection-in-pytorch copied to clipboard

Parameter initialization is wrong.

Open ljpadam opened this issue 6 years ago • 1 comments

In the network.py:

nn.init.constant_(self.pos_conv.bias, 0)
nn.init.constant_(self.reg_conv.bias, -math.log(0.99/0.01))

This causes a large loss at the beginning.

These two lines should be amended as:

nn.init.constant_(self.pos_conv.bias,  -math.log(0.99/0.01))
nn.init.constant_(self.reg_conv.bias, 0)

ljpadam avatar Jun 19 '19 14:06 ljpadam

yes, you are right. But a very interesting thing is the wrong initialization doesn't harm the result, though the loss is large at the beginning. This is a finding in my experiment, which is a little bit wired.

lwpyr avatar Sep 30 '19 07:09 lwpyr