[Dune-devel] Small Patch for parallel UG

Oliver Sander sander at igpm.rwth-aachen.de
Mon Sep 16 17:19:13 CEST 2013


Hi Christoph,
I am not saying the patch is bad, but I will not consider patches sent to me
without any reason given.
In other words: yes I object to it being committed until I have understood
what it does precisely.  The parallel UGGrid code is delicate stuff.  If you
cannot give me a better explanation than "fixes an error that should not occur"
I need to have a closer look myself first.
Cheers,
Oliver

Am 16.09.2013 16:55, schrieb Christoph GrĂ¼ninger:
> Hi Oliver,
> the patch is necessary to run an MFEM code in parallel that works fine with SPgrid. The missing case causes an error that should not occur. But maybe I am missing something and the Patch works for
> Jizhou only by coincidence.
>
> Bye
> Christoph
>
> Am 15.09.2013 19:05, schrieb Oliver Sander:
>> Well, what is the patch supposed to do?
>> --
>> Oliver
>>
>> Am 15.09.2013 18:05, schrieb Christoph GrĂ¼ninger:
>>> Hi there,
>>> I got a small UG patch for parallel UG from Jizhou Li and I
>>> adapted it to the current trunk. As I did not test the patch
>>> myself, I'd like to ask if anyone objects committing it.
>>>
>>> Bye
>>> Christoph
>





More information about the Dune-devel mailing list