[PR #404] [CLOSED] Fix dictionary hashing with pydantic models as keys #440

Closed
opened 2026-02-16 01:17:21 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/llama_deploy/pull/404
Author: @apost71
Created: 12/13/2024
Status: Closed

Base: mainHead: main


📝 Commits (10+)

  • dfbc3cf fix: fix dict hashing with pydantic models as keys
  • 00e7616 fix: use str and add test
  • bb37c5d chore: reformatting
  • 0c47465 Merge branch 'main' into main
  • a160a6e Merge branch 'main' into main
  • c23a6cf Merge branch 'main' into main
  • 9cf05ae Merge branch 'main' into main
  • 049dd98 fix: use model validator for hash
  • db0c382 Merge branch 'main' into main
  • c8f8b02 Merge branch 'main' into main

📊 Changes

2 files changed (+125 additions, -15 deletions)

View changed files

📝 llama_deploy/services/workflow.py (+15 -13)
📝 tests/services/test_workflow_service.py (+110 -2)

📄 Description

PR to address issue https://github.com/run-llama/llama_deploy/issues/392


🔄 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/404 **Author:** [@apost71](https://github.com/apost71) **Created:** 12/13/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (10+) - [`dfbc3cf`](https://github.com/run-llama/llama_deploy/commit/dfbc3cfe649e55b5ec8c79588d4d3e0e20b8a876) fix: fix dict hashing with pydantic models as keys - [`00e7616`](https://github.com/run-llama/llama_deploy/commit/00e7616b09320c57defd118a7237045539e148bd) fix: use str and add test - [`bb37c5d`](https://github.com/run-llama/llama_deploy/commit/bb37c5d9f1c61cb4c48bad8935df71480d68d7bd) chore: reformatting - [`0c47465`](https://github.com/run-llama/llama_deploy/commit/0c474656ea154e5e34d70f856bfaa9480fda920b) Merge branch 'main' into main - [`a160a6e`](https://github.com/run-llama/llama_deploy/commit/a160a6e83e9c115fee05e491ad7d5ee6cf72b368) Merge branch 'main' into main - [`c23a6cf`](https://github.com/run-llama/llama_deploy/commit/c23a6cf61153439e5cdb8caece38bea89480483c) Merge branch 'main' into main - [`9cf05ae`](https://github.com/run-llama/llama_deploy/commit/9cf05ae45041d24a3ceff7d270de3701c1ecd379) Merge branch 'main' into main - [`049dd98`](https://github.com/run-llama/llama_deploy/commit/049dd980acc5cd5af55d8416f3487395f79c8000) fix: use model validator for hash - [`db0c382`](https://github.com/run-llama/llama_deploy/commit/db0c382c16691e66100d9b583071cc06bceffee6) Merge branch 'main' into main - [`c8f8b02`](https://github.com/run-llama/llama_deploy/commit/c8f8b02776d824a5b6b94084f15cd41be6f1f3d0) Merge branch 'main' into main ### 📊 Changes **2 files changed** (+125 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `llama_deploy/services/workflow.py` (+15 -13) 📝 `tests/services/test_workflow_service.py` (+110 -2) </details> ### 📄 Description PR to address issue https://github.com/run-llama/llama_deploy/issues/392 --- <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:21 -05:00
yindo closed this issue 2026-02-16 01:17:21 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/llama_deploy#440