On 8 November 2017 at 05:51, Smith, Barry F. <[email protected]> wrote: > >> On Nov 7, 2017, at 1:33 AM, Lisandro Dalcin <[email protected]> wrote: >> >> 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. >
Note however that things looks easy to refactor. I'll try to team up with Matt to improve things. -- 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
