[Dolfin] patch for boundary indicators (e.g. VMTK) in parallel

2011-04-04 Thread Kent-Andre Mardal
Please consider the attached patch. Kent # Bazaar merge directive format 2 (Bazaar 0.90) # revision_id: kent-...@simula.no-20110404064913-n944nlrq5w2oylgy # target_branch: bzr+ssh://bazaar.launchpad.net/%2Bbranch/dolfin/ # testament_sha1: eca4dcac14855f7fa199e2a4a8a809ede8a378b6 # timestamp:

Re: [Dolfin] patch for boundary indicators (e.g. VMTK) in parallel

2011-04-04 Thread Garth N. Wells
I would prefer to not apply this just yet. I'm in the process if re-factoring some aspects of parallel mesh functionality. Some classes and functions have gotten out of hand. Register a branch and a merge request so that it doesn't get forgotten. Unit tests are also required before patches will

Re: [Dolfin] patch for boundary indicators (e.g. VMTK) in parallel

2011-04-04 Thread Kent-Andre Mardal
I have a unit test that I can include. BTW: A couple of issues concerning this patch. 1. I have used a std::map for a global - local mapping. I don't know if this the most efficient thing to do, but it is certainly convenient. 2. The boundary indicators are read on every process because

Re: [Dolfin] patch for boundary indicators (e.g. VMTK) in parallel

2011-04-04 Thread Garth N. Wells
On 04/04/11 09:49, Kent-Andre Mardal wrote: I have a unit test that I can include. BTW: A couple of issues concerning this patch. 1. I have used a std::map for a global - local mapping. I don't know if this the most efficient thing to do, but it is certainly convenient.

Re: [Dolfin] patch for boundary indicators (e.g. VMTK) in parallel

2011-04-04 Thread Kent-Andre Mardal
On 4 April 2011 11:25, Garth N. Wells gn...@cam.ac.uk wrote: On 04/04/11 09:49, Kent-Andre Mardal wrote: I have a unit test that I can include. BTW: A couple of issues concerning this patch. 1. I have used a std::map for a global - local mapping. I don't know if this the most

Re: [Dolfin] patch for boundary indicators (e.g. VMTK) in parallel

2011-04-04 Thread Anders Logg
On Mon, Apr 04, 2011 at 10:25:02AM +0100, Garth N. Wells wrote: On 04/04/11 09:49, Kent-Andre Mardal wrote: I have a unit test that I can include. BTW: A couple of issues concerning this patch. 1. I have used a std::map for a global - local mapping. I don't know if this the

Re: [Dolfin] patch for boundary indicators (e.g. VMTK) in parallel

2011-04-04 Thread Garth N. Wells
On 04/04/11 11:23, Anders Logg wrote: On Mon, Apr 04, 2011 at 10:25:02AM +0100, Garth N. Wells wrote: On 04/04/11 09:49, Kent-Andre Mardal wrote: I have a unit test that I can include. BTW: A couple of issues concerning this patch. 1. I have used a std::map for a global - local mapping.

Re: [Dolfin] patch for boundary indicators (e.g. VMTK) in parallel

2011-04-04 Thread Garth N. Wells
On 04/04/11 10:36, Kent-Andre Mardal wrote: On 4 April 2011 11:25, Garth N. Wells gn...@cam.ac.uk mailto:gn...@cam.ac.uk wrote: On 04/04/11 09:49, Kent-Andre Mardal wrote: I have a unit test that I can include. BTW: A couple of issues concerning this patch.