Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Use FEValues extractors, avoid explicit use of view. #16179

Merged
merged 1 commit into from Oct 27, 2023
Merged

Use FEValues extractors, avoid explicit use of view. #16179

merged 1 commit into from Oct 27, 2023

Conversation

bangerth
Copy link
Member

step-81 is the only program that does things differently. Since the views are expensive, we should not be creating them by hand -- FEValues has an internal cache.

Fixes #15846 .

@kronbichler kronbichler merged commit 8bc8079 into dealii:master Oct 27, 2023
15 checks passed
@bangerth bangerth deleted the 81 branch October 27, 2023 19:45
@bangerth bangerth added this to the Release 9.6 milestone Oct 27, 2023
@tamiko tamiko mentioned this pull request Oct 28, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kronbichler kronbichler kronbichler approved these changes

Assignees
No one assigned
Labels
ready to test Tutorials
Projects
None yet
Milestone
Release 9.6
Development

Successfully merging this pull request may close these issues.

Don't explicitly use FEValuesViews in step-81.
2 participants
@bangerth @kronbichler