-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Deposit improvements #4347
Deposit improvements #4347
Conversation
Are you sure it’s safe to trust the trade statistics data? AFAIK it’s not reliable and can be easily manipulated |
I think it's safe enough to trust it to give a suggested deposit value. This value can still be changed if the trader doesn't like it, and it's still in the 15%-50% range so I don't really see a problem there. |
Yes to making it more prominent in the offer book. Why can I not have a high security deposit as a BTC BUYER / offer maker, and set a low one for the BTC seller? |
There seems to be support for symmetric deposits in bisq-network/proposals#233. It's still easy to just change the constant |
I'm probably not helping. And these are probably dumb questions. But why does there even need to be an algorithm? Choice please. Flexibility. That is the only real answer imo. Why not indicate the things to consider: i.e. consequences of a low % security deposit? (Less capital tied up for maker, and taker?, more risk for taker? [thus less likely to accept?]) Consequences of a high % deposit? (more capital tied up, but more reassurance for taker?) shrugs those are all poorly formulated, but hopefully the gist is clear... maker sets security deposit %'s (10%-100%), and let the market (individual) decide whether its acceptable or not. |
@wiz the previous PR raised the minimum to 30%, this leaves it at 15%. It's just suggesting a deposit that the user can change. |
@Conza88 this is basically what's happening now, except the range is 15% - 50%. Widening the range might be problematic during deployment due to the trade protocol, but can be done of course. There is a problem that the cost of mediation and arbitration is not easily priced for traders so we're basically trying to suggest a reasonable deposit that includes that risk. I suspect many Bisq users don't understand the implication of the deposit as the price of an option that not only depend on the price movement of the underlying but also on the arbitration counterparty risk. |
As a mediator, I have very few BSQ trades that go into mediation. There’s no need to suggest users to more than triple the security deposits, why would you do that? Please only suggest users to use higher security deposits for the markets and payment methods that need them. BSQ and JPY don’t need higher deposits, those can be left alone. The markets that always go into mediation like XMR and USD with Zelle are probably the ones that could benefit from higher deposit. Didn’t we discuss this in the proposal already |
Don't include outliers (20% deviation from moving average) in moving average calculation. It's quite likely that low liquidity markets or markets with large spreads can't calculate deposit suggestion and will then suggest deposit from preferences. Added test for moving average class
The security deposit needs to be higher for volatile markets, that's what it's pricing, not the safety of the payment method. During very volatile times even 50% might be low so leaving that as max for now is quite reasonable. The latest changes improves the moving average calculation by excluding prices that deviate more than 20% (not sure if too high, 10%?) from the the moving average. This might cause a failure to calculate a suggestion for illiquid markets and suggest the default which seems reasonable. Issues in new or illiquid markets are likely not from suggesting a low deposit. |
So if someone wants to manipulate your security deposit suggestion algorithm into always suggesting 50% max deposit for all markets, they just need to make 1 self-trade that's way off the current market price? How often do they need to do this? |
They would need to continuously do this at price levels that are less than 20% off the moving average to manipulate the suggested price. Not saying it can't happen, but it seems unlikely as there is little benefit to doing so for the tape painter, and any user can change the suggested deposit if they don't like it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK - Tested it on Regtest and Mainnet and the values seem reasonable (e.g before in EUR/BTC market 15% vs 17%). Of course long living offers might not have a good adapted deposit amount, but as the deposit can be adapted by the maker and is shown in the offerbook its fine IMO.
Why is there even a need for suggested deposits algorithm %?! Explain the consequences of low and high security deposit to maker & taker. |
@wiz are you testing this live or regtest? When I create an offer live I get the expected 15% since there aren't enough trades in the near past to get an estimate Also make sure you didn't set the deposit to 50% at some point which saves that value to settings as your preferred minimum deposit. |
This was mainnet testing of the release branch |
Do you still get this when you have preferences set to 15%? It's not obvious that this is set to preferences, but that's how it's apparently always been, if you set the deposit manually, that value will be saved as your preferred value. |
Implements suggestion in bisq-network/proposals#233 (comment)
Increased visibility of security deposit in offer overview.
Set suggested deposit to double of trade range over the trade period.
Both traders have the same security deposit.