-
Notifications
You must be signed in to change notification settings - Fork 664
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
Krypton deployment + adjustments #1896
Conversation
120, | ||
30, | ||
20, | ||
3333333333333333) // total liquid supply is 40000000000000000 µSTX |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, 8.3% of liquid uSTX can disable PoX?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Am I misunderstanding #1882?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
8.3% is fine by me -- I'm in favor of a low threshold so the community can easily test it out. I was just curious if there was a particular reason for the precise number.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, 8.3% works for me -- if someone griefs the PoX anchor blocks, we can always bump it up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks @lgalabru
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
8dc4f24
to
d6049d0
Compare
@@ -934,9 +934,9 @@ impl PeerHost { | |||
#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] | |||
pub struct RPCPeerInfoData { | |||
pub peer_version: u32, | |||
pub burn_consensus: ConsensusHash, | |||
pub pox_consensus: ConsensusHash, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pox_consensus makes sense here 👍
Additional adjustments: