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

HIP for changing consensus group deselection dynamics #490

Merged
merged 2 commits into from
Oct 14, 2022

Conversation

PaulVMo
Copy link
Contributor

@PaulVMo PaulVMo commented Oct 13, 2022

This HIP introduces a change to the consensus election dynamics on the Helium L1 blockchain to provide additional tunability for the deselection scoring of current consensus group members. The aim is to more quickly and reliable eject poor performers from the group.

Code for this change has already been merged as blockchain-core PR 1468 helium/blockchain-core#1468

@vincenzospaghetti
Copy link
Contributor

Hi @PaulVMo, does this need to be a HIP if it's already deployed? Please correct me if I have that wrong.

@abhay
Copy link
Contributor

abhay commented Oct 13, 2022

It's deployed (or in the process. see image below. left two bars are the viable versions) but behind a chain variable that's bound to the 1.15.x release. I believe it's a fair thing to have a HIP and a quick vote on (at least by validator operators) since it changes election dynamics.

image

@vincenzospaghetti
Copy link
Contributor

Okay. I likely see others having the same question I just asked. Can we put that in the context of this HIP? I added this as two comments to be added to the Summary portion. Otherwise looks great and, we'll get it out!

@PaulVMo
Copy link
Contributor Author

PaulVMo commented Oct 14, 2022

@vincenzospaghetti great questions and feedback. I updated the summary to address the points you brought up. Please let me know if you have any more questions.

@abhay
Copy link
Contributor

abhay commented Oct 14, 2022

I’d be supportive of this coming to a full discussion (for a few days), some education from the author and the foundation, and a relatively quick vote. Ideally we could activate this variable around Nov 1.

@vincenzospaghetti
Copy link
Contributor

All of that sounds good @abhay. I see this definitely needing some education from you and @PaulVMo. How can the Foundation help education? If you have a proposal of that we can add in the comments here that would be ideal.

Let's go ahead and open a channel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants