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

LA: optimize update of system matrix #360

Merged
merged 1 commit into from
Nov 15, 2022
Merged

Conversation

andrea-iob
Copy link
Member

If the sizes of PETSc data types match the sizes of data types used by bitpit a direct update can be performed, otherwise the matrix is updated using intermediate data storages.

@andrea-iob andrea-iob self-assigned this Nov 5, 2022
@andrea-iob andrea-iob force-pushed the LA.optimize.matrix.update branch 2 times, most recently from 4e97c41 to d4d9b52 Compare November 6, 2022 20:12
Copy link
Contributor

@lucask81 lucask81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested the changes by using immerflow. In this case, the changes have a small contribution to the simulation performance. I approve this PR since all bitpit tests passed.

@andrea-iob andrea-iob merged commit 9e78c24 into master Nov 15, 2022
@andrea-iob andrea-iob deleted the LA.optimize.matrix.update branch November 15, 2022 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants