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

Test-time augmentation: noise, blur, brightness, contrast, geometric scaling, etc. #106

Closed
remtav opened this issue Oct 24, 2019 · 3 comments · Fixed by #193
Closed

Test-time augmentation: noise, blur, brightness, contrast, geometric scaling, etc. #106

remtav opened this issue Oct 24, 2019 · 3 comments · Fixed by #193
Labels
P2 Medium priority
Milestone

Comments

@remtav
Copy link
Collaborator

remtav commented Oct 24, 2019

Wouldn't trained models gain robustness if data augmentations (utils/augmentation.py) included brightness and contrast shifts (ex.: random, 10-15%)? I imagine it could also help with overfitting issues.

@mpelchat04
Copy link
Collaborator

That's true for satellite imagery. But since brightness and contrast shifts are only histograms modifications, there are a few considerations:

  1. EO data other than satellite or aerial imagery (e.g. derived products from DEMs [Slope, aspect, etc.]) have specific data range and I'm not convinced that messing with these histograms would really help.
  2. We could add this feature as a parameter in the config file. But we would have to be able to turn it on/off for each band. For example, let say we use as input data images with RGB+slope. We'd like to apply contrast and/or brightness shifts on the RGB bands but probably not on the slope band...
  3. We would probably have to manage those functions ourselves, since data augmentations libraries such as Albumentations only manage 3bands data (for now).

It's worth the discussion, tough. I think the first step would be to test the first hypothesis and work from there. I'll add this to our "tests wishlist".

Math

@mpelchat04 mpelchat04 added the P3 Low priority label Oct 24, 2019
@remtav
Copy link
Collaborator Author

remtav commented May 1, 2020

Augmentration techniques should be reviewed (noise, blur, geometric scaling, etc.). Overall, there could be a slight improvement of results if different augmentation strategies were tested. For example, test-time augmentation could be interesting to try out.

This paper deserves a look: https://link.springer.com/article/10.1186/s40537-019-0197-0

Edited June 25th 2020:
Mathieu, the point you raised is important to keep in mind. Currently random radiometric trim augmentation is applied to all bands at once. We'll have to think at how it could be applied to only a portion of those bands. First, I imagine we'd have to inform GDL what bands it will be seeing by their name in what order (e.g. RGB, not band 012 or 123), then identify which bands will need to be augmented. To be discussed.

@remtav remtav marked this as a duplicate of #132 May 1, 2020
@remtav remtav closed this as completed May 1, 2020
@remtav remtav reopened this May 1, 2020
@remtav remtav marked this as not a duplicate of #132 May 1, 2020
@remtav remtav changed the title Data augmentation: brightness and contrast Data augmentation techniques: test-time augmentation, noise, blur, brightness, contrast, geometric scaling, etc. May 1, 2020
@remtav remtav added P2 Medium priority and removed P3 Low priority labels Aug 11, 2020
@valhassan valhassan changed the title Data augmentation techniques: test-time augmentation, noise, blur, brightness, contrast, geometric scaling, etc. Test-time augmentation: noise, blur, brightness, contrast, geometric scaling, etc. Aug 11, 2020
@valhassan valhassan added this to the 1.3.0 milestone Aug 13, 2020
@remtav
Copy link
Collaborator Author

remtav commented Aug 13, 2020

@bstdenis bstdenis linked a pull request Mar 8, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Medium priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants