-
Notifications
You must be signed in to change notification settings - Fork 577
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
Update has_mask method for mmdet models #1054
Conversation
…slicing COCO Solution to obss#1011 by adding a filter for Polygon only
Ummm, probably the CI test ran on an old style of mmdet config, leading to an error. I will try to make that compatible |
The test was indeed on mmdet v2.0, how do you like to change this? |
@Alias-z you can drop the support for mmdet 2.0 and update it with mmdet 3.0 models/configs |
Well, the CI test is tough ... @fcakyon I initially used mmengine to combine all config inheritances into one config for the three test models; then I realized that there would many more code to modify. So i reverted back to current CI test configs, while updated Tested the new
So it should work now... |
@Alias-z a very elegant solution 💯 |
Solve the issue mentioned in #1053, which leads to empty instance segmentation results with mmdet models