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

replace Focus with Conv #998

Open
ljmiao opened this issue Dec 16, 2021 · 3 comments
Open

replace Focus with Conv #998

ljmiao opened this issue Dec 16, 2021 · 3 comments
Assignees

Comments

@ljmiao
Copy link

ljmiao commented Dec 16, 2021

It is recommended to replace Focus with conv, because Focus is extremely unfriendly to deployment, and doing so has almost no negative impact on the model, which has been confirmed in yolov5.
link: ultralytics/yolov5#4825

@FateScript
Copy link
Member

Thanks, I will have try. I will inform you If anything updates.

@FateScript
Copy link
Member

Using kernel_size 6x6 in Conv2d is not a good way to replace Focus. Anyway, since so many users are willing to replace Focus with Conv2d, we will try to do such a thing in a different way.

@FateScript FateScript self-assigned this Dec 17, 2021
@scsolaris
Copy link

scsolaris commented Jan 14, 2022

Using kernel_size 6x6 in Conv2d is not a good way to replace Focus. Anyway, since so many users are willing to replace Focus with Conv2d, we will try to do such a thing in a different way.

@FateScript any progress?

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

3 participants