<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 5 Jul 2019 at 09:11, Smith, Barry F. <<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</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"><br>
   Lisandro,<br>
<br>
    Both plans look good to me. Remove DMCreateAggregates completely and reactor DMHasCreateInjection.<br>
<br></blockquote><div><br></div><div>Barry, I need a bit of help. What should we do for DMDA? </div><div><br></div><div>1) Should the code used in DMCreateAggregates_DA be used to implement a new DMCreateRestriction_DA (which is not currently implemented), or should I just kill DMCreateAggregates_DA ? This approach is dangerous, right now any code using DMDA implicitly uses the transpose of interpolation to define restriction (because DMCreateRestriction_DA is not available).</div><div><br></div><div>2) Or maybe rename DMCreateAggregates_DA to a new public DMDACreateAggregates() routine, just in case some third party is using it? And then add a deprecation for DMCreateAggregates pointing to the DMDACreateAggregates().</div><div><br></div><div>I'm inclined to go with option (2), and I'll start working on it. If you prefer (1), it is trivial to refactor.</div><div><br></div><div><br></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Lisandro Dalcin<br>============<br>Research Scientist<br>Extreme Computing Research Center (ECRC)<br>King Abdullah University of Science and Technology (KAUST)<br><a href="http://ecrc.kaust.edu.sa/" target="_blank">http://ecrc.kaust.edu.sa/</a><br></div></div></div></div>