Page MenuHomePhabricator

[policy] Delete block policy estimator.
ClosedPublic

Authored by deadalnix on Dec 3 2018, 06:28.

Details

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 Buildbot (legacy)
Build 6568: arc lint + arc unit

Event Timeline

Remove writing/reading of fee stats.

Remove test_framework dependency on silly file

Clean up some unnecessary globals

Remove last vestigates of block policy estimator

Remove an extraneous whitespace

Delete yet another vestige

deadalnix edited reviewers, added: schancel; removed: deadalnix.
markblundeberg added a subscriber: markblundeberg.

Great idea, code changes look good.

src/policy/fees.h
28 ↗(On Diff #6739)

will it be possible to get rid of this too (and FEE_SPACING above) ?

This revision is now accepted and ready to land.Jan 26 2019, 05:57
This revision was automatically updated to reflect the committed changes.