Skip to content

Performance testing#48

Merged
sebastiankreutzer merged 1 commit intotudasc:develfrom
sebastiankreutzer:feat/perf_test
Jul 7, 2025
Merged

Performance testing#48
sebastiankreutzer merged 1 commit intotudasc:develfrom
sebastiankreutzer:feat/perf_test

Conversation

@sebastiankreutzer
Copy link
Member

This adds performance tests for the graph lib. I wrote this as a weekend project to be able to evaluate the performance differences of the upcoming graphlib re-write.
As a follow-up, this can be integrated into the CI to catch performance regressions.

Usage: ./perftest <num_nodes> <num_edges> <rng_seed>

Results for current devel on my laptop:

Test results for 10000 nodes and 20000 edges (seed: 463248923)
---------------------------
Test Name         Time (ms)
---------------------------
read V2 format        40736
write V2 format         866
read V3 format          230
write V3 format         580
insert edges            131
insert nodes             31
---------------------------

@sebastiankreutzer sebastiankreutzer requested a review from jplehr July 6, 2025 12:24
@sebastiankreutzer sebastiankreutzer added the enhancement New feature or request label Jul 6, 2025
Copy link
Member

@jplehr jplehr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, that's cool stuff.

For the future: we can also think of generating some long strings as names, since the older formats are showing real slowness with long strings and they are quite common for C++ mangled names.

@jplehr
Copy link
Member

jplehr commented Jul 6, 2025

I forgot one thing: can you update the commit title to be something like

[Graph] Add initial performance tests

Or something like that? Thank you.

@sebastiankreutzer
Copy link
Member Author

Thanks, that's cool stuff.

For the future: we can also think of generating some long strings as names, since the older formats are showing real slowness with long strings and they are quite common for C++ mangled names.

Yes, I'll try that out. Thanks for the quick review!

@sebastiankreutzer sebastiankreutzer merged commit 2ce9326 into tudasc:devel Jul 7, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants