From b54dedcc8563286861b2ccda68bc246ad61338c0 Mon Sep 17 00:00:00 2001 From: ismaelsadeeq Date: Mon, 1 Sep 2025 13:32:13 +0200 Subject: [PATCH] fees: reduce `MIN_BUCKET_FEERATE` to 100 - The DEFAULT_MIN_RELAY_TX_FEE has been reduced to 100 in v30. This change updates block policy fee estimator to reflect that change. --- src/policy/fees/block_policy_estimator.h | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/policy/fees/block_policy_estimator.h b/src/policy/fees/block_policy_estimator.h index ed561d9a6e5..4524911c299 100644 --- a/src/policy/fees/block_policy_estimator.h +++ b/src/policy/fees/block_policy_estimator.h @@ -181,13 +181,15 @@ private: static constexpr double SUFFICIENT_TXS_SHORT = 0.5; /** Minimum and Maximum values for tracking feerates - * The MIN_BUCKET_FEERATE should just be set to the lowest reasonable feerate we - * might ever want to track. Historically this has been 1000 since it was - * inheriting DEFAULT_MIN_RELAY_TX_FEE and changing it is disruptive as it - * invalidates old estimates files. So leave it at 1000 unless it becomes - * necessary to lower it, and then lower it substantially. + * The MIN_BUCKET_FEERATE should just be set to the lowest reasonable feerate. + * MIN_BUCKET_FEERATE has historically inherited DEFAULT_MIN_RELAY_TX_FEE. + * It is hardcoded because changing it is disruptive, as it invalidates existing fee + * estimate files. + * + * Whenever DEFAULT_MIN_RELAY_TX_FEE changes, this value should be updated + * accordingly. At the same time CURRENT_FEES_FILE_VERSION should be bumped. */ - static constexpr double MIN_BUCKET_FEERATE = 1000; + static constexpr double MIN_BUCKET_FEERATE = 100; static constexpr double MAX_BUCKET_FEERATE = 1e7; /** Spacing of FeeRate buckets