Remove code that is no longer necessary given the removal of IsWorthPolling(), and which is likely the cause of the test failures.

Fix accidental change-stack cross-contamination.

Remove incorrect, racy test assertion.

Looks good. Left a comment to consider but not something that should hold up this changeset.

Move logging into lambda, make unparkBlock lambda, and add to tests.

Remove isAccepted contructor property and IsWorthPolling.

@Fabien Let me know if there's something more you'd like to see regarding testing/test-plan for this change.

Superseded by D11367.

Looks correct other than a small comment typo.

Everything looks correctly implemented and the logic all makes sense, except I'm not clear on the reasoning about the quorum params.

This is incorrect based on a misunderstanding with the peermanager and local proof.

300 is a lot more reasonable than 16 for a larger network.

The idea and implementation are great, but I have a minor bikeshed-nit about naming. I don't think that nit should hold up acceptance but is worth considering.

This flag will be useful and the implementation looks correct.

Starting on line 2370 there is a breakdown of the overall algorithm here. I think we should update it to include this new behavior because it's specific enough that this change would make it incorrect.

The concept and implementation look good. A few nits with the test.

Update test name.

Use MiniWallet for tests instead of the node's wallet.

Use a clean chain in tests.

Nice, this looks a good way to catch-all cleanup all disconnections.

The code makes sense and seems to be working. I can't tell why the build is failing it it appears to be unrelated to this change.

This makes sense. Other nodes must always validate the proofs anyways.

Makes sense and I don't see any reason it should affect the test logically.

Great idea so the quorum detection logic can stay encapsulated in the server.

In hindsight, this seems obvious. I think this fix and the tests are good.

The concept and implementation look correct and well-tested.

Change looks correct.

Remove all changes to peermanager tests.

Remove unrelated change.

Use BOOST_CHECK instead of BOOST_CHECK_EQUAL to avoid passing the processor.

Debugging failure

Address feedback. Add range checks to params, add failure cases, simplify tests, and remove unrelated cleanup changes.

Fix remaining feedback, fix locale issues, and use proof orphaning to remove peers instead of needing their ids.

Fix all review items except marking isQuorumEstablished as const, pending discussion.

Update variable and method names.

Fix tests and rebase onto latest D11118.

Fix unit test name.

Rename properties and improve tests.

Redesign to use pre-computed score amounts from D11181.

This functionality was included in a different changeset that landed.

Spacing and test cleanup.

Fix missing newline at end of file.

Make comment style consistent and add space.

Update abc_p2p_getavaddr to allow a 0-stake quorum so node changes are processed.

Fix getavalancheinfo test by setting quorum min stake to 0. Otherwise, the eventloop never gets to the point of trying the disconnected nodes and never removes them from them so it keeps reporting 10.

Update handling of min stake arg to avoid parsing issues with ToString()'d version.

Debugging remaining test failures.

Rebase onto master after dependency landed.

Include chrono in avalanche.h

Fix locale function linting issue by not trying to format double as a string.

Need to address new lint issue.

Rebase onto D11133 and address all oustanding feedback items.

Concept and implementation both look good.

Make tests correctly disconnect peer nodes without hacks.

Fix proof voting test.

Fix node vs peer stake comparison.
Implement ratio of connected nodes checking.
Update default parameters.

Update arg names, fix unit tests, remove by_score handling, and start on peer stake ratio checking.