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

Add benchmarks for block validation #1715

Open
CjS77 opened this issue Apr 16, 2020 · 2 comments
Open

Add benchmarks for block validation #1715

CjS77 opened this issue Apr 16, 2020 · 2 comments
Labels
C-performance adds no new functionality but improves performance. Benchmarks will be included. C-tests Category - tests E-good_first_issue Experience Level - Good for newcomers

Comments

@CjS77
Copy link
Collaborator

CjS77 commented Apr 16, 2020

Add a benchmark test that times some "standard blocks"

  • Empty block
  • Full block - each transaction having 1 - 5 inputs and 1 - 20 outputs
  • One massive coinjoin (~1,000 inputs, 1 output)
  • One huge coin split (1 output, ~1,000 outputs)
  • An invalid block - invalid proof of work
  • An invalid block - kernel signature
  • Invalid blocks - invalid UTXO (double spend, and does not exist)
  • Invalid block - insufficient difficulty
@CjS77 CjS77 added E-good_first_issue Experience Level - Good for newcomers C-performance adds no new functionality but improves performance. Benchmarks will be included. C-tests Category - tests labels Apr 16, 2020
@willyrgf
Copy link
Contributor

willyrgf commented Aug 30, 2022

Hi @CjS77, I'm interested in try to solve this issue. There's some time since the last update here. Is this issue still valid?

@stringhandler
Copy link
Collaborator

Hi @willyrgf, yep, still valid.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-performance adds no new functionality but improves performance. Benchmarks will be included. C-tests Category - tests E-good_first_issue Experience Level - Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants