Skip to content

Add benchmark for gix-blame - #3124

Merged
Sebastian Thiel (Byron) merged 2 commits into
GitoxideLabs:mainfrom
cruessler:add-benchmark-for-gix-blame
Oct 25, 2026
Merged

Sebastian Thiel (Byron) merged 2 commits into
GitoxideLabs:mainfrom
cruessler:add-benchmark-for-gix-blame

Conversation

@cruessler

Copy link
Copy Markdown
Contributor

This PR adds a benchmark for gix-blame. This will hopefully be the basis for some upcoming work on gix-blame that I intend to do.

The benchmark is deliberately simple, and it deliberately does not set up a synthetic repository, but measures gix_blame::file() on a set of files from the gitoxide repository itself. I hope that the files I chose are somewhat representative of real-world workloads.

The benchmark only measures gix_blame::file(), excluding the time it takes to construct and destruct the resource cache.

# to capture a baseline
cargo bench -p gix-blame --bench blame -- --save-baseline baseline

# to compare a change against a captured baseline
cargo bench -p gix-blame --bench blame -- --baseline baseline

Assisted-by: GPT 6.1 Sol
@cruessler
Christoph Rüßler (cruessler) marked this pull request as ready for review October 25, 2026 17:19
@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review 🔄 Running since 2026-10-25T17:19:43.912308Z 950f408 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Byron

Byron commented Oct 25, 2026

Copy link
Copy Markdown
Member

Thanks!

In future, I'd hope to see what's in the PR description in the commit as well, as I still think that having as much context as possible in the commit history is valuable, also in the long run. Also no need to rely on information that is on GitHub exclusively.

@Byron
Sebastian Thiel (Byron) merged commit 9f805e2 into GitoxideLabs:main Oct 25, 2026
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants