-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Replacing deprecated alpaca_trade_api library with alpaca #1378
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
base: master
Are you sure you want to change the base?
Conversation
Removed 'alpaca-trade-api' from dependencies.
Removed alpaca_trade_api dependency from requirements.
Removed unused import statements for alpaca_trade_api.
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joe1083
I've been digging through the code some more, it looks like the FinRL_PaperTrading_Demo.ipynb will need to be updated as well. I am looking into this. |
Removed unnecessary installation commands for wrds, swig, and condacolab.
for more information, see https://pre-commit.ci
Removed unused import of the alpaca module.
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Sorry for all the commits on this pull request. There was a lot more areas of the code that needed updated to the new A couple of things to note:
Thank you for your patience on this! Let me know if you have any questions about the changes made! |
Design
alpaca_trade_api
(Github Repo) has been deprecated and is no longer compatible with Google Colab due to it referencing libraries that are now out of date. This change is to replace the old library with the newer versionalpaca
(Github Repo). This Pull Request is related to Issue #1376.A separate pull request will be open for the FinRL Tutorial repository to handle this update.
Google Colab Error