Skip to content

Add tps calculation based on instant blocks#2813

Merged
stevenlanders merged 5 commits intomainfrom
steven/add-theroetical-tps
Feb 10, 2026
Merged

Add tps calculation based on instant blocks#2813
stevenlanders merged 5 commits intomainfrom
steven/add-theroetical-tps

Conversation

@stevenlanders
Copy link
Contributor

Describe your changes and provide context

  • calculates theoreticalTPS so we can better forecast what autobahn will yield

Testing performed to validate your change

  • unit test
  • see new log attributes emitted by scripts/benchmark.sh

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

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

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedFeb 10, 2026, 3:03 PM

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

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

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedFeb 5, 2026, 9:12 PM

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.76%. Comparing base (28bc1c0) to head (ada2f8b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2813      +/-   ##
==========================================
- Coverage   56.77%   56.76%   -0.02%     
==========================================
  Files        2070     2070              
  Lines      168274   168281       +7     
==========================================
- Hits        95542    95526      -16     
- Misses      64247    64265      +18     
- Partials     8485     8490       +5     
Flag Coverage Δ
sei-chain 40.84% <100.00%> (-0.02%) ⬇️
sei-cosmos 48.13% <ø> (+0.01%) ⬆️
sei-db 68.72% <ø> (ø)
sei-tendermint 58.71% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
app/benchmark/logger.go 93.33% <100.00%> (+0.41%) ⬆️

... and 30 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.

if blockCount <= 0 || avgBlockProcessMs <= 0 {
return 0.0
}
avgTxsPerBlock := float64(txCount) / float64(blockCount)

Check notice

Code scanning / CodeQL

Floating point arithmetic Note test

Floating point arithmetic operations are not associative and a possible source of non-determinism
return 0.0
}
avgTxsPerBlock := float64(txCount) / float64(blockCount)
return avgTxsPerBlock * 1000 / float64(avgBlockProcessMs)

Check notice

Code scanning / CodeQL

Floating point arithmetic Note test

Floating point arithmetic operations are not associative and a possible source of non-determinism
return 0.0
}
avgTxsPerBlock := float64(txCount) / float64(blockCount)
return avgTxsPerBlock * 1000 / float64(avgBlockProcessMs)

Check notice

Code scanning / CodeQL

Floating point arithmetic Note test

Floating point arithmetic operations are not associative and a possible source of non-determinism
@stevenlanders stevenlanders enabled auto-merge (squash) February 10, 2026 15:06
@stevenlanders stevenlanders merged commit 077f083 into main Feb 10, 2026
43 checks passed
@stevenlanders stevenlanders deleted the steven/add-theroetical-tps branch February 10, 2026 15:17
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