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
Work on ShapeInfo::is_supported() #14953
Conversation
dynamic_cast<const FE_SimplexPoly<dim, dim> *>( | ||
&fe_in.base_element(base_element_number)) != nullptr || | ||
dynamic_cast<const FE_WedgePoly<dim, dim> *>( | ||
&fe_in.base_element(base_element_number)) != nullptr || | ||
dynamic_cast<const FE_PyramidPoly<dim, dim> *>( | ||
&fe_in.base_element(base_element_number)) != nullptr) |
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.
This was not correct and not an issue, since the first condition normally is true.
@@ -383,7 +383,20 @@ namespace internal | |||
const unsigned int base_element = 0); | |||
|
|||
/** | |||
* Return which kinds of elements are supported by MatrixFree. | |||
* Return whether an element is supported by MatrixFree. |
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.
@kronbichler Have I missed any finite elements?
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.
Looks good as far as I can see.
ade1d7e
to
80754a7
Compare
/rebuild |
80754a7
to
2c0f378
Compare
@@ -16,7 +16,7 @@ DEAL::FE_FaceP<2>(1) supported by MatrixFree: false | |||
DEAL::FE_FaceQ<2>(1) supported by MatrixFree: false | |||
DEAL::FE_P1NC supported by MatrixFree: false | |||
DEAL::FE_Nedelec<2>(1) supported by MatrixFree: false | |||
DEAL::FE_Nothing<2>() supported by MatrixFree: false | |||
DEAL::FE_Nothing<2>() supported by MatrixFree: true |
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.
2 new supported elements :)
No description provided.