Skip to content
This repository has been archived by the owner on Feb 21, 2019. It is now read-only.

Potentially incorrect matching in block 2059269 #1459

Closed
theoreticalbts opened this issue Mar 18, 2015 · 4 comments
Closed

Potentially incorrect matching in block 2059269 #1459

theoreticalbts opened this issue Mar 18, 2015 · 4 comments
Assignees
Milestone

Comments

@theoreticalbts
Copy link
Contributor

btswildpig has brought to our attention a potential market matching issue demonstrated in block 2059269.

@ghost
Copy link

ghost commented Mar 18, 2015

image
This was the original short order with 100K BTS collateral (Now the collateral is decreased 0.83604 because partially filled today , look at the next photo )

image

it filled at odd price (first 3 in the history )
the block number is 2059269 .

And the guy finally used 0.0035 USD to cover this partial short order and get 29 BTS in return . (no photo about it at this point ) .

@theoreticalbts
Copy link
Contributor Author

The issue where the short is matched despite not being at the top of the book should be fixed with the upcoming hardfork.

I think the strange price is legit. It seems to be due to the ask orders specifying a very low price:

>>> blockchain_market_order_book USD BTS
     BIDS (* Short, + Relative, - Relative Limit)                            |                                   ASKS                                  
TOTAL                     QUANTITY                                     PRICE | PRICE                                        QUANTITY                     TOTAL   COLLATERAL
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
16.9999 USD               2,091.32769 BTS                 0.008128759551 USD | 0.000125500000 USD                       10.00000 BTS                0.0012 USD
2.2391 USD                275.96907 BTS                   0.008113590264 USD | 0.000125600000 USD                       11.00000 BTS                0.0013 USD
100.0000 USD              12,324.99999 BTS                0.008113590264 USD | 0.008130000000 USD                       21.68218 BTS                0.1762 USD
1,934.9218 USD            238,962.85227 BTS               0.008097165992 USD*| 0.008403361345 USD                    2,022.98810 BTS               16.9999 USD

There is another more subtle matching issue here as well, it is known and a fix for it has been developed and is currently being tested.

I will double check the numbers on this, will close the ticket once I do that and have a reference to the ticket for the other issue.

@pmconrad
Copy link
Contributor

See also the last 4 comments in #1277

@vikramrajkumar vikramrajkumar added this to the dvs/0.10.0 milestone Mar 30, 2015
@vikramrajkumar
Copy link
Contributor

I believe this was fixed by 0f34cb3 and/or 371343c, which have since been backported to develop.

@vikramrajkumar vikramrajkumar modified the milestones: dvs/0.10.0, dvs/0.9.0 Apr 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants