Closed leehuwuj closed 1 month ago
Latest commit: c7e530f8547f622ac112127b12a3d8895dedc8c0
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
The changes primarily focus on the LlamaCloudConfig
class within the index.py
file. A new __init__
method has been introduced to allow for flexible initialization of class attributes using kwargs
, while also retrieving values from environment variables if necessary. The previous from_env
class method has been removed, and the instantiation of LlamaCloudConfig
in the IndexConfig
class and the get_client
function has been updated to reflect this new initialization approach.
File Path | Change Summary |
---|---|
templates/components/vectordbs/python/llamacloud/index.py | - Added __init__(self, **kwargs) method in LlamaCloudConfig class.- Removed from_env class method in LlamaCloudConfig class.- Modified llama_cloud_pipeline_config to use a lambda for instantiation instead of from_env .- Updated get_client function to instantiate LlamaCloudConfig using the new constructor. |
In the clouds where llamas play,
A new config leads the way.
With keys from here and there,
It finds its home, a breath of air.
No more from_env to call,
Just simple kwargs, one and all! 🐰✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
New Features
LlamaCloudConfig
, allowing initialization from environment variables.Bug Fixes
from_env
method for environment variable handling, streamlining the client creation process.