Skip to content

Subrounds timing by epoch#7881

Open
sstanculeanu wants to merge 3 commits into
feat/testnet-fixesfrom
subrounds-timing-by-epoch
Open

Subrounds timing by epoch#7881
sstanculeanu wants to merge 3 commits into
feat/testnet-fixesfrom
subrounds-timing-by-epoch

Conversation

@sstanculeanu

Copy link
Copy Markdown
Collaborator

Reasoning behind the pull request

Proposed changes

Testing procedure

Pre-requisites

Based on the Contributing Guidelines the PR author and the reviewers must check the following requirements are met:

  • was the PR targeted to the correct branch?
  • if this is a larger feature that probably needs more than one PR, is there a feat branch created?
  • if this is a feat branch merging, do all satellite projects have a proper tag inside go.mod?

@sstanculeanu sstanculeanu self-assigned this Jun 16, 2026
@sstanculeanu sstanculeanu added the ignore-for-release-notes Do not include item in release notes label Jun 16, 2026
@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.23810% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.58%. Comparing base (818563b) to head (55a0eb3).

Files with missing lines Patch % Lines
consensus/spos/bls/proxy/subroundsHandler.go 80.00% 1 Missing and 1 partial ⚠️
common/configs/commonConfigs.go 80.00% 1 Missing ⚠️
Additional details and impacted files
@@                  Coverage Diff                   @@
##           feat/testnet-fixes    #7881      +/-   ##
======================================================
- Coverage               77.59%   77.58%   -0.01%     
======================================================
  Files                     884      884              
  Lines                  125249   125273      +24     
======================================================
+ Hits                    97181    97188       +7     
- Misses                  21618    21634      +16     
- Partials                 6450     6451       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

*optional: keep the constants for v1, and update the factory only for v2

Comment on lines +113 to +117
SubroundStartStartTime = 0.0,
SubroundStartEndTime = 0.05,
SubroundBlockStartTime = 0.05,
SubroundBlockEndTime = 0.25,
SubroundSignatureStartTime = 0.25,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

*optional: remove Subround prefix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release-notes Do not include item in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants