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

DW_Conv #5

Open
Jinchen2028 opened this issue Jul 18, 2023 · 2 comments
Open

DW_Conv #5

Jinchen2028 opened this issue Jul 18, 2023 · 2 comments

Comments

@Jinchen2028
Copy link

Thank you very much for sharing your great work, in the paper I noticed that DW_Conv were used in the SAM module. If I use vanilla convolution instead of DW_Conv, will it have a performance improvement in addition to growing parameters?

@AFeng-x
Copy link
Owner

AFeng-x commented Jul 18, 2023

Thank you for your interest! We initially attempted to use vanilla convolution, but we found that it led to a larger number of parameters in the module. As a result, we ultimately opted for depthwise convolution. Theoretically, vanilla convolution has stronger modeling capabilities than depthwise convolution and can potentially improve accuracy. However, we are uncertain about the extent of this improvement in practice. Therefore, we suggest that you run an experiment to see for yourself, and we welcome you to share the results with us. Thank you!

@Jinchen2028
Copy link
Author

Thank you so much for your answer!

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