[petsc-dev] preallocation checks

[Barry Smith]

  This is NOT the right patch. 

   The whole reason for Jed putting in the flag is to clean up the bad no preallocation in PETSc. 

    Please do not do this kind of stuff.

    This needs to be fixed by putting the correct preallocation in. And why is converting to SeqAIJ using a basic converter? Isn't there a converter to  SeqAIJ that does preallocation?


   Barry

On Jan 20, 2012, at 10:36 AM, Satish Balay wrote:

> On Fri, 20 Jan 2012, Matthew Knepley wrote:
> 
>> On Fri, Jan 20, 2012 at 10:25 AM, Stefano Zampini <stefano.zampini at gmail.com
>>> wrote:
>> 
>>> I pulled the latest tip and recompiling my application I got this error
>>> when converting from SEQDENSE to SEQAIJ
>>> 
>>> [0]PETSC ERROR:
>>> ------------------------------------------------------------------------
>>> [0]PETSC ERROR: MatSetValues_SeqAIJ() line 331 in
>>> src/mat/impls/aij/seq/aij.c
>>> [0]PETSC ERROR: MatSetValues() line 1119 in src/mat/interface/matrix.c
>>> [0]PETSC ERROR: MatConvert_Basic() line 37 in src/mat/utils/convert.c
>>> [0]PETSC ERROR: MatConvert() line 3859 in src/mat/interface/matrix.c
>>> 
>>> Is there a way to disable from command line new nonzeros errors?
>> 
>> 
>> This is not the whole error. It looks like this is a problem with
> 
> that was for MatView().
> 
> Try the attached patch - and if it works - I'll push to petsc-dev
> 
> cd petsc-dev
> patch -Np1 < matconvert.patch
> 
> Satish<matconvert.patch>