Page MenuHomePhabricator

[policy] Delete block policy estimator.
Needs ReviewPublic

Authored by schancel on Mon, Dec 3, 06:28.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Summary

This commit removes the block policy estimator. It's currently basically useless,
and the algorithm for determining minimum fee isn't correct either. We will replace this
estimator using something that examines the active mempool.

Depends on D2037
Complete T499

Test Plan
make VERBOSE=1 check && ./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
delete-fee-estimator (branched from master)
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 4252
Build 6569: Bitcoin ABC Teamcity Staging
Build 6568: arc lint + arc unit

Event Timeline

schancel created this revision.Mon, Dec 3, 06:28
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Dec 3, 06:28
schancel updated this revision to Diff 6247.Mon, Dec 3, 06:43

Remove writing/reading of fee stats.

schancel updated this revision to Diff 6249.Mon, Dec 3, 06:52

Remove test_framework dependency on silly file

schancel updated this revision to Diff 6250.Mon, Dec 3, 06:54

Include change to docs

schancel updated this revision to Diff 6252.Mon, Dec 3, 07:05

Clean up some unnecessary globals

schancel updated this revision to Diff 6253.Mon, Dec 3, 07:08

Remove last vestigates of block policy estimator

schancel updated this revision to Diff 6255.Mon, Dec 3, 07:22

Remove an extraneous whitespace

schancel updated this revision to Diff 6258.Mon, Dec 3, 09:03

Delete yet another vestige