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

pikers.broker.ib subpackage #335

Merged
merged 6 commits into from
Jun 7, 2022
Merged

pikers.broker.ib subpackage #335

merged 6 commits into from
Jun 7, 2022

Conversation

goodboy
Copy link
Contributor

@goodboy goodboy commented Jun 6, 2022

A conversion of piker.brokers.ib.py into a sub-package with the following new modules:

  • ib.api.py containing the core ib_insync wrapper Client and low level trio-asyncio wrapping / translation layer using tractor.to_asyncio
  • ib.feed.py which contains all data related endpoints for historical and real time feeds
  • ib.broker.py which contains the required trade endpoint(s) needed to integrate with piker's EMS~

goodboy added 5 commits June 6, 2022 19:33
The single-file module was getting way out of hand size-wise with the
new flex report parsing stuff so this starts the process of breaking
things up into smaller modules oriented around trade, data, and ledger
related endpoints.

Add support for backends to declare sub-modules to enable in
a `__enable_modules__: list[str]` module var which is parsed by the
daemon spawning code passed to `tractor`'s `enable_modules: list[str]`
input.
@goodboy goodboy changed the title Ib subpkg pikers.broker.ib subpackage Jun 6, 2022
@goodboy goodboy marked this pull request as ready for review June 6, 2022 23:39
Base automatically changed from flexxin to master June 7, 2022 13:42
@goodboy goodboy merged commit 633fa7c into master Jun 7, 2022
@goodboy goodboy deleted the ib_subpkg branch June 7, 2022 15:41
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