[petsc-dev] proposed minor PetscPartitioner changes

[Smith, Barry F.]

> On Nov 7, 2017, at 1:33 AM, Lisandro Dalcin <dalcinl at gmail.com> wrote:
> 
> On 6 November 2017 at 16:37, Matthew Knepley <knepley at gmail.com> wrote:
>> On Mon, Nov 6, 2017 at 8:34 AM, Smith, Barry F. <bsmith at 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