-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
[Transforms] ApplySomeOf #7544
Comments
Thanks for the suggestion @marcown . I think we should be able to support this functionality by extending
(We can't have the default be 1 because that would allow zero transforms to be applied, which would break BC by default). If we want we can also allow |
+1 for extending
@marcown could you link the paper / reference implementation for this? Is |
Adding it to RandomChoice sounds good to me, with the options None (pick one) and N (pick a subset of maximum size N). Furthermore, applying zero transformations is useful I think, randomly having images that are near the real world use case is quite helpful for training I don't know any paper that did a large scale test of this function, I just used it a lot in the imgaug library: https://github.com/aleju/imgaug |
Can I help with submitting the PR for this new feature? |
Thanks for proposing to help @ambujpawar. Feel free to submit a PR. Let's start by including this feature in
Something I missed earlier is that the new |
Thanks for the heads up. I will work on it and let you know if I encounter some issues |
🚀 The feature
Hey,
I found it useful to have a someof class e.g. in imgaug to randomly apply a subset of transformations out of a set of transformation with a parameter "max_transforms". The number of transformations in the subset is randomly sampled with random.randint(0, max_number).
I could not found something similar in the repo so here is an example class for v2:
What do you think? Could this be integrated? Or did I miss a similar functionality?
Motivation, pitch
Imgaug has it. It is quite useful.
Alternatives
No response
Additional context
No response
cc @vfdev-5
The text was updated successfully, but these errors were encountered: