Skip to content

Fix package names and mark build-dependent deps as optional - #89

Open
AmelieLang wants to merge 1 commit into
AI4Finance-Foundation:masterfrom
AmelieLang:master
Open

Fix package names and mark build-dependent deps as optional#89
AmelieLang wants to merge 1 commit into
AI4Finance-Foundation:masterfrom
AmelieLang:master

Conversation

@AmelieLang

Copy link
Copy Markdown

Summary

  • Fix finnhub -> finnhub-python: the original package name does not exist on PyPI and causes pip install to fail for all new users
  • Mark torch>=2.0.0 as optional comment — saves ~2GB on default install
  • Mark bt>=1.2.0 as optional comment — requires Microsoft C++ Build Tools to compile on Windows

Test plan

  • pip install -r requirements.txt completes without errors on a clean Python 3.11+ environment
  • import finnhub works after install
  • Users who need bt or torch can install them separately

- Fix finnhub -> finnhub-python (correct PyPI package name, original causes install failure)
- Mark torch as optional comment (saves ~2GB, install separately if DRL needed)
- Mark bt as optional comment (requires Microsoft C++ Build Tools to build on Windows)
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.

1 participant