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

Use adjusted difficulty for trees in active transactions #1810

Merged
merged 11 commits into from
Mar 21, 2019

Conversation

SergiySW
Copy link
Contributor

@SergiySW SergiySW commented Mar 7, 2019

  • add adjusted difficulty calculation for active transactions
  • make list of dependent blocks in election

Better alternative to #1691

SergiySW added 2 commits March 8, 2019 00:08
* add adjusted difficulty calculation
* make list of dependent blocks in election
@SergiySW SergiySW added the functionality quality improvements This item indicates the need for or supplies a better way to implement existing functionality label Mar 7, 2019
@SergiySW SergiySW added this to the V19.0 milestone Mar 7, 2019
@SergiySW SergiySW self-assigned this Mar 7, 2019
@SergiySW SergiySW requested review from argakiig and cryptocode March 7, 2019 21:33
@zhyatt zhyatt requested a review from wezrule March 19, 2019 15:57
@SergiySW SergiySW merged commit 5670252 into nanocurrency:master Mar 21, 2019
guilhermelawless pushed a commit to guilhermelawless/nano-node that referenced this pull request Apr 15, 2019
…y#1810)

* Use adjusted difficulty for trees in active transactions

* add adjusted difficulty calculation
* make list of dependent blocks in election

* Adjust difficulty for dependent on deleted blocks elections

* Potential overflow check

* Using reference to items in remaining_blocks

* correct words

* Replace push_back with emplace_back for deque of pairs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
functionality quality improvements This item indicates the need for or supplies a better way to implement existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants