Pull request to merge kim_pull_request into master

Created by: kliegeois

Here is the pull request following long delays.

Waves has been tested successfully on Gaston with a freshly rebuilt Trilinos.

All tests have passed:

ctest -j 6
...
100% tests passed, 0 tests failed out of 76

Total Test time (real) = 240.93 sec

A memory leak remains somewhere in the beggining of the code resulting in some crash at the begining or a the destruction or when running several test in parallel.

Those leaks are however partially corrected since the last pull request and must be fully corrected in the future.

@rboman @acrovato

Created by: kliegeois

@acrovato @rboman
The Trilinos library is installed in the folder /home/liegeois/local/trilinos on gaston. Let met know if you cannot access it.

linuxbin has been updated so that Kim's trilinos binaries can be found by cmake

Created by: kliegeois

@acrovato Thanks for the review, @rboman is currently running the tests to verify that it works on his session.

I agree that some improvements should be made.

I have opened an issue as discussed.

Created by: acrovato

Great work!

Concernant les branches qui restent, peut-on supprimer "merge_kim" et "merge_kim_update_trilinos"? Quid de "bowman" et "bowman_ER" qui sont pas mergées?

Created by: kliegeois

Je vais passer tout ça en revue demain matin, en théorie elles pourront toutes être supprimées. Le seul "souci" c'est qu'il me faudrait un moyen facile de ne compiler que ce que je dois compiler sur les clusters (où je n'ai pas installé mumps). Soit je rajoute une option dans le fichier de configuration qui disable tout et je reste entièrement cohérent avec master, soit je vais changer les valeurs dans le CMakeList global et je dois à chaque fois rechanger ça avant tout push/pull vers master.

tu peux desactiver tout ce qui depend de mumps dans ton script de config. si tu veux, commite ton script de config dans le repertoire avec tous les scripts (utilities_to_be_sorted).

Le dim. 10 févr. 2019 à 14:07, Kim Liegeois notifications@github.com a écrit :

Je vais passer tout ça en revue demain matin, en théorie elles pourront toutes être supprimées. Le seul "souci" c'est qu'il me faudrait un moyen facile de ne compiler que ce que je dois compiler sur les clusters (où je n'ai pas installé mumps). Soit je rajoute une option dans le fichier de configuration qui disable tout et je reste entièrement cohérent avec master, soit je vais changer les valeurs dans le CMakeList global et je dois à chaque fois rechanger ça avant tout push/pull vers master.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ulgltas/waves/pull/21#issuecomment-462131403, or mute the thread https://github.com/notifications/unsubscribe-auth/AClo5xTn-NibQaZHJuq2L_uGFPY5LgPJks5vMBmVgaJpZM4avp9U .