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

first stub #11

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

first stub #11

wants to merge 10 commits into from

Conversation

sixtyninesigma
Copy link

  • opening pr to track ibkr work. i will amend as things progress, no need to merge as of now.

@AdoNunes
Copy link
Owner

AdoNunes commented Sep 1, 2023

let's try step by step does:

ibkr = Ibkr()
ibkr.get_session()

print out success?

@sixtyninesigma
Copy link
Author

let's try step by step does:

ibkr = Ibkr()
ibkr.get_session()

print out success?

yes.

the constructor work works fine so far. adding account info, need to add some parse logic and dict formatiing to abide to definition of acc_inf

@AdoNunes
Copy link
Owner

AdoNunes commented Feb 9, 2024

I was going to finish implementing IBKR but it is a pain, their methods are weird as, and the documentation is not clear... Instead, I am working on tradestation, very good and straightforward API and have api demo account

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