search

liyunsheng13 / BDL

MIT License
222 stars 29 forks source link

Bug in the DeepLab ResNet101 atrous convolution forward function #41

Closed akshaykulkarni07 closed 4 years ago

akshaykulkarni07 commented 4 years ago

In this file, line numbers 115 to 119,

    def forward(self, x):
        out = self.conv2d_list[0](x)
        for i in range(len(self.conv2d_list) - 1):
            out += self.conv2d_list[i + 1](x)
            return out

The return statement is inside the for loop, instead of outside the for loop.

liyunsheng13 commented 4 years ago

Thanks for letting me know. I have fixed it.

akshaykulkarni07 commented 4 years ago

Could you also tell me whether this bug affects your paper results? Because atrous convolutions are important to get good segmentation outputs.

liyunsheng13 commented 4 years ago

I don't think it will influence the result a lot. At least removing the bug won't worsen the result.