From 95496d90c79ad841c728998f6b51ecf9277adb5c Mon Sep 17 00:00:00 2001 From: Fahim Ahmed <68152521+fahimahmedx@users.noreply.github.com> Date: Tue, 2 Jan 2024 15:02:42 -0500 Subject: [PATCH] docs: add column for governance modifiable params in resource_pricing (#2974) ## Overview Addresses #2966 by using the governance modifiable params tests in PR #2973. All four of the params are modifiable by governance, which matches the specs in [params.md](https://github.com/celestiaorg/celestia-app/blob/main/specs/src/specs/params.md). ## Checklist - [x] New and updated code has appropriate documentation - [x] New and updated code has new and/or updated testing - [x] Required CI checks are passing - [x] Visual proof for any user facing features like CLI or documentation updates - [ ] Linked issues closed with keywords --- specs/src/specs/resource_pricing.md | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/specs/src/specs/resource_pricing.md b/specs/src/specs/resource_pricing.md index e41d5b7135..30355d777e 100644 --- a/specs/src/specs/resource_pricing.md +++ b/specs/src/specs/resource_pricing.md @@ -141,15 +141,15 @@ updated. ## Parameters -There are three parameters that can be modified via governance to modify gas +There are four parameters that can be modified via governance to modify gas usage. -| Parameter | Default Value | Description | -|-----------------|---------------|-----------------------------------------| -| consensus/max_gas | -1 | The maximum gas allowed in a block. Default of -1 means this value is not capped. | -| auth/tx_size_cost_per_byte | 10 | Gas used per each byte used by the transaction. | -| auth/sig_verify_cost_secp256k1 | 1000 | Gas used per verifying a secp256k1 signature | -| blob/gas_per_blob_byte | 8 | Gas used per byte used by blob. Note that this value is applied to all encoding overhead, meaning things like the padding of the remaining share and namespace. See PFB gas estimation section for more details. | +| Parameter | Default Value | Description | Changeable via Governance | +|-----------------|---------------|-----------------------------------------|---------------------------| +| consensus/max_gas | -1 | The maximum gas allowed in a block. Default of -1 means this value is not capped. | True | +| auth/tx_size_cost_per_byte | 10 | Gas used per each byte used by the transaction. | True | +| auth/sig_verify_cost_secp256k1 | 1000 | Gas used per verifying a secp256k1 signature | True | +| blob/gas_per_blob_byte | 8 | Gas used per byte used by blob. Note that this value is applied to all encoding overhead, meaning things like the padding of the remaining share and namespace. See PFB gas estimation section for more details. | True | ## Gas Limit