Skip to content
Snippets Groups Projects

Katropton: Add supports for Gmsh 4

Merged Liegeois Kim requested to merge gmsh_4_mpi into master
1 unresolved thread

As observed by @R.Boman, katropton was not working with MPI using Gmsh 4. This was due to the fact that Gmsh 4 does not save the ghost cells by default.

To save those cells, we have to use the option -part_ghosts as discussed in the Gmsh doc.

However, this option is not supported by Gmsh 3 (gmsh -h | grep "ghost" does not find any line). Therefore, to solve this issue, the code must test the version of Gmsh used and, if the major version is 4 or greater, must add the -part_ghosts argument.

This MR includes the correction of if statements in plat*.geo which prevented the use of newer Gmsh than 3.0.6.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
16 16 import numpy as np
17 17 from katropton.eigenvalues import *
18 18
19 def get_gmsh_version():
20 command_line = 'gmsh --version'
21 tmp = shlex.split(command_line)
22 fileout = open('gmsh_version.log', 'w')
  • Liegeois Kim added 1 commit

    added 1 commit

    • 9eaeec57 - Katropton: use checkout_output instead of a temporary file to get gmsh version

    Compare with previous version

  • Boman Romain approved this merge request

    approved this merge request

  • Boman Romain enabled an automatic merge when the pipeline for 9eaeec57 succeeds

    enabled an automatic merge when the pipeline for 9eaeec57 succeeds

  • Liegeois Kim aborted the automatic merge because source branch was updated

    aborted the automatic merge because source branch was updated

  • Liegeois Kim added 1 commit

    added 1 commit

    • 1b9f93fe - Katropton: resolve a string bug with python3 in the get_gmsh_version function

    Compare with previous version

  • Boman Romain approved this merge request

    approved this merge request

  • merged

  • Please register or sign in to reply
    Loading