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

Standardize LUT table calls throughout library #542

Merged
merged 2 commits into from Jan 7, 2020
Merged

Standardize LUT table calls throughout library #542

merged 2 commits into from Jan 7, 2020

Conversation

aleju
Copy link
Owner

@aleju aleju commented Dec 30, 2019

This patch adds common methods for mappings based on lookup tables.
The functions imgaug.imgaug.apply_lut() and imgaug.imgaug.apply_lut_() are added.
All augmenters are switched to the new functions.

@codecov-io
Copy link

codecov-io commented Dec 30, 2019

Codecov Report

Merging #542 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #542      +/-   ##
==========================================
+ Coverage    96.4%   96.41%   +0.01%     
==========================================
  Files          38       38              
  Lines       13839    13839              
==========================================
+ Hits        13341    13342       +1     
+ Misses        498      497       -1

This patch adds common methods for mappings based
on lookup tables.
The functions imgaug.imgaug.apply_lut() and
imgaug.imgaug.apply_lut_() are added.
This patch switches all augmenters from cv2.lut() to
imgaug's apply_lut() and apply_lut_() functions.
This decreases code duplication and likely fixes
some bugs that were not yet discovered.
@aleju aleju merged commit 3c98119 into master Jan 7, 2020
@aleju aleju deleted the unify_lut branch January 7, 2020 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants