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

why lconv and cconv in SPEC do not have padding value? #13

Closed
skidsaint opened this issue May 28, 2020 · 2 comments
Closed

why lconv and cconv in SPEC do not have padding value? #13

skidsaint opened this issue May 28, 2020 · 2 comments

Comments

@skidsaint
Copy link

Thanks for the great work and code!
I want to understand the details of the SEPC so I check the code for the details of the new module. I found that in line37 of the spec.py the code is
self.lconv = spec_conv(256, 256, kernel_size=3, dilation=1, part_deform=lcconv_deform)
The padding value is seem not specialized here. So my question is whether the padding is zero in lconv and cconv? If it is true, padding=0 in lconv and cconv is important for SEPC?

@jshilong
Copy link
Owner

I'm sorry for the confusion, It's just my fault. It's not important for the final results, But you may need to train It again for padding=1

@skidsaint
Copy link
Author

Thanks for your reply !

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