Skip to content
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

BEP19: Introduce Maker and Taker for Match Engine #19

Merged
merged 2 commits into from
Jun 28, 2019
Merged

Conversation

rickyyangz
Copy link
Contributor

@rickyyangz rickyyangz commented Jun 18, 2019

This BEP describes a new on-chain match engine, Maker/Taker concepts are introduced to enhance the current periodic auction match algorithm.

Motivation

On not-so-liquid or highly volatile markets, the classic periodic auction match methodology would result in much worse average trade price to aggressive orders (i.e. Taker orders in the continuous match context), because all the trades would be marked with one final price, instead of the average price of all the waiting orders from the market. This is not friendly to active traders and amateur clients that are more familiar with continuous match markets. This causes extra effort to split aggressive orders and much confusion as Binance DEX matches as fast as continuous matching exchange.

@rickyyangz rickyyangz changed the title BEP19: match engine revision [WIP] BEP19: match engine revision Jun 18, 2019
@rickyyangz rickyyangz changed the title [WIP] BEP19: match engine revision BEP19: match engine revision Jun 28, 2019
@rickyyangz rickyyangz changed the title BEP19: match engine revision BEP19: Introduce Maker and Taker for Match Engine Jun 28, 2019
@darren-liu darren-liu merged commit c7ee904 into master Jun 28, 2019
Copy link

@jonesk7734 jonesk7734 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😀

@unclezoro unclezoro deleted the matcheng branch November 28, 2023 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants