Skip to content

feat/cleanup report#455

Open
zeroXbrock wants to merge 7 commits intomainfrom
feat/cleanup-report
Open

feat/cleanup report#455
zeroXbrock wants to merge 7 commits intomainfrom
feat/cleanup-report

Conversation

@zeroXbrock
Copy link
Member

Motivation

clean up visual artifacts & bugs in reports

Solution

  • cleanup report charts
    • show slot when hovering heatmap
    • remove heatmap idle labels
    • limit number of X-axis rows in Time To Inclusion chart
    • log metrics for batched transactions
      • this normally happens in the RPC client middleware but for batches we don't use that so we have to do it manually in TestScenario
  • fix unchecked error in reports (if txs are not found)
  • handle bad params in fill_block to prevent panic
  • collect metrics for batched eth_sendRawTransaction calls

PR Checklist

  • Added Tests
  • Added Documentation
  • Ran cargo +nightly clippy --workspace --lib --examples --tests --benches --all-features --locked --fix
  • Ran cargo fmt --all
  • Note breaking changes in PR description, if applicable
  • update changelogs
    • Update CHANGELOG.md in each affected crate
    • add a high-level description in the root changelog

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.

1 participant