From 4f26e6affe86a88eff7c63024df10525131af186 Mon Sep 17 00:00:00 2001 From: Rootul Patel Date: Mon, 25 Nov 2024 16:29:17 -0500 Subject: [PATCH] docs(specs): add missing versioned consts --- pkg/appconsts/v1/app_consts.go | 14 ++++++++++---- pkg/appconsts/v2/app_consts.go | 7 +++++-- pkg/appconsts/v3/app_consts.go | 2 +- specs/src/parameters_v1.md | 9 +++++---- specs/src/parameters_v2.md | 11 ++++++----- specs/src/parameters_v3.md | 18 +++++++++++------- 6 files changed, 38 insertions(+), 23 deletions(-) diff --git a/pkg/appconsts/v1/app_consts.go b/pkg/appconsts/v1/app_consts.go index 873d3ec18a..fee33747e3 100644 --- a/pkg/appconsts/v1/app_consts.go +++ b/pkg/appconsts/v1/app_consts.go @@ -6,10 +6,16 @@ const ( Version uint64 = 1 SquareSizeUpperBound int = 128 SubtreeRootThreshold int = 64 - TimeoutPropose = time.Second * 10 - TimeoutCommit = time.Second * 11 + // Note: TimeoutPropose and TimeoutCommit weren't actually constants in v1, + // they were defaults for user-configurable timeouts. TODO: consider + // removing them from this package. + TimeoutPropose = time.Second * 10 + TimeoutCommit = time.Second * 11 // UpgradeHeightDelay is the number of blocks after a quorum has been - // reached that the chain should upgrade to the new version. Assuming a block - // interval of 12 seconds, this is 7 days. + // reached that the chain should upgrade to the new version. Assuming a + // block interval of 12 seconds, this is 7 days. + // + // TODO: why does this constant exist in v1? v1 does not contain the signal + // module. UpgradeHeightDelay = int64(7 * 24 * 60 * 60 / 12) // 7 days * 24 hours * 60 minutes * 60 seconds / 12 seconds per block = 50,400 blocks. ) diff --git a/pkg/appconsts/v2/app_consts.go b/pkg/appconsts/v2/app_consts.go index d02a97079b..78d6603668 100644 --- a/pkg/appconsts/v2/app_consts.go +++ b/pkg/appconsts/v2/app_consts.go @@ -6,8 +6,11 @@ const ( Version uint64 = 2 SquareSizeUpperBound int = 128 SubtreeRootThreshold int = 64 - TimeoutPropose = time.Second * 10 - TimeoutCommit = time.Second * 11 + // Note: TimeoutPropose and TimeoutCommit weren't actually constants in v1, + // they were defaults for user-configurable timeouts. TODO: consider + // removing them from this package. + TimeoutPropose = time.Second * 10 + TimeoutCommit = time.Second * 11 // UpgradeHeightDelay is the number of blocks after a quorum has been // reached that the chain should upgrade to the new version. Assuming a block // interval of 12 seconds, this is 7 days. diff --git a/pkg/appconsts/v3/app_consts.go b/pkg/appconsts/v3/app_consts.go index 3f9279518d..e5b660bf2e 100644 --- a/pkg/appconsts/v3/app_consts.go +++ b/pkg/appconsts/v3/app_consts.go @@ -13,6 +13,6 @@ const ( TimeoutCommit = time.Millisecond * 4200 // UpgradeHeightDelay is the number of blocks after a quorum has been // reached that the chain should upgrade to the new version. Assuming a block - // interval of 12 seconds, this is 7 days. + // interval of 6 seconds, this is 7 days. UpgradeHeightDelay = int64(7 * 24 * 60 * 60 / 6) // 7 days * 24 hours * 60 minutes * 60 seconds / 6 seconds per block = 100,800 blocks. ) diff --git a/specs/src/parameters_v1.md b/specs/src/parameters_v1.md index 436b8cb471..ff95bd47b6 100644 --- a/specs/src/parameters_v1.md +++ b/specs/src/parameters_v1.md @@ -8,10 +8,11 @@ hardcoded in the application or they are blocked by the `x/paramfilter` module. ## Global parameters -| Parameter | Default | Summary | Changeable via Governance | -|-------------------|---------|------------------------------------------------------------------------------------------------------------------------|---------------------------| -| MaxBlockSizeBytes | 100MiB | Hardcoded value in CometBFT for the protobuf encoded block. | False | -| MaxSquareSize | 128 | Hardcoded maximum square size determined per shares per row or column for the original data square (not yet extended). | False | +| Parameter | Value | Summary | Changeable via Governance | +|----------------------|---------|------------------------------------------------------------------------------------------------------------------------------------|---------------------------| +| SquareSizeUpperBound | 128 | Hardcoded maximum square size which limits the number of shares per row or column for the original data square (not yet extended). | False | +| SubtreeRootThreshold | 64 | See ADR 13 for more details. | False | +| MaxBlockSizeBytes | 100 MiB | Hardcoded value in CometBFT for the protobuf encoded block. | False | ## Module parameters diff --git a/specs/src/parameters_v2.md b/specs/src/parameters_v2.md index ec93c9acda..a9b258f965 100644 --- a/specs/src/parameters_v2.md +++ b/specs/src/parameters_v2.md @@ -8,11 +8,12 @@ hardcoded in the application or they are blocked by the `x/paramfilter` module. ## Global parameters -| Parameter | Default | Summary | Changeable via Governance | -|--------------------|---------|------------------------------------------------------------------------------------------------------------------------|---------------------------| -| MaxBlockSizeBytes | 100MiB | Hardcoded value in CometBFT for the protobuf encoded block. | False | -| MaxSquareSize | 128 | Hardcoded maximum square size determined per shares per row or column for the original data square (not yet extended). | False | -| UpgradeHeightDelay | 50400 | Height based delay after a successful `MsgTryUpgrade` has been submitted. | False | +| Parameter | Value | Summary | Changeable via Governance | +|----------------------|--------------|------------------------------------------------------------------------------------------------------------------------------------|---------------------------| +| SquareSizeUpperBound | 128 | Hardcoded maximum square size which limits the number of shares per row or column for the original data square (not yet extended). | False | +| SubtreeRootThreshold | 64 | See ADR 13 for more details. | False | +| UpgradeHeightDelay | 50400 blocks | Height based delay after a successful `MsgTryUpgrade` has been submitted. | False | +| MaxBlockSizeBytes | 100 MiB | Hardcoded value in CometBFT for the protobuf encoded block. | False | ## Module parameters diff --git a/specs/src/parameters_v3.md b/specs/src/parameters_v3.md index 6f8d84fccb..c2912afb3d 100644 --- a/specs/src/parameters_v3.md +++ b/specs/src/parameters_v3.md @@ -8,11 +8,15 @@ hardcoded in the application or they are blocked by the `x/paramfilter` module. ## Global parameters -| Parameter | Default | Summary | Changeable via Governance | -|--------------------|---------|------------------------------------------------------------------------------------------------------------------------|---------------------------| -| MaxBlockSizeBytes | 100MiB | Hardcoded value in CometBFT for the protobuf encoded block. | False | -| MaxSquareSize | 128 | Hardcoded maximum square size determined per shares per row or column for the original data square (not yet extended). | False | -| UpgradeHeightDelay | 100800 | Height based delay after a successful `MsgTryUpgrade` has been submitted. | False | +| Parameter | Value | Summary | Changeable via Governance | +|----------------------|---------------|------------------------------------------------------------------------------------------------------------------------------------|---------------------------| +| SquareSizeUpperBound | 128 | Hardcoded maximum square size which limits the number of shares per row or column for the original data square (not yet extended). | False | +| SubtreeRootThreshold | 64 | See ADR 13 for more details. | False | +| MaxTxSize | 2 MiB | Maximum size of a transaction in bytes. | False | +| TimeoutPropose | 3500 ms | Specifies the time that validators wait during the proposal phase of the consensus process. See CometBFT specs for more details. | False | +| TimeoutCommit | 4200 ms | Specifies the duration that validators wait during the Commit phase of the consensus process. See CometBFT specs for more details. | False | +| UpgradeHeightDelay | 100800 blocks | Height based delay after a successful `MsgTryUpgrade` has been submitted. | False | +| MaxBlockSizeBytes | 100 MiB | Hardcoded value in CometBFT for the protobuf encoded block. | False | ## Module parameters @@ -22,9 +26,9 @@ hardcoded in the application or they are blocked by the `x/paramfilter` module. | auth.SigVerifyCostED25519 | 590 | Gas used to verify Ed25519 signature. | True | | auth.SigVerifyCostSecp256k1 | 1000 | Gas used to verify secp256k1 signature. | True | | auth.TxSigLimit | 7 | Max number of signatures allowed in a multisig transaction. | True | -| auth.TxSizeCostPerByte | 10 | Gas used per transaction byte. | False | +| auth.TxSizeCostPerByte | 10 | Gas used per transaction byte. | False | | bank.SendEnabled | true | Allow transfers. | False | -| blob.GasPerBlobByte | 8 | Gas used per blob byte. | False | +| blob.GasPerBlobByte | 8 | Gas used per blob byte. | False | | blob.GovMaxSquareSize | 64 | Governance parameter for the maximum square size of the original data square. | True | | consensus.block.MaxBytes | 1974272 bytes (~1.88 MiB) | Governance parameter for the maximum size of the protobuf encoded block. | True | | consensus.block.MaxGas | -1 | Maximum gas allowed per block (-1 is infinite). | True |