JuliaLinearAlgebra / AlgebraicMultigrid.jl

Algebraic Multigrid in Julia
Other
117 stars 23 forks source link

Tests for Smooth Aggregation #22

Closed ranjanan closed 6 years ago

ranjanan commented 6 years ago

Fixes #16

codecov-io commented 6 years ago

Codecov Report

Merging #22 into master will increase coverage by 0.9%. The diff coverage is 96.96%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #22     +/-   ##
=========================================
+ Coverage   90.06%   90.96%   +0.9%     
=========================================
  Files          10       10             
  Lines         493      509     +16     
=========================================
+ Hits          444      463     +19     
+ Misses         49       46      -3
Impacted Files Coverage Δ
src/multilevel.jl 75% <ø> (ø) :arrow_up:
src/strength.jl 98.5% <100%> (+1.58%) :arrow_up:
src/smoother.jl 72.72% <100%> (+1.06%) :arrow_up:
src/aggregation.jl 95.74% <100%> (+0.39%) :arrow_up:
src/gallery.jl 100% <100%> (ø) :arrow_up:
src/utils.jl 96.15% <85.71%> (+6.35%) :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 a219756...091a92d. Read the comment docs.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.9%) to 90.963% when pulling 091a92d20e24036ec80d891ee391024319bb8d2a on satest into a21975687f77e2d25ef6bd06785a43a04a355023 on master.

ranjanan commented 6 years ago

I've switched to a local weighting metric for the Jacobi prolongation because I suspect the spectral radius calculation was always unstable since it has a random component.