-
-
Notifications
You must be signed in to change notification settings - Fork 16.6k
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
Two Successive Linear Layer is same with one linear layer. #5580
Comments
@developer0hye that's a good question, but I can't answer that myself since @dingyiwei is the C3TR author in PR# #2333. I know 2 convolution2d layers in series are reducible to 1 if it were not for the activation layer in between, does the same hold true for fully connected layers with no activation in between? |
If the kernel size of conv2d layer is one, your statement is true! |
Hi @developer0hye , thanks to point out this issue! It should be an activation between the two successive linear layer. I'll run experiments on that and post results here. |
👋 Hello, this issue has been automatically marked as stale because it has not had recent activity. Please note it will be closed if no further activity occurs. Access additional YOLOv5 🚀 resources:
Access additional Ultralytics ⚡ resources:
Feel free to inform us of any other issues you discover or feature requests that come to mind in the future. Pull Requests (PRs) are also always welcomed! Thank you for your contributions to YOLOv5 🚀 and Vision AI ⭐! |
I'm waiting your experiments. |
In this comment I mentioned why I removed the activation function. |
Search before asking
Question
Hi!, @glenn-jocher and @dingyiwei !
I am trying to improve the performance of object detection model with transformer layer. And I found that @dingyiwei already experimented it on this project.
I have one question about your method.
yolov5/models/common.py
Line 66 in 79bca2b
Why do you design the transformer layer with two successive linear layer?
Two Successive Linear Layer is same with one linear layer theorectically.
Additional
No response
The text was updated successfully, but these errors were encountered: