Decay `historical_estimated_channel_liquidity_*` result to `None`
authorMatt Corallo <git@bluematt.me>
Tue, 6 Jun 2023 04:08:32 +0000 (04:08 +0000)
committerMatt Corallo <git@bluematt.me>
Fri, 15 Sep 2023 17:27:29 +0000 (17:27 +0000)
commitf7f524f19a29652296e1af507cad9e9b9749f6fc
tree5bcf8aa18e133c7ec87c8b2da8dbaa72fa76b2fe
parentb7d1e5f516ff340f8ba512df3e62d502b18e92fd
Decay `historical_estimated_channel_liquidity_*` result to `None`

`historical_estimated_channel_liquidity_probabilities` previously
decayed to `Some(([0; 8], [0; 8]))`. This was thought to be useful
in that it allowed identification of cases where data was previously
available but is now decayed away vs cases where data was never
available. However, with the introduction of
`historical_estimated_payment_success_probability` (which uses the
existing scoring routines so will decay to `None`) this is
unnecessarily confusing.

Given data which has decayed to zero will also not be used anyway,
there's little reason to keep the old behavior, and we now decay to
`None`.

We also take this opportunity to split the overloaded
`get_decayed_buckets`, removing uneccessary code during scoring.
lightning/src/routing/scoring.rs