Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Question about the baseline model #27

Closed
jqsun98 opened this issue Apr 2, 2022 · 1 comment
Closed

Question about the baseline model #27

jqsun98 opened this issue Apr 2, 2022 · 1 comment

Comments

@jqsun98
Copy link

jqsun98 commented Apr 2, 2022

Dear Authors,
Congratulations to you for the paper has been acepted as Oral in CVPR 2022.
Here, I have a question about the code.
In Section 4.5 'Ablation Studies', Table.7 compares the performance of baseline, multi-head attention, FFN and overall model. The baseline's component is Unet with Resblocks (EDSR model). Is the number of blocks [4, 6, 6, 8] in overall network same as the number of blocks in baseline model? And where can I find the baseline model code? Is that in 'https://github.com/xinntao/BasicSR/blob/master/basicsr/archs/edsr_arch.py'?

@swz30
Copy link
Owner

swz30 commented Apr 4, 2022

Hi @cambridgeinch,

Yes, the number of blocks is [4,6,6,8] in the baseline as well; but we set dim=40, as the regular Convs are heavier. Here is the code snippet for Resblock:

class Resblock(nn.Module):
    def __init__(self, dim, bias):
        super(Resblock, self).__init__()

        self.body = nn.Sequential(nn.Conv2d(dim, dim, kernel_size=3, stride=1, padding=1, bias=False),
                                  nn.LeakyReLU(0.2),
                                  nn.Conv2d(dim, dim, kernel_size=3, stride=1, padding=1, bias=False)
                                 )

    def forward(self, x):
        x = x + self.body(x)
        return x

@swz30 swz30 closed this as completed Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants