Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix benchmarks #1086

Merged
merged 1 commit into from
Aug 27, 2024
Merged

Fix benchmarks #1086

merged 1 commit into from
Aug 27, 2024

Conversation

prymitive
Copy link
Collaborator

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request was validated by pint.

✔️ No problems found

Stats

Stat Value
Version v0.64.0-53-gbc99058
Number of rules parsed 4
Number of rules checked 4
Number of problems found 0
Number of offline checks 0
Number of online checks 0
Checks duration 0

Problems

No problems reported

Copy link

github-actions bot commented Aug 27, 2024

Benchmark diff:

goos: linux
goarch: amd64
pkg: github.com/cloudflare/pint/cmd/pint
cpu: AMD EPYC 7763 64-Core Processor                
              │   old.txt   │              new.txt               │
              │   sec/op    │   sec/op     vs base               │
FindEntries-4   39.80m ± 1%   39.73m ± 1%       ~ (p=0.912 n=10)
CheckRules-4                   8.525 ± 2%
geomean         39.80m        582.0m       -0.17%

              │   old.txt    │               new.txt               │
              │     B/op     │     B/op      vs base               │
FindEntries-4   27.41Mi ± 0%   27.41Mi ± 0%       ~ (p=0.165 n=10)
CheckRules-4                   978.6Mi ± 0%
geomean         27.41Mi        163.8Mi       +0.01%

              │   old.txt   │              new.txt               │
              │  allocs/op  │  allocs/op   vs base               │
FindEntries-4   131.9k ± 0%   131.9k ± 0%       ~ (p=0.098 n=10)
CheckRules-4                  14.20M ± 0%
geomean         131.9k        1.368M       +0.00%

@codecov-commenter
Copy link

codecov-commenter commented Aug 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.89%. Comparing base (7d0d534) to head (bc99058).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1086   +/-   ##
=======================================
  Coverage   94.89%   94.89%           
=======================================
  Files         102      102           
  Lines       10184    10188    +4     
=======================================
+ Hits         9664     9668    +4     
  Misses        365      365           
  Partials      155      155           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@prymitive prymitive merged commit 6630e3d into main Aug 27, 2024
20 checks passed
@prymitive prymitive deleted the cmd branch August 27, 2024 07:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants