Skip to content
This repository has been archived by the owner on Aug 16, 2021. It is now read-only.

[WIP] [FN] Fee estimator #4040

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Conversation

zeptin
Copy link
Contributor

@zeptin zeptin commented Oct 26, 2019

Reopening this as a new PR with the latest master branch merged in.

Original work by AndrewZvvv in #1486

Still needs to be reviewed in its entirety, particularly the estimation algorithm portion. The merge may also not have been entirely clean due to the length of time involved.

@zeptin
Copy link
Contributor Author

zeptin commented Oct 26, 2019

My initial understanding of this is that the Algorithm014 folder represents the implementation as it existed in Bitcoin Core 0.14. There were changes in 0.15 described here:

https://gist.github.com/morcos/d3637f015bc4e607e1fd10d8351e9f41

So I think review effort should concentrate on the TxConfirmStats and BlockPolicyEstimator that are not in the Algorithm014 folder. I think at some point the extra files can be deleted entirely, but I want to be sure we don't need them for anything first.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants