add GenericFenwickTree

Open eulerkochy opened this issue 5 years ago • 4 comments

modify code
add tests
change in docs

Apply suggestions from #497 in this PR. Cleaned up a bit of code.

@kmsquire review?

Mar 07 '20 06:03 eulerkochy

Codecov Report

Merging #589 into master will increase coverage by 0.02%. The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #589      +/-   ##
==========================================
+ Coverage   95.01%   95.04%   +0.02%     
==========================================
  Files          33       33              
  Lines        2831     2846      +15     
==========================================
+ Hits         2690     2705      +15     
  Misses        141      141

Impacted Files	Coverage Δ
src/DataStructures.jl	`100% <ø> (ø)`	:arrow_up:
src/fenwick.jl	`100% <100%> (ø)`	:arrow_up:
src/queue.jl	`100% <0%> (ø)`	:arrow_up:

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 d321175...376fcfa. Read the comment docs.

Mar 07 '20 06:03 codecov[bot]

Bump

Mar 27 '20 13:03 eulerkochy

CI is failing

Sep 13 '20 17:09 oxinabox

I was going to try to review this finally tonight, but it got too late here, so I'll try to do it first thing tomorrow.

Sep 14 '20 09:09 kmsquire