[Dune-devel] [SOCIS2015] junior jobs, METIS issue

[Emmanouil Kiagias]

Hello again,

Here is the updated patch[0] (more detailed commit description and >= 5 in
METIS_VAR_MAJOR comparison ) following  gruenich's comments on github.

Kind regards,
Emmanouil

[0]:
https://github.com/alamages/dune-istl/commit/c8c540a23c21ff6325d178fa7930885b9126c039

On Mon, Mar 23, 2015 at 4:26 PM, Emmanouil Kiagias <e.kiagias at gmail.com>
wrote:

> Hello,
>
> I wrote a patch [0] in case METIS 5 is found. I also posted a comment in
> the bugtracker [1]. About the patch:
>
> Below I paste a quote from METIS 5.1.0 manual regarding the migration
> issues (section 2.2.1):
>
> "Since the calling sequence of all the API routines and in some cases
> their names has changed, migrating to the new API will require code
> modifications. To ensure that these modifications are minimal, the new API
> routines allow users to provide NULL as the argument to many of the
> parameters for which there are reasonable defaults"
>
> Following this I used NULL(where possible) in the new
> METIS_PartGraphRecursive's parameters. Only one new parameter could not be
> set to NULL, it was "ncon" (second argument). In the manual it says: "ncon
> The number of balancing constraints. It should be at least 1." so I used 1
> as its value ( I am not sure though about its meaning).
>
> In the current code the options passed to METIS_PartGraphRecursive have
> options[0]=0 which means that the default option values are used, so
> options[1], options[2], options[3] values are ignored. I also passed the
> options using the new layout, using the default values, same as the current
> code. I only manually added the METIS_OPTION_NUMBERING = numflag which is
> the same numflag parameter passed in the old version of
> METIS_PartGraphRecursive function.
>
> METIS_PartGraphRecursive is not called elsewhere in the code. What is left
> to completely fix the existing issue with METIS 5.x.x? Also I just compiled
> my patch, I am not sure how else I can test my code for the moment.
>
> Kind regards,
> Emmanouil
>
> [0]:
> https://github.com/alamages/dune-istl/commit/87516d49485e7a8991235d503d1252b65568fd3e
> [1]: https://dune-project.org/flyspray/index.php?do=details&task_id=1212
>
> On Sun, Mar 22, 2015 at 12:44 PM, Christoph Grüninger <
> christoph.grueninger at iws.uni-stuttgart.de> wrote:
>
>> Hi Emmanouil,
>> welcome to Dune! Good to hear that you are interested in our
>> project ideas and that you figured it out by yourself, how to
>> configure Dune with ParMETIS.
>>
>> If you have additional questions, let us know.
>>
>> Bye
>> Christoph
>>
>> --
>> Je n'ai rien contre les étrangers, quelques uns de mes meilleurs amis
>> sont des étrangers. Mais ces étrangers là ne sont pas de chez nous!
>> [Agecanonix, Astérix - Le cadeau de César]
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.dune-project.org/pipermail/dune-devel/attachments/20150323/7ab78536/attachment.htm>