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

Not so sure about the patch embedding kernel size. #16

Open
Asagami-Fujino opened this issue Aug 23, 2023 · 1 comment
Open

Not so sure about the patch embedding kernel size. #16

Asagami-Fujino opened this issue Aug 23, 2023 · 1 comment

Comments

@Asagami-Fujino
Copy link

Asagami-Fujino commented Aug 23, 2023

Thanks a lot for your work,it helps a lot.
In your paper you mentioned that the convolution kernel size is a conbination of 1x14x14 and 14x1x1, but in the code it seems that the kernel size is 1x16x16 and 16x1x1. So which is correct?

@peterant330
Copy link
Collaborator

Thanks a lot for your work,it helps a lot. In your paper you mentioned that the convolution kernel size is a conbination of 1x14x14 and 14x1x1, but in the code it seems that the kernel size is 1x16x16 and 16x1x1. So which is correct?

Hi,

It should be 16. There is a typo in the paper. Thanks for pointing this out.

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