[Dune] UG

[Oliver Sander]

Hi Rebecca,
I just committed a bugfix.  For me test-parallel-ug compiles and runs now.
Please write me if there are further problems.
cheers,
Oliver

Am 15.12.2011 10:15, schrieb Rebecca Neumann:
> Hey Oliver,
>
> for me also revision r7799 does not work,
> test-parallel-ug complains for example:
> test_parallel_ug-test-parallel-ug.o: In function
> `Dune::UGMakeableEntity<0, 2, Dune::UGGrid<2>
> const>::setToTarget(UG::D2::element*)':
> /home/rebecca/dune/dune-grid/dune/grid/test/../../../dune/grid/uggrid/uggridentity.hh:50:
> undefined reference to `Dune::UGGridEntity<0, 2, Dune::UGGrid<2>
> const>::setToTarget(UG::D2::element*)'
>
> For me the last revision that works with test-parallel-ug is r7630, I
> will go back there!
> Thanks,
> Rebecca
>
> On 14.12.2011 16:36, Oliver Sander wrote:
>> Hi Rebecca,
>> you are right, I can reproduce this. However the problem is another
>> commit:
>>
>> ------------------------------------------------------------------------
>> r7800 | sander | 2011-12-10 21:43:49 +0100 (Sa, 10. Dez 2011) | 17 Zeilen
>>
>> Fix a bug in the UGGrid handling of boundary segment indices.
>>
>> This has been reported by Lars Lubkoll: If you use more than
>> one UGGrid, then the boundary segment indices of the grids
>> get corrupted.
>>
>> The fix is simply, but still the patch is invasive. To fix
>> the bug, UG's global currBVP variable needs to be set to the
>> current grid to obtain the correct indices. That's one
>> additional line of code. However, the pointer to the current
>> grid is not available in the UGGridIntersections class...
>>
>> This is what makes the present patch so long: it basically
>> make the grid known to almost all UGGrid data structures.
>> This adds a little more overhead, but I really don't see
>> any other way to fix the bug.
>>
>>
>> Let me try to think how to fix that. If you need Bernd's edge
>> communication
>> now you can go back before 7800.
>>
>> cheers,
>> Oliver
>>
>>
>>
>>
>>
>> Am 14.12.2011 16:18, schrieb Rebecca Neumann:
>>> Hallo Oli,
>>>
>>> seit diesem commit läuft ug bei mir nicht mehr, ich glaube es fehlen ein
>>> paar includes.
>>> ------------------------------------------------------------------------
>>> r7760 | sander | 2011-11-11 17:32:05 +0100 (Fr, 11. Nov 2011) | 5 Zeilen
>>>
>>> Implement edge communication for UGGrid.
>>> Only 3D is working so far. Hopefully 2D will follow soon.
>>>
>>> Patch by Bernd Flemisch and Alexander Schlaich.
>>>
>>> ------------------------------------------------------------------------
>>>
>>> Man sieht es zB an /dune/grid/test/test-parallel-ug.cc, das kompiliert
>>> nicht mehr.
>>>
>>> zB fehlt in dune/grid/uggrid.hh
>>> #include <dune/grid/uggrid/uggridentity.hh>
>>> damit die UGMakeableEntitys bekannt sind, aber das ist noch nicht alles.
>>>
>>> Vielleicht kannst du das ja nochmal anschauen?
>>> Danke und viele Grüße,
>>> Rebecca