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

修复rtmdet-ins tiny没有开启recompute_bbox导致bbox和mask对不上的bug #10349

Open
wants to merge 1 commit into
base: dev-3.x
Choose a base branch
from

Conversation

zylo117
Copy link
Contributor

@zylo117 zylo117 commented May 18, 2023

Motivation

害,搞了很久,原来是你们的bug,l m s x都沿用的l的配置,唯独rtmdet-ins tiny是没有开启recompute_bbox的!没有开启就不会根据mask重新调整bbox的位置,和bbox对不上还怎么训练。好奇你们当初是怎么用错误的配置训练出来的?

Modification

开启rtmdet-ins tiny的recompute_bbox

@mm-assistant
Copy link

mm-assistant bot commented May 18, 2023

We recommend using English or English & Chinese for pull requests so that we could have broader discussion.

@zylo117
Copy link
Contributor Author

zylo117 commented May 18, 2023

哦,你们只是stage1写错了,所以stage2时还是沿用的l的配置,在stage2实例分割部分还是会自己修复回来。是不是说修复了配置后rtmdet-ins tiny的map可以更高呢?

@hhaAndroid
Copy link
Collaborator

哦,你们只是stage1写错了,所以stage2时还是沿用的l的配置,在stage2实例分割部分还是会自己修复回来。是不是说修复了配置后rtmdet-ins tiny的map可以更高呢?

是的,其实我们已经发现了,并且在修复,性能都会涨,但是因为一些其他事情所以一直没有发布

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

Successfully merging this pull request may close these issues.

3 participants