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 library top-level imports #1296

Merged
merged 5 commits into from
Nov 28, 2024
Merged

Conversation

rlouf
Copy link
Member

@rlouf rlouf commented Nov 28, 2024

Users are currently running into install issues. After a clean install of outlines they get an error message that asks for transformers to be installed. This should not be the case, as the library is not required for every integration. In this PR we remove transformers and datasets top-level imports, and add per-integration optional dependencies.

TODO

  • Test import outlines from clean install
  • Test installing outlines with vLLM optional dependencies
  • Test installing outlines with MLX optional dependencies
  • Test installing outlines with transformers optional dependencies
  • Test installing outlines with llama-cpp optional dependencies
  • Test installing outlines with exllamav2 optional dependencies
  • Test installing outlines with openai optional dependencies
  • Update the documentation

Supersedes #1295. Fixes #1263.

@rlouf rlouf added bug enhancement dependencies Pull requests that update a dependency file labels Nov 28, 2024
@rlouf rlouf force-pushed the fix-library-imports branch 3 times, most recently from b51bcb6 to 2ef76ca Compare November 28, 2024 18:57
@rlouf rlouf marked this pull request as ready for review November 28, 2024 19:07
@rlouf rlouf merged commit 36f1bf2 into dottxt-ai:main Nov 28, 2024
5 of 6 checks passed
@rlouf rlouf deleted the fix-library-imports branch November 28, 2024 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug dependencies Pull requests that update a dependency file enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing dependency transformers upon fresh install
1 participant