codecov-commenter
commented
9 months ago Codecov Report
Merging #565 (6764c16) into main (b20aeb0) will increase coverage by
0.17%. Report is 1 commits behind head on main. The diff coverage is96.42%.
@@ Coverage Diff @@
## main #565 +/- ##
==========================================
+ Coverage 87.94% 88.11% +0.17%
==========================================
Files 40 40
Lines 3334 3383 +49
==========================================
+ Hits 2932 2981 +49
Misses 402 402 | Files | Coverage Δ | |
|---|---|---|
| src/LookupArrays/LookupArrays.jl | 100.00% <ø> (ø) |
|
| src/LookupArrays/lookup_traits.jl | 97.72% <ø> (ø) |
|
| src/LookupArrays/selector.jl | 87.09% <100.00%> (+0.27%) |
:arrow_up: |
| src/LookupArrays/show.jl | 84.72% <100.00%> (ø) |
|
| src/LookupArrays/lookup_arrays.jl | 85.86% <95.65%> (+2.19%) |
:arrow_up: |
... and 2 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!
This PR adds a
Cycliclookup type that modifies selectors to cycle the lookup below and above its range.This works for
At,NearandContainsI'm not yet sure how to support
... It probably needs to return aVector{Int}but that would need special handling to have the correctOrderedunrolled cyclic lookup afterwards rather than just repeats of the cycle.