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
Extend docu of MatrixFree::initialize_dof_vector() #13484
Conversation
static_assert(IsBlockVector<VectorType>::value == false, | ||
"This function is not supported for block vectors."); | ||
|
||
Assert(task_info.n_procs == 1, | ||
ExcMessage("This function can only be used in serial.")); |
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 also added here some asserts. Hope my understanding about serial non-block vector
is correct.
* defines which component is to be used. | ||
* Initialize function for a general serial non-block vector. | ||
* The length of the vector is equal to the total number of | ||
* degrees in the DoFHandler. Vector entries are initialized with zero. |
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.
* degrees in the DoFHandler. Vector entries are initialized with zero. | |
* degrees of freedom in the DoFHandler. Vector entries are initialized with zero. |
* @p dof_handler_index defines which component is to be used. | ||
* | ||
* @note Serial vectors also include Trilinos and PETSc vectors; however | ||
* in these cases, MatrixFree has to be in a serial context, i.e., size |
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.
* in these cases, MatrixFree has to be in a serial context, i.e., size | |
* in these cases, MatrixFree has to be used in a serial context, i.e., the size |
df1637d
to
32c8566
Compare
@kronbichler I have made the modifications! |
* The length of the vector is equal to the total number of | ||
* degrees of freedom in the DoFHandler. Vector entries are initialized |
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.
Maybe say "After a call to this function, the length of the vector..." to make it clear that this function sets the length of the vector to that value?
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 we can get this merged if you fix @bangerth's comment.
32c8566
to
100abfb
Compare
@bangerth @kronbichler I have made the modification. |
follow-up to #13481
FYI @nfehn