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

Implementation of convolutional projection #1

Closed
leoxiaobin opened this issue Mar 31, 2021 · 7 comments
Closed

Implementation of convolutional projection #1

leoxiaobin opened this issue Mar 31, 2021 · 7 comments

Comments

@leoxiaobin
Copy link

hi, @rishikksh20

Thank you for your quick implementation!

I notice that your depth-wise separable convolution is implemented as depth-wise conv --> point-wise conv. In the paper, CvT's depth-wise seprable convolution is implemented as depth-wise conv --> bn --> point-wise conv.
image

@rishikksh20
Copy link
Owner

@leoxiaobin oh my bad I just reuse my older SepCon2d class in this repo, I will update that class.

@rishikksh20
Copy link
Owner

rishikksh20 commented Mar 31, 2021

Done, do let me know if you find any other vulnerabilities in code.

@huixiancheng
Copy link

@leoxiaobin Hi! When will the official implementation be open source?

@youngwanLEE
Copy link

@leoxiaobin
The official code URL denoted in the paper is broken. Do you have a plan to open the code and model weights?

@leoxiaobin
Copy link
Author

hi, @youngwanLEE, you can get the pre-trained model at https://github.com/microsoft/CvT

@leoxiaobin
Copy link
Author

@leoxiaobin Hi! When will the official implementation be open source?

Please find it at https://github.com/microsoft/CvT

@rishikksh20
Copy link
Owner

Thanks @leoxiaobin I am adding repo on ReadMe.

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

4 participants