Tzanio Kolev
Tzanio Kolev
This PR is now [under review](https://github.com/mfem/mfem/blob/master/CONTRIBUTING.md#pull-requests) (see the table in the PR description). To help with the review process, please **do not force push** to the branch.
Lets vote on how to move forward here: https://github.com/orgs/mfem/discussions/4223
@ebchin can you address @sebastiangrimberg's suggestion above and also make the `real_t`-related changes in this PR? We will merge it before https://github.com/mfem/mfem/pull/4054.
Merged in `next` for testing...
@ebchin, can you add that ☝️ change in #4054 ?
This PR is now [under review](https://github.com/mfem/mfem/blob/master/CONTRIBUTING.md#pull-requests) (see the table in the PR description). To help with the review process, please **do not force push** to the branch.
ping: @jandrej , @sohailreddy
Now that PR https://github.com/mfem/mfem/pull/3922 has been merged in `master`, please update this PR by: 1. merging `master` in here and resolving conflicts 2. replacing use of `double` with `real_t` 3....
As a compromise: is it just possible to just `ParMesh::GenerateBoundaryElements = delete` which will explicitly expose this as a bug to the users? @sebastiangrimberg, @v-dobrev and @najlkin -- what do...
OK, then I vote to implement it as `{ MFEM_ABORT(" ..."); }` and note that users shouldn't be calling that function.