-
Notifications
You must be signed in to change notification settings - Fork 707
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
EvaluatorTensorProduct::apply_face for higher dimensions #10896
Conversation
@@ -303,7 +303,8 @@ namespace internal | |||
template <int face_direction, | |||
bool contract_onto_face, | |||
bool add, | |||
int max_derivative> | |||
int max_derivative, | |||
bool lex_faces = 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.
Could you please document the new template parameter?
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.
Done!
/rebuild |
* sorted: lexicographically or deal.II-style (special | ||
* treatment of orientation 1 in 3D). Per default | ||
* deal.II-order is enabled. Note: that the deal.II | ||
* ordering is only working for dimensions up to 3. |
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 would not call it deal.Il
orientation but rather right-hand-system
numbering or something like that (which is a 3D concept). The idea is to let axis direction wrap around in defining the order, rather than going lexicographically from the faster running tensor index to the slower running one. What do you think?
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.
What do you think?
I agree. I just did not know how to call it. I have made the change!
References hyperdeal/hyperdeal#47 and hyperdeal/hyperdeal#48.