search

jipolanco / BSplineKit.jl

A collection of B-spline tools in Julia
https://jipolanco.github.io/BSplineKit.jl/dev/
MIT License
50 stars 9 forks source link

Use custom LazyVcatVector instead of LazyArrays #77

Closed jipolanco closed 1 year ago

jipolanco commented 1 year ago

This fixes allocation tests. It seems like some recent version of LazyArrays results in extra allocations when constructing a lazy array as in ApplyArray(vcat, x, y, z). It's not the first time this happens, so we define a minimal custom type instead...

codecov-commenter commented 1 year ago

Codecov Report

Merging #77 (80af66a) into master (07b8464) will decrease coverage by 0.27%. The diff coverage is 90.90%.

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
- Coverage   95.34%   95.08%   -0.27%     
==========================================
  Files          35       31       -4     
  Lines        2191     2175      -16     
==========================================
- Hits         2089     2068      -21     
- Misses        102      107       +5
Files Changed Coverage Δ
src/Recombinations/splines.jl 96.66% <90.90%> (-3.34%) :arrow_down:

... and 12 files with indirect coverage changes