gridap/P4est_wrapper.jl

TODO

victorsndvg opened this issue · 1 comments

victorsndvg commented

Current implementation is almost done, but some details are still pending:

  1. Increase coverage/tests/examples to stress bindings as much as possible
  2. test/example_3.jl: replace malloc/free by julia array
  3. P4est_jll integration: Currently, this bindings only supports the parallel version of p4est. P4est_jlll artifact contains a non-MPI version. I think the main difference between both is the MPI_Comm representation, Ptr{Cvoid} (8 bytes) for parallel version, Cint(?) (4 bytes) for sequential version. We should deal with this difference during package build.
victorsndvg commented
  1. still pending
  2. Done in 5a013ee
  3. implemented in ce5023b . A new variable P4EST_ENABLE_MPI is set during build (manually from ENV or with a default value). MPI_Comm datatype changes depending on its value. P4est_jll case is correctly managed. Tests are also adapted to exemplify its usage.