Tzanio Kolev
Tzanio Kolev
Any updates on this?
Discussion continued in #3170.
Hi @IdoAkkerman, This slipped through the cracks because it was not listed as `ready-for-review` or `wip`. If it is ready to go, can you merge `master` in it and mark...
ping: @IdoAkkerman
Please mark as `ready-for-review` when this is ready.
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.
@IdoAkkerman and @JacobLotz any chance you can pick this up again? I'd like to merge it in, now that mfem-4.4 is out 😄 Thanks! Tzanio
I tried to fix merging and the new check for shadow variables, but one of the tests is failing... can you take a look?
Hi @ptsuji, When this is ready, can you mark it with the `ready-for-review` label? See https://github.com/mfem/mfem/blob/master/CONTRIBUTING.md#quick-summary for more details. Tzanio
Thanks @ptsuji, I started looking at it and didn't get a chance to continue, but it looks good now. We will start the review.