<div dir="ltr">This has been updated now in barry/tao-fixes-for-petsc4py<div><br></div><div style>Jason</div><div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 18, 2014 at 11:32 AM, Smith, Barry F. <span dir="ltr"><<a href="mailto:bsmith@anl.gov" target="_blank">bsmith@anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im HOEnZb"><br>
On Feb 18, 2014, at 10:12 AM, Jason Sarich <<a href="mailto:jason.sarich@gmail.com">jason.sarich@gmail.com</a>> wrote:<br>
<br>
</div><div class="im HOEnZb">> Yes, I'll rename them under Tao and make them PETSC_INTERNAL.<br>
<br>
</div><div class="im HOEnZb">   Thanks, move the declarations to taoimpl.h also and do it in the barry/tao-fixes-for-petsc4py branch. Just put Tao in front of them.<br>
<br>
   We should eventually clean this stuff up and merge into VecGetSubVector() and MatGetSubMatrix() and unify usage in all of PETSc but will require some thought and care so I have added an issue <a href="https://bitbucket.org/petsc/petsc/issue/58/cleanup-taovecmatgetsubvec-and-unify-into" target="_blank">https://bitbucket.org/petsc/petsc/issue/58/cleanup-taovecmatgetsubvec-and-unify-into</a> instead of doing it rapidly and poorly now.<br>

<br>
   It is generally useful stuff and after cleanup should be used by SNESVI also.<br>
<br>
   Barry<br>
<br>
<br>
><br>
</div><div class="HOEnZb"><div class="h5">> I don't think using the VI routines is a good idea at the moment, those are attached to SNES objects.<br>
><br>
> Jason<br>
><br>
><br>
> On Tue, Feb 18, 2014 at 10:06 AM, Jed Brown <<a href="mailto:jed@jedbrown.org">jed@jedbrown.org</a>> wrote:<br>
> Jason Sarich <<a href="mailto:jason.sarich@gmail.com">jason.sarich@gmail.com</a>> writes:<br>
><br>
> > These functions are for eliminating the variables corresponding to active<br>
> > sets when performing matrix, vector operations. I don't think they were<br>
> > meant to be in the public interface, only for utilities in active set<br>
> > algorithms.<br>
><br>
> Okay, can you either make them PETSC_INTERNAL and namespace them or<br>
> replace them?<br>
><br>
<br>
</div></div></blockquote></div><br></div>