chia-blockchain icon indicating copy to clipboard operation
chia-blockchain copied to clipboard

New peak eos edge case

Open almogdepaz opened this issue 11 months ago • 9 comments

Purpose:

when choosing between two peaks with the same height chose the chain with less iterations even if peaks are on different sub_slots

Current Behavior:

we will chose the peak that was discovered first

New Behavior:

prefer the chain with lower iterations

Testing Notes:

almogdepaz avatar Mar 25 '24 13:03 almogdepaz

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Mar 25 '24 13:03 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Mar 25 '24 15:03 github-actions[bot]

Don't we want the chain with more iterations or is the inference that less iterations in the same epoch implies a better proof of space?

hoffmang9 avatar Mar 25 '24 15:03 hoffmang9

Don't we want the chain with more iterations or is the inference that less iterations in the same epoch implies a better proof of space?

this avoids some possible orphaning strategies. the motivation is that the lower iterations chain might be heavier at the same iterations if we keep on it and wait for the next block, this only happens when the weight of the two chains is the same, otherwise we still always prefer the heavier chain

almogdepaz avatar Mar 25 '24 16:03 almogdepaz

File Coverage Missing Lines
chia/full_node/full_node.py 87.5% lines 2184-2185
Total Missing Coverage
17 lines 2 lines 88%

github-actions[bot] avatar Mar 26 '24 15:03 github-actions[bot]

This PR has been flagged as stale due to no activity for over 60 days. It will not be automatically closed, but it has been given a stale-pr label and should be manually reviewed by the relevant parties.

github-actions[bot] avatar Jul 20 '24 11:07 github-actions[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Jul 29 '24 18:07 github-actions[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Jul 29 '24 18:07 github-actions[bot]

This PR has been flagged as stale due to no activity for over 60 days. It will not be automatically closed, but it has been given a stale-pr label and should be manually reviewed by the relevant parties.

github-actions[bot] avatar Sep 13 '24 11:09 github-actions[bot]