search

JuliaLinearAlgebra / AlgebraicMultigrid.jl

Algebraic Multigrid in Julia
Other
116 stars 23 forks source link

Printed Grid Complexity is wrong (just copies Operator Complexity) #86

Closed learning-chip closed 2 years ago

learning-chip commented 2 years ago

The line ground = round(op, digits = 3) below should be ground = round(g, digits = 3) instead:

https://github.com/JuliaLinearAlgebra/AlgebraicMultigrid.jl/blob/b40945a7c5c724f65b92b1a2611ca264df483101/src/multilevel.jl#L66-L67

https://github.com/JuliaLinearAlgebra/AlgebraicMultigrid.jl/blob/b40945a7c5c724f65b92b1a2611ca264df483101/src/multilevel.jl#L83-L90

Package version

AlgebraicMultigrid@0.4.2

ranjanan commented 2 years ago

Thank you! I'll fix this shortly.

ranjanan commented 2 years ago

Fixed by #88