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 support for bbo quotes in Databento adapter #1990

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

faysou
Copy link
Collaborator

@faysou faysou commented Oct 7, 2024

Pull Request

Add support for bbo-1s and bbo-1m sampled quotes in databento adapter

Type of change

  • New feature (non-breaking change which adds functionality)

How has this change been tested?

usage example on prototype (use bbo-1s or bbo-1m here as schema https://github.com/nautechsystems/nautilus_trader/blob/develop/examples/backtest/databento_option_greeks.py) , added tests

Note sure if something needs to be added for live databento data for this new data type, or how I would subscribe to live databento data from a live node.

Copy link
Member

@cjdsellers cjdsellers left a comment

Choose a reason for hiding this comment

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

Great work, thank you for adding the tests as well!

@cjdsellers cjdsellers changed the title Add support for bbo-1s and bbo-1m quotes in databento adapter Add support for bbo quotes in databento adapter Oct 7, 2024
@cjdsellers cjdsellers changed the title Add support for bbo quotes in databento adapter Add support for bbo quotes in Databento adapter Oct 7, 2024
@cjdsellers cjdsellers merged commit a0664a0 into nautechsystems:develop Oct 7, 2024
10 checks passed
@faysou
Copy link
Collaborator Author

faysou commented Oct 7, 2024

Thanks I've been eagerly waiting for this data type, this has given me time to learn and contribute to nautilus, which is quite important.

There's only one trick to know to contribute to nautilus: make build-debug :))))

@faysou faysou deleted the bbo branch October 7, 2024 18:27
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.

2 participants