Jacob Lotz

If I understand correctly, after this the periodic boundaries are read correctly, but the actual connecting of the boundaries should be done manually after creating a mesh/NURBSExtension. This was partly...

BTW. I think that a work around for me would be to specify boundary attribute 1 to a dummy boundary in the original mesh and not specify a boundary condition...

Hi @dylan-copeland, thanks for your explanation and examples! I will try to find a similar workaround. I will leave this issue open for reference for others encountering similar issues.

@dreamer2368: I have rebased with respect to the current master branch and the tests seem to pass now. @dylan-copeland: I think you accomplish the same. However, I think that the...

> All the CI tests were passing, but now the AppVeyor build fails. I ran this unit test on linux and mac, even with valgrind, with no errors. Does anyone...

@tzanio @dylan-copeland sorry for being a bit absent in this one. I felt a bit uncomfortable in a PR with such a big impact. The work looks very good to...

Thanks @dylan-copeland! I have tested it and looks sound! For the approval I want document here that I have focused on the parts of MFEM that I know, nurbs.cpp and...

> Is there an example mesh to illustrate what this PR fixes? A unit test would be ideal. If a unit test cannot be made easily, can you share some...