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

fix(pyproject.toml): remove eval from all dependencies #817

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Pouyanpi
Copy link
Collaborator

@Pouyanpi Pouyanpi commented Oct 22, 2024

Description

Setuptools is unable to resolve the dependency graph when eval is included in the all dependencies list. This issue is primarily caused by a limitation of pip and a conflict with streamlit. Removing eval from the all dependencies list resolves this issue. Eval can be installed separately if needed.

Related Issue(s)

Checklist

  • I've read the CONTRIBUTING guidelines.
  • I've updated the documentation if applicable.
  • I've added tests if applicable.
  • @mentions of the person or team responsible for reviewing proposed changes.

@Pouyanpi Pouyanpi requested a review from drazvan October 22, 2024 11:44
@Pouyanpi Pouyanpi self-assigned this Oct 22, 2024
@Pouyanpi
Copy link
Collaborator Author

@drazvan , I am not sure why eval is included in the all optional dependencies. Pip fails resolving it, tested with poetry and it works, also if you install all excluding eval, and then eval it works, so I removed it from the all section.

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