Closed timholy closed 3 years ago
Merging #201 (4ca2542) into master (83ac3ed) will decrease coverage by
0.58%
. The diff coverage is95.83%
.
@@ Coverage Diff @@
## master #201 +/- ##
==========================================
- Coverage 91.59% 91.00% -0.59%
==========================================
Files 9 10 +1
Lines 1380 1412 +32
==========================================
+ Hits 1264 1285 +21
- Misses 116 127 +11
Impacted Files | Coverage Δ | |
---|---|---|
src/ImageFiltering.jl | 77.27% <ø> (ø) |
|
src/kernelfactors.jl | 91.15% <90.00%> (-2.42%) |
:arrow_down: |
src/precompile.jl | 94.11% <94.11%> (ø) |
|
src/border.jl | 93.02% <100.00%> (-1.13%) |
:arrow_down: |
src/kernel.jl | 98.40% <100.00%> (-1.60%) |
:arrow_down: |
src/mapwindow.jl | 86.22% <100.00%> (-0.45%) |
:arrow_down: |
src/utils.jl | 87.30% <0.00%> (-1.59%) |
:arrow_down: |
... and 2 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 83ac3ed...4ca2542. Read the comment docs.
I think we can merge this as-is, and continue in separate efforts.
On the current release, the first call looks like this:
On this branch, it looks like this:
Quite a nice reduction in latency.
I should note this requires https://github.com/JuliaMath/AbstractFFTs.jl/pull/48 and also some changes to FFTW.jl that I haven't yet submitted since they rely on the changes in AbstractFFTs.