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

Conversation

@ceyonur
Copy link
Collaborator

@ceyonur ceyonur commented Sep 16, 2025

Why this should be merged

It adds relevant acp-226 math very similar to acp 176.

ACP: https://github.com/avalanche-foundation/ACPs/blob/main/ACPs/226-dynamic-minimum-block-times/README.md#activation-parameters-for-the-c-chain

Closes: #1230

How this works

Copies acp-176 math and applies for acp-226

How this was tested

added tests

Need to be documented?

no

Need to update RELEASES.md?

no

@ceyonur ceyonur force-pushed the ceyonur/acp-226-min-block-delay branch from 4803282 to b5ed22c Compare September 16, 2025 22:51
@ceyonur ceyonur changed the title Ceyonur/acp 226 min block delay acp-226: dynamic block delay math Sep 17, 2025
},
{
name: "500ms_target",
excess: 6_516_490, // TargetConversion (2^20) * ln(500) + 2
Copy link
Collaborator Author

@ceyonur ceyonur Sep 17, 2025

Choose a reason for hiding this comment

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

had to add +2, probably due to rounding in fake exponential?

@ceyonur ceyonur marked this pull request as ready for review September 17, 2025 16:40
@ceyonur ceyonur requested a review from a team as a code owner September 17, 2025 16:40
@ceyonur ceyonur added acp-226 granite Items related to the granite release. requires-subnet-evm-port labels Sep 17, 2025
@StephenButtolph
Copy link
Contributor

PR got moved to: ava-labs/avalanchego#4289

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

Labels

acp-226 granite Items related to the granite release. requires-subnet-evm-port

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add acp226 math package

3 participants