From 8e3b727d07aedff07e1a2038ccb72adfa256d6a4 Mon Sep 17 00:00:00 2001 From: wiz Date: Fri, 2 Oct 2020 05:06:55 +0900 Subject: [PATCH] Increase our mempool fee estimation from "medium" to "high" priority Bisq currently uses the mempool.space API for fee estimation, selecting the "medium" priority fee, which generally works very well. However, if your timing is bad, sometimes the mempool can quickly fill up and your TX can get stuck for several hours or even days, which degrades the UX of Bisq. This PR changes the Pricenode code to always use "high" priority fees, which would at the time of writing use 99 sat/vB instead of 85 sat/vB, and hopefully prevent the above-mentioned issue from occuring. Of course if the mempool is empty the minimum fees would still be used, as this only changes the "priority" from medium to high. Fee estimates: https://mempool.space/ API endpoint: https://mempool.space/api/v1/fees/recommended --- .../bisq/price/mining/providers/MempoolFeeRateProvider.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pricenode/src/main/java/bisq/price/mining/providers/MempoolFeeRateProvider.java b/pricenode/src/main/java/bisq/price/mining/providers/MempoolFeeRateProvider.java index b9944d0383..faa988cd62 100644 --- a/pricenode/src/main/java/bisq/price/mining/providers/MempoolFeeRateProvider.java +++ b/pricenode/src/main/java/bisq/price/mining/providers/MempoolFeeRateProvider.java @@ -91,7 +91,7 @@ abstract class MempoolFeeRateProvider extends FeeRateProvider { private long getEstimatedFeeRate() { return getFeeRatePredictions() - .filter(p -> p.getKey().equalsIgnoreCase("halfHourFee")) + .filter(p -> p.getKey().equalsIgnoreCase("fastestFee")) .map(Map.Entry::getValue) .findFirst() .map(r -> {