You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
ragbits-core should list LiteLLM as its direct dependency.
Conditional imports of litellm should be changes to regular imports
Make LiteLLM available from the root ragbits.core.llms module
Test that LiteLLM can imported without full path by LLM.subclass_from_config by adding test similar to tests.unit.embeddings.test_from_config.test_subclass_from_config_default_path to tests.unit.llms.test_from_config
All places that mention ragbits[litellm] in the documentation (e.g., Quickstart 1) should be adjusted accordingly
The text was updated successfully, but these errors were encountered:
Currently
ragbits
andragbits-core
are installed without installing LiteLLM (which requires explicitely installingragbits[litellm]
.Motivation
default_llm_factories
#154, Ragbits comes pre configured with default LLM values which use LiteLLM. This currently leads to Ragbits not being able to use its own defaults by default ;)Steps
ragbits-core
should list LiteLLM as its direct dependency.litellm
should be changes to regular importsLiteLLM
available from the rootragbits.core.llms
moduleLiteLLM
can imported without full path byLLM.subclass_from_config
by adding test similar totests.unit.embeddings.test_from_config.test_subclass_from_config_default_path
totests.unit.llms.test_from_config
ragbits[litellm]
in the documentation (e.g., Quickstart 1) should be adjusted accordinglyThe text was updated successfully, but these errors were encountered: