[PR #358] [MERGED] fix: (FastAPI) Using LlamaCloud parameters does not use the configured value in the environment. #450

Closed
opened 2026-02-15 19:17:43 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/create-llama/pull/358
Author: @leehuwuj
Created: 10/9/2024
Status: Merged
Merged: 10/9/2024
Merged by: @marcusschiesser

Base: mainHead: fix/fastapi-llamacloud-env


📝 Commits (2)

  • 575b71e use env directly in constructor
  • c7e530f enable llamacloud tests for fastapi

📊 Changes

2 files changed (+16 additions, -13 deletions)

View changed files

📝 .github/workflows/e2e.yml (+1 -1)
📝 templates/components/vectordbs/python/llamacloud/index.py (+15 -12)

📄 Description

Summary by CodeRabbit

  • New Features

    • Enhanced configuration flexibility for LlamaCloudConfig, allowing initialization from environment variables.
  • Bug Fixes

    • Removed reliance on the deprecated from_env method for environment variable handling, streamlining the client creation process.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/create-llama/pull/358 **Author:** [@leehuwuj](https://github.com/leehuwuj) **Created:** 10/9/2024 **Status:** ✅ Merged **Merged:** 10/9/2024 **Merged by:** [@marcusschiesser](https://github.com/marcusschiesser) **Base:** `main` ← **Head:** `fix/fastapi-llamacloud-env` --- ### 📝 Commits (2) - [`575b71e`](https://github.com/run-llama/create-llama/commit/575b71e52fea8c1d1f50eb535779bc308419992c) use env directly in constructor - [`c7e530f`](https://github.com/run-llama/create-llama/commit/c7e530f8547f622ac112127b12a3d8895dedc8c0) enable llamacloud tests for fastapi ### 📊 Changes **2 files changed** (+16 additions, -13 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/e2e.yml` (+1 -1) 📝 `templates/components/vectordbs/python/llamacloud/index.py` (+15 -12) </details> ### 📄 Description <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Enhanced configuration flexibility for `LlamaCloudConfig`, allowing initialization from environment variables. - **Bug Fixes** - Removed reliance on the deprecated `from_env` method for environment variable handling, streamlining the client creation process. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 19:17:43 -05:00
yindo closed this issue 2026-02-15 19:17:43 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/create-llama#450