-
Notifications
You must be signed in to change notification settings - Fork 264
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
make max-feerate-mismatch configurable and set default value to 500% #62
Conversation
@@ -38,6 +38,10 @@ eclair { | |||
fee-base-msat = 546000 | |||
fee-proportional-millionth = 10 | |||
|
|||
// mamixmum local vs remote feerate mismatch; 1.0 means 100% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -38,6 +38,10 @@ eclair { | |||
fee-base-msat = 546000 | |||
fee-proportional-millionth = 10 | |||
|
|||
// mamixmum local vs remote feerate mismatch; 1.0 means 100% | |||
// actuel check is abs((local feerate - remote fee rate) / (local fee rate + remote fee rate)/2) > fee rate mismatch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that comment should rather be in the scaladoc of function isFeeDiffTooHigh
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -29,12 +29,6 @@ object Globals { | |||
* than this ratio. | |||
*/ | |||
val UPDATE_FEE_MIN_DIFF_RATIO = 0.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should UPDATE_FEE_MIN_DIFF_RATIO
also be increased?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is now in application.conf
there can be a significant gap between the fee rate estimated by different bitcoin clients (see estimatesmartfee) so we must set a reasonably high value for the threshold above which we consider htat local and remote fee rates are too different and close the channel.
it becomes symmetrical and easier to reason with, and also more forgiving.
f571fda
to
801e8c2
Compare
there can be a significant gap between the fee rate estimated by different bitcoin
clients (see estimatesmartfee) so we must set a reasonably high value for the threshold
above which we consider htat local and remote fee rates are too different and close the
channel.