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
Improve documentation for MultigridTransfer::copy_to_mg #5104
Conversation
@@ -154,7 +154,9 @@ class MGLevelGlobalTransfer : public MGTransferBase<VectorType> | |||
|
|||
/** | |||
* Transfer from a vector on the global grid to vectors defined on each of | |||
* the levels separately, i.a. an @p MGVector. | |||
* the levels separately, i.e. a @p MGVector, for the active degrees of |
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 can go further and say that in a globally refined mesh without renumbering, it is a one-to-one copy to the finest level vector.
@@ -154,7 +154,9 @@ class MGLevelGlobalTransfer : public MGTransferBase<VectorType> | |||
|
|||
/** | |||
* Transfer from a vector on the global grid to vectors defined on each of | |||
* the levels separately, i.a. an @p MGVector. | |||
* the levels separately, i.e. a @p MGVector, for the active degrees of |
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 are you trying to say with MGVector
? I don't think this exists as a type? Do you mean "multilevel vector"?
4d4f52d
to
bea0183
Compare
@tjhei Here you go. |
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.
LGTM. @tjhei maybe we should add one of those 2D pictures you showed me that explains what copy_to_mg
actually does w.r.t. each level and its dofs.
typo in "untouched". Nitpicking: It only is a plain copy if you did not renumber the DoFs. (but I don't care if you want to fix this). |
bea0183
to
543282d
Compare
Fixed the typo. |
Clarify that
copy_to_mg
only acts on active degrees of freedoms. The current documentation led to the assumption that this function would do similar things asrestrict_add
for theMultigridTransfer
classes.