New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MatrixFree: distinguish internally between mixed and non-mixed meshes #13042
Conversation
const auto &reference_cells = triangulation.get_reference_cells(); | ||
const bool is_mixed_mesh = reference_cells.size() > 1 || | ||
((reference_cells[0].is_hyper_cube() || | ||
reference_cells[0].is_simplex()) == false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it make sense to move this into Triangulation
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is simple enough to keep it here, as long as it is not used more widely. But it's good to keep an eye on it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe that such a function could be useful in the future for other purposes, so I think it would be well suited for Triangulation
. Also, we might forget that this functionality already exists here, and duplicate code in the future.
I am pro Triangulation::is_mixed_mesh()
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/rebuild |
be3e31b
to
1488718
Compare
No description provided.