-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Reversible Resize #974
base: main
Are you sure you want to change the base?
Reversible Resize #974
Conversation
@Dipet thanks for the PR! This would be super useful. For the TTA use case we do not need to reverse the images, only the boxes/keypoints. What's the use case for reversing image transforms? Aren't many image transforms non-reversible in any case? |
At the moment, this is just a blank.
We usually use the same function for segmentation masks, this function is only needed for them. |
@Dipet oh got it, ok! The main thing i was worried about was the extra cost in reversing a full image transform, but if only the keypoints are reversed and not the full images that's perfect. |
Yes, only provided data will be reversed. Perhaps I will remove the |
Is this still planned? It would super useful to have this feature. |
+1, this would be great to have |
I tried to implement the interface for Test Time Augmentations (TTA) using
ReplayCompose
API.If some transform could support reverse operation we must implement these functions:
get_reverse_args
- to save all needed parameters to do reverse transformationreverse_image
- to reverse images and masksreverse_bbox
- to reverse bboxesreverse_keypoint
- to reverse keypoints.Example: