-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Description
In a future adaptation, the GMGPolar solver should only handle solver information and we should think of better separation of responsibilities.
GMGPolar seems to be a very big class, it would be clearer if it just takes care of the solving part. Someting like gmgpolar.solve(lhs, rhs, tol, maxIt). You could also consider a templated class to avoid several switch statements and code repetition inside the class : GMGPolar<SmootherOpLevel0, RestrictionOpLevel0, ProlongationOpLevel0, SmootherOp, RestrictionOp, ProlongationOp, MGCycle> ( just an idea )
Originally posted by @ds-aakash in #65 (comment)
Metadata
Metadata
Assignees
Labels
No labels