David Wells

Those failures are spurious - I think we just fixed that in our CI.

Could we put these in `MatrixTools` instead? We don't use that namespace much but I would prefer not to introduce a new one that is similar.

Another CGAL problem we have is that it may depend on GMP and MPFR but we don't seem to propagate those dependencies along (I get missing symbol errors since we...

On second thought, this might break things in a weird way (it at least substantially changes the behavior of loading mixed meshes) so lets hold off until 9.5. If necessary...

Hm. We might want to keep throwing that exception - it is a lot more informative than an internal error. I'll see if I can get that to work.

Not yet. I've been traveling for most of the last two weeks and I haven't had time to look at this.

That was pretty easy to address - all we need to do is switch to the correct exception type here.

/rebuild

Can someone take another look?

Related: we need something like `CellInfo::max_faces_per_cell` or equivalent instead of hard-coding in things like #12126.