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

Fix InteractiveBrokers historical bar data bug #1499

Merged
merged 3 commits into from
Feb 14, 2024

Conversation

benjaminsingleton
Copy link
Contributor

Pull Request

User "d2b" in Discord reported the following error:

Error on `_run_internal_msg_queue`: AttributeError("'coroutine' object has no attribute 'keywords'")

The issue originates from a bug in the historicalDataUpdate EWrapper override method of the InteractiveBrokersClientMarketDataMixin class. The root cause is the incorrect usage of a functools.partial object, which encapsulates the subscribe_historical_bars function. The functools.partial was being called rather than simply accessing the keywords attribute.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How has this change been tested?

Tested locally.

@cjdsellers
Copy link
Member

Looks like CI only failing due to a temporary GitHub runner glitch:

Error output:
No Python at '"C:\hostedtoolcache\windows\Python\3.11.7\x64\python.exe'

This normally clears after a few hours - I'll re-run the failing jobs later.

@cjdsellers cjdsellers merged commit bf32484 into nautechsystems:develop Feb 14, 2024
9 of 11 checks passed
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