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

Tracking: all the consensus parameters #565

Closed
6 of 18 tasks
yaahc opened this issue Jun 30, 2020 · 4 comments
Closed
6 of 18 tasks

Tracking: all the consensus parameters #565

yaahc opened this issue Jun 30, 2020 · 4 comments
Labels
A-consensus Area: Consensus rule updates C-cleanup Category: This is a cleanup C-tracking-issue Category: This is a tracking issue for other tasks

Comments

@yaahc
Copy link
Contributor

yaahc commented Jun 30, 2020

Right now as part of equihash verification we've hard coded:

  • n=200 and k=9 in one of the functions used to verify equihash

These types of constants should be centralized as much as possible to ease maintenance and make it easier to look up the consensus parameters that zebra uses. This type should probably be defined and instantiated in zebra-consensus.


Here are some other things we'll want in our consensus parameters:

It might be helpful to look at zcashd's chain and consensus parameters:

We might also need some parts of the network upgrade info:

To do some network integration tests (#466), we might need a "regression test" mode like zcashd. So we might end up with 3 sets of parameters.


Yeah, I think that we should think about how we want to represent consensus parameters, maybe in an object in zebra-consensus or something?

Originally posted by @hdevalence in #550

@teor2345
Copy link
Contributor

teor2345 commented Jul 2, 2020

(Moved to description, so we can track items on the checklist.)

@teor2345 teor2345 added the A-consensus Area: Consensus rule updates label Jul 2, 2020
@teor2345
Copy link
Contributor

teor2345 commented Jul 14, 2020

I think we want to interface to the consensus parameters using functions that takea a block height, and a network (mainnet, testnet, and maybe regtest).

@teor2345 teor2345 changed the title Create a consensus parameter type Create a consensus parameter module Jul 22, 2020
@dconnolly dconnolly changed the title Create a consensus parameter module Tracking: all the consensus parameters Aug 13, 2020
@dconnolly dconnolly added the C-tracking-issue Category: This is a tracking issue for other tasks label Aug 13, 2020
@mpguerra mpguerra removed this from the Wallet Support 💰 milestone Jan 5, 2021
@mpguerra mpguerra added the Epic Zenhub Label. Denotes a theme of work under which related issues will be grouped label May 25, 2021
@mpguerra mpguerra added the C-cleanup Category: This is a cleanup label Nov 23, 2021
@mpguerra mpguerra removed the Epic Zenhub Label. Denotes a theme of work under which related issues will be grouped label Jan 21, 2022
@dconnolly
Copy link
Contributor

@conradoplg @teor2345 I think this is redundant now, should we close it?

@teor2345
Copy link
Contributor

Yep, we have all the parameters we need for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-consensus Area: Consensus rule updates C-cleanup Category: This is a cleanup C-tracking-issue Category: This is a tracking issue for other tasks
Projects
None yet
Development

No branches or pull requests

4 participants