Skip to content

Conversation

@arvidn
Copy link
Contributor

@arvidn arvidn commented Dec 23, 2025

This is a collection of a few minor fixes to benchmark-clvm-cost.

  • every ChiaDialect used to benchmark is created with flags 0. Create an abstraction to easily change the flags for all dialect objects.
  • One assumption in measuring the cost-per-byte is that 0 bytes have 0 cost. When performing the linear regression, add a bias towards (0,0).
  • Extend write_gnuplot_header() to allow better titles to the plots
  • Fix issue in the sha256 operator test, where the parameter was hard-coded rather than substituted by the test atoms. This caused the cost-per-byte to always be 0.

per-byte benchmarks

before

time: per-byte: 2.78ns

sha256-per-byte

after

time: per-byte: 2.86ns

sha256-per-byte

@coveralls-official
Copy link

coveralls-official bot commented Dec 23, 2025

Pull Request Test Coverage Report for Build 20473952212

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 91.549%

Totals Coverage Status
Change from base Build 20434068006: 0.0%
Covered Lines: 6294
Relevant Lines: 6875

💛 - Coveralls

@arvidn arvidn requested a review from matt-o-how December 23, 2025 23:13
@arvidn arvidn force-pushed the benchmark-clvm-fix branch from 5715beb to d8774ec Compare December 23, 2025 23:30
@arvidn arvidn merged commit e4850f3 into main Jan 5, 2026
30 checks passed
@arvidn arvidn deleted the benchmark-clvm-fix branch January 5, 2026 11:47
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