-
Notifications
You must be signed in to change notification settings - Fork 917
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
[Internal][Executor] Use _kwargs to pass all common kwargs to SpawnedForkProcessManager instead of specifying them explicitly #2312
Conversation
…pawnedForkProcessManager
…pawnedForkProcessManager
promptflow-tracing unit test result devs/peiwen/refine_process_pool 4 files 4 suites 22s ⏱️ Results for commit 6652b49. |
SDK CLI Global Config Test Result devs/peiwen/refine_process_pool2 tests 2 ✅ 51s ⏱️ Results for commit 6652b49. |
promptflow-tracing e2e test result devs/peiwen/refine_process_pool 4 files 4 suites 1m 44s ⏱️ Results for commit 6652b49. |
promptflow SDK CLI Azure E2E Test Result devs/peiwen/refine_process_pool 4 files 4 suites 3m 49s ⏱️ Results for commit 6652b49. |
Executor Unit Test Result devs/peiwen/refine_process_pool754 tests 754 ✅ 3m 54s ⏱️ Results for commit 6652b49. |
Executor E2E Test Result devs/peiwen/refine_process_pool210 tests 208 ✅ 6m 9s ⏱️ Results for commit 6652b49. |
SDK CLI Test Result devs/peiwen/refine_process_pool 12 files 12 suites 40m 34s ⏱️ Results for commit 6652b49. |
Description
Use _kwargs to pass all common kwargs to SpawnedForkProcessManager instead of instead of specifying them explicitly.
All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines