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

Update coco.py #503

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JavierClearImageAI
Copy link

attempt to debug "getImgIds" for multiple categories ids

attempt to debug "getImgIds" for multiple categories ids
@nghiahhnguyen
Copy link

nghiahhnguyen commented Apr 20, 2021

I welcome this change. The original implementation does not get all the corresponding images.

@nghiahhnguyen
Copy link

nghiahhnguyen commented Apr 20, 2021

I think the reason they use &= is that the authors want getImgIds to get the intersection of classes (i.e., each retrieved image must have all the categories) while what we want is the union of classes (i.e., each retrieved image must have at least one category). I think it's more of a misunderstanding than a bug. And I hope that your fix can be a new function instead of modifying this function.

Someone explained it in this issue: #114 (comment)

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.

2 participants