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
Time In Force #23
Time In Force #23
Conversation
current_slot, | ||
current_unix_timestamp, |
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.
weird to sometimes pass around the get_clock_fn
and sometimes pass around the values, no?
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.
If get_clock_fn
were an actual clock (ticking while executing) you really want the snapshot of that clock to be taken at the time of place. That makes passing its outputs into the private function match_order
a reasonable decision.
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.
ya but it's not, and never will be
#[inline] | ||
/// Quote lot budget with fees adjusted (sells) | ||
/// | ||
/// The desired result is adjusted_quote_lots / (1 - fee_bps). We approach this result by taking |
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.
explain in comment the use case for this one, it's definitely not obvious
current_slot, | ||
current_unix_timestamp, |
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.
ya but it's not, and never will be
Time-in-force (TIF) orders for Phoenix v1. Changes: - Uses the padding on the FIFORestingOrder to store and expiration slot and an expiration time - Modifies the OrderPacket struct to enable specification of expiration - Injects logic into the matching engine to skip orders that are expired
Time-in-force (TIF) orders for Phoenix v1. Changes: - Uses the padding on the FIFORestingOrder to store and expiration slot and an expiration time - Modifies the OrderPacket struct to enable specification of expiration - Injects logic into the matching engine to skip orders that are expired
Draft implementation of time-in-force (TIF) orders for Phoenix v1.
Added a standard test in test_market.rs. Could use more coverage
Changes: