8000 Move some variable by xvjiarui · Pull Request #19 · open-mmlab/cocoapi · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Move some variable #19

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 20, 2020
Merged

Move some variable #19

merged 2 commits into from
Sep 20, 2020

Conversation

xvjiarui
Copy link
Collaborator

This PR removed self.img_ann_map and self.cat_img_map from self.createIndex(). So that when a class inherits from COCO, user only needs to take care of self.catToImgs and self.imgToAnns.

@xvjiarui xvjiarui requested a review from hellock September 17, 2020 11:28
@xvjiarui
Copy link
Collaborator Author

@OceanPang

@OceanPang
Copy link

@xvjiarui Can we merge the PR and release a new version of mmpycocotools?

@xvjiarui xvjiarui merged commit 01f805c into master Sep 20, 2020
@xvjiarui
Copy link
Collaborator Author

@OceanPang Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hellock hellock hellock approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xvjiarui @OceanPang @hellock
0