Skip to content

Show time spent per block in benchmark#3019

Merged
cody-littley merged 1 commit intomainfrom
cody-littley/time-per-block
Mar 4, 2026
Merged

Show time spent per block in benchmark#3019
cody-littley merged 1 commit intomainfrom
cody-littley/time-per-block

Conversation

@cody-littley
Copy link
Contributor

Describe your changes and provide context

Screenshot 2026-03-04 at 12 42 51 PM

Adds a new visualization that shows time spent per block doing various operations. This is amortized time, since we don't, for example, commit after every block.

Testing performed to validate your change

Local testing.

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMar 4, 2026, 6:45 PM

@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.39%. Comparing base (e45e0b9) to head (086caeb).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3019      +/-   ##
==========================================
- Coverage   58.40%   58.39%   -0.02%     
==========================================
  Files        2110     2109       -1     
  Lines      175009   174941      -68     
==========================================
- Hits       102220   102155      -65     
+ Misses      63761    63755       -6     
- Partials     9028     9031       +3     
Flag Coverage Δ
sei-db 70.62% <ø> (+0.21%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 32 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cody-littley cody-littley merged commit ca6816a into main Mar 4, 2026
40 of 42 checks passed
@cody-littley cody-littley deleted the cody-littley/time-per-block branch March 4, 2026 20:08
yzang2019 pushed a commit that referenced this pull request Mar 19, 2026
## Describe your changes and provide context

<img width="1603" height="921" alt="Screenshot 2026-03-04 at 12 42
51 PM"
src="https://github.com/user-attachments/assets/ac57c9dc-e4a9-40bb-a6e5-15994d52a489"
/>

Adds a new visualization that shows time spent per block doing various
operations. This is amortized time, since we don't, for example, commit
after every block.

## Testing performed to validate your change

Local testing.

Co-authored-by: Cody Littley <cody.littley@seinetwork.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants