← Back to team overview

dolfin team mailing list archive

Re: [noreply@xxxxxxxxxxxxx: [Branch ~dolfin-core/dolfin/main] Rev 6357: Fix subdomain assemble for SystemAssembler]

 

On Tue, Oct 18, 2011 at 06:22:20AM -0700, Johan Hake wrote:
> On Monday October 17 2011 23:41:03 Anders Logg wrote:
> > On Mon, Oct 17, 2011 at 11:27:59PM -0700, Johan Hake wrote:
> > > On Monday October 17 2011 23:25:06 Anders Logg wrote:
> > > > This has already been fixed (including new unit tests) but is likely
> > > > sitting in my branch. Are these the same fixes I made or did you make
> > > > some other changes?
> > >
> > > Not sure. I manage to uncomment a unittest which was failing becuase the
> > > subdomains where not handled correctly for SystemAssemble
> >
> > Yes, but I fixed all that (I think):
> >
> > http://bazaar.launchpad.net/~dolfin-core/dolfin/logg/revision/6332
> > http://bazaar.launchpad.net/~dolfin-core/dolfin/logg/revision/6333
> > http://bazaar.launchpad.net/~dolfin-core/dolfin/logg/revision/6334
>
> Yes it looks like we did some over lapping work there :P. Not sure you got the
> global facet number correct, nor the correct domains from the mesh ;) Did you
> test it against the outcommented test?

The tests are not outcommented in my branch. I did some reorganization
of the unit tests in my branch and partly reimplemented some new tests
for SystemAssembler.

I will attempt a merge later, but need to sort out some book things
first.

Seems it is important to merge early with trunk!

--
Anders


References