On 7/9/07, Barry Smith <bsmith at mcs.anl.gov> wrote: > I have pushed a change to the MatSetOption() function.
Barry, a couple of comments * I think MAT_NO_NEW_NONZERO_LOCATIONS should be MAT_NEW_NONZERO_LOCATIONS, for symmetry with the other option names. * I think all the MAT_INODE_LIMIT_[1..5] could be perhaps removed from MatOption, and add a new function MatSetInodeLimit(Mat A, PetscInt limit). Not sure what to do with MAT_USE_INODE. Perhpas it can be also removed, and use the previous function (with limit==0, no inodes, and with 1 <= limit <=5, use inodes with that limit) * You removed the MAT_[ROW|COLUMN]_SORTED stuff. I think you should keep it, even when this feature is not yet implemented. -- 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
