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

Is the definition of SeparableConv2d wrong ? #6

Closed
sh1neon opened this issue May 6, 2020 · 1 comment
Closed

Is the definition of SeparableConv2d wrong ? #6

sh1neon opened this issue May 6, 2020 · 1 comment

Comments

@sh1neon
Copy link

sh1neon commented May 6, 2020

hi~, I find that the forward function of SeparableConv2d class is different from Xception paper definition.
In Xception, the order of separable convolution operation should be 1x1 convolution first(pointwise convolution) and then depthwise convolution.
what do you think?

@tstandley
Copy link
Owner

See the last few paragraphs on the second page of their current arxiv paper. Inception does pointwise first. Xception does channelwise first.

This code was designed to use the same weights and architecture of the keras version exactly.

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