run-llama/llama_deploy
1
0
Fork 0
mirror of https://github.com/run-llama/llama_deploy.git synced 2026-08-24 21:01:26 -04:00
Code Issues 37 Packages Projects Releases 10 Wiki Activity

[PR #318] [MERGED] Update full stack example to have state-management #392

New Issue
Closed
opened 2026-02-16 01:17:10 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 01:17:10 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/run-llama/llama_deploy/pull/318
Author: @logan-markewich
Created: 10/14/2024
Status: ✅ Merged
Merged: 10/16/2024
Merged by: @logan-markewich

Base: main ← Head: logan/state_management

📝 Commits (10+)

📊 Changes

20 files changed (+414 additions, -149 deletions)

View changed files

➕ e2e_tests/basic_session/__init__.py (+0 -0)
➕ e2e_tests/basic_session/conftest.py (+55 -0)
📝 e2e_tests/basic_session/test_run_client.py (+31 -52)
➕ e2e_tests/basic_session/workflow.py (+16 -0)
➖ e2e_tests/run_all_e2e_tests.sh (+0 -11)
📝 examples/python_fullstack/docker-compose.yml (+1 -1)
📝 examples/python_fullstack/frontend/frontend/frontend.py (+21 -22)
➕ examples/python_fullstack/frontend/frontend/session_list/__init__.py (+0 -0)
➕ examples/python_fullstack/frontend/frontend/session_list/component.py (+74 -0)
➕ examples/python_fullstack/frontend/frontend/session_list/state.py (+68 -0)
📝 examples/python_fullstack/frontend/frontend/state.py (+37 -21)
📝 examples/python_fullstack/python_fullstack.yaml (+6 -19)
📝 examples/python_fullstack/workflows/agent_workflow.py (+22 -9)
📝 llama_deploy/apiserver/routers/deployments.py (+32 -5)
📝 llama_deploy/cli/deploy.py (+5 -1)
📝 llama_deploy/client/async_client.py (+15 -0)
📝 llama_deploy/client/sync_client.py (+13 -0)
📝 llama_deploy/control_plane/server.py (+2 -2)
📝 llama_deploy/services/workflow.py (+14 -4)
📝 tests/apiserver/routers/test_deployments.py (+2 -2)

📄 Description

Oh boy, this was a doozy. I need to get better at frontend patterns for components/state lol

This PR

  • adds state management to the UI of the full stack example
  • fixes how things are serialized (to avoid too many json.dumps calls)
  • fixes a random bug with reporting errors in the cli tooling
  • fixes e2e tests for state management (should have included this in the previous PR, whoops)

🔄 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/llama_deploy/pull/318 **Author:** [@logan-markewich](https://github.com/logan-markewich) **Created:** 10/14/2024 **Status:** ✅ Merged **Merged:** 10/16/2024 **Merged by:** [@logan-markewich](https://github.com/logan-markewich) **Base:** `main` ← **Head:** `logan/state_management` --- ### 📝 Commits (10+) - [`0f72731`](https://github.com/run-llama/llama_deploy/commit/0f727310e0dbae455bb726f8a30665a6a54353f1) wip - [`fc3185d`](https://github.com/run-llama/llama_deploy/commit/fc3185dc30e9a7e7df7e5d7bc431eed157ea37d8) wip - [`ad82924`](https://github.com/run-llama/llama_deploy/commit/ad82924e5eb081cf9dc6e70aac50e610c71f1cb4) add e2e tests - [`c7cc04e`](https://github.com/run-llama/llama_deploy/commit/c7cc04e0711bb6cc85e6ad977cf84a718d5e230d) add session_id to test task_def - [`220bf64`](https://github.com/run-llama/llama_deploy/commit/220bf647e7fc228655b7e832d3c494543328b02d) bump core dep - [`2ed628a`](https://github.com/run-llama/llama_deploy/commit/2ed628a20cb6a10f6e532c42d9970fc58fcd690f) Merge branch 'main' into logan/state_management - [`d7c3c8f`](https://github.com/run-llama/llama_deploy/commit/d7c3c8fc3583638f85cff76a6034ff86aea5683f) merge main - [`1878cd0`](https://github.com/run-llama/llama_deploy/commit/1878cd05375997657c5e2e224cd3e4dcc9f8e1f9) fix e2e tests - [`673efcb`](https://github.com/run-llama/llama_deploy/commit/673efcb563602a5f26606f5cce278066e6f2ef16) update fullstack example to have state management - [`1fafc16`](https://github.com/run-llama/llama_deploy/commit/1fafc16b08830ab7aab0ff08aaac7df3cbcad32b) merge main ### 📊 Changes **20 files changed** (+414 additions, -149 deletions) <details> <summary>View changed files</summary> ➕ `e2e_tests/basic_session/__init__.py` (+0 -0) ➕ `e2e_tests/basic_session/conftest.py` (+55 -0) 📝 `e2e_tests/basic_session/test_run_client.py` (+31 -52) ➕ `e2e_tests/basic_session/workflow.py` (+16 -0) ➖ `e2e_tests/run_all_e2e_tests.sh` (+0 -11) 📝 `examples/python_fullstack/docker-compose.yml` (+1 -1) 📝 `examples/python_fullstack/frontend/frontend/frontend.py` (+21 -22) ➕ `examples/python_fullstack/frontend/frontend/session_list/__init__.py` (+0 -0) ➕ `examples/python_fullstack/frontend/frontend/session_list/component.py` (+74 -0) ➕ `examples/python_fullstack/frontend/frontend/session_list/state.py` (+68 -0) 📝 `examples/python_fullstack/frontend/frontend/state.py` (+37 -21) 📝 `examples/python_fullstack/python_fullstack.yaml` (+6 -19) 📝 `examples/python_fullstack/workflows/agent_workflow.py` (+22 -9) 📝 `llama_deploy/apiserver/routers/deployments.py` (+32 -5) 📝 `llama_deploy/cli/deploy.py` (+5 -1) 📝 `llama_deploy/client/async_client.py` (+15 -0) 📝 `llama_deploy/client/sync_client.py` (+13 -0) 📝 `llama_deploy/control_plane/server.py` (+2 -2) 📝 `llama_deploy/services/workflow.py` (+14 -4) 📝 `tests/apiserver/routers/test_deployments.py` (+2 -2) </details> ### 📄 Description Oh boy, this was a doozy. I need to get better at frontend patterns for components/state lol This PR - adds state management to the UI of the full stack example - fixes how things are serialized (to avoid too many json.dumps calls) - fixes a random bug with reporting errors in the cli tooling - fixes e2e tests for state management (should have included this in the previous PR, whoops) --- <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-16 01:17:10 -05:00
yindo closed this issue 2026-02-16 01:17:10 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
P0
P1
P2
bug
documentation
documentation
enhancement
example
good first issue
invalid
pull-request
Mirrored from GitHub Pull Request
 question
topic:CLI
topic:SDK
topic:SDK
topic:SDK
topic:apiserver
topic:control_plane
topic:message_queues
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/llama_deploy#392
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?