-
Notifications
You must be signed in to change notification settings - Fork 147
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
Add summary of the rustworkx benchmarks to the documentation #736
Conversation
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.
Thanks for taking care of this. I have three minor comments regarding the images:
- we might need to increase the axis on the Time to create a weighted directed graph
- we need to update the captions to say
rustworkx
notretworkx
- Because we are on the web I think saving the image as an
.svg
might be better than using.png
Pull Request Test Coverage Report for Build 3510505910
💛 - Coveralls |
I made changes according to your comments. Let me know if anything else pending |
Can you run I will take over and add these details and the citations for where we grabbed the datasets, then we should be good to go |
build from main branch https://git.skewed.de/count0/graph-tool.git, Debian GNU/Linux 11 (bullseye), Intel i9-9900K 32Gb of ram
|
benchmark of https://github.com/mtreinish/retworkx-comparison-benchmarks . Refer to issue #550