-
Notifications
You must be signed in to change notification settings - Fork 289
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
EPIC: Parameter picking Tracking issue #669
Labels
epic
item groups other items for easier tracking
Comments
41 tasks
This was referenced Sep 19, 2022
Closed
In our giant param call, we came to a few decisions that we can track here as well. Mempool Config Defaults
RPC Config
Genesis Params
Gas
|
we should also fix celestiaorg/cosmos-sdk#283 before closing this issue |
This was referenced Nov 16, 2022
rootulp
added a commit
that referenced
this issue
Nov 16, 2022
Addresses #669 (comment) by copying the consensus config overrides from celestiaorg/cosmos-sdk. This PR should be merged in parallel with celestiaorg/cosmos-sdk#284 Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
evan-forbes
changed the title
Parameter picking Tracking issue
EPIC: Parameter picking Tracking issue
Nov 17, 2022
rach-id
pushed a commit
to rach-id/celestia-app
that referenced
this issue
Nov 17, 2022
Addresses celestiaorg#669 (comment) by copying the consensus config overrides from celestiaorg/cosmos-sdk. This PR should be merged in parallel with celestiaorg/cosmos-sdk#284 Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
evan-forbes
added a commit
that referenced
this issue
Dec 8, 2022
Addresses #669 (comment) by copying the consensus config overrides from celestiaorg/cosmos-sdk. This PR should be merged in parallel with celestiaorg/cosmos-sdk#284 Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
can close since all parameters have been chosen |
0xchainlover
pushed a commit
to celestia-org/celestia-app
that referenced
this issue
Aug 1, 2024
Addresses celestiaorg/celestia-app#669 (comment) by copying the consensus config overrides from celestiaorg/cosmos-sdk. This PR should be merged in parallel with celestiaorg/cosmos-sdk#284 Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We need to do a lot of investigation based work in order to ensure that we pick correct parameters. This will involve individuals selecting specific sets of parameters, doing some investigation, including performing preliminary network tests, and summarizing this work for comment on other teams. Our goal is to have an official list of parameters decided on by the end of onsite, where we will discuss the params and their respective investigations synchronously.
Max
andMinSquareSize
as a parameter to the payment module #183The text was updated successfully, but these errors were encountered: