And MAT_FACTOR_LU is also being set in some MatILUDTFactor_XXX routines... On Wed, Nov 5, 2008 at 6:52 PM, Lisandro Dalcin <dalcinl at gmail.com> wrote: > Barry, After a fast look at the new factorization implementation, I've > noticed the following > > 1) In MatILUFactor_SeqAIJ and MatILUFactor_SeqBAIJ it is written: > > inA->factor = MAT_FACTOR_LU > > Why not MAT_FACTOR_ILU ?? > > 2) It seems that in MatICCFactor_SeqSBAIJ, the 'factor' field is never filled > > > As I said, my review was a fast one, and perhaps I'm missing something. > > > -- > Lisandro Dalc?n > --------------- > Centro Internacional de M?todos Computacionales en Ingenier?a (CIMEC) > Instituto de Desarrollo Tecnol?gico para la Industria Qu?mica (INTEC) > Consejo Nacional de Investigaciones Cient?ficas y T?cnicas (CONICET) > PTLC - G?emes 3450, (3000) Santa Fe, Argentina > Tel/Fax: +54-(0)342-451.1594 >
-- Lisandro Dalc?n --------------- Centro Internacional de M?todos Computacionales en Ingenier?a (CIMEC) Instituto de Desarrollo Tecnol?gico para la Industria Qu?mica (INTEC) Consejo Nacional de Investigaciones Cient?ficas y T?cnicas (CONICET) PTLC - G?emes 3450, (3000) Santa Fe, Argentina Tel/Fax: +54-(0)342-451.1594
