Skip to content

Order book methods #8

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Order book methods #8

wants to merge 10 commits into from

Conversation

rundef
Copy link
Owner

@rundef rundef commented Apr 28, 2025

No description provided.

@DeclanDunneIRL
Copy link

Any when this PR will be complete? Ive been using it but id love to see the docs. well done on this project so far.

@rundef
Copy link
Owner Author

rundef commented Jun 1, 2025

Hey, thanks for the kind words.

I didn't realize anyone was already using this branch!
I will try to wrap up this PR and the docs in the next 1-2 months max.

@DeclanDunneIRL
Copy link

Almost! 😄

@socialgrowthlabs
Copy link

Hey @rundef How are you tracking with the Market Depth, is it ready to be used?

@rundef
Copy link
Owner Author

rundef commented Jun 26, 2025

Hey @rundef How are you tracking with the Market Depth, is it ready to be used?

I'm hoping to release this feature in a week or so

@rundef
Copy link
Owner Author

rundef commented Jun 30, 2025

@DeclanDunneIRL @socialgrowthlabs The feature is ready, you can try this branch and let me know what you think.

The doc is here: https://github.com/rundef/async_rithmic/blob/feat/order_book/docs/realtime_data.rst#order-book

@DeclanDunneIRL
Copy link

DeclanDunneIRL commented Jun 30, 2025

I’m away this week but will get to it next week. Thanks for the update and hard work

@rundef rundef marked this pull request as ready for review July 23, 2025 21:25
@rundef
Copy link
Owner Author

rundef commented Jul 23, 2025

I’m away this week but will get to it next week. Thanks for the update and hard work

Did you have time to test it @DeclanDunneIRL ? If not it's okay, I'll just merge it

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.

3 participants