Skip to content

New synthetic generator for benchmarks#710

Open
richardkiss wants to merge 2 commits intomainfrom
bench_large_generator
Open

New synthetic generator for benchmarks#710
richardkiss wants to merge 2 commits intomainfrom
bench_large_generator

Conversation

@richardkiss
Copy link
Contributor

@richardkiss richardkiss commented Feb 24, 2026

This pull request expands the benchmark coverage in the project by adding a new data block (5.generator) to both the intern and serialize benchmarks.

This generator was created using https://github.com/richardkiss/chia-scan and is extremely large (and probably actually exceeds the cost if we were to run it).


Note

Low Risk
Bench-only change that adds another benchmark input; main risk is longer/more resource-intensive benchmark runs due to a very large generator file.

Overview
Extends benchmark coverage by adding a new 5.generator input and including it in the iteration sets for both intern and serialize Criterion benchmark groups.

This increases the range of generator sizes exercised by the benchmarks without changing the benchmark logic itself.

Written by Cursor Bugbot for commit 3117d43. This will update automatically on new commits. Configure here.

Copilot AI review requested due to automatic review settings February 24, 2026 01:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR expands existing Criterion benchmark coverage by adding a new, additional generator input (5.generator) to the intern and serialize benchmark suites.

Changes:

  • Add 5.generator as an embedded benchmark input in benches/serialize.rs
  • Add 5.generator as an embedded benchmark input in benches/intern.rs

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File Description
benches/serialize.rs Includes 5.generator and adds it to the serialize benchmark input set.
benches/intern.rs Includes 5.generator and adds it to the intern benchmark input set.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coveralls-official
Copy link

Pull Request Test Coverage Report for Build 22333171523

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 88.222%

Totals Coverage Status
Change from base Build 22327026712: 0.0%
Covered Lines: 6831
Relevant Lines: 7743

💛 - Coveralls

@richardkiss richardkiss enabled auto-merge (squash) February 25, 2026 22:36
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.

3 participants