<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, 11 Dec 2018 at 07:47, Stefano Zampini via petsc-dev <<a href="mailto:petsc-dev@mcs.anl.gov">petsc-dev@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"><div dir="ltr">Yes, I have also noticed it.<div>IMHO, we should enforce the  TSGetDM + DMCreateGlobalVector pattern instead of duplicating ts->vec_sol in all the implementations.</div><div>I'll open a bitbucket issue if you agree</div></div><br></blockquote><div><br></div><div>As long as you do not break third party code that do not set a DM on TS, I would not object. Otherwise, I'll replace the sugar in your desk with salt, so you can have a really tasty afternoon coffee.</div><div><br></div></div>-- <br><div dir="ltr" class="gmail_signature">Lisandro Dalcin<br>============<br>Research Scientist<br>Computer, Electrical and Mathematical Sciences & Engineering (CEMSE)<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><br>4700 King Abdullah University of Science and Technology<br>al-Khawarizmi Bldg (Bldg 1), Office # 0109<br>Thuwal 23955-6900, Kingdom of Saudi Arabia<br><a href="http://www.kaust.edu.sa" target="_blank">http://www.kaust.edu.sa</a><br><br>Office Phone: +966 12 808-0459</div></div>