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

mining: Update blk templ diff for too few voters. #3241

Merged
merged 1 commit into from
May 4, 2024

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented May 3, 2024

This updates the block template generation path for handling too few voters to also update the required difficulty bits in the created block template since the new ASERT difficulty algorithm changes every block and is influenced by the timestamp.

This is necessary to prevent the possibility of generating block templates that fail to validate and have to be discarded resulting in unnecessary additional work while aggressive mining in the aforementioned case of not having enough voters.

This updates the block template generation path for handling too few
voters to also update the required difficulty bits in the created block
template since the new ASERT difficulty algorithm changes every block
and is influenced by the timestamp.

This is necessary to prevent the possibility of generating block
templates that fail to validate and have to be discarded resulting in
unnecessary additional work while aggressive mining in the
aforementioned case of not having enough voters.
@davecgh davecgh force-pushed the mining_update_diff_too_foo_voters branch from 912ce6a to 8de792c Compare May 4, 2024 02:30
@davecgh davecgh merged commit 8de792c into decred:master May 4, 2024
2 checks passed
@davecgh davecgh deleted the mining_update_diff_too_foo_voters branch May 4, 2024 02:32
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.

None yet

2 participants