
<html>


<head>


<meta http-equiv="Content-Type" content="text/html; charset=utf-8">


</head>


<body>


<div dir="ltr">


<div dir="ltr">Done. See <a href="https://gitlab.com/petsc/petsc/commit/85ec510f49531057ebfe1fb641fe93a36371878e">https://gitlab.com/petsc/petsc/commit/85ec510f49531057ebfe1fb641fe93a36371878e</a><br>


</div>


<div>Hong</div>


</div>


<br>


<div class="gmail_quote">


<div dir="ltr" class="gmail_attr">On Mon, Sep 23, 2019 at 11:32 AM Pierre Jolivet <<a href="mailto:pierre.jolivet@enseeiht.fr" target="_blank">pierre.jolivet@enseeiht.fr</a>> wrote:<br>


</div>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


<div>Hong,


<div>You should probably cherry pick <a href="https://gitlab.com/petsc/petsc/commit/93d7d1d6d29b0d66b5629a261178b832a925de80?merge_request_iid=2069" target="_blank">https://gitlab.com/petsc/petsc/commit/93d7d1d6d29b0d66b5629a261178b832a925de80?merge_request_iid=2069</a> (and


 remove the MatNest part).</div>


<div>This fixes a similar issue in MatTransposeMatMult with nontrivial LDAs.</div>


<div>Since this commit is part of a feature MR that is unlikely to be ready for tomorrow, this fix (as of now) is also unlikely to be in master for the release.</div>


<div><br>


</div>


<div>Thanks,</div>


<div>Pierre<br>


<div><br>


<blockquote type="cite">


<div>On 23 Sep 2019, at 6:02 PM, Zhang, Hong <<a href="mailto:hzhang@mcs.anl.gov" target="_blank">hzhang@mcs.anl.gov</a>> wrote:</div>


<br class="gmail-m_5576923788568520606gmail-m_-3703444396135516478Apple-interchange-newline">


<div>


<div>


<div dir="ltr">


<div dir="ltr">Barry:</div>


<div class="gmail_quote">


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


    As a hack for this release could you have the Numeric portion of the multiply routines check if the symbolic data is there and if not just call the symbolic an attach the needed data? You might need to have a utility function that does all the symbolic


 part except the allocation of the matrix and then call this from the numeric part as well as the real symbolic part.<br>


</blockquote>


<div> </div>


<div>I'm working on this now.  I was not aware of MatSeqDenseSetLDA() which changes pattern of data access in seqdense matrix. </div>


<div><span style="font-family:Roboto,RobotoDraft,Helvetica,Arial,sans-serif;font-size:14px">Pierre's patch:</span><br>


</div>


<div>"change Bm here <a href="https://www.mcs.anl.gov/petsc/petsc-dev/src/mat/impls/aij/mpi/mpimatmatmult.c.html#line549" target="_blank">https://www.mcs.anl.gov/petsc/petsc-dev/src/mat/impls/aij/mpi/mpimatmatmult.c.html#line549</a> to the LDA of B" </div>


<div>fix this bug. I'll further test it and submit a pull request. </div>


<div>Then, I'll check slepc's bug report.</div>


<div>Hong</div>


</div>


</div>


</div>


</div>


</blockquote>


</div>


<br>


</div>


</div>


</blockquote>


</div>


</body>


</html>