Differential D14963

[avalanche] Lower the remote proofs threshold
ClosedPublic
Actions

Authored by Fabien on Dec 11 2023, 15:48.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC148a52484b7c: [avalanche] Lower the remote proofs threshold

Summary

Make it closer to the vote.
Depends on D14962.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

remote_proofs_lower_threshold

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25918
Build 51411: Build Diff	lint-circular-dependencies · build-without-wallet · build-clang-tidy · build-diff · build-clang · build-debug
Build 51410: arc lint + arc unit

Event Timeline

Fabien created this revision.Dec 11 2023, 15:48

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 11 2023, 15:48

Fabien requested review of this revision.Dec 11 2023, 15:48

Harbormaster completed remote builds in B25915: Diff 43544.Dec 11 2023, 16:08

PiRK accepted this revision.Dec 11 2023, 19:53

This revision is now accepted and ready to land.Dec 11 2023, 19:53

Fix node_count for the local proof being off by one, and fix the test not using the proper node id

Fabien planned changes to this revision.Dec 11 2023, 21:33

Rebase

This revision is now accepted and ready to land.Dec 11 2023, 21:42

Harbormaster completed remote builds in B25918: Diff 43548.Dec 11 2023, 21:44

Harbormaster completed remote builds in B25920: Diff 43550.Dec 11 2023, 22:01

Closed by commit rABC148a52484b7c: [avalanche] Lower the remote proofs threshold (authored by Fabien). · Explain WhyDec 12 2023, 21:17

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC148a52484b7c: [avalanche] Lower the remote proofs threshold.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

peermanager.cpp

15 lines

test/

peermanager_tests.cpp

36 lines

Diff 43548

View Options

src/avalanche/peermanager.cpp