level_5 module

[DxqS]

which one is right？

[Jam-G]

@DxqS The code you shown in the second picture, is pytorch implementation of PyNET，and the tensorflow implementation of PyNET is same to the description in papers.

And I checked the pytorch implementation of PyNET ， the autor @aiff22 modified the model exactly and declared three major modifications.
But he didn't mention the modifications about dropping the skip connections of level5.
If @aiff22 have tried such modifications and it has better effects？ or just omitted?
It confused me too.

[aiff22]

Hi @DxqS, @Jam-G,

The right one is in the paper and in this repository. In the PyTorch repository, it was just omitted by mistake. You can add skip-connections in the 5th level by changing the above code to:

 def level_5(self, pool4):

        conv_l5_d1 = self.conv_l5_d1(pool4)
        conv_l5_d2 = self.conv_l5_d2(conv_l5_d1) + conv_l5_d1
        conv_l5_d3 = self.conv_l5_d3(conv_l5_d2) + conv_l5_d2
        conv_l5_d4 = self.conv_l5_d4(conv_l5_d3)

        conv_t4a = self.conv_t4a(conv_l5_d4)
        conv_t4b = self.conv_t4b(conv_l5_d4)

        conv_l5_out = self.conv_l5_out(conv_l5_d4)
        output_l5 = self.output_l5(conv_l5_out)

        return output_l5, conv_t4a, conv_t4b