-
Notifications
You must be signed in to change notification settings - Fork 420
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] Extends the list of supported data augmentations #730
Comments
@frgfm only @SiddhantBahuguna PR left or do you have other augmentations in mind ? :) |
For now, I think the random perspective would be the last one required yeah 👍 |
Hi @SiddhantBahuguna any way in near future that you finish your draft PR ? :) |
Greetings @felixdittrich92 , I am really sorry for the unwanted delay. |
Sounds nice 🤗 👍 |
@SiddhantBahuguna Do you think you could finish your PR that we can close this ? :) |
@odulcy-mindee I think we can close this ? |
Yeah, this issue can be closed, that's fine |
As discussed in #654, the artefact detection needs to improve its robustness. In order to do so and prevent overfitting, I would suggest gradually extending the list of our supported transformations:
Ideally, a given transform should be implemented in
doctr/transforms/modules
so that with the corresponding backend, we can do:for transformations that only change the image.
and:
for transformations that alter the target. In doing so, that will work nicely with our Dataloaders 👍
The text was updated successfully, but these errors were encountered: