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

Add benchmarks for Reclass::render_inventory() #47

Merged
merged 2 commits into from
Jan 10, 2024
Merged

Conversation

simu
Copy link
Member

@simu simu commented Sep 21, 2023

We add two standalone benchmarks for single-threaded and multi-threaded inventory rendering. The benchmarks are implemented with criterion.

We need to have two separate benchmark binaries, since we're currently using the global rayon thread pool which can't be reinitialized.

Checklist

  • The PR has a meaningful title. The title will be used to auto generate the changelog
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency, internal
    as they show up in the changelog

We add two standalone benchmarks for single-threaded and multi-threaded
inventory rendering. The benchmarks are implemented with `criterion`.

We need to have two separate benchmark binaries, since we're currently
using the global `rayon` thread pool which can't be reinitialized.
@simu simu force-pushed the feat/bench-inventory branch 5 times, most recently from 86a8ee5 to 79b84a9 Compare January 9, 2024 16:14
@simu simu requested a review from a team January 10, 2024 09:17
@simu
Copy link
Member Author

simu commented Jan 10, 2024

See #75 for an additional GitHub action job which runs the benchmarks for PRs and posts the result as a comment

@simu simu merged commit c69f978 into main Jan 10, 2024
17 checks passed
@simu simu deleted the feat/bench-inventory branch January 10, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants