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

DO NOT SQUASH: Specify sparsity block-wise #3288

Merged
merged 17 commits into from
Mar 1, 2024
Merged

Conversation

ksagiyam
Copy link
Contributor

@ksagiyam ksagiyam commented Dec 10, 2023

Depends on OP2/PyOP2#713.

  • refactor assemble.py,
  • make sparsity block-wise.

@ksagiyam ksagiyam force-pushed the ksagiyam/sparsity branch 6 times, most recently from ee777cc to 6094ccf Compare December 21, 2023 04:00
@ksagiyam ksagiyam changed the title DROP BEFORE MERGE Specify sparsity block-wise Dec 21, 2023
@ksagiyam ksagiyam marked this pull request as ready for review December 21, 2023 12:14
@ksagiyam ksagiyam requested a review from connorjward January 3, 2024 16:28
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
tests/regression/test_assemble.py Outdated Show resolved Hide resolved
tests/regression/test_assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Show resolved Hide resolved
@ksagiyam ksagiyam force-pushed the ksagiyam/sparsity branch 4 times, most recently from f8793f8 to cfc3c5f Compare January 10, 2024 13:55
firedrake/assemble.py Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
@ksagiyam ksagiyam force-pushed the ksagiyam/sparsity branch 4 times, most recently from 0eb3b1d to 87bb6e0 Compare February 7, 2024 15:23
@ksagiyam ksagiyam force-pushed the ksagiyam/sparsity branch 9 times, most recently from fd42652 to a585d93 Compare February 21, 2024 11:32
dham
dham previously requested changes Feb 28, 2024
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
@ksagiyam ksagiyam force-pushed the ksagiyam/sparsity branch 2 times, most recently from 1a27598 to 8ccb56f Compare February 29, 2024 01:24
@ksagiyam
Copy link
Contributor Author

ksagiyam commented Mar 1, 2024

Can we merge this and the linked PyOP2 PR? I need to drop branch pointer before merging.

Copy link
Contributor

@connorjward connorjward left a comment

Choose a reason for hiding this comment

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

I think the documentation needs a little clarifying but otherwise I'm happy to approve this.

firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
firedrake/assemble.py Outdated Show resolved Hide resolved
@ksagiyam ksagiyam force-pushed the ksagiyam/sparsity branch from 8ccb56f to 66baa64 Compare March 1, 2024 17:04
connorjward
connorjward previously approved these changes Mar 1, 2024
@ksagiyam ksagiyam force-pushed the ksagiyam/sparsity branch from 66baa64 to f69a6a8 Compare March 1, 2024 21:21
@ksagiyam ksagiyam merged commit db30b35 into master Mar 1, 2024
8 checks passed
@ksagiyam ksagiyam deleted the ksagiyam/sparsity branch March 1, 2024 21:22
@ksagiyam
Copy link
Contributor Author

ksagiyam commented Mar 1, 2024

Thanks. Merged.

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

Successfully merging this pull request may close these issues.

3 participants