<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 20, 2015 at 11:46 PM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Mark,<br>
<br>
   What's the rational for keeping the Chebyshev tuning inside GAMG that duplicates that is already support by KSPChebyshev ? I'd like to remove this chunk of code from GAMG since it is duplicative and instead make sure that the Chebyshev tuning  inside the KSPChebyshev provides everything that is needed.<br>
<br>
   Also in particular what about the /* zeroing out BC rows -- needed for crazy matrices */ should this be ported to KSPChebyshev ?<br></blockquote><div><br></div><div>This is a fix for a matrix that has a norm << X, but X is put on the diagonal for BCs.  You then get X as the eigen estimate.  We should probably remove this ...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
    /* create cheby smoothers */<br>
    for (lidx = 1, level = pc_gamg->Nlevels-2; lidx <= fine_level; lidx++, level--) {<br>
      KSP       smoother;<br>
      PetscBool flag,flag2;<br>
      PC        subpc;<br>
<br>
      ierr = PCMGGetSmoother(pc, lidx, &smoother);CHKERRQ(ierr);<br>
      ierr = KSPGetPC(smoother, &subpc);CHKERRQ(ierr);<br>
<br>
      /* do my own cheby */<br>
<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888"><br>
   Barry<br>
<br>
<br>
</font></span></blockquote></div><br></div></div>