Skip to content

Cleanup of prototype transforms #6492

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

Merged
merged 14 commits into from
Aug 25, 2022
Merged

Conversation

pmeier
Copy link
Collaborator

@pmeier pmeier commented Aug 25, 2022

Each commit in this PR addresses some points flagged in #6486.

@pmeier pmeier marked this pull request as ready for review August 25, 2022 12:25
@pmeier pmeier requested review from datumbox and vfdev-5 and removed request for datumbox August 25, 2022 12:25
Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

All changes look good. I'll wait for the update on AutoAugment as discussed offline.

@pmeier pmeier requested a review from datumbox August 25, 2022 13:23
Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

LGTM, just some minor optional nits.

def forward(self, *inputs: Any) -> Any:
sample = inputs if len(inputs) > 1 else inputs[0]

id, image = self._extract_image(sample)
Copy link
Contributor

Choose a reason for hiding this comment

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

Rant: Not massive fan of using image naming here but we can change once we introduce videos.

Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

Still LGTM, thanks!

@datumbox datumbox merged commit becaba0 into pytorch:main Aug 25, 2022
@pmeier pmeier deleted the proto-transform-cleanup branch August 26, 2022 06:38
facebook-github-bot pushed a commit that referenced this pull request Aug 30, 2022
Summary:
* fix passtrough on transforms and add dispatchers for five and ten crop

* Revert "cleanup prototype auto augment transforms (#6463)"

This reverts commit d8025b9.

* use legacy kernels in deprecated Grayscale and RandomGrayscale transforms

* fix default type for Lambda transform

* fix default type for ToDtype transform

* move simple_tensor to features module

* [skip ci]

* Revert "move simple_tensor to features module"

This reverts commit 7043b6e.

* cleanup

* reinstate valid AA changes

* address review

* Fix linter

Reviewed By: NicolasHug

Differential Revision: D39131014

fbshipit-source-id: 0237a0e2a8256cf7ec5f5bc3b529e471c465ea04

Co-authored-by: Vasilis Vryniotis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants