New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bugfix][Executor] Fix the issue of duplicate line numbers in the chat group run scenario #3145
Conversation
promptflow SDK CLI Azure E2E Test Result devs/peiwen/fix_chatgroup_run 4 files 4 suites 4m 10s ⏱️ Results for commit 9b639b8. ♻️ This comment has been updated with latest results. |
SDK CLI Global Config Test Result devs/peiwen/fix_chatgroup_run6 tests 6 ✅ 1m 14s ⏱️ Results for commit 9b639b8. ♻️ This comment has been updated with latest results. |
promptflow-core test result0 tests 0 ✅ 0s ⏱️ Results for commit 9b639b8. ♻️ This comment has been updated with latest results. |
Executor Unit Test Result devs/peiwen/fix_chatgroup_run796 tests 796 ✅ 3m 46s ⏱️ Results for commit 9b639b8. ♻️ This comment has been updated with latest results. |
Executor E2E Test Result devs/peiwen/fix_chatgroup_run243 tests 238 ✅ 5m 24s ⏱️ Results for commit 9b639b8. ♻️ This comment has been updated with latest results. |
SDK CLI Test Result devs/peiwen/fix_chatgroup_run 4 files 4 suites 1h 3m 35s ⏱️ Results for commit 9b639b8. ♻️ This comment has been updated with latest results. |
Description
In a chat group run, the
line_number
remains consistent within each round, which can lead to the current flow run within a round possibly getting the result of the previous one. To ensure the uniqueness of each request and retrieve the correct execution results, we userequest_id (uuid)
as the key for the result dictionary.All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines