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

[Feature] Add annotation conversion tool from Labelbee to COCO #1306

Open
wants to merge 1 commit into
base: dev-0.26
Choose a base branch
from

Conversation

wanghao-007
Copy link

@wanghao-007 wanghao-007 commented Apr 9, 2022

Motivation

Modification

BC-breaking (Optional)

Use cases (Optional)

Checklist

Before PR:

  • I have read and followed the workflow indicated in the CONTRIBUTING.md to create this PR.
  • Pre-commit or linting tools indicated in CONTRIBUTING.md are used to fix the potential lint issues.
  • Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
  • New functionalities are covered by complete unit tests. If not, please add more unit tests to ensure correctness.
  • The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

  • CLA has been signed and all committers have signed the CLA in this PR.

@CLAassistant
Copy link

CLAassistant commented Apr 9, 2022

CLA assistant check
All committers have signed the CLA.

@ly015 ly015 changed the base branch from master to dev-0.26 April 9, 2022 08:38
@ly015 ly015 changed the title add labelbee annotation conversion tool[WIP] [Feature] Add annotation conversion tool from Labelbee to COCO Apr 9, 2022
@ly015 ly015 marked this pull request as draft April 9, 2022 08:39
@ly015 ly015 marked this pull request as ready for review April 9, 2022 08:40
@ly015 ly015 self-requested a review April 9, 2022 08:41
@codecov
Copy link

codecov bot commented Apr 14, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!


def main():
parser = argparse.ArgumentParser()
parser.add_argument('--input-file', type=str, default='/home/wanghao/Work/labelbee_json_2_MMPose_COCO_json/', help='输入labelbee json的根路径')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make the default path anonymous.



if __name__=='__main__':
main()
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR has some linting problems. Please follow
https://github.com/open-mmlab/mmpose/blob/master/.github/CONTRIBUTING.md to setup pre-commit.

import argparse

'''
将 labelbee多步标注 后的关键点数据进行标签转化,转至mmpose模型需要的coco格式。 数据集先划分好。
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ly015 Do we need an English version of the docstring?

"id":1,
"name":"Attribution-NonCommercial-ShareAlike License"}]

anno_info["categories"] = [{"supercategory":"person",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that only COCO 17-kpt body pose definition is supported. The generality is a little bit concerning.

@OpenMMLab-Assistant003
Copy link

Hi @wanghao-007!We are grateful for your efforts in helping improve this open-source project during your personal time.
Welcome to join OpenMMLab Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us :https://discord.gg/UjgXkPWNqA
If you have a WeChat account,welcome to join our community on WeChat. You can add our assistant :openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends:)
Thank you again for your contribution❤ @wanghao-007

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.

None yet

4 participants