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.

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

CeedOperatorLinearAssembleQFunction[PointBlock]Diagonal #1671

Open
jeremylt opened this issue Sep 23, 2024 · 0 comments
Open

CeedOperatorLinearAssembleQFunction[PointBlock]Diagonal #1671

jeremylt opened this issue Sep 23, 2024 · 0 comments

Comments

@jeremylt
Copy link
Member

jeremylt commented Sep 23, 2024

Ok, turns out that for diagonal assembly we're only using the point block diagonal of the assembled QFunction, so we should add CeedOperatorLinearAssembleQFunctionPointBlockDiagonal

Need to investigate a bit, but a point block diagonal assembly by component should be sufficient and would cut memory usage for our big projections (Ratel diagnostic) without talking about QF coupling yet. Though, it would need to be swapped for the full QFunction when doing assembly, such as for multi grid, so it might not actually save us much in practice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant