Skip to content

Babe/time#1568

Merged
turuslan merged 8 commits intomasterfrom
babe/time
Apr 20, 2023
Merged

Babe/time#1568
turuslan merged 8 commits intomasterfrom
babe/time

Conversation

@turuslan
Copy link
Copy Markdown
Contributor

Referenced issues

Description of the Change

  • Start block production 2 seconds (one third of slot) earlier.
  • Save timestamp at start of slot.

Benefits

Possible Drawbacks

Signed-off-by: turuslan <turuslan.devbox@gmail.com>
Signed-off-by: turuslan <turuslan.devbox@gmail.com>
Signed-off-by: turuslan <turuslan.devbox@gmail.com>
Comment thread core/consensus/babe/impl/babe_impl.cpp Outdated
Comment thread core/consensus/babe/impl/babe_impl.cpp Outdated
Comment thread core/consensus/babe/impl/babe_impl.hpp Outdated
Signed-off-by: turuslan <turuslan.devbox@gmail.com>
@turuslan turuslan requested a review from xDimon April 17, 2023 13:10
xDimon
xDimon previously approved these changes Apr 17, 2023
Copy link
Copy Markdown
Contributor

@kamilsa kamilsa left a comment

Choose a reason for hiding this comment

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

Please add a condition in proposer that when the deadline (4 seconds from the beginning of the slot) is reached we do not add new transactions to the block

Signed-off-by: turuslan <turuslan.devbox@gmail.com>
Signed-off-by: turuslan <turuslan.devbox@gmail.com>
@turuslan turuslan enabled auto-merge (squash) April 19, 2023 09:41
@turuslan turuslan merged commit ef9d8ba into master Apr 20, 2023
@turuslan turuslan deleted the babe/time branch April 20, 2023 07:35
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.

3 participants