Skip to content

Update installing requirements section#9

Open
BakungaBronson wants to merge 2 commits into
yoheinakajima:mainfrom
BakungaBronson:patch-2
Open

Update installing requirements section#9
BakungaBronson wants to merge 2 commits into
yoheinakajima:mainfrom
BakungaBronson:patch-2

Conversation

@BakungaBronson

Copy link
Copy Markdown

Changed the command to install all the requirements and not just litellm.

Changed the command to install all the requirements and not just litellm.
@jamiekawabata

jamiekawabata commented Oct 21, 2024

Copy link
Copy Markdown

Maybe also change the "Python 3.7 or higher" prerequisite.

I think litellm requires Python 3.8 at a minimum.

The packages in the requirements.txt do not all work on Python 3.8, so demanding everything from requirements.txt would further raise the python requirement.

The readme could strike the explicit python prerequisite and people are on their own.

Edit: It looks like Python 3.9 is enough to install everything in requirements.txt

Minimum Python version that can install the requirements without errors.
@BakungaBronson

Copy link
Copy Markdown
Author

I use Python 3.10, so I was oblivious to this. Thank you for pointing it out. I have added the edit too.

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