Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1 second finality #4820

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

1 second finality #4820

wants to merge 4 commits into from

Conversation

Frozen
Copy link
Contributor

@Frozen Frozen commented Dec 20, 2024

1 second finality

  1. Created config IsOneSecondEpoch
  2. Reward reduced 2 times.

@Frozen Frozen self-assigned this Jan 14, 2025
@sophoah
Copy link
Contributor

sophoah commented Feb 4, 2025

@Frozen could you rebase with the latest dev so we can test it with the latest code changes

@Frozen
Copy link
Contributor Author

Frozen commented Feb 4, 2025

updated

@Frozen
Copy link
Contributor Author

Frozen commented Mar 12, 2025

rollback to 1 second

@sophoah
Copy link
Contributor

sophoah commented Mar 18, 2025

@Frozen could you

  1. rebase with the latest dev so we can test it with the latest code changes
  2. change this PR from draft to ready

we'll then get everyone to review/approve

then schedule the devnet HF activation

@Frozen Frozen marked this pull request as ready for review March 18, 2025 20:01
@Frozen
Copy link
Contributor Author

Frozen commented Mar 18, 2025

@Frozen could you

  1. rebase with the latest dev so we can test it with the latest code changes
  2. change this PR from draft to ready

we'll then get everyone to review/approve

then schedule the devnet HF activation

Done

@sophoah sophoah requested review from sunwavesun and sophoah March 19, 2025 06:18
@sophoah
Copy link
Contributor

sophoah commented Mar 19, 2025

i am good with the PR. When we add the devnet (partner network) HF, let's also add a IsOneSecondEpoch: epochTBD, for testnet and mainnet.

@sunwavesun please review/approve

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.

2 participants