Jan Nikl

Results 71 comments of Jan Nikl

Ok, as discussed at the meeting, I am reverting that move of the initialization of shared structures, which was meant to make the code cleaner and better maintainable. So it...

Yes, that is where we have started 😄 . I edited the header of #4240 to mention it avoids this as well 😉 . Please keep this PR minimal, so...

> Just need to reemphasize that my suggestion to remove the boundary element generation in the first part of https://github.com/mfem/mfem/pull/4128#issuecomment-2037631281 would be one way of remove that limitation and make...

@v-dobrev , I thought the decision was to split this PR, as I did, moving the potentially controversial change of the initialization of some structures to #4240 . The implementation...

@jandrej , have you tested this PR is working? That is all we need now, the discussion related to the API changes should go to #4240 .

@sebastiangrimberg , it must be without `virtual` (and `override`)

I think that behavior is correct in the sense that when you explicitly use the local sub-mesh, you can do many harmful things to the distributed mesh for sure, but...

Hmm, well, it is not critical, because that happens only when you do not have any boundary elements globally, which was not the case for the user and most other...

Ok, so followed the pathway of @tzanio 's [comment](https://github.com/mfem/mfem/pull/4128#issuecomment-2050703154) 😉 .