Rebase on D17660 to address the feedback about splitting up setContenderStatusForLocalWinners(). It should be more clear now that addToReconcile() is only called once per block.

Avoid unused pollableContenders at call sites for now

Is there a reason -simplifygbt is a command line arg and not a parameter to getblocktemplate?

nvm. compared the wrong lines side-by-side

Make local acceptance match what this node would eventually vote

Add comments to make it more clear that the test is checking the item is not being polled for

Remove erroneous image file change

feedback

Remove errant newline

Use assert_equal

Fix wait_until_helper -> self.wait_until

Add back accidentally deleted comment

Bump mocktime beyond the max registration time so there is not a bunch of flaky winners in the functional test

Reuse numPollable

Fix circular dependency

Cleanup unnecessary nesting when sorting contenders and make the sorting work the same as staking rewards selection.

• Fix rare sorting case where two proofs have the same rank.
• Make num pollable a parameter so the processor will have control over how many contenders to poll for.
• Return number of contenders instead of bool.
• Fix test not actually checking the contender order.

Move static_assert to processor.h in order to avoid circular dependency