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

[Image] Add bbox support #120

Closed
wants to merge 15 commits into from

Conversation

zpapakipos
Copy link
Contributor

@zpapakipos zpapakipos commented Sep 16, 2021

Summary: Added support for passing in bounding boxes to all image augmentations to see how the bboxes will be transformed by the augmentation.

Differential Revision: D30991086

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Sep 16, 2021
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D30991086

@zpapakipos zpapakipos changed the title Remove bboxes from fb augmentations [Image] Add bbox support Sep 17, 2021
zpapakipos pushed a commit to zpapakipos/AugLy that referenced this pull request Sep 17, 2021
Summary:
Pull Request resolved: facebookresearch#120

Defines BaseTransform separately for `fb/` augmentations so we don't need to add bbox support (until requested by an internal user at some point). Also defines a separate `base_unit_test.py` that doesn't pass in or expect bbox args.

Differential Revision: D30991086

fbshipit-source-id: ff516f181a9649f6a21c5628cf1031c09c5bd4c2
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D30991086

zpapakipos pushed a commit to zpapakipos/AugLy that referenced this pull request Sep 20, 2021
Summary:
Pull Request resolved: facebookresearch#120

Defines BaseTransform separately for `fb/` augmentations so we don't need to add bbox support (until requested by an internal user at some point). Also defines a separate `base_unit_test.py` that doesn't pass in or expect bbox args.

Reviewed By: jbitton

Differential Revision: D30991086

fbshipit-source-id: 722de4491c1330b6673fbc1551435cc0a042c986
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D30991086

zpapakipos pushed a commit to zpapakipos/AugLy that referenced this pull request Sep 20, 2021
Summary:
Pull Request resolved: facebookresearch#120

Defines BaseTransform separately for `fb/` augmentations so we don't need to add bbox support (until requested by an internal user at some point). Also defines a separate `base_unit_test.py` that doesn't pass in or expect bbox args.

Reviewed By: jbitton

Differential Revision: D30991086

fbshipit-source-id: 0b7cc459e302b17e64a6c9957edf90fbbdbd7c42
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D30991086

zpapakipos pushed a commit to zpapakipos/AugLy that referenced this pull request Sep 20, 2021
Summary:
Pull Request resolved: facebookresearch#120

Defines BaseTransform separately for `fb/` augmentations so we don't need to add bbox support (until requested by an internal user at some point). Also defines a separate `base_unit_test.py` that doesn't pass in or expect bbox args.

Reviewed By: jbitton

Differential Revision: D30991086

fbshipit-source-id: c140c6d3ba083e6573bc46982a5e8880e7227725
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D30991086

Zoe Papakipos added 13 commits September 20, 2021 07:10
Differential Revision: D30639160

fbshipit-source-id: 40519dc0c670a0684c339119341b9bcfae8300c6
Differential Revision: D30573948

fbshipit-source-id: 7ae0f3f45d8c41fc79f22cebcd78d873ee811a63
Differential Revision: D30575975

fbshipit-source-id: 6ff5ba8f8dab54d257a598d54695b2b3d5b91024
Differential Revision: D30638997

fbshipit-source-id: a77170f1320e2a65ffc8bd8225d0dcf5d8b0bbac
Differential Revision: D30639851

fbshipit-source-id: c99745e059d7cd0d61a9ec6ab1b39021e687cf06
Differential Revision: D30640755

fbshipit-source-id: d1988f95501c5dfd8536e36f7bf0926e213b5ac4
Differential Revision: D30641283

fbshipit-source-id: faf1af7348762efd875bd55f0327b7fb784eb6ee
Differential Revision: D30666857

fbshipit-source-id: 06e53f4608b8726274b5ad2d1987af4795f0915c
Differential Revision: D30667422

fbshipit-source-id: 639ba07b602d98b5f891152b58623f0bdeb18fd3
Differential Revision: D30696182

fbshipit-source-id: 38b04d6b1fcd5adcf1ad45cd37a86dc7ba1a4f38
Differential Revision: D30698243

fbshipit-source-id: 2c9e77c68b5641da6d3f1c9d8dba59eef5f70c71
Differential Revision: D30726817

fbshipit-source-id: 7ae25a60830b6289b1df8719036d4f8d791a37ec
Differential Revision: D30766662

fbshipit-source-id: cb855c2bf9bd696ee7a1e233b1a0d7e502d0c4ae
Zoe Papakipos added 2 commits September 20, 2021 07:10
Differential Revision: D31015613

fbshipit-source-id: 4f22c89426ee22e22c6c685ef64a72fa4969501e
Summary:
Pull Request resolved: facebookresearch#120

Defines BaseTransform separately for `fb/` augmentations so we don't need to add bbox support (until requested by an internal user at some point). Also defines a separate `base_unit_test.py` that doesn't pass in or expect bbox args.

Reviewed By: jbitton

Differential Revision: D30991086

fbshipit-source-id: 85227eb7005f482f6b6ef5ed7f2b960d87c980aa
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D30991086

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 67db232.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants