-
Notifications
You must be signed in to change notification settings - Fork 96
Does recursive Bisection support breadth-first algorithm in the future? #144
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
Comments
Hey @lucifer4011, Out of curiosity: What values of In general, and if |
@SebastianSchlag |
Thanks a lot for your explanation!
Yes that's correct. To KaHyPar, both of the partitions in your figure are identical. I'm not entirely sure if it would help in your case, but one thing you could maybe look into is using fixed vertices in your hypergraph to encode some locality information. |
@SebastianSchlag |
The problem which you describe is an application of process mapping. I already describe this problem in another issue (see Issue #126). You can also find there some useful references. We currently plan to integrate this feature into Mt-Kahypar, which is the shared-memory parallelisation of KaHyPar. I will keep you updated once it is available. |
@kittobi1992 thaks! The cause of the previous problem has been found. It is because kahypar is the concept of a logical part, which means that the interval between each part is the same. That's why the above problem occurs. I have found another way to solve it. Thanks! |
The process mapping problem takes the distance between two blocks into account and has also a rich literature. But it is great to hear that you found another solution 😉 |
The code in func (recursive_bisection) only supports Depth-first algorithm. it means that the partition processes one side completely by recursive bisection , and then processes the other side. But i find the result is very worse when the part is large.
Does recursive Bisection Support support breadth-first algorithm in the future? I guess the feature will improve performance significantly.
thanks for your reply!
The text was updated successfully, but these errors were encountered: