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

Simple Benchmark #4

Merged
merged 1 commit into from
May 29, 2023
Merged

Simple Benchmark #4

merged 1 commit into from
May 29, 2023

Conversation

jairbubbles
Copy link
Contributor

@jairbubbles jairbubbles commented May 18, 2023

  • Simple benchmark that compares GitReader perf to libGit2Sharp
    • Repository is opened and we get all branches / stashes / tags
    • Tests are run for multiple frameworks
    • GitReader does not provide all info yet (for instance, it's missings tracked branches, tracking details etc...)
  • Use PolySharp to allow using list patterns on older frameworks not only nullability

NB: I'm not too experienced with Benchmark.Net but there's probably room for improvements to make tests run faster without affecting the results

Here is an example of the output summary:

Method Runtime Mean Error StdDev
GitReader .NET 6.0 4.865 ms 0.0512 ms 0.0454 ms
GitReaderStructured .NET 6.0 69.200 ms 1.3674 ms 1.4631 ms
LibGit2sharp .NET 6.0 22.208 ms 0.0958 ms 0.0748 ms
GitReader .NET 7.0 4.853 ms 0.0210 ms 0.0196 ms
GitReaderStructured .NET 7.0 68.018 ms 0.8555 ms 0.8002 ms
LibGit2sharp .NET 7.0 22.183 ms 0.1585 ms 0.1405 ms
GitReader .NET Framework 4.8 8.096 ms 0.0148 ms 0.0138 ms
GitReaderStructured .NET Framework 4.8 249.047 ms 1.9543 ms 1.8281 ms
LibGit2sharp .NET Framework 4.8 22.679 ms 0.1812 ms 0.1513 ms

@jairbubbles jairbubbles changed the base branch from main to develop May 18, 2023 08:15
@jairbubbles jairbubbles mentioned this pull request May 18, 2023
@kekyo
Copy link
Owner

kekyo commented May 28, 2023

If all is well, please activate the review request. If you still have work to do, please continue.

@jairbubbles jairbubbles marked this pull request as ready for review May 28, 2023 12:58
@kekyo kekyo merged commit 93c87d8 into kekyo:develop May 29, 2023
@kekyo
Copy link
Owner

kekyo commented May 29, 2023

Merged!

There were a few problems, which will be corrected later.

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.

None yet

2 participants