dolfin team mailing list archive

["Martin Sandve Alnæs" <martinal@xxxxxxxxx>]

2007/11/29, Anders Logg <logg@xxxxxxxxx>:
> On Wed, Nov 28, 2007 at 05:44:00PM -0600, Matthew Knepley wrote:
> > On Nov 27, 2007 9:40 AM, Anders Logg <logg@xxxxxxxxx> wrote:
> > > On Tue, Nov 27, 2007 at 09:35:52AM -0600, Matthew Knepley wrote:
> > > > On Nov 27, 2007 9:33 AM, Anders Logg <logg@xxxxxxxxx> wrote:
> > > > > On Tue, Nov 27, 2007 at 04:30:43PM +0100, Thomas Witkowski wrote:
> > > > > > Hi,
> > > > > >
> > > > > > I want to use dolfin for solving eigenvalue problems. I've installed
> > > > > > dolfin 0.7.1 with petsc and slepc. The matrix, for which the eigenvalues
> > > > > > should be computed, is defined in the following way:
> > > > > >
> > > > > > PETScMatrix C(TS, TS);
> > > > > > C.set(.....);
> > > > > > C.set(.....);
> > > > >
> > > > > Try calling
> > > > >
> > > > >   C.apply();
> > > >
> > > > Why change the name from assemble()? You even used it in your description,
> > > > and apply() is easily confused with operator application.
> > > >
> > > >   Matt
> > >
> > > Because we refer to "compute the matrix from the bilinear form" as
> > > "assemble".
> > >
> > > And yes, it looks like we're inconsistent with the comment
> > > ("/// Finalise assembly of matrix")...
> >
> > I would be happy with assembleParallel() since that is exactly what it does,
> > and nothing else can be confused with it I think.
>
> That also looks confusing to me, since a call to assembleParallel()
> would be needed also when running on only one processor?
>
> --
> Anders

What about finalizeAssembly()?

-- 
Martin