stanfordnlp / dspy

DSPy: The framework for programming—not prompting—foundation models
https://dspy-docs.vercel.app/
MIT License
16.67k stars 1.29k forks source link

Conflicting openai dependencies #583

Open mwildehahn opened 6 months ago

mwildehahn commented 6 months ago

There are two dependency declarations for openai:

I'm trying to install this into my poetry managed app and it's complaining about the one pinned to 0.28.1.

It seems like the answer to this: https://github.com/stanfordnlp/dspy/blob/c2d639fb1c1dafa7d6a12b3fe730b3564580fbfe/pyproject.toml#L21 is yes? Happy to contribute moving off the old dependencies array to poetry if that is desired.

arnavsinghvi11 commented 6 months ago

@mwildehahn agreed that including both dependencies can cause internal conflicts. would be great if you could help with the dependencies migration! tagging @okhat as well

darinkishore commented 6 months ago

updated the poetry dependency.

but i think keeping both dependencies is best -> tools.poetry.dependencies is used for poetry users to be able to just poetry install things.

however, the project dependencies section functions similarly to requirements.txt ->

see: https://stackoverflow.com/questions/62408719/download-dependencies-declared-in-pyproject-toml-using-pip

tldr: python -m pip install . will install dependencies defined in pyproject.toml under the first dependencies list