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

[CHIA-786] Update tools to work with the hard-fork #581

Merged
merged 3 commits into from
Jun 20, 2024
Merged

Conversation

arvidn
Copy link
Contributor

@arvidn arvidn commented Jun 19, 2024

The main change is to make analyze-chain use run_block_generator() and to also make test-block-generators correctly switch to run_block_generator2() at the hard fork block height.

Since analyze-chain is changed to use run_block_generator() instead of run_program_with_counters() it can no longer collect operator- and stack size counters. The output is changed and the log parser scripts are updated to account for this.

Example output from analyze-chain:

block-wall-clock-delta-cdf
block-execution-time-cdf

@arvidn arvidn requested a review from Rigidity June 19, 2024 09:28
Copy link

coveralls-official bot commented Jun 19, 2024

Pull Request Test Coverage Report for Build 9579407440

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 573 unchanged lines in 19 files lost coverage.
  • Overall coverage decreased (-4.1%) to 78.715%

Files with Coverage Reduction New Missed Lines %
crates/chia-consensus/src/gen/run_puzzle.rs 1 98.31%
crates/chia-traits/src/streamable.rs 1 94.03%
crates/chia-consensus/src/gen/run_block_generator.rs 2 94.83%
crates/chia-bls/src/parse_hex.rs 3 60.0%
crates/chia-consensus/src/gen/validation_error.rs 4 24.19%
wheel/src/adapt_response.rs 4 0.0%
crates/chia-consensus/src/gen/conditions.rs 5 99.37%
crates/clvm-traits/src/clvm_decoder.rs 6 83.33%
crates/chia-protocol/src/lazy_node.rs 9 64.0%
wheel/src/run_program.rs 10 70.59%
Totals Coverage Status
Change from base Build 9562939778: -4.1%
Covered Lines: 11098
Relevant Lines: 14099

💛 - Coveralls

@arvidn arvidn changed the title Update tools to work with the hard-fork [CHIA-786] Update tools to work with the hard-fork Jun 20, 2024
@arvidn arvidn merged commit a3bd807 into main Jun 20, 2024
64 checks passed
@arvidn arvidn deleted the update-tools branch June 20, 2024 11:16
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