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
Adding Boruvka algorithm + tests for Minimum Spanning Trees #2676
Conversation
@Panquesito7 @realstealthninja reviews appreciated, thank you! |
@realstealthninja I've addressed your suggestions. Also changed from int -> uint32_t in the union find, where negative values also don't make sense. Let me know if there are any other issues. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contributions @Panquesito7 could you have a look as well?
Adding comment on the self-test function Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
@realstealthninja iterated on the last round of reviews! |
@realstealthninja can you approve the workflows? Thank you, |
@realstealthninja @Panquesito7 can you please approve the required workflows? I've addressed all the comments/suggested changes. Thank you, |
@realstealthninja @Panquesito7 any updates? I'd like to merge this if possible |
@realstealthninja @Panquesito7 can you kindly take a look? I have addressed all the changes requested and this is ready to merge. I can't initiate the checks and I am not currently a contributor. |
This pull request has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Please ping one of the maintainers once you commit the changes requested or make improvements on the code. If this is not the case and you need some help, feel free to ask for help in our Gitter channel or our Discord server. Thank you for your contributions! |
Description of Change
Checklist
Notes: