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

The latest versions of deepeval and Instructor can't be installed together #1100

Open
milest opened this issue Oct 18, 2024 · 5 comments
Open

Comments

@milest
Copy link

milest commented Oct 18, 2024

To Reproduce
Steps to reproduce the behavior:

  1. pip install deepeval==1.4.1 instructor==1.6.2
  2. See error

Expected behavior
Both libraries install

Screenshots

Collecting instructor==1.6.2
  Downloading instructor-1.6.2-py3-none-any.whl.metadata (17 kB)
Collecting deepeval==1.4.1
  Downloading deepeval-1.4.1-py3-none-any.whl.metadata (977 bytes)
ERROR: Cannot install deepeval==1.4.1 and instructor==1.6.2 because these package versions have conflicting dependencies.

The conflict is caused by:
    deepeval 1.4.1 depends on tenacity~=8.4.1
    instructor 1.6.2 depends on tenacity<10.0.0 and >=9.0.0

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

The latest version of Tenacity made a breaking change to the statistics attribute. I did a quick search in deepeval and couldn't find an obvious usage of that attribute.

@yuriykuzin
Copy link

Same. It could be much helpful to have a fix on that.

@stevermeister
Copy link

same

@penguine-ip
Copy link
Contributor

@stevermeister @yuriykuzin @milest I tried tenacity ^9.0.0 but it will be conflicting with langchain.

From poetry:

... because langchain (0.1.6) depends on tenacity (>=8.1.0,<9.0.0)

I think the better solution would be to downgrade instructor?

@milest
Copy link
Author

milest commented Nov 1, 2024

Ah, I see. langchain==0.1.6 is a fairly old version, isn't it? The most recent release is 0.3.6.

Downgrading Instructor is definitely an option. Would another option be to require a newer version of langchain?

@yuriykuzin
Copy link

yuriykuzin commented Nov 12, 2024

Looks like even langchain 0.3.7 still requires older tenacity 😞

And because deepeval depends on langchain (^0.3.7), 
every version of llama-index-llms-langchain requires tenacity (>=8.2.0,<9.0.0).

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

No branches or pull requests

4 participants