Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Fix] Avoid mapping unnecessary key to albu format #1773

Merged
merged 1 commit into from
Oct 27, 2022

Conversation

liqikai9
Copy link
Collaborator

Motivation

Same as #1716

img_v2_2b7b0727-de34-4743-b7cb-1b6091936e8g

The original results of data info may contain the key like keypoints which may trigger unexpected functions and raise error in albu like apply_to_keypoint.

Modification

Modify mmpose/datasets/transforms/common_transforms.py, only map those keys that are specified in keymap_to_albu to albu format.

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.

@ly015 ly015 merged commit 33ecde2 into open-mmlab:dev-1.x Oct 27, 2022
ly015 pushed a commit to ly015/mmpose that referenced this pull request Feb 21, 2023
shuheilocale pushed a commit to shuheilocale/mmpose that referenced this pull request May 6, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants