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.

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

[BENCH] Benchmark Welsh-Powell Algorithm #149

Closed
2 tasks
bobluppes opened this issue Oct 10, 2023 · 4 comments
Closed
2 tasks

[BENCH] Benchmark Welsh-Powell Algorithm #149

bobluppes opened this issue Oct 10, 2023 · 4 comments
Labels
good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed no-issue-activity performance Benchmarks or performance improvements

Comments

@bobluppes
Copy link
Owner

bobluppes commented Oct 10, 2023

Benchmark Welsh-Powell

The goal of this issues is to add a benchmark for the Welsh-Powell algorithm. The algorithm implementation is located under include/graaflib/algorithm/coloring/welsh_powell.h. Benchmarks are vital to our library and allow us to measure the impact of future performance improvements.

We use the Google benchmark framework. For inspiration, please take a look at the existing benchmarks in /perf.

The benchmark should be added under /perf in a directory which resembles the file structure of the original algorithm. i.e. if the algorithm is implemented in include/graaflib/algorithm/coloring/greedy_graph_coloring.h then the benchmark should be added to perf/graaflib/algorithm/coloring/greedy_graph_coloring_benchmark.cpp.

The benchmark should measure the runtime performance of the algorithm for increasing input sizes.

Running Benchmarks

If you IDE has the necessary integrations for it, all benchmarks can be run in the IDE from the perf/graaflib/benchmark.cpp file.

Otherwise, we can run the benchmarks from the command line:

# run all benchmarks
cd build/perf && ./Graaf_perf

To run an individual benchmark:

./Graaf_perf --benchmark_filter=YOUR_BENCHMARK_NAME

For more options, pass the --help flag.

Definition of Done

  • A benchmark is added for the algorithm
  • Benchmark results (copy past of the output is fine) is added to the PR
@bobluppes bobluppes added help wanted Extra attention is needed good first issue Good for newcomers hacktoberfest performance Benchmarks or performance improvements labels Oct 10, 2023
@github-actions
Copy link
Contributor

Stale issue message

@KathanrDave
Copy link

@bobluppes if you are working on this issue ?
if yes can you assign it to me

Copy link
Contributor

Stale issue message

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Nov 21, 2023
@bobluppes bobluppes reopened this Nov 25, 2023
Copy link
Contributor

Stale issue message

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed no-issue-activity performance Benchmarks or performance improvements
Projects
None yet
Development

No branches or pull requests

2 participants