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

Improve Performance of Elementwise Multiplication #615

Merged
merged 4 commits into from
May 2, 2020

Conversation

aleju
Copy link
Owner

@aleju aleju commented Feb 16, 2020

This patch improves the performance of
imgaug.augmenters.arithmetic.multiply_elementwise(). The
performance improvement is roughly between 1.5x and 10x.
The effect is stronger for smaller images and denser
matrices of multipliers (i.e. (H,W,C) instead of (H,W)).
This change affects MultiplyElementwise.

Add functions:

  • imgaug.augmenters.arithmetic.multiply_elementwise_().

@codecov-io
Copy link

codecov-io commented Feb 16, 2020

Codecov Report

Merging #615 into master will decrease coverage by 0.00%.
The diff coverage is 96.00%.

@@            Coverage Diff             @@
##           master     #615      +/-   ##
==========================================
- Coverage   96.35%   96.34%   -0.00%     
==========================================
  Files          41       41              
  Lines       14923    14938      +15     
==========================================
+ Hits        14378    14392      +14     
- Misses        545      546       +1     

aleju added 2 commits May 2, 2020 21:39
This patch improves the performance of
`imgaug.augmenters.arithmetic.multiply_elementwise()`. The
performance improvement is roughly between 1.5x and 10x.
The effect is stronger for smaller images and denser
matrices of multipliers (i.e. `(H,W,C)` instead of `(H,W)`).
This change affects `MultiplyElementwise`.

Add functions:
* `imgaug.augmenters.arithmetic.multiply_elementwise_()`.
@aleju aleju merged commit faf3352 into master May 2, 2020
@aleju aleju deleted the improve_perf_mul_elmtw branch May 2, 2020 20:36
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.

2 participants