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
Refer to FESystem as such (not FE_System). #7138
Conversation
This fixes some doxygen links.
@@ -1994,7 +1994,7 @@ FESystem<dim, spacedim>::get_subface_interpolation_matrix( | |||
FullMatrix<double> & interpolation_matrix) const | |||
{ | |||
AssertThrow( | |||
(x_source_fe.get_name().find("FE_System<") == 0) || | |||
(x_source_fe.get_name().find("FESystem<") == 0) || |
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.
Oy, how did this work? What does FESystem
's get_name()
function return?
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.
It returns the correct value. We are saved by the dynamic cast on the next line :)
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.
Ha, bizarre!
/run-tests |
@@ -534,7 +534,7 @@ class MatrixFree : public Subscriptor | |||
* pointers to the initialization function. Note that the finite element | |||
* underlying an DoFHandler must either be scalar or contain several copies | |||
* of the same element. Mixing several different elements into one | |||
* @p FE_System is not allowed. | |||
* FESystem is not allowed. |
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 realize that this comment is outdated since #6300 or one of the related commits from #5667. Can you replace the part starting from Note that the finite element underlying...
by the following text (I do not have the time right now to prepare it myself):
As an alternative, a system of several components can also be represented by a single
DoFHandler based on an FESystem. The prerequisite for this case is that each base element
of the FESystem must be compatible with the present class, such as the FE_Q or FE_DGQ
classes.
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.
369a922
to
1fdde1f
Compare
This fixes some doxygen links.