-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Quadrature permutations for DG and some negating of vector-valued spaces #201
Conversation
This reverts commit 0884f12.
const double* restrict coordinate_dofs, const int* entity_local_index, | ||
const uint8_t* restrict quadrature_permutation, | ||
const bool* edge_reflections, const bool* face_reflections, | ||
const uint8_t* face_rotations); |
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.
You didn't document face_rotations
. Also, it feels a bit weird now. If I understand correctly, in general, for vector dofs you need to know face_rotations
for a cell integrals too, right? So for a interior facet integrals, isnt now information about rotations of codim 1 entities, facets, duplicated?
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.
quadrature_permutation
has information about the rotation from two cells either side, while the other inputs are information about the different entities on one cell. But I agree that restructuring what these inputs are at some point is a good idea.
…fcx into mscroggs/move_base_perms_to_ffc
dof_permutations.py
UFC_HEADER_DECL
fromufc.h
instead of duplicating it.entity_block_size
is removed, as this is not constant for higher order vector spaces. Instead, this functionality is now included in the calculate of the base dof permutations.This pull request resolves FEniCS/dolfinx#742, and partially covers FEniCS/dolfinx#736 and FEniCS/dolfinx#737
This is an updated version of #190, and is twinned with FEniCS/dolfinx#770.