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

Draft: Add Maximum matching algorithm in experimental #254

Draft
wants to merge 60 commits into
base: v1.2
Choose a base branch
from
Draft

Draft: Add Maximum matching algorithm in experimental #254

wants to merge 60 commits into from

Conversation

kchristin22
Copy link

This PR introduces the Maximum Matching algorithm as presented in the paper: Distributed-Memory Algorithms for Maximum Cardinality Matching in Bipartite Graphs, using GraphBLAS operations.

TODO:

  • add push-pull optimization
  • convert to uint32 when the data set is small enough
  • create a separate vanilla version or add pragma to differentiate the function calls based on the GraphBLAS implementation used

@kchristin22
Add initial commit of maximum matching
3a9d316
@kchristin22
Fix initialization of frontierC and add test for debugging purposes
2906e70
@kchristin22
Add test of semiring in MCM
e2bb41a
@kchristin22
Discard visited rows and Set parents of newly visited rows
d8b48fa
@kchristin22
Compute mateR through inverting mateC
9f49b4c
@kchristin22
Update pathC
ee80f27
@kchristin22
Fix vector_build usage
efbc6ea
@kchristin22
Draft: Add PRUNE function
402f381
@kchristin22
Finish PRUNE function
23d648b
@kchristin22
Improve PRUNE, finish inner loop body and clean up
8c761f7
@kchristin22
Implement comments
5112c8d
@kchristin22
Draft: fix previous build warning, add augment and test multiple iter…
d28e141 
…ations case
@kchristin22
Use mateR copy for each iteration and check if vectors are empty befo…
18f23b2 
…re inverting
@kchristin22
Improve descriptor in ur assignment and add explanation in a comment
80a08e4
@kchristin22
Fix how parents of fR are updated
582a59c
@kchristin22
Delete unnecessary allocations and wrong deallocations to avoid memor…
6671d1e 
…y leaks
@kchristin22
Use jumble flag of pack when inverting and allow indices to be unsort…
c9b5670 
…ed when not required
@kchristin22
Fix LAGraph_Free calls
42479d7
@kchristin22
Make I a bool vector, add another vector to copy to output and free w…
610304f 
…ork in the end
@kchristin22
Add the burble and evaluation of the matching in MCM test
2c8c11c
@kchristin22
Fix shared jumbled var among vectors bug
f872354
@kchristin22
Test with a matrix from the data set of LAGraph
f3a5c0b
@kchristin22
Add second test and comment out any debugging prints
4032a53
@kchristin22
Specify nvals name per vector
bddfc36
@kchristin22
Parametrize tests of multiple data sets
eaf9d0e
@kchristin22
Change input graph to matrix to support rectangular adjacency matrices
52fce12
@kchristin22
Change input graph of MaximumMatching to matrix
4091ddb
@kchristin22
Use ur vector to compute mateR instead of inverting mateC
31c67d1
@kchristin22
Fix and improve the algorithm
325e9bc
@kchristin22
Merge remote-tracking branch 'upstream/v1.2' into maximum-matching
3b1b6a6
kchristin22 and others added 19 commits July 11, 2024 12:18
@kchristin22
Add push-pull optimization and enable it in tests and benchmarks
dea5618
@DrTimothyAldenDavis
comments and timing
cedcc81
@kchristin22
Fix mateC initialization on the test
8b086f7
@DrTimothyAldenDavis
starting to add A,AT options
bd45f2e
@DrTimothyAldenDavis
.
6a40f82
@DrTimothyAldenDavis
add check of input/output of invert
141df7b
@kchristin22
Fix setting of mateR and inverting of vr in the augment function
1f18d60
@DrTimothyAldenDavis
debugging for Orkut matrix, added check_matching
96839fa
@kchristin22
Print number of matches in the benchmark
ce5ead5
@kchristin22
Merge remote-tracking branch 'refs/remotes/origin/maximum-matching' i…
e41787e 
…nto maximum-matching
@kchristin22
Clear frontierC before updating it and clean up
d63aa7e
@kchristin22
Add check of the matching in benchmark and fix memory leakage in tests
dd62fd4
@kchristin22
Revert mateR update method and use one trial for the benchmarks
9aed507
@kchristin22
Apply mask separately in vxm
df0c3e0
@kchristin22
Use 3 trails in benchamrking for a better sample than having just one
138d4bc
@kchristin22
Fix push-only method
7373fb4
@kchristin22
Allow mateR as initial matching and add mateR as output
6b593a9
@kchristin22
First half of vanilla version: missing invert2
7788238
@kchristin22
Merge vxm and mask operations again for testing
edb3df9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kchristin22 @DrTimothyAldenDavis