Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Dynamically adjust PoH hashes per tick based on cluster participants #4332

Closed
2 tasks done
mvines opened this issue May 17, 2019 · 4 comments
Closed
2 tasks done

Dynamically adjust PoH hashes per tick based on cluster participants #4332

mvines opened this issue May 17, 2019 · 4 comments
Labels
stale [bot only] Added to stale content; results in auto-close after a week.
Milestone

Comments

@mvines
Copy link
Contributor

mvines commented May 17, 2019

As of #4292, the number of hashes per tick to achieve the cluster's target tick duration is set statically in the genesis block. But hardware will vary, generally should get faster over time, so that same hashes-per-tick value should cause tick durations to decrease (or otherwise vary) over time.

The underlying assumption is that we would prefer the wallclock duration of each tick to remain as constant as possible throughout the life of the cluster.

This means that:

  1. Periodically (perhaps within a vote-like transaction), validators should report their actual tick duration
  2. Periodically (perhaps at the end of an epoch), the hashes-per-tick value for the next epoch should be computed based on the stake weighted average of the reported tick durations from the previous epoch.

NB: Congestion-driven fees are likely to require a similar mechanism to agree on a value, see
#4167 (comment)

As a part of fixing this issue enable real PoH on (eg. 20b2be6)

  • The testnet testnet
  • The TdS testnet
@mvines mvines added this to the Trestles v0.16.0 milestone May 17, 2019
@aeyakovenko
Copy link
Member

Dynamic measurement is a big attack surface.

  1. Add a wallclock to the vote instruction
  2. Gossip/leaders should honor the wallclock and drop old votes. This puts pressure on nodes to give an honest wallclock.
  3. Compute the stake weighted median from all the deltas between votes. That should be set as a network constant for the next epoch.

@aeyakovenko
Copy link
Member

And make sure the rate doesn’t adjust by more than 33% per epoch

@stale
Copy link

stale bot commented Jan 14, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale [bot only] Added to stale content; results in auto-close after a week. label Jan 14, 2021
@stale
Copy link

stale bot commented Jan 23, 2021

This stale issue has been automatically closed. Thank you for your contributions.

@stale stale bot closed this as completed Jan 23, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale [bot only] Added to stale content; results in auto-close after a week.
Projects
None yet
Development

No branches or pull requests

2 participants