Skip to content

Conversation

@Shourya742
Copy link
Collaborator

@Shourya742 Shourya742 commented Dec 19, 2025

Closes: #2041

This PR adds noise benches, targeting the handshake and post-handshake scenario's.

Copy link
Contributor

@lucasbalieiro lucasbalieiro left a comment

Choose a reason for hiding this comment

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

this is one of my first times working with bench code, so I added a few conceptual questions to make sure my understanding is correct. If you have time, I’d appreciate your thoughts on them.

Also, I was able to run all the benches locally without problems @ 6011a60

@Shourya742 Shourya742 force-pushed the 2025-12-19-add-noise-bench branch 2 times, most recently from 11ba2d9 to cf62086 Compare January 19, 2026 14:01
@Shourya742 Shourya742 force-pushed the 2025-12-19-add-noise-bench branch from 6b7ec1d to c69339b Compare January 20, 2026 16:50
@plebhash
Copy link
Member

plebhash commented Jan 20, 2026

it would be good to add some words about this to the noise_sv2 crate's README.md

also a BENCHES.md with results

@Shourya742 Shourya742 requested a review from GitGab19 January 21, 2026 05:32
Copy link
Contributor

@lucasbalieiro lucasbalieiro left a comment

Choose a reason for hiding this comment

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

tACK

944c2c7

@Shourya742 Shourya742 force-pushed the 2025-12-19-add-noise-bench branch from 944c2c7 to fc93767 Compare January 21, 2026 15:45
@GitGab19 GitGab19 merged commit b3c550b into stratum-mining:main Jan 21, 2026
15 checks passed
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.

Benchmarking noise-sv2

4 participants