> On Nov 7, 2017, at 1:33 AM, Lisandro Dalcin <dalc...@gmail.com> wrote: > > On 6 November 2017 at 16:37, Matthew Knepley <knep...@gmail.com> wrote: >> On Mon, Nov 6, 2017 at 8:34 AM, Smith, Barry F. <bsm...@mcs.anl.gov> wrote: >>> >>> MatPartitioning is NOT about partitioning MATRICES it is about >>> partitioning GRAPHS, so it can support anything related to graphs including >>> dual graphs etc. Again, you didn't even try to extend/improve MatPartioning >>> you just wrote all your own duplicative code, and not even particularly >>> well. >> >> >> By "not particularly well, I assume there is some sort of deficiency there. >> However, I do not believe that there is. >> > > The only concern I have about PetscPartitioner is that the API depends > on DM (PetscPartitionerPartition_<TYPE> routines). Maybe > PetscPartitioner should eventually move to became more agnostic, and > that way it can be used to partition matrices and meshes.
This is certainly a serious flaw if PetscPartitioner is intended as THE API to use for partitioning. If it is not intended as THE API for partitioning then that is also a problem, because why have multiple APIs for what is essentially one set of abstractions. > > > -- > Lisandro Dalcin > ============ > Research Scientist > Computer, Electrical and Mathematical Sciences & Engineering (CEMSE) > Extreme Computing Research Center (ECRC) > King Abdullah University of Science and Technology (KAUST) > http://ecrc.kaust.edu.sa/ > > 4700 King Abdullah University of Science and Technology > al-Khawarizmi Bldg (Bldg 1), Office # 0109 > Thuwal 23955-6900, Kingdom of Saudi Arabia > http://www.kaust.edu.sa > > Office Phone: +966 12 808-0459