-
Notifications
You must be signed in to change notification settings - Fork 895
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
Leverage async batch run for first async-enabled evaluator - FluencyEvaluator #3542
Conversation
promptflow SDK CLI Azure E2E Test Result users/ninhu/async_evaluators 4 files 4 suites 4m 17s ⏱️ Results for commit 38b77b3. ♻️ This comment has been updated with latest results. |
SDK CLI Global Config Test Result users/ninhu/async_evaluators6 tests 6 ✅ 1m 24s ⏱️ Results for commit 38b77b3. ♻️ This comment has been updated with latest results. |
Executor Unit Test Result users/ninhu/async_evaluators798 tests 798 ✅ 3m 46s ⏱️ Results for commit 38b77b3. ♻️ This comment has been updated with latest results. |
Executor E2E Test Result users/ninhu/async_evaluators246 tests 240 ✅ 5m 5s ⏱️ Results for commit 38b77b3. ♻️ This comment has been updated with latest results. |
SDK CLI Test Result users/ninhu/async_evaluators 4 files 4 suites 1h 6m 43s ⏱️ Results for commit 38b77b3. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good to me.
Description
This PR aims to improve the performance of the evaluate API by leveraging async batch run to eliminate the overhead associated with using multiple processes. The key changes include:
For more details, please check the "Run Evaluators Asynchronously" section in this document.
Results:
All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines