-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Closed
Labels
P3might get fixed, nice to havemight get fixed, nice to havebrainstormingLong term ideas/discussion/requests for feedbackLong term ideas/discussion/requests for feedbackfeesRelated to the fees paid for transactions (both LN and funding/commitment transactions)Related to the fees paid for transactions (both LN and funding/commitment transactions)optimizationutxo sweeping
Description
The sweeper clusters input with similar fee rates together to save on fees. However, in such a cluster all inputs won't necessarily have the exact same fee preference, and hence we must choose a "compromise fee rate" to use when sweeping them together.
Currently we take just the average among the inputs in the cluster:
Line 816 in 8ec4697
| // calculating the average fee rate of the inputs within each set. |
As pointed out by @cfromknecht (see #4779 (comment)) this is probably not optimal, for several reasons:
- a high pref input might get a lower fee rate than we want, in case it gets clustered together with lower pri inputs.
- it is not weight aware, meaning that the fee rate for heavy inputs will impact the final fee rate no more than less heavy inputs
- the input might be lingering for a while, so the fee estimates might not be up-to-date. Could make sense to update them/or take most recent fee estimate.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P3might get fixed, nice to havemight get fixed, nice to havebrainstormingLong term ideas/discussion/requests for feedbackLong term ideas/discussion/requests for feedbackfeesRelated to the fees paid for transactions (both LN and funding/commitment transactions)Related to the fees paid for transactions (both LN and funding/commitment transactions)optimizationutxo sweeping