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

[Wallet] Enable miner with mnsync incomplete #1069

Merged
merged 1 commit into from
Nov 10, 2019

Conversation

random-zebra
Copy link

Instead of outright preventing the generation of new blocks when mnsync is incomplete, use a different logic: prevent the creation of new block when:

  • mnsync is incomplete
    AND
  • either the spork list is not synced (we are at the very start of mnsync) or one between sporks 8,9 or 13 is active

In other words, try to stake a new block only if

  • mnsync is complete
    OR
  • the spork list is synced and all three sporks (8,9 and 13) are not active.

when sporks 8, 9 and 13 are disabled
@random-zebra random-zebra added Masternode Block Generation Mining/Staking related issues labels Oct 29, 2019
@random-zebra random-zebra added this to the 4.0.0 milestone Oct 29, 2019
@random-zebra random-zebra self-assigned this Oct 29, 2019
Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK fe550d4

Copy link

@Warrows Warrows left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK fe550d4

random-zebra added a commit that referenced this pull request Nov 10, 2019
fe550d4 [Wallet] Enable miner with mnsync incomplete (random-zebra)

Pull request description:

  Instead of outright preventing the generation of new blocks when mnsync is incomplete, use a different logic: prevent the creation of new block when:
  - mnsync is incomplete
  **AND**
  - _either_ the spork list is not synced (we are at the very start of mnsync) _or one_ between sporks 8,9 or 13 is active

  In other words, try to stake a new block only if
  - mnsync is complete
  **OR**
  - the spork list is synced _and all three_ sporks (8,9 and 13) are not active.

ACKs for top commit:
  furszy:
    utACK fe550d4
  Warrows:
    utACK fe550d4

Tree-SHA512: f31ab8d1679f49edd5aa6bae88bef67701bd923b95761a7a5853a9110932df0972bb05e3413a981a32bbd62561285329d0ef4cfe264a62e3773cf9e52f2f8979
@random-zebra random-zebra merged commit fe550d4 into PIVX-Project:master Nov 10, 2019
@Fuzzbawls Fuzzbawls added the Needs Release Notes Placeholder tag for anything needing mention in the "Notable Changes" section of release notes label Dec 3, 2019
@Fuzzbawls Fuzzbawls removed the Needs Release Notes Placeholder tag for anything needing mention in the "Notable Changes" section of release notes label Dec 14, 2019
@random-zebra random-zebra deleted the 2019_miner_mnsync branch September 24, 2020 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Block Generation Mining/Staking related issues Masternode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants