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

Add a per-amount base penalty in the ProbabilisticScorer #1617

Merged
merged 2 commits into from Jul 25, 2022

Commits on Jul 25, 2022

  1. Add a per-amount base penalty in the ProbabilisticScorer

    There's not much reason to not have a per-hop-per-amount penalty in
    the `ProbabilisticScorer` to go along with the per-hop penalty to
    let it scale up to larger amounts, so we add one here.
    
    Notably, we use a divisor of 2^30 instead of 2^20 (like the
    equivalent liquidity penalty) as it allows for more flexibility,
    and there's not really any reason to worry about us not being able
    to create high enough penalties.
    
    Closes lightningdevkit#1616
    TheBlueMatt committed Jul 25, 2022
    Copy the full SHA
    fe39a89 View commit details
    Browse the repository at this point in the history
  2. Rename amount penalty to liquidity_penalty_amount_multiplier_msat

    This makes our `ProbabilisticScorer` field names more consistent,
    as we add more types of penalties, referring to a penalty as only
    the "amount penalty" no longer makes sense - we not have several
    amount multiplier penalties.
    TheBlueMatt committed Jul 25, 2022
    Copy the full SHA
    7f80972 View commit details
    Browse the repository at this point in the history