Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[draft] 10 minute confirmation target #12912

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kiminuo
Copy link
Collaborator

@kiminuo kiminuo commented Apr 22, 2024

Fixes #11358
Depends on #12910

Introduction

The problem here is that conf_target in estimatesmartfee conf_target ( "estimate_mode" ) cannot be 1 because:

The request target will be clamped between 2 and the highest target fee estimation is able to return based on how long it has been running.

[Source]

Testing

Given that RegTest does not really have history and thus it is handles specially, we need to test RegTest as well.

Resources

Copy link
Collaborator

@turbolay turbolay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm unsure of this concept. I think if we are doing something like this we should at least try to do it properly.

I like this approach of the problem by OXT: https://code.samourai.io/oxt/one_dollar_fee_estimator/-/blob/master/README.md
I think it produces great results. It is server side.

A simplification of this algorithm that keeps the same idea is what @MaxHillebrand suggested in the first issue to take 0.5 MB of the mempool.

I'm sorry to suggest a more complex approach, but an estimation you can't trust is an estimation you don't want to use. I see no value, and even harm, of multiplying by a random coefficient.

@kiminuo
Copy link
Collaborator Author

kiminuo commented Apr 23, 2024

I just tried to do what seemed to be the request.

If there is an agreement on a better solution, we can certainly work on a better solution.

@molnard Do you have an opinion on this?

@molnard
Copy link
Collaborator

molnard commented Apr 25, 2024

I like this approach of the problem by OXT:

The link is not available anymore :( I guess I know why. I cannot see how that should work.

The fundamental idea is to make sure to be in the next block. If I need to do this manually: I would look at the mempool, check the txs in the next expected block, find the minimum fee, and try to aim a bit above that.

Questions:

  • Do we have the mempool on the client side? No, just partially. It is impossible to calculate this on the client side.
  • Can we guess the next block? I am not aware of any functions that can do that in bitcoind. Let me know if there is.
  • Even if we would have the expected block, aiming a bit above would result in using some kind of random coefficient.

to take 0.5 MB of the mempool.

Can work, it is coming from "above" (from the highest fee direction). The coefficient here is xMb. Take xMb of the mempool means, ordering txs in descending order according to FeeRate without any other logic (transaction chains, etc.)?

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

Successfully merging this pull request may close these issues.

Add 10 Min Conf Target
3 participants