Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Dec 16, 2017

fallbackfee is impossible to keep up-to-date by hardcoding a default value. This removes the default value and hands the responsibility to the user to determine an appropriate fallbackfee in case fee estimates are not available.

Todo:

  • Properly rework the GUI when no fee estimates are available and no fallbackfee is set. (TBD: Upcoming in a later pull request)

@maflcko maflcko force-pushed the Mf1712-NoFallBackFee branch from fa33f64 to faad60c Compare December 16, 2017 19:27
@aikq35
Copy link

aikq35 commented Dec 16, 2017

#11882

@laanwj
Copy link
Member

laanwj commented Dec 20, 2017

Concept ACK. I think this is is an obviously good idea. If no fee estimate can be made, users should set a manual fee. Hardcoding a fallback fee is not realistic anymore.

@promag
Copy link
Contributor

promag commented Dec 20, 2017

+1

@jonasschnelli
Copy link
Contributor

Concept ACK... the only bothering thing is, that regtest mode gets quite useless without passing in a -fallbackfee... and for that reason #11882 (removes default fallbackfee on mainnet only) may make more sense.

@maflcko
Copy link
Member Author

maflcko commented Dec 20, 2017

My understanding is that regtest mode becomes more and more useless as the number of differences to mainnet increase. If we special case that on mainnet, there'd be no way to test other than running a mainnet node.

Especially since we plan to rework the gui based on the fallback settings, it is important to be able to test in a non-mainnet setting.

@morcos
Copy link
Contributor

morcos commented Dec 20, 2017

@MarcoFalke But how do you avoid anyone doing manual regtest avoiding passing -fallbackfee every time they launch?

@maflcko
Copy link
Member Author

maflcko commented Dec 20, 2017

They could specify it in their bitcoin.conf, assuming that it exists. If no bitcoin.conf exists, the only way I see is by passing it in on the command line.

@maflcko
Copy link
Member Author

maflcko commented Dec 20, 2017

Closing, since #11882 is now similar and there is little value in keeping both open.

@maflcko maflcko closed this Dec 20, 2017
@maflcko maflcko deleted the Mf1712-NoFallBackFee branch December 20, 2017 20:24
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants