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
Avoid use of old AffineConstraints functions. #16034
Conversation
Only the indent check fails, but for reasons I do not understand (and seem unrelated). Anyone got an idea? |
* Return the index set describing which part of the degrees of freedom to | ||
* which this object stores constraints are "locally owned". Typically, | ||
* these would be the | ||
* @ref GlossLocallyOwnedDoFs "locally owned degrees of freedom". |
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.
* @ref GlossLocallyOwnedDoFs "locally owned degrees of freedom". | |
* @ref GlossLocallyOwnedDoF "locally owned degrees of freedom". |
should make doxygen
happy.
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.
The indentation CI is failing because of
/home/runner/work/dealii/dealii/include/deal.II/lac/affine_constraints.h:711: warning: unable to resolve reference to 'GlossLocallyOwnedDoFs' for \ref command
Ah, so close yet oh so far! |
constraints.reinit(index_set); | ||
constraints.reinit(constraints.get_locally_owned_indices(), index_set); |
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.
For completeness (in hindsight), this change was wrong. It is fixed in #16053.
This is a companion to #16032, but for places in the library itself. It turns out that I need access to which DoFs the
AffineConstraint
object considers as "locally owned" in a couple of places, so I'm adding an accessor function.Follow-up to #15789.
In reference to #15375.