-
Notifications
You must be signed in to change notification settings - Fork 16
Direct Solver: Multilevel Proxy-based Solver #174
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
Open
alexfikl
wants to merge
2
commits into
inducer:main
Choose a base branch
from
alexfikl:direct-solver-solve
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
Author
|
This also contains #164 at the moment and is still quite WIP. For one, only implemented a forward matvec, not the actual solver 😢 |
5e7f654 to
3164799
Compare
a00d5a0 to
6efa838
Compare
ae83dd5 to
0105f84
Compare
0105f84 to
9ed57c2
Compare
871c491 to
c742491
Compare
Collaborator
Author
|
This now implements both a matvec and a direct solver with recursive skeletonization. Both seem to work pretty close to design parameters. Still need to stress test it more though! |
7d5bb22 to
ddc4a8b
Compare
ddc4a8b to
d3f168c
Compare
1524bc9 to
af9c2d0
Compare
af9c2d0 to
28c1c92
Compare
28c1c92 to
8813bce
Compare
8813bce to
be87bed
Compare
64a779a to
158afc9
Compare
d6fc4ac to
ee7fb16
Compare
47c06b1 to
83658ca
Compare
83658ca to
0e74b01
Compare
7211102 to
bf42a83
Compare
bf42a83 to
ae80999
Compare
ae80999 to
7736d36
Compare
d13c752 to
7ee43b6
Compare
b887731 to
6303f9e
Compare
6303f9e to
5d37d51
Compare
6042a67 to
6a6db24
Compare
6a6db24 to
ee6670b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Attempting to break up the direct solver MR into smaller, more reviewable pieces. As a rundown
linalg.proxy).The code for this is mostly in the deprecated https://gitlab.tiker.net/inducer/pytential/-/merge_requests/137.