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

Add order book documentation #85

Merged
merged 9 commits into from
Nov 23, 2023
Merged

Add order book documentation #85

merged 9 commits into from
Nov 23, 2023

Conversation

Chralt98
Copy link
Member

@Chralt98 Chralt98 commented Nov 8, 2023

No description provided.

Copy link

vercel bot commented Nov 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2023 10:10am

Comment on lines 20 to 33
This function is used to introduce a new order into the market. When traders
decide to buy or sell an asset, they use `place_order` to specify the details of
their order, the quantity they wish to buy the `taker_asset` for `taker_amount`
and sell the `maker_asset` for `maker_amount`. This order is then added to the
order book, where it waits to be matched by taker(s).

It is important to mention that the users of the Zeitgeist orderbook do specify
the price of the assets implicitly. This means that the `place_order` does not
take a price as an argument, but rather the `taker_amount` and `maker_amount`.
This means the caller of the `place_order` function does willingly sell the
`maker_amount` of the `maker_asset` to buy the given `taker_amount` of the
`taker_asset`. After the execution of `place_order` the `maker_amount` of the
`maker_asset` is reserved for the order. This means it is not available for the
user to use in other transactions.
Copy link
Member

Choose a reason for hiding this comment

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

Would be nice to have a formula for the price here.

Copy link
Member Author

Choose a reason for hiding this comment

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

docs/learn/order-book.md Show resolved Hide resolved
docs/learn/order-book.md Outdated Show resolved Hide resolved
docs/learn/order-book.md Outdated Show resolved Hide resolved
trade, transferring the specified asset between parties at the agreed price, and
update the order book to reflect the completion of the trade.

The taker has the ability to partially fill the order for a given
Copy link
Member

Choose a reason for hiding this comment

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

I'm confused by this description. I think you're confusing maker and taker here. The taker is the one who fills the order, so I'm not quite sure what "This amount specifies the partial amount of what the maker wants to fill." is referring to.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, I agree it's confusing. Your sentence "The taker is the one who fills the order" and my sentence is just the perspective of the maker "This amount specifies the partial amount of what the maker wants to fill". But shall we leave this sentence rather out?

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

What I'm saying is that the sentence "... the maker wants to fill" doesn't really work because the maker doesn't fill orders.

Copy link
Member Author

Choose a reason for hiding this comment

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

Then it's the maker wants to get filled

Copy link
Member Author

Choose a reason for hiding this comment

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

But anyways, I deleted it. Can you approve?

Copy link
Member

Choose a reason for hiding this comment

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

The next sentence also mentions the maker filling an order. ;)

Copy link
Member Author

Choose a reason for hiding this comment

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

docs/learn/order-book.md Outdated Show resolved Hide resolved
trade, transferring the specified asset between parties at the agreed price, and
update the order book to reflect the completion of the trade.

The taker has the ability to partially fill the order for a given
Copy link
Member

Choose a reason for hiding this comment

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

The next sentence also mentions the maker filling an order. ;)

Co-authored-by: Malte Kliemann <[email protected]>
@Chralt98 Chralt98 merged commit 4fd2845 into main Nov 23, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants