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

Tweak channel fees to make base higher and proportional lower #62

Closed
wants to merge 2 commits into from

Conversation

ZmnSCPxj
Copy link
Owner

This follows the "forwarder is payment aggregator" argument, which is that the forwarding node is in the business of aggregating multiple small payments via the shortest path (through their own node) and using longer paths to ensure their shortest-path channels remain balanced. In order to earn, the forwarding node depends on the base fee and not the proportional fee. #56 (comment)

This is not an alternate to #59 or #61 , I will likely merge this in regardless, but it might be useful for others to actually have a look at it and check if the logic in the "forwarder is payment aggregator" argument makes sense.

…o provide separate multipliers for base and proportional.
@hosiawak
Copy link
Contributor

@ZmnSCPxj I won't complain but sooner or later someone will that micropayments are no longer possible. I think eventually clboss will need a few knobs for setting various fee strategies to satisfy this and other needs.

I'll give this a test too and report back once I have something.

@ZmnSCPxj
Copy link
Owner Author

Due to the modular nature of CLBOSS, it may be possible to just include an option to outright prevent particular modules from being instantiated. Of course, the relative lack of C++ introspection makes that difficult.

In any case I intend to merge this in later after we have a 59/61/neither decision.

@ZmnSCPxj
Copy link
Owner Author

Closing since this may not make much sense.

I may change channel fee system after 0.12.

@ZmnSCPxj ZmnSCPxj closed this Apr 25, 2022
@ZmnSCPxj ZmnSCPxj deleted the by-forwarder branch May 5, 2022 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants