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

Updates to benchmark random inputs #275

Merged
merged 4 commits into from
Aug 31, 2022
Merged

Updates to benchmark random inputs #275

merged 4 commits into from
Aug 31, 2022

Conversation

chfast
Copy link
Owner

@chfast chfast commented Aug 30, 2022

No description provided.

@chfast chfast requested review from gumb0 and rodiazet August 30, 2022 12:27
Base automatically changed from benchmark_fixes to master August 31, 2022 06:31
chfast added 4 commits August 31, 2022 08:32
Create lt_256 set without modifying {x,y}_256 sets. Otherwise, the
distribution of the later is changed.
Use separate sample sets for arguments, include mixed set.
@codecov-commenter
Copy link

codecov-commenter commented Aug 31, 2022

Codecov Report

Merging #275 (aa76fa5) into master (aa76fa5) will not change coverage.
The diff coverage is n/a.

❗ Current head aa76fa5 differs from pull request most recent head 3b099f5. Consider uploading reports for the commit 3b099f5 to get more accurate results

@@            Coverage Diff            @@
##            master      #275   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines         1941      1941           
=========================================
  Hits          1941      1941           
Flag Coverage Δ
32bit 100.00% <0.00%> (ø)
gcc 99.47% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@chfast chfast merged commit 1e6dd26 into master Aug 31, 2022
@chfast chfast deleted the benchmarks branch August 31, 2022 06:49
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